*** naohirot has joined #openstack-ironic | 00:01 | |
*** Marga_ has quit IRC | 00:02 | |
*** yuikotakada has joined #openstack-ironic | 00:03 | |
*** spandhe has quit IRC | 00:06 | |
*** yuikotakada has quit IRC | 00:08 | |
*** Marga_ has joined #openstack-ironic | 00:11 | |
*** rloo has quit IRC | 00:16 | |
*** penick has quit IRC | 00:16 | |
*** smoriya has joined #openstack-ironic | 00:20 | |
*** jamielennox|away is now known as jamielennox | 00:35 | |
*** mtanino has quit IRC | 00:56 | |
*** praneshp has joined #openstack-ironic | 00:56 | |
*** praneshp has quit IRC | 00:56 | |
*** jamielennox is now known as jamielennox|away | 01:00 | |
*** jamielennox|away is now known as jamielennox | 01:08 | |
*** mitchjameson has quit IRC | 01:21 | |
*** Haomeng|2 has quit IRC | 01:23 | |
*** Marga_ has quit IRC | 01:29 | |
*** Marga_ has joined #openstack-ironic | 01:30 | |
*** rbudden has quit IRC | 01:39 | |
*** rbudden has joined #openstack-ironic | 01:39 | |
*** rloo has joined #openstack-ironic | 01:52 | |
*** rloo_ has joined #openstack-ironic | 01:54 | |
*** rloo has quit IRC | 01:54 | |
openstackgerrit | Yuiko Takada proposed openstack/ironic-inspector: Load authenticate token from HTTP header https://review.openstack.org/174231 | 02:03 |
---|---|---|
*** yuikotakada has joined #openstack-ironic | 02:03 | |
openstackgerrit | Jamie Lennox proposed openstack/python-ironicclient: Cleanup session creation https://review.openstack.org/185850 | 02:06 |
*** mtanino has joined #openstack-ironic | 02:07 | |
*** rloo_ has quit IRC | 02:14 | |
*** jamielennox is now known as jamielennox|away | 02:20 | |
openstackgerrit | Peter Martini proposed openstack/bifrost: Move explicit URLs in ironic-install to defaults https://review.openstack.org/192580 | 02:23 |
*** Shrews has quit IRC | 02:39 | |
*** Shrews has joined #openstack-ironic | 02:39 | |
*** ramineni has joined #openstack-ironic | 02:42 | |
openstackgerrit | Merged openstack/bifrost: Initial changes for testing VM creation for CentOS https://review.openstack.org/191504 | 02:51 |
*** Marga_ has quit IRC | 02:51 | |
*** Marga_ has joined #openstack-ironic | 02:52 | |
*** spandhe has joined #openstack-ironic | 02:53 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Add a new boot section 'trusted_boot' for PXE https://review.openstack.org/191661 | 02:53 |
*** Marga_ has quit IRC | 03:01 | |
*** jamielennox|away is now known as jamielennox | 03:03 | |
*** jamielennox is now known as jamielennox|away | 03:15 | |
*** lazy_prince has joined #openstack-ironic | 03:22 | |
*** rbudden has quit IRC | 03:22 | |
*** rbudden has joined #openstack-ironic | 03:23 | |
*** jamielennox|away is now known as jamielennox | 03:25 | |
*** Marga_ has joined #openstack-ironic | 03:31 | |
*** Marga_ has quit IRC | 03:33 | |
*** Marga__ has joined #openstack-ironic | 03:33 | |
*** rbudden has quit IRC | 03:39 | |
*** rbudden has joined #openstack-ironic | 03:39 | |
*** penick has joined #openstack-ironic | 03:43 | |
*** spandhe has quit IRC | 03:43 | |
*** mtanino has quit IRC | 03:54 | |
*** achanda has joined #openstack-ironic | 04:07 | |
*** Marga__ has quit IRC | 04:10 | |
*** korean101 has quit IRC | 04:14 | |
*** achanda has quit IRC | 04:23 | |
*** coolsvap|away is now known as coolsvap | 04:30 | |
*** achanda has joined #openstack-ironic | 04:33 | |
*** rbudden has quit IRC | 04:33 | |
*** rbudden has joined #openstack-ironic | 04:33 | |
*** achanda_ has joined #openstack-ironic | 04:39 | |
*** achanda_ has quit IRC | 04:40 | |
*** pradipta has joined #openstack-ironic | 04:40 | |
*** achanda has quit IRC | 04:41 | |
*** zz_natorious is now known as natorious | 04:43 | |
*** boris-42 has joined #openstack-ironic | 04:44 | |
*** natorious is now known as zz_natorious | 04:48 | |
*** ramineni has quit IRC | 04:53 | |
*** Marga_ has joined #openstack-ironic | 05:11 | |
*** Marga_ has quit IRC | 05:16 | |
*** jamielennox is now known as jamielennox|away | 05:23 | |
*** jamielennox|away is now known as jamielennox | 05:26 | |
*** coolsvap is now known as coolsvap|away | 05:28 | |
*** coolsvap|away is now known as coolsvap | 05:36 | |
*** chlong has quit IRC | 05:38 | |
*** chlong has joined #openstack-ironic | 05:53 | |
*** radek_ has joined #openstack-ironic | 05:56 | |
*** Nisha has joined #openstack-ironic | 06:06 | |
*** Marga_ has joined #openstack-ironic | 06:12 | |
*** dtantsur|sick is now known as dtantsur | 06:13 | |
dtantsur | Morning Ironic! | 06:14 |
*** Marga_ has quit IRC | 06:16 | |
*** spandhe has joined #openstack-ironic | 06:19 | |
*** rameshg87 has joined #openstack-ironic | 06:20 | |
rameshg87 | good morning ironic | 06:21 |
*** pradipta has quit IRC | 06:23 | |
*** spandhe has quit IRC | 06:24 | |
*** spandhe_ has joined #openstack-ironic | 06:24 | |
dtantsur | rameshg87, o/ | 06:27 |
yuikotakada | dtantsur, rameshg87: o/ | 06:28 |
dtantsur | hey yuikotakada! :) | 06:28 |
rameshg87 | dtantsur: yuikotakada: o/ | 06:28 |
dtantsur | yuikotakada, we got https://github.com/openstack/python-ironic-inspector-client yesterday, please keep an eye on it from now on as well :) | 06:29 |
*** mitchjameson has joined #openstack-ironic | 06:37 | |
yuikotakada | dtantsur, OK, I have not catch up with it at all. I'll check it from now :) | 06:38 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Add gitreview file https://review.openstack.org/192988 | 06:41 |
yuikotakada | dtantsur, checking... It seems np, well, how did you decide port no? It's too dififcult to review lol | 06:43 |
dtantsur | yuikotakada, you mean this patc above? I've copy-pasted ironic-inspector .gitreview and fixed the repo name :) | 06:45 |
dtantsur | no magic actually | 06:45 |
yuikotakada | dtantsur, yep, I mean so. Ah, that's true. I didn't know all projects have same port no (now I checked Nova's one also) | 06:48 |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Add gitreview file https://review.openstack.org/192988 | 06:48 |
yuikotakada | dtantsur, BTW, are you OK? You were sick yesterday, right? | 06:49 |
*** subscope has joined #openstack-ironic | 06:50 | |
*** mmorais_ has quit IRC | 06:51 | |
dtantsur | yuikotakada, yeah thanks, I'm better now :) | 06:52 |
*** coolsvap is now known as coolsvap|away | 06:53 | |
*** mmorais has joined #openstack-ironic | 06:53 | |
yuikotakada | dtantsur, fine :) take care. | 06:54 |
dtantsur | thanks | 06:54 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Sync remaining changes from ironic-inspector tree https://review.openstack.org/192989 | 06:55 |
*** saripurigopi has joined #openstack-ironic | 07:01 | |
*** coolsvap|away is now known as coolsvap | 07:01 | |
*** spandhe_ has quit IRC | 07:03 | |
*** e0ne has joined #openstack-ironic | 07:04 | |
*** subscope has quit IRC | 07:05 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: After-moving code reorganization https://review.openstack.org/192993 | 07:05 |
*** penick has quit IRC | 07:07 | |
yuikotakada | dtantsur, I'm checking https://review.openstack.org/#/c/192989 , it seems np. then, how to handle https://review.openstack.org/#/c/192190/ ? | 07:07 |
*** mitchjameson has quit IRC | 07:07 | |
dtantsur | yuikotakada, ah good question, for some reason I thought we approved it already :) I'm going to approve them, and then drop client code in a separate patch. WDYT? | 07:08 |
*** jcoufal has joined #openstack-ironic | 07:09 | |
yuikotakada | dtantsur, yeah, I'm OK. just confused | 07:10 |
*** lazy_prince has quit IRC | 07:10 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: After-moving code reorganization https://review.openstack.org/192993 | 07:11 |
*** Marga_ has joined #openstack-ironic | 07:12 | |
*** Marga_ has quit IRC | 07:17 | |
*** chlong has quit IRC | 07:18 | |
openstackgerrit | Merged openstack/ironic-inspector: Change the way error are returned from API to JSON https://review.openstack.org/192190 | 07:21 |
*** dasm|afk is now known as dasm | 07:23 | |
*** ifarkas has joined #openstack-ironic | 07:23 | |
*** e0ne has quit IRC | 07:23 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Basic support for API versions https://review.openstack.org/192996 | 07:28 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: After-moving code reorganization https://review.openstack.org/192993 | 07:30 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Basic support for API versions https://review.openstack.org/192996 | 07:30 |
*** subscope has joined #openstack-ironic | 07:30 | |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Sync remaining changes from ironic-inspector tree https://review.openstack.org/192989 | 07:30 |
*** coolsvap is now known as coolsvap|away | 07:34 | |
*** jlvillal has quit IRC | 07:37 | |
*** jrist has quit IRC | 07:44 | |
*** jrist has joined #openstack-ironic | 07:44 | |
*** Haomeng has joined #openstack-ironic | 07:45 | |
*** mmorais has quit IRC | 07:47 | |
*** mmorais has joined #openstack-ironic | 07:48 | |
*** dlpartain has joined #openstack-ironic | 07:48 | |
*** stendulker has joined #openstack-ironic | 07:48 | |
*** boris-42 has quit IRC | 07:52 | |
*** rbudden has quit IRC | 07:52 | |
*** rbudden has joined #openstack-ironic | 07:52 | |
*** yog_ has joined #openstack-ironic | 07:53 | |
*** jlvillal has joined #openstack-ironic | 07:54 | |
*** ramineni has joined #openstack-ironic | 07:54 | |
*** athomas has joined #openstack-ironic | 07:58 | |
*** gzhai has joined #openstack-ironic | 08:00 | |
gzhai | dtantsur:hello | 08:08 |
*** Nisha has quit IRC | 08:13 | |
*** Marga_ has joined #openstack-ironic | 08:13 | |
gzhai | dtantsur: Do you have time to review my BP after modification? @ https://review.openstack.org/#/c/176109/ | 08:13 |
*** dlpartain has left #openstack-ironic | 08:14 | |
*** dguerri` is now known as dguerri | 08:15 | |
dtantsur | gzhai, hi, will do (a bit later today) | 08:16 |
*** Marga_ has quit IRC | 08:18 | |
gzhai | dtantsur: Thanks! | 08:18 |
*** sambetts_ has joined #openstack-ironic | 08:21 | |
*** romainh has joined #openstack-ironic | 08:22 | |
sambetts_ | Morning Ironic o/ | 08:23 |
yuikotakada | sambetts, o/ | 08:24 |
*** MattMan has joined #openstack-ironic | 08:26 | |
*** ndipanov has joined #openstack-ironic | 08:27 | |
*** mdbooth has joined #openstack-ironic | 08:28 | |
*** viktors|afk is now known as viktors | 08:28 | |
*** lucas-dinner has joined #openstack-ironic | 08:28 | |
dtantsur | sambetts_, morning! | 08:30 |
dtantsur | sambetts_, also FYI: we have https://github.com/openstack/python-ironic-inspector-client now \o/ | 08:31 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add iRMC Virtual Media Deploy module for iRMC Driver https://review.openstack.org/151958 | 08:31 |
openstackgerrit | Merged openstack/python-ironic-inspector-client: After-moving code reorganization https://review.openstack.org/192993 | 08:32 |
*** subscope has quit IRC | 08:32 | |
*** Nisha has joined #openstack-ironic | 08:33 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Refactor node's and driver's vendor passthru to a common place https://review.openstack.org/190121 | 08:33 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Allow vendor methods to serve static files https://review.openstack.org/189716 | 08:33 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Change return value of [driver_]vendor_passthru to dict https://review.openstack.org/190120 | 08:33 |
*** lucas-dinner is now known as lucasagomes | 08:33 | |
dtantsur | lucasagomes, it was a long dinner, wasn't it? :D | 08:33 |
lucasagomes | dtantsur, haha good morning | 08:34 |
lucasagomes | it was indeed | 08:34 |
dtantsur | morning :) | 08:34 |
yuikotakada | lucasagomes, g'morning | 08:34 |
lucasagomes | yuikotakada, good ugt morning | 08:36 |
*** mdbooth has quit IRC | 08:39 | |
*** derekh has joined #openstack-ironic | 08:40 | |
*** mdbooth has joined #openstack-ironic | 08:40 | |
*** yuikotakada is now known as yuikot_dinner | 08:40 | |
*** yuikot_dinner has quit IRC | 08:40 | |
*** zhenguo has quit IRC | 08:44 | |
*** mdbooth has quit IRC | 08:45 | |
*** sambetts_ has quit IRC | 08:46 | |
*** e0ne has joined #openstack-ironic | 08:48 | |
*** subscope has joined #openstack-ironic | 08:49 | |
*** sambetts_ has joined #openstack-ironic | 08:53 | |
*** mdbooth has joined #openstack-ironic | 08:53 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add localboot support for iscsi_irmc driver https://review.openstack.org/163738 | 08:54 |
openstackgerrit | Anusha Ramineni proposed openstack/ironic: Add option to configure passes in erase_devices https://review.openstack.org/191750 | 08:57 |
*** e0ne is now known as e0ne_ | 08:58 | |
*** pelix has joined #openstack-ironic | 09:00 | |
*** e0ne_ is now known as e0ne | 09:00 | |
sambetts_ | dtantsur: thats awesome we've got a client :D | 09:00 |
*** coolsvap|away is now known as coolsvap | 09:02 | |
*** oomichi has quit IRC | 09:05 | |
*** dtantsur is now known as dtantsur|brb | 09:05 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add whole disk image support for iscsi_irmc driver https://review.openstack.org/181771 | 09:07 |
*** romcheg has joined #openstack-ironic | 09:09 | |
*** Marga_ has joined #openstack-ironic | 09:14 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add IPA support for iscsi_irmc driver https://review.openstack.org/181807 | 09:17 |
*** Marga_ has quit IRC | 09:19 | |
openstackgerrit | Anusha Ramineni proposed openstack/ironic-python-agent: IPA:'shred' utility to use configured iterations https://review.openstack.org/191751 | 09:25 |
openstackgerrit | Anusha Ramineni proposed openstack/ironic-python-agent: IPA:'shred' utility to use configured iterations https://review.openstack.org/191751 | 09:26 |
openstackgerrit | Anusha Ramineni proposed openstack/ironic: Add option to configure passes in erase_devices https://review.openstack.org/191750 | 09:27 |
lucasagomes | btw gate is broken https://bugs.launchpad.net/ironic/+bug/1466405 | 09:35 |
openstack | Launchpad bug 1466405 in Ironic "broken ACL tests " [Undecided,New] - Assigned to Lucas Alvares Gomes (lucasagomes) | 09:35 |
lucasagomes | something changed in the dependecies I think and broke us | 09:35 |
*** Marga_ has joined #openstack-ironic | 09:39 | |
*** Marga_ has quit IRC | 09:43 | |
*** Marga_ has joined #openstack-ironic | 09:44 | |
rameshg87 | lucasagomes: sambetts: morning | 09:46 |
rameshg87 | yeah gate seems broken :( | 09:46 |
lucasagomes | rameshg87, it's the new version of keystonemiddleware that was release yesterday | 09:47 |
lucasagomes | :-/ | 09:47 |
* lucasagomes fixing | 09:47 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Rename ilo_power._attach_boot_iso to improve readability https://review.openstack.org/193042 | 09:53 |
*** e0ne is now known as e0ne_ | 09:57 | |
*** naohirot has quit IRC | 09:59 | |
*** e0ne_ is now known as e0ne | 09:59 | |
*** jcoufal has quit IRC | 10:02 | |
*** dtantsur|brb is now known as dtantsur | 10:08 | |
*** yuanying has quit IRC | 10:10 | |
dtantsur | lucasagomes, so, keystonemiddleware.. I wonder if it broke inspector too... | 10:11 |
lucasagomes | dtantsur, right idk | 10:11 |
lucasagomes | I don't know what changed that broke ironic yet | 10:11 |
lucasagomes | I suspect https://review.openstack.org/#/c/174196 | 10:11 |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Basic support for API versions https://review.openstack.org/192996 | 10:15 |
*** martini has joined #openstack-ironic | 10:16 | |
martini | Good morning Ironic | 10:16 |
*** Nisha has quit IRC | 10:17 | |
*** romainh has quit IRC | 10:18 | |
*** achanda has joined #openstack-ironic | 10:20 | |
MattMan | morning ironic | 10:27 |
MattMan | dtantsur: I have a question related to inspection... specifically the naming of discovered properties | 10:29 |
MattMan | Essential properties that I can see are memory_gb, local_gb, cpus and cpu_arch right ? | 10:30 |
lucasagomes | ok found the problem in gate... fixing | 10:32 |
dtantsur | MattMan, hi! yes (just memory_mb not memory_gb) | 10:33 |
MattMan | ah ok thank | 10:33 |
MattMan | A driver can add any other properties it needs as well I presume | 10:34 |
dtantsur | MattMan, that's properties Nova needs from us. We also find out MAC addresses (and probably LLDP info in the future) | 10:34 |
MattMan | For instance SPARC hardware may have a small number of CPU's but it's the number of cores per CPU that makes the real difference, so we would need "cpu_cores" | 10:35 |
dtantsur | MattMan, if your driver depends on something introspectable, of course you can do it | 10:35 |
MattMan | great | 10:35 |
dtantsur | MattMan, note that cpus is used by Nova, not Ironic | 10:35 |
MattMan | nod | 10:35 |
*** saripurigopi has quit IRC | 10:36 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Switch to pbr https://review.openstack.org/193056 | 10:36 |
MattMan | Now where does capabilities fit into this / | 10:36 |
dtantsur | MattMan, depending. you can introspect capabilities like ilo does: http://specs.openstack.org/openstack/ironic-specs/specs/kilo/ilo-properties-capabilities-discovery.html | 10:37 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix broken ACL tests https://review.openstack.org/193057 | 10:38 |
lucasagomes | ^ this is fixing gate | 10:38 |
lucasagomes | (hopefully) so please cores take a look | 10:38 |
MattMan | dtantsur, is there a predefined list of capabilities as well ? | 10:38 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Switch to pbr https://review.openstack.org/193056 | 10:38 |
dtantsur | MattMan, only what ilo folks decided for them | 10:39 |
*** saripurigopi has joined #openstack-ironic | 10:39 | |
MattMan | ok, from what I have been reading they can be set on the flavor and passed down to ironic in that manner.. I'll look at ilo driver to see if what they do makes sense for us as well... thanks for your help | 10:40 |
dtantsur | lucasagomes, ouch, I believe we should not use it in tests | 10:40 |
lucasagomes | dtantsur, remove those tests? | 10:40 |
lucasagomes | dtantsur, I mean... we had it before, I'm just fixing it to be compatible with the new version | 10:40 |
dtantsur | lucasagomes, remove anything touching keystonemiddleware internals | 10:40 |
dtantsur | why do we even have them? | 10:40 |
lucasagomes | dtantsur, I can't recall | 10:41 |
lucasagomes | it seems to be old stuff | 10:41 |
dtantsur | good time to drop it ;) | 10:41 |
*** achanda has quit IRC | 10:41 | |
lucasagomes | 71c2525d (Roman Prykhodchenko 2013-07-15 20:29:19 +0300 34) class FakeMemcache(object): | 10:41 |
lucasagomes | 2013 | 10:41 |
lucasagomes | :-) | 10:41 |
lucasagomes | dtantsur, right... I mean, let's fix it. And then we can discuss whether drop it or not | 10:42 |
dtantsur | lucasagomes, fine | 10:43 |
dtantsur | lemme know when tests pass :) | 10:43 |
lucasagomes | dtantsur, ack, just as a FYI I'm having this same discussion in the keystone channel | 10:44 |
lucasagomes | <samueldmq> lucasagomes, and yes, fixing that is fair enough for now | 10:44 |
lucasagomes | <lucasagomes> samueldmq, yup | 10:44 |
lucasagomes | <samueldmq> lucasagomes, nice | 10:44 |
lucasagomes | <lucasagomes> yeah I will fix it and then bring it up to the community whether we should drop it or not | 10:44 |
lucasagomes | <lucasagomes> I don't wanna fix the gate by deleting the tests, I think it's unfair | 10:44 |
lucasagomes | <samueldmq> lucasagomes, yes it is, fix that to have the gate working asap, then revisit if they're really needed | 10:44 |
lucasagomes | <lucasagomes> samueldmq, yes, I think it sounds like a good approach | 10:44 |
lucasagomes | <lucasagomes> samueldmq, thanks much! | 10:44 |
dtantsur | ack | 10:44 |
*** athomas has quit IRC | 10:47 | |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Switch to pbr https://review.openstack.org/193056 | 10:47 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Manual update from global requirements https://review.openstack.org/193059 | 10:49 |
*** e0ne is now known as e0ne_ | 10:50 | |
*** coolsvap is now known as coolsvap|away | 10:56 | |
*** athomas has joined #openstack-ironic | 10:56 | |
*** Haomeng|2 has joined #openstack-ironic | 10:57 | |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Manual update from global requirements https://review.openstack.org/193059 | 10:57 |
dasm | o/ | 10:58 |
dasm | i have an question about ironic baremetal provisioning. using rdo, i've encountered a problem with provisioning hosts. | 10:59 |
dasm | in heat stack-show i see failure with traceback: "No valid host was found. Exceeded max scheduling " | 10:59 |
*** Haomeng has quit IRC | 10:59 | |
*** Marga_ has quit IRC | 11:00 | |
dasm | in the mean time, nova-scheduler shows me sth like: | 11:00 |
*** e0ne_ has quit IRC | 11:00 | |
dasm | Selected host: WeighedHost [host: (gklab-44-126, 04672bd8-5e8e-41e9-be5d-8b89b4c13c8b) ram:262144 disk:1142784 io_ops:0 instances:0, weight: 0.666666666667] _schedule /usr/lib/python2.7/site-packages/nova/scheduler/filter_scheduler.py:158 | 11:00 |
dasm | 2015-06-17 17:58:37.129 61895 WARNING nova.scheduler.host_manager [req-45297851-3cec-4052-84a5-7ca0d2e2c4ab 6fcd049dbb6c4eb4b6e30a1b58656c27 58d12f25b9664589875c097668963a54 - - -] Host 04672bd8-5e8e-41e9-be5d-8b89b4c13c8b has more disk space than database expected (1116gb > 1076gb) | 11:00 |
*** ramineni has quit IRC | 11:00 | |
dasm | is it connected, each other or just notification? | 11:00 |
dtantsur | lucasagomes, cool, I've set https://launchpad.net/python-ironic-inspector-client maintainers to ironic-drivers by mistake and it locked me out of maitaining it | 11:01 |
dtantsur | lucasagomes, as you're ironic-drivers member, could you revert it back to me? | 11:01 |
dasm | how ironic supposed to work, when starting baremetal provisioning? could anyone try to explain it in short words? | 11:01 |
lucasagomes | dtantsur, hah sure | 11:01 |
* lucasagomes reverts | 11:01 | |
lucasagomes | dtantsur, you want me to set it to you directly? Or a specific group? | 11:02 |
dtantsur | lucasagomes, to me for now | 11:02 |
lucasagomes | there's a inspected-members or something? | 11:02 |
lucasagomes | ack | 11:02 |
lucasagomes | dtantsur, done | 11:02 |
dtantsur | lucasagomes, thanks a lot! | 11:02 |
lucasagomes | no problem | 11:02 |
*** Marga_ has joined #openstack-ironic | 11:03 | |
lucasagomes | dtantsur, the beauty of launchpad :D | 11:03 |
dtantsur | lucasagomes, I will form a group and invite you there just in case, ok? | 11:03 |
lucasagomes | dtantsur, ack | 11:03 |
* rameshg87 goes home | 11:06 | |
*** rameshg87 has quit IRC | 11:06 | |
*** Marga_ has quit IRC | 11:08 | |
*** Marga_ has joined #openstack-ironic | 11:08 | |
*** stendulker has quit IRC | 11:09 | |
* dtantsur makes a release | 11:12 | |
*** saripurigopi has quit IRC | 11:17 | |
yuriyz | morning Ironic | 11:24 |
dtantsur | yuriyz, morning | 11:24 |
yuriyz | lucasagomes thanks for the unit tests fix but it's py3 incompatible | 11:25 |
*** sambetts_ has quit IRC | 11:28 | |
*** jlvillal has quit IRC | 11:30 | |
lucasagomes | yuriyz, oh :-( fixing that too | 11:31 |
*** jlvillal has joined #openstack-ironic | 11:31 | |
lucasagomes | yuriyz, thanks for the review | 11:32 |
*** amotoki_ has joined #openstack-ironic | 11:33 | |
yuriyz | lucasagomes maybe you should set up local py34 env :) | 11:33 |
lucasagomes | yeah I have too | 11:34 |
lucasagomes | I'm using f20 yet, cause I'm a bit stuck so there's no py34 | 11:34 |
lucasagomes | plus unittests on py34 for newer versions of python34 are all broken | 11:34 |
lucasagomes | due a mock bug | 11:34 |
lucasagomes | summarizing py34 is a piece of s* | 11:34 |
*** e0ne has joined #openstack-ironic | 11:36 | |
openstackgerrit | Monty Taylor proposed openstack/python-ironic-inspector-client: Add venv environment for tox https://review.openstack.org/193078 | 11:36 |
*** romainh has joined #openstack-ironic | 11:46 | |
TheJulia | Haomeng|2: By chance are you awake? | 11:49 |
*** Marga_ has quit IRC | 11:52 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Add venv environment for tox https://review.openstack.org/193078 | 11:55 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Use pbr without explicit version https://review.openstack.org/193087 | 11:55 |
*** jrist has quit IRC | 11:56 | |
*** jrist has joined #openstack-ironic | 11:56 | |
*** Marga_ has joined #openstack-ironic | 11:57 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix broken ACL tests https://review.openstack.org/193057 | 11:57 |
lucasagomes | yuriyz, ^ thanks for the suggestions inline | 11:58 |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Use pbr without explicit version https://review.openstack.org/193087 | 12:01 |
*** achanda has joined #openstack-ironic | 12:02 | |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Add venv environment for tox https://review.openstack.org/193078 | 12:02 |
openstackgerrit | Merged openstack/bifrost: Fix when logic for boolean handling https://review.openstack.org/191979 | 12:04 |
TheJulia | Yay, merge conflicts :( | 12:05 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Explicitly depend on pbr https://review.openstack.org/193090 | 12:08 |
*** trown|outttypeww is now known as trown | 12:08 | |
martini | dang - I rebased ~14 hours ago (at least I think I did) | 12:08 |
*** dprince has joined #openstack-ironic | 12:09 | |
martini | Oh, I *definitely* need to add docs on how to clone ansible offline. | 12:09 |
*** yuikotakada has joined #openstack-ironic | 12:10 | |
martini | It's a bit tedious because of the submodules - git clone recursive will clone the top level locally, then try to clone the submodules from github. What I end up doing each time I do this is: cd /opt/stack; git clone /vagrant/git/ansible; cd ansible; cp /vagrant/gitmodules .gitmodules; git submodule init; git submodule update; git checkout -- .gitmodules; then go back and env_setup works fine | 12:11 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Move explicit URLs in ironic-install to defaults https://review.openstack.org/192580 | 12:12 |
TheJulia | martini: I merged a fix for a nasty issue that had a neighboring line change | 12:12 |
TheJulia | so... merge conflcit :( | 12:12 |
martini | Ah, such is timing :-) | 12:13 |
TheJulia | I've heard submodules are a nightmare at times, but have never really messed with them. | 12:13 |
TheJulia | well, one +2 :) | 12:15 |
*** Shrews has quit IRC | 12:17 | |
openstackgerrit | Merged openstack/python-ironic-inspector-client: Explicitly depend on pbr https://review.openstack.org/193090 | 12:18 |
*** Shrews has joined #openstack-ironic | 12:18 | |
*** jjohnson2 has joined #openstack-ironic | 12:25 | |
*** Marga_ has quit IRC | 12:27 | |
*** e0ne is now known as e0ne_ | 12:32 | |
openstackgerrit | Zhenguo Niu proposed openstack/ironic: When boot option is not persisted, set boot on next power on https://review.openstack.org/177642 | 12:34 |
*** radek_ has quit IRC | 12:36 | |
*** radek_ has joined #openstack-ironic | 12:37 | |
*** e0ne_ is now known as e0ne | 12:39 | |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Add monotonic time handling for Windows https://review.openstack.org/193108 | 12:50 |
openstackgerrit | Jarrod Johnson proposed stackforge/pyghmi: Add monotonic time handling for Windows https://review.openstack.org/193108 | 12:51 |
openstackgerrit | Merged stackforge/pyghmi: Acknowledge PET alert on decode https://review.openstack.org/189392 | 12:52 |
*** rloo has joined #openstack-ironic | 12:57 | |
*** smoriya has quit IRC | 13:01 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Change job logging such that successful jobs save logs https://review.openstack.org/193114 | 13:06 |
*** pradipta has joined #openstack-ironic | 13:11 | |
*** achanda has quit IRC | 13:12 | |
*** athomas_ has joined #openstack-ironic | 13:15 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: API: Get a subset of fields from Ports and Chassis https://review.openstack.org/183549 | 13:19 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: API: Get a subset of fields from Nodes https://review.openstack.org/183491 | 13:19 |
lucasagomes | jroll, dtantsur rloo if you guys are around, please take a look at https://review.openstack.org/#/c/193057/ | 13:20 |
openstackgerrit | Christopher Dearborn proposed openstack/ironic: Add DRAC BIOS config vendor passthru API https://review.openstack.org/150488 | 13:20 |
* dtantsur looks | 13:20 | |
lucasagomes | to unblock the gate, tests are passing | 13:20 |
rloo | lucasagomes, dtantsur: hi | 13:20 |
rloo | lucasagomes: you're too fast for me. I just replied to an older version of 183491! | 13:21 |
rloo | lucasagomes: will look | 13:21 |
lucasagomes | rloo, oh hah ack | 13:21 |
dtantsur | rloo, hi! | 13:21 |
rloo | lucasagomes: so for 193057, we keep supporting keystonemiddleware until the requirements change to >= 2.0.0? | 13:25 |
*** yuikotakada has quit IRC | 13:25 | |
*** athomas_ has quit IRC | 13:26 | |
rloo | lucasagomes: I mean keep supporting the non-hash-vesion | 13:26 |
lucasagomes | rloo, yeah pretty much... currently the requirements.txt is >=1.5.0 | 13:26 |
lucasagomes | so we gotta support the non-hash | 13:26 |
lucasagomes | rloo, yeah | 13:26 |
rloo | lucasagomes: ok, might have been nice to have a comment about removing the other stuff later, but dtantsur has already approved :) | 13:26 |
lucasagomes | I don't think people are pinning the global requirements to >=2.0.0 just yet tho, cause it doesn't break all projects. It broke Ironic and Ceilometer AFAICT | 13:27 |
lucasagomes | because we do ACL tests | 13:27 |
lucasagomes | apparently other projects doesn't bother on doing it | 13:27 |
NobodyCam | morning from grants Pass Ironicers | 13:27 |
lucasagomes | so it just works for them | 13:27 |
rloo | lucasagomes: yay for us! sort of ;) | 13:27 |
lucasagomes | yeah | 13:27 |
rloo | morning NobodyCam | 13:28 |
lucasagomes | rloo, well we also could think about removing it tho, it's sorta of a functional test that we have there | 13:28 |
rloo | lucasagomes: we can't as long as it isn't pinned to >2.0.0 | 13:28 |
lucasagomes | but let's fix first to unblock the gate and then talk about whether we should remove it or not | 13:28 |
rloo | >= I mean | 13:28 |
lucasagomes | rloo, I mean removing the whole tests | 13:29 |
lucasagomes | for ACL | 13:29 |
rloo | lucasagomes: oh. hmm. hadn't thought of why we have the acl tests in the first place. if it is just testing keystone... yeah, do we need it. | 13:29 |
lucasagomes | yeah it's old stuff from 2013 | 13:30 |
dtantsur | NobodyCam, morning! | 13:30 |
lucasagomes | I also don't rememeber the reasonings about having it | 13:30 |
NobodyCam | morning rloo lucasagomes and dtantsur | 13:30 |
lucasagomes | rloo, and tempest will test it for us, we probably don't need to bother having it on unittest | 13:31 |
lucasagomes | but anyway | 13:31 |
lucasagomes | ... | 13:31 |
lucasagomes | NobodyCam, good ugt morning | 13:31 |
NobodyCam | :) | 13:32 |
lucasagomes | rloo, re 183491 replied to ur replies | 13:32 |
lucasagomes | thanks | 13:32 |
rloo | lucasagomes: ok. yw :) | 13:32 |
lucasagomes | I've most changed the names according to your suggestions before _DEFAULT_RETURN_FIELDS and check_allow_specify_fields() | 13:33 |
rloo | lucasagomes: btw, I think you mean 'polling', not 'pooling' | 13:33 |
lucasagomes | and agreed with chassis_id being PITA | 13:33 |
lucasagomes | :-( yup I do | 13:34 |
* lucasagomes fix the bug | 13:34 | |
rloo | lucasagomes: i noticed in the nova bug but forgot to mention it to you | 13:34 |
*** Marga_ has joined #openstack-ironic | 13:34 | |
lucasagomes | rloo, thx, fixing both | 13:34 |
dtantsur | lucasagomes, can you convince me not to -1 you for check_allow_specify_fields? ;) | 13:35 |
rloo | lucasagomes: ok, i'm fine with your comments. will wait for jenkins first, i haven't yet looked at the test changes | 13:36 |
* dtantsur is not fond of "let's blindly copy Nova" approach | 13:36 | |
lucasagomes | dtantsur, heh I probably can't... But so far I think people in the API-WG have being towards to agree that we should bump the micro version for new features too | 13:37 |
lucasagomes | dtantsur, well yeah, but we should follow the API-WG it was created to make APIs more consistent | 13:37 |
dtantsur | lucasagomes, I'm not asking about bumping the microversion | 13:37 |
lucasagomes | no reason to diverge from it | 13:37 |
dtantsur | lucasagomes, reason: it's insane and they ignore critics? | 13:37 |
rloo | dtantsur: i think until we come to an agreement, we should continue to do what we've been doing | 13:37 |
lucasagomes | dtantsur, right yeah, but I will keep doing as we did before | 13:37 |
lucasagomes | if turns out we should not do it we can remove | 13:38 |
dtantsur | *facepalm* | 13:38 |
lucasagomes | just like we would remove it from name and so on | 13:38 |
dtantsur | ok, do whatever you want | 13:38 |
* dtantsur removes patch from his watch list | 13:38 | |
lucasagomes | heh fair enuff | 13:38 |
lucasagomes | dtantsur, also jroll was talking about it yesterday, threating micro version just like releases of a library | 13:38 |
dtantsur | it's clearly not even remotely the same | 13:39 |
lucasagomes | cause you don't bump a version of a lib and release it when things are not backward compat | 13:39 |
rloo | dtantsur: it seems like there are several 'issues'. 1. What the guidelines will be; 2. whether Ironic will follow all/some of those guidelines; 3. whether ironic agrees with you | 13:39 |
lucasagomes | yeah well it's an analogy | 13:39 |
dtantsur | lucasagomes, "bump a version of a lib and release it when things are not backward compat" is exactly what everyone does and will do | 13:40 |
lucasagomes | dtantsur, they also bump the version and release when new features are added | 13:40 |
lucasagomes | that's the thinking | 13:40 |
dtantsur | lucasagomes, we also bump version and release, but how is it related to API? | 13:41 |
dtantsur | API is not library. it's rather like requiring #define API_VERSION 10 to use new features. it's done extremely rarely | 13:41 |
lucasagomes | dtantsur, it was an analogy | 13:42 |
lucasagomes | jroll, may be able to explain it better... | 13:42 |
dtantsur | well, analogies are nice, but I'm still waiting for at least one use case for that. without use case we're jsut doing additional work | 13:43 |
dtantsur | like rloo says: do everything, unless it requires me to do more work :) | 13:43 |
lucasagomes | heh | 13:43 |
lucasagomes | yeah I understand, we are in the middle of the discussions about when to bump the micro version etc... so for now I will continue to do what we did before for other things | 13:44 |
lucasagomes | until we get a consensus | 13:44 |
rloo | dtantsur: ++. That's partly why I'm ignoring, not participating in the decision of what to do. that means having to think about it etc. | 13:44 |
dtantsur | i.e. I'm not against us playing weird toys, I'm against it making Ironic harder for contributors and for reviewers :) | 13:44 |
rloo | dtantsur: sometimes I prefer to be a lamb. | 13:44 |
lucasagomes | dtantsur, sure, I understand | 13:44 |
lucasagomes | but in doubt fall back to the current default | 13:45 |
lucasagomes | so... that's what I did | 13:45 |
rloo | dtantsur: "just tell me what to do so we can get the code in and things are consistent" :) | 13:45 |
dtantsur | the current default, to be honest, was not properly decided and properly approved by the tean | 13:45 |
* lucasagomes brb coffee | 13:46 | |
dtantsur | ok, the last thing that pisses me off is that people are talking about making developers happy. I am the developer for Ironic, but they ignore me :) | 13:46 |
dtantsur | and enough with this, time to get some job done :) | 13:46 |
rloo | dtantsur: agreed (wrt lack of decision process) | 13:46 |
rloo | dtantsur: i didn't realize they wanted to make developers happy. if they wanted to do that, we should let everyone merge their code. | 13:47 |
rloo | dtantsur: except that might make me unhappy. hmm, guess we can't make everyone happy. | 13:48 |
trown | lol | 13:48 |
trown | indeed | 13:48 |
trown | good ugt moring Ironic | 13:48 |
dtantsur | heh, merging random code won't make developers happy :) | 13:48 |
dtantsur | trown, morning :) | 13:48 |
lucasagomes | trown, morning | 13:48 |
dtantsur | didn't I say g'morning to you already? | 13:48 |
NobodyCam | morning (UGT) trown | 13:49 |
NobodyCam | lol | 13:49 |
rloo | dtantsur: we should ask on the mailing list. if any non-cores reply, it might be interesting to read. | 13:49 |
trown | dtantsur: ya, in #rdo :p | 13:49 |
rloo | hi trown | 13:49 |
trown | good morning rloo lucasagomes NobodyCam | 13:49 |
openstackgerrit | Merged openstack/bifrost: Fix DHCP conditionals https://review.openstack.org/192174 | 13:49 |
dtantsur | rloo, if we make even majority of cores reply to anything on ML, I would call it a great success :D | 13:49 |
rloo | dtantsur: oh, I rarely reply unless I feel strongly about something and no one else has commented. I don't see it as a voting thing (others do I guess) and I feel like it adds more noise if I reply to say I agree, so I don't bother. | 13:51 |
*** achanda has joined #openstack-ironic | 13:52 | |
openstackgerrit | Merged openstack/bifrost: Allow a user to define the OS for the Disk Image https://review.openstack.org/191980 | 13:52 |
lucasagomes | dtantsur, ignoring ur comments... Have you posted it on the microversion review ? | 13:56 |
lucasagomes | I think you will get some input from there instead of ML | 13:56 |
lucasagomes | (just an idea, I haven't actually checked if you did or not) | 13:56 |
openstackgerrit | Merged openstack/ironic: Fix broken ACL tests https://review.openstack.org/193057 | 13:57 |
lucasagomes | w00t gate is back | 13:57 |
*** Marga_ has quit IRC | 13:58 | |
rloo | lucasagomes: wow, that was fast! | 13:59 |
lucasagomes | rloo, yeah, well took me a couple of hours in the morning | 13:59 |
lucasagomes | to actually find the problem | 13:59 |
rloo | lucasagomes: oh yeah, thank you for doing that :D | 14:00 |
lucasagomes | thanks to some folks in the keystone channel helped with it | 14:00 |
NobodyCam | lucasagomes: :) w00t | 14:01 |
* lucasagomes opens a beer | 14:01 | |
lucasagomes | jk :-( I wish | 14:01 |
*** achanda has quit IRC | 14:02 | |
rloo | lucasagomes: fwiw, I was thinking that we should pay more attention to new library releases. so I looked at the email - release for keystonemiddleware, and I would not have known from that, if anything would break. | 14:02 |
*** achanda has joined #openstack-ironic | 14:03 | |
*** Marga_ has joined #openstack-ironic | 14:03 | |
*** achanda has quit IRC | 14:03 | |
lucasagomes | rloo, yeah, problem is to know when it will break u or not... I think the new library release email is good to check when things break | 14:04 |
rloo | lucasagomes: it would have been useful if there had been a 'non-backwards-compat' section in the email or somewhere... | 14:05 |
dtantsur | lucasagomes, the fun thing is that nobody openly says about hiding the features. the guidelines do not contain it, it's considered to be implied :) | 14:05 |
lucasagomes | rloo, that would be awesome! | 14:05 |
lucasagomes | rloo, I'm not sure people are actually tracking it... perhaps once a change is merged which is non backward compat | 14:06 |
lucasagomes | they could send an email | 14:06 |
lucasagomes | same for us | 14:06 |
dtantsur | hmm, interesting | 14:06 |
dtantsur | lucasagomes, the problem is, changing internals is not considered a breaking change | 14:06 |
lucasagomes | yeah | 14:07 |
rloo | lucasagomes: i was thinking since dhellmann (or whoever) sends out an email about releases, that the info should be in that email or linked from it. | 14:07 |
lucasagomes | for this case | 14:07 |
lucasagomes | dtantsur, +1 | 14:07 |
lucasagomes | rloo, right could be, people would need to add this info somewhere as things are getting merged so they don't forget | 14:07 |
rloo | lucasagomes: true. but if you're going to release something with a major version bump... | 14:08 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Switch to pbr https://review.openstack.org/193145 | 14:10 |
dtantsur | can't believe I'm doing it ^^^ | 14:10 |
dtantsur | :) | 14:10 |
lucasagomes | rloo, yeah... we could think about something along those lines for Ironic | 14:10 |
lucasagomes | for our new release model | 14:10 |
lucasagomes | everytime we release something, put up an email with the changes and so on | 14:10 |
rloo | lucasagomes: oh, yeah, whoever is handling the releases should let the rest of us know what they might want :) | 14:11 |
lucasagomes | hah fair | 14:12 |
trown | dtantsur: haha | 14:12 |
trown | dtantsur: I tried the 'postversioning', and I like it better | 14:13 |
trown | seems more intuitive to me | 14:13 |
dtantsur | trown, a bit annoying that your version will be patch one, until you tag | 14:14 |
dtantsur | i.e. it won't understand that 2.1.0 will follow 2.0.0 on master | 14:15 |
dtantsur | by default | 14:15 |
trown | ya I suppose for stuff that is > 1.0.0, it makes more sense for preversion...you know what you want in the next version | 14:17 |
trown | I kept screwing up preversion when packaging though | 14:18 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Move client out of tree https://review.openstack.org/193150 | 14:18 |
*** bethelwell has joined #openstack-ironic | 14:19 | |
dtantsur | sambetts, are we calling https://blueprints.launchpad.net/ironic-inspector/+spec/node-not-found-hook done? | 14:22 |
*** dasm is now known as dasm|afk | 14:23 | |
* dtantsur assumes we do | 14:24 | |
openstackgerrit | Merged openstack/ironic-inspector: Switch to pbr https://review.openstack.org/193145 | 14:26 |
*** rameshg87 has joined #openstack-ironic | 14:31 | |
rameshg87 | morning folks | 14:32 |
thiagop | morning rameshg87 | 14:32 |
thiagop | rameshg87: I did some rebase on your patch, have you seen it? | 14:33 |
rameshg87 | thiagop: o/ | 14:33 |
* rameshg87 checks | 14:33 | |
rameshg87 | thiagop: I saw that. looks like jenkins has failed. | 14:33 |
rameshg87 | thiagop: simple rebase might not help. more changes might be needed. | 14:33 |
thiagop | Yeah, some tests are failing because of the mocks on PXEDeploy | 14:34 |
rameshg87 | thiagop: I see all pxe_ssh tests also failed | 14:34 |
thiagop | rameshg87: in fact, I think it needs a rework on tests to see how it'll work with the new interface | 14:34 |
rameshg87 | thiagop: AttributeError: 'PXEBoot' object has no attribute 'tear_down_deploy_ramdisk_boot' | 14:35 |
NobodyCam | morning rameshg87 | 14:35 |
rameshg87 | thiagop: yeah, the methods need to synced as per the spec | 14:35 |
rameshg87 | NobodyCam: o/ | 14:35 |
thiagop | rameshg87: This one I haven't seen... | 14:35 |
rameshg87 | thiagop: http://logs.openstack.org/13/166513/5/check/check-tempest-dsvm-ironic-pxe_ssh/d5bb66b/logs/screen-ir-cond.txt.gz | 14:35 |
thiagop | tks | 14:36 |
rameshg87 | thiagop: I will plan to put up a new patch today or tomorrow for pxe driver | 14:37 |
*** e0ne is now known as e0ne_ | 14:37 | |
*** e0ne_ is now known as e0ne | 14:37 | |
rameshg87 | thiagop: I see you are waiting for this ? is there some dependency on your work on this ? | 14:37 |
thiagop | rameshg87: Nice. I have some internal issues to take care of today, but I can help as soon as it ends. | 14:38 |
thiagop | Yes, I intend to implement the OneView driver compliant to the new interface | 14:38 |
rameshg87 | thiagop: oh okay | 14:38 |
jroll | dtantsur: I haven't read all of the scrollback, but I think when folks talk about developers and API versioning, they are talking about developers that are developing against the API, not ironic developers | 14:39 |
dtantsur | jroll, I'm both :) | 14:39 |
jroll | dtantsur: aha, true :) | 14:39 |
jroll | fair enough. | 14:39 |
jroll | also good morning :) | 14:41 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Override PXE kernel cmdline via Glance property https://review.openstack.org/178229 | 14:42 |
lucasagomes | jroll, good morning | 14:43 |
lucasagomes | ugt | 14:43 |
jroll | \o | 14:43 |
rameshg87 | lucasagomes: hi there | 14:45 |
rameshg87 | lucasagomes: In https://review.openstack.org/#/c/178229/14/ironic/drivers/modules/pxe.py L351 | 14:45 |
rameshg87 | lucasagomes: I meant someone could still give boot_option == 'local' and give a glance image that has custom kernel_cmdline | 14:46 |
lucasagomes | rameshg87, hi there | 14:47 |
* lucasagomes checks | 14:47 | |
lucasagomes | rameshg87, right but that glance metadata will be copied to the instance_info right? | 14:47 |
*** romcheg has quit IRC | 14:48 | |
rameshg87 | lucasagomes: but only after validate() right ? | 14:48 |
* rameshg87 checks | 14:48 | |
* lucasagomes checks too | 14:49 | |
*** mtanino has joined #openstack-ironic | 14:49 | |
rameshg87 | lucasagomes: yeah it's in prepare | 14:49 |
rameshg87 | lucasagomes: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L398 | 14:50 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Move client out of tree https://review.openstack.org/193150 | 14:50 |
lucasagomes | rameshg87, oh right... I will fix that | 14:50 |
rameshg87 | okay | 14:50 |
*** zz_jgrimm is now known as jgrimm | 14:51 | |
*** romcheg has joined #openstack-ironic | 14:51 | |
*** romainh has left #openstack-ironic | 14:54 | |
lucasagomes | rameshg87, _build_pxe_config_options() will be called after deploy() right? | 14:55 |
*** romainh has joined #openstack-ironic | 14:56 | |
lucasagomes | so the instance_info will be populated | 14:56 |
* lucasagomes looks | 14:56 | |
*** rameshg87 has quit IRC | 14:57 | |
lucasagomes | rameshg87, problem with validate() it's not only called when the node is being dpeloyed | 14:57 |
lucasagomes | it can be called any time, even on our API | 14:57 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Override PXE kernel cmdline via Glance property https://review.openstack.org/178229 | 15:00 |
lucasagomes | ^ just remove some unrelated changes from pxe.py | 15:00 |
NobodyCam | morning jroll | 15:01 |
jroll | heya NobodyCam | 15:01 |
NobodyCam | :) | 15:01 |
jroll | :D | 15:01 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs: Switch periodic tasks to the Futurist library https://review.openstack.org/191710 | 15:02 |
*** achanda has joined #openstack-ironic | 15:04 | |
*** e0ne has quit IRC | 15:04 | |
*** boris-42 has joined #openstack-ironic | 15:10 | |
*** subscope has quit IRC | 15:11 | |
*** achanda has quit IRC | 15:13 | |
*** rwsu has joined #openstack-ironic | 15:18 | |
*** subscope has joined #openstack-ironic | 15:26 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Finish switch to inspector and inspector-client https://review.openstack.org/193176 | 15:26 |
dtantsur | folks, I will appreciate fast review of ^^^ very much, otherwise it's blocking getting client out of inspector itself (aka https://review.openstack.org/193150) | 15:27 |
openstackgerrit | Merged openstack/ironic: Add Wake-On-Lan driver documentation https://review.openstack.org/192252 | 15:28 |
trown | dtantsur: I will give it a look shortly | 15:28 |
dtantsur | thnx! | 15:28 |
*** spandhe has joined #openstack-ironic | 15:30 | |
*** Marga_ has quit IRC | 15:31 | |
*** achanda has joined #openstack-ironic | 15:39 | |
*** mdbooth has quit IRC | 15:40 | |
*** achanda_ has joined #openstack-ironic | 15:41 | |
* dtantsur has new bass pickups to install, so he's heading home :) | 15:41 | |
dtantsur | see you tomorrow | 15:41 |
NobodyCam | ahhh | 15:42 |
NobodyCam | dtantsur: i was just going to ask if you had a quick second to look at https://review.openstack.org/#/c/191109 | 15:42 |
NobodyCam | I ask because you have already lookd at it :) | 15:43 |
openstackgerrit | Merged openstack/bifrost: Move explicit URLs in ironic-install to defaults https://review.openstack.org/192580 | 15:43 |
*** achanda has quit IRC | 15:44 | |
*** mdbooth has joined #openstack-ironic | 15:45 | |
dtantsur | NobodyCam, I'm not that good in bifrost anyway :) | 15:45 |
NobodyCam | :) hehehe | 15:45 |
NobodyCam | dtantsur: your just "good" | 15:46 |
NobodyCam | :-p | 15:46 |
dtantsur | lol true :D | 15:46 |
*** dtantsur is now known as dtantsur|afk | 15:46 | |
*** ifarkas has quit IRC | 15:47 | |
NobodyCam | have a goodnight dtantsur|afk :) | 15:47 |
*** achanda_ has quit IRC | 15:51 | |
*** achanda has joined #openstack-ironic | 15:52 | |
*** Marga_ has joined #openstack-ironic | 15:56 | |
*** kozhukalov_ has joined #openstack-ironic | 15:57 | |
*** victor_lowther has left #openstack-ironic | 15:59 | |
devananda | g'morning, all | 16:02 |
lucasagomes | devananda, morning | 16:02 |
NobodyCam | mornign devananda | 16:02 |
*** ukalifon has joined #openstack-ironic | 16:04 | |
openstackgerrit | Merged openstack/ironic: Validate capability in properties and instance_info https://review.openstack.org/174785 | 16:07 |
*** zz_natorious is now known as natorious | 16:08 | |
*** jjohnson2 has quit IRC | 16:13 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Change job logging such that successful jobs save logs https://review.openstack.org/193114 | 16:17 |
NobodyCam | brb | 16:17 |
openstackgerrit | Merged openstack/bifrost: Support for AMT https://review.openstack.org/191109 | 16:20 |
*** ukalifon has quit IRC | 16:20 | |
*** openstackgerrit has quit IRC | 16:22 | |
*** openstackgerrit has joined #openstack-ironic | 16:23 | |
*** ijw_ has joined #openstack-ironic | 16:25 | |
*** ukalifon1 has joined #openstack-ironic | 16:25 | |
*** ukalifon1 has quit IRC | 16:27 | |
*** romcheg has quit IRC | 16:28 | |
*** rbudden has quit IRC | 16:32 | |
*** rbudden_ has joined #openstack-ironic | 16:32 | |
*** rbudden_ has quit IRC | 16:33 | |
*** Marga_ has quit IRC | 16:34 | |
*** Marga_ has joined #openstack-ironic | 16:34 | |
*** rbudden has joined #openstack-ironic | 16:37 | |
*** yog_ has quit IRC | 16:39 | |
*** ukalifon has joined #openstack-ironic | 16:41 | |
*** teju has joined #openstack-ironic | 16:42 | |
*** romcheg has joined #openstack-ironic | 16:43 | |
*** Marga_ has quit IRC | 16:43 | |
*** Marga_ has joined #openstack-ironic | 16:43 | |
*** derekh has quit IRC | 16:48 | |
*** teju has quit IRC | 16:49 | |
jroll | devananda: welcome back? :) | 16:51 |
devananda | jroll: thanks :) | 16:51 |
devananda | it's nice to be back in seattle. I know I can eat the food here :) | 16:51 |
lucasagomes | heh | 16:54 |
lucasagomes | yeah that's important | 16:54 |
lucasagomes | anyhoo, folks I'm calling it a day | 16:54 |
lucasagomes | came to the office so gotta catch the train back home yet | 16:54 |
lucasagomes | have a good night everyone! | 16:54 |
jroll | later lucasagomes :) | 16:55 |
lucasagomes | see y'all | 16:55 |
*** lucasagomes has quit IRC | 16:55 | |
NobodyCam | have a good night lucas | 16:55 |
*** ijw_ has quit IRC | 17:00 | |
*** bethelwell has quit IRC | 17:01 | |
*** ndipanov has quit IRC | 17:01 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add pluggable credentials storage https://review.openstack.org/185074 | 17:04 |
*** ijw_ has joined #openstack-ironic | 17:04 | |
*** pradipta has quit IRC | 17:06 | |
openstackgerrit | Merged openstack/ironic-specs: Minor grammar cleanup of enroll-node-state https://review.openstack.org/191967 | 17:06 |
*** achanda has quit IRC | 17:08 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add credentials migration script. https://review.openstack.org/193218 | 17:08 |
*** mitchjameson has joined #openstack-ironic | 17:11 | |
*** achanda has joined #openstack-ironic | 17:11 | |
*** ijw_ has quit IRC | 17:13 | |
*** ijw_ has joined #openstack-ironic | 17:14 | |
*** krtaylor has quit IRC | 17:14 | |
*** krtaylor has joined #openstack-ironic | 17:15 | |
*** jjohnson2 has joined #openstack-ironic | 17:15 | |
*** natorious is now known as zz_natorious | 17:16 | |
*** amotoki_ has quit IRC | 17:18 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs: Add pluggable credentials storage https://review.openstack.org/186056 | 17:18 |
*** achanda has quit IRC | 17:19 | |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add credentials migration script https://review.openstack.org/193218 | 17:23 |
*** Marga_ has quit IRC | 17:23 | |
*** achanda has joined #openstack-ironic | 17:24 | |
*** dguerri is now known as dguerri` | 17:30 | |
*** rbudden has quit IRC | 17:30 | |
*** rbudden has joined #openstack-ironic | 17:30 | |
martini | TheJulia: just realized I can post another commit to update the README for the ironic-install role ;-) | 17:34 |
martini | TheJulia: and I meant to ask, it looks like latest_os_ironic_url and latest_os_ironic_node_url aren't actually used anymore, just by a quick grep. (I didn't do a run through to delete them). Were those made irrelevant by some refactor, or am I missing something silly? | 17:35 |
*** trown is now known as trown|lunch | 17:38 | |
TheJulia | martini: oh, please rip them out, they were made irrelevant and I keep forgetting | 17:38 |
*** zz_natorious is now known as natorious | 17:39 | |
martini | Cool, will do this evening | 17:39 |
TheJulia | awesome | 17:39 |
TheJulia | I have to step away, but I'll be back on this evening as well. | 17:39 |
*** athomas has quit IRC | 17:42 | |
*** absubram has joined #openstack-ironic | 17:46 | |
*** openstackgerrit has quit IRC | 17:50 | |
*** openstackgerrit has joined #openstack-ironic | 17:51 | |
*** marzif has joined #openstack-ironic | 17:52 | |
*** Marga_ has joined #openstack-ironic | 17:54 | |
*** Marga_ has quit IRC | 17:59 | |
*** Marga_ has joined #openstack-ironic | 18:07 | |
openstackgerrit | Shilla Saebi proposed openstack/ironic: changes and edit of the ironic install guide https://review.openstack.org/191900 | 18:10 |
*** achanda has quit IRC | 18:15 | |
*** zer0c00l has quit IRC | 18:18 | |
*** zer0c00l has joined #openstack-ironic | 18:18 | |
*** amotoki_ has joined #openstack-ironic | 18:18 | |
*** zer0c00l has quit IRC | 18:18 | |
*** zer0c00l has joined #openstack-ironic | 18:19 | |
openstackgerrit | Merged openstack/ironic: Enable flake8 checking of ironic/nova/* https://review.openstack.org/191183 | 18:19 |
*** pelix has quit IRC | 18:23 | |
*** amotoki_ has quit IRC | 18:23 | |
openstackgerrit | Richard H proposed openstack/ironic: Copy editing of ironic docs https://review.openstack.org/192889 | 18:28 |
*** _gridinv has joined #openstack-ironic | 18:28 | |
*** kozhukalov_ has quit IRC | 18:37 | |
*** marzif has quit IRC | 18:37 | |
*** marzif has joined #openstack-ironic | 18:40 | |
*** marzif has quit IRC | 18:47 | |
*** Sukhdev has joined #openstack-ironic | 18:49 | |
openstackgerrit | Jeremy Stanley proposed openstack/ironic: Switch from MySQL-python to PyMySQL https://review.openstack.org/184378 | 18:50 |
JayF | jroll: ^ talk about a patch we'll want to test well before going for it | 18:52 |
jroll | JayF: ohhhh ya. | 18:53 |
jroll | JayF: that patch is just docs/tox though :) | 18:53 |
*** spandhe has quit IRC | 18:54 | |
*** Marga_ has quit IRC | 18:55 | |
*** Marga_ has joined #openstack-ironic | 18:55 | |
*** ijw__ has joined #openstack-ironic | 18:57 | |
*** ijw_ has quit IRC | 19:00 | |
*** bethelwell has joined #openstack-ironic | 19:02 | |
*** bethelwell has quit IRC | 19:07 | |
*** rloo has quit IRC | 19:10 | |
*** rloo has joined #openstack-ironic | 19:11 | |
*** romcheg has quit IRC | 19:11 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: Add spec for user-settable Node affinity https://review.openstack.org/193263 | 19:13 |
devananda | JayF: ^ | 19:13 |
*** rloo_ has joined #openstack-ironic | 19:16 | |
*** ukalifon has quit IRC | 19:18 | |
*** rloo_ has quit IRC | 19:18 | |
*** rloo_ has joined #openstack-ironic | 19:18 | |
*** romcheg has joined #openstack-ironic | 19:18 | |
*** romainh has quit IRC | 19:19 | |
*** rloo has quit IRC | 19:20 | |
*** ukalifon has joined #openstack-ironic | 19:20 | |
*** _gridinv has quit IRC | 19:22 | |
*** uschreiber_ has joined #openstack-ironic | 19:23 | |
*** uschreiber_ has quit IRC | 19:25 | |
*** lauramoore has joined #openstack-ironic | 19:26 | |
*** ukalifon has quit IRC | 19:34 | |
*** Marga_ has quit IRC | 19:37 | |
*** dprince has quit IRC | 19:45 | |
*** r-daneel has joined #openstack-ironic | 19:47 | |
*** Marga_ has joined #openstack-ironic | 19:47 | |
*** Marga_ has quit IRC | 19:53 | |
*** trown|lunch is now known as trown | 19:53 | |
*** achanda has joined #openstack-ironic | 19:54 | |
openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Use host_device mode for qemu-img https://review.openstack.org/193284 | 19:58 |
openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Use host_device mode for qemu-img https://review.openstack.org/193284 | 20:00 |
*** jjohnson2 has quit IRC | 20:02 | |
*** natorious is now known as zz_natorious | 20:07 | |
*** ijw__ has quit IRC | 20:25 | |
*** achanda has quit IRC | 20:30 | |
*** thiagop has quit IRC | 20:32 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: Update nodes-tagging spec https://review.openstack.org/192935 | 20:36 |
*** marzif has joined #openstack-ironic | 20:41 | |
*** sambetts has quit IRC | 20:43 | |
*** Marga_ has joined #openstack-ironic | 20:44 | |
*** sambetts has joined #openstack-ironic | 20:46 | |
mrda | Morning | 20:49 |
*** Sukhdev has quit IRC | 20:49 | |
*** e0ne has joined #openstack-ironic | 20:59 | |
openstackgerrit | Joshua Harlow proposed openstack/ironic: Transition state machine to use automaton oslo lib. https://review.openstack.org/191985 | 20:59 |
*** jamielennox is now known as jamielennox|away | 21:01 | |
*** marzif has quit IRC | 21:03 | |
rloo_ | morning mrda :) | 21:06 |
devananda | morning! | 21:06 |
mrda | o/ | 21:07 |
*** jamielennox|away is now known as jamielennox | 21:09 | |
*** thiagop has joined #openstack-ironic | 21:16 | |
*** Marga_ has quit IRC | 21:23 | |
*** ijw_ has joined #openstack-ironic | 21:25 | |
*** trown is now known as trown|outttypeww | 21:26 | |
*** Marga_ has joined #openstack-ironic | 21:29 | |
*** Marga_ has quit IRC | 21:34 | |
*** Marga_ has joined #openstack-ironic | 21:34 | |
rloo_ | so ironic's official name & capitalization is 'Bare metal service'. That seems odd to me. https://wiki.openstack.org/wiki/Documentation/Conventions#Service_and_project_names | 21:37 |
JayF | Yeah; shouldn't it be Bear? | 21:37 |
rloo_ | which means we'd see something like 'The Bare metal service is really useful' vs 'The Bare Metal service is even better'? Yeah, maybe we should change to Bear metal service. | 21:38 |
JayF | I agree that the M should be capitalized as well | 21:38 |
*** Marga__ has joined #openstack-ironic | 21:39 | |
rloo_ | just mentioning cuz there are some patches that are proposing that. https://review.openstack.org/#/c/191900/ | 21:39 |
*** Marga_ has quit IRC | 21:42 | |
rloo_ | devananda: ^^ do you know why it is 'Bare metal service' vs 'Bare Metal service'? | 21:44 |
* devananda checks something | 21:45 | |
rloo_ | oh, its from http://git.openstack.org/cgit/openstack/governance/tree/reference/projects.yaml | 21:46 |
*** romcheg has quit IRC | 21:46 | |
devananda | yep | 21:46 |
devananda | that's what i just checked | 21:46 |
devananda | easy to change that | 21:46 |
*** romcheg has joined #openstack-ironic | 21:46 | |
rloo_ | sigh. | 21:46 |
devananda | however, the entry in keystone catalog is what's critical | 21:46 |
rloo_ | thx devananda. if we have to review those patches, might as well do it just once. | 21:47 |
*** romcheg has quit IRC | 21:47 | |
devananda | rloo_: so it's registered in keystone as "baremetal" | 21:48 |
rloo_ | (would have been nice for some writer person to point out that it might not have been ideal...) | 21:48 |
devananda | no spaces or caps | 21:48 |
rloo_ | devananda: so that 'string' is meant to be the same everywhere, eg in documentation and keystone catalog? | 21:48 |
devananda | literally, the only thing that actually matters is the entry in keystone catalog | 21:48 |
devananda | changing that breaks users | 21:48 |
devananda | changing a doc string somewhere else is just noise in the review queue :) | 21:49 |
rloo_ | devananda: gotcha. | 21:49 |
rloo_ | devananda: yeah, i want to minimize the noise in the review queue. | 21:49 |
devananda | rloo_: feel free to propose any changes to cApItAlIzation where you feel it's needed and I'll +[1,2] | 21:50 |
rloo_ | devananda: I think we need to change that govenance.../projects.yaml file. | 21:50 |
rloo_ | devananda: cuz ... umm, I think people are using that and may not deviate if you know what I mean | 21:50 |
devananda | exactly | 21:51 |
devananda | "Baremetal" || "Bare Metal" ? | 21:51 |
rloo_ | devananda: I'd actually prefer 'Baremetal' but I don't know if there is an official/correct version. | 21:51 |
morganfainberg | devananda: i have a fix mostly rolled up to eliminate the subclassing of keystonemiddleware for you. I *think* it'll work. hope to have it posted this week. | 21:52 |
morganfainberg | should be a relatively small-ish change | 21:52 |
devananda | morganfainberg: \o/ | 21:53 |
devananda | rloo_: google tells me that "bare metal" is much more prevalent than "baremetal" in our community's usage | 21:53 |
devananda | caps notwithstandting | 21:53 |
devananda | also - i need to catch a bus very soon | 21:53 |
devananda | bbl | 21:53 |
rloo_ | devananda: yeah, unfortunately. no worries, no hurry on it :) | 21:53 |
*** jgrimm is now known as zz_jgrimm | 21:55 | |
*** ijw_ has quit IRC | 21:55 | |
*** lauramoore has quit IRC | 21:56 | |
*** lauramoore has joined #openstack-ironic | 21:56 | |
*** lauramoore has quit IRC | 21:56 | |
*** Marga__ has quit IRC | 22:05 | |
openstackgerrit | Doug Hellmann proposed openstack/ironic-lib: Drop use of 'oslo' namespace package https://review.openstack.org/193338 | 22:07 |
*** thiagop has quit IRC | 22:11 | |
*** Marga_ has joined #openstack-ironic | 22:11 | |
*** thrash is now known as thrash|g0ne | 22:13 | |
*** ijw_ has joined #openstack-ironic | 22:28 | |
*** e0ne has quit IRC | 22:31 | |
*** radek_ has quit IRC | 22:32 | |
openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/192372 | 22:34 |
openstackgerrit | Merged openstack/ironic: Added documentation to Vagrantfile https://review.openstack.org/191912 | 22:41 |
*** davidlenwell is now known as david_lenwell | 22:44 | |
*** david_lenwell is now known as davidlenwell | 22:44 | |
*** absubram has quit IRC | 22:47 | |
*** absubram has joined #openstack-ironic | 22:48 | |
*** kevinbenton has quit IRC | 22:48 | |
*** mitchjameson has quit IRC | 22:49 | |
*** ijw__ has joined #openstack-ironic | 22:50 | |
*** TheJulia has quit IRC | 22:50 | |
*** kevinbenton has joined #openstack-ironic | 22:51 | |
*** boris-42 has quit IRC | 22:52 | |
*** ijw_ has quit IRC | 22:53 | |
* SpamapS gets bifrosty | 22:54 | |
*** openstackgerrit has quit IRC | 22:55 | |
*** vdrok has quit IRC | 22:55 | |
*** EmilienM has quit IRC | 22:55 | |
cinerama | devananda, rloo_ : the openstack glossary suggests "bare metal" http://docs.openstack.org/glossary/content/glossary.html | 22:58 |
*** TheJulia has joined #openstack-ironic | 22:58 | |
cinerama | which is harder for me to write without making a typo that references our lovable mascot :) | 22:59 |
*** ijw__ is now known as ijw_ | 23:00 | |
*** ijw_ has quit IRC | 23:00 | |
*** Marga_ has quit IRC | 23:01 | |
*** bethelwell has joined #openstack-ironic | 23:03 | |
SpamapS | ugh | 23:04 |
SpamapS | TheJulia: hey I'm playing with BiFrost | 23:04 |
SpamapS | and the iPXE is failing | 23:04 |
SpamapS | TheJulia: 10.10.16.131 - - [18/Jun/2015:16:03:07 -0700] "GET /boot.ipxe HTTP/1.0" 200 269 "-" "iPXE/1.0.0+" | 23:04 |
jroll | SpamapS: always blame the network :P | 23:04 |
SpamapS | TheJulia: its downloading, but something fails after that and then my ilo switches to booting from disk | 23:05 |
SpamapS | I suppose I should see if I can make the ilo keep trying network only | 23:05 |
*** openstackgerrit has joined #openstack-ironic | 23:06 | |
*** vdrok has joined #openstack-ironic | 23:06 | |
*** EmilienM has joined #openstack-ironic | 23:06 | |
SpamapS | jroll: inorite | 23:06 |
jroll | can you get a console and see where it's failing? | 23:06 |
SpamapS | jroll: I'm at the console, and no, currently the fail is wiped by "Attempting Boot From Hard Drive" | 23:06 |
SpamapS | jroll: I'll see if I can et the BIOS to not do that. | 23:07 |
jroll | grr | 23:07 |
jroll | sounds like maybe an image download thing | 23:07 |
jroll | I assume apache serves images, do you see that happening? | 23:07 |
SpamapS | nginx | 23:08 |
SpamapS | and no | 23:08 |
jroll | or whatever :P | 23:08 |
jroll | hrm, can you paste boot.ipxe? | 23:08 |
*** bethelwell has quit IRC | 23:08 | |
SpamapS | jroll: http://git.openstack.org/cgit/openstack/bifrost/tree/playbooks/roles/ironic-install/files/boot.ipxe | 23:10 |
*** marzif has joined #openstack-ironic | 23:10 | |
SpamapS | funny enough, these mellanox cards already have ipxe loaded on them | 23:11 |
SpamapS | or something | 23:11 |
jroll | hrm | 23:11 |
SpamapS | I don't really understand how it all works | 23:11 |
jroll | maybe a lame version of ipxe that doesn't support ${mac:hexraw} ? | 23:11 |
jroll | I've never heard good things about mellanox | 23:11 |
*** mitchjameson has joined #openstack-ironic | 23:11 | |
SpamapS | actually | 23:12 |
SpamapS | theres nothing in my pxelinux.cfg dir | 23:12 |
jroll | oh, that'll do it. | 23:12 |
martini | quit | 23:13 |
*** martini has quit IRC | 23:13 | |
SpamapS | unrecognized option --autofree | 23:13 |
SpamapS | thats the error | 23:13 |
jroll | :/ | 23:14 |
jroll | yeah, may be an ipxe version thing | 23:14 |
SpamapS | but I thought we were chain-loading a new one on? I'm confused now. :_P | 23:14 |
SpamapS | it says iPXE 1.0.0 | 23:14 |
SpamapS | anyway I suspect something has borked writing my pxelinux.cfg too | 23:14 |
jroll | full disclosure: I still haven't used bifrost :/ | 23:15 |
SpamapS | thats ok | 23:15 |
SpamapS | you've used ipxe | 23:15 |
jroll | heh | 23:15 |
SpamapS | looks like --autofree is not part of 1.0.0 | 23:15 |
jroll | yes, but very differently | 23:15 |
jroll | right | 23:15 |
*** zhenguo has joined #openstack-ironic | 23:16 | |
jroll | so I don't think this is chainloading to a different ipxe but I may be wrong | 23:16 |
SpamapS | README on bifrost claims it is | 23:16 |
jroll | huh. | 23:17 |
*** Marga_ has joined #openstack-ironic | 23:18 | |
SpamapS | http://git.openstack.org/cgit/openstack/bifrost/tree/README.rst#n55 | 23:18 |
jroll | well | 23:18 |
jroll | so that's for normal pxe booting | 23:18 |
SpamapS | dhcp-match=set:ipxe,175 # iPXE sends a 175 option. | 23:18 |
SpamapS | dhcp-boot=tag:!ipxe,undionly.kpxe | 23:18 |
SpamapS | dhcp-boot=http://10.10.16.130:8080/boot.ipxe | 23:18 |
jroll | but the response depends on the dhcp options in the request | 23:18 |
jroll | so to the dhcp server it looks like it's already chain-loaded to ipxe | 23:18 |
*** alexpilotti has quit IRC | 23:19 | |
SpamapS | oh thats annoying | 23:19 |
*** Marga_ has quit IRC | 23:19 | |
SpamapS | TheJulia: have you run into this yet? | 23:20 |
jroll | I suspect nobody has used bifrost yet with ipxe on the NIC ROM | 23:20 |
SpamapS | entirely possible | 23:20 |
jroll | it's something I've always wanted to do, though, just to skip a step | 23:21 |
SpamapS | well lets see what happens if I remove the tag from the undionly.kpxe bit | 23:22 |
*** Marga__ has joined #openstack-ironic | 23:23 | |
* SpamapS wishes his hardware booted faster than 8 minutes | 23:23 | |
*** Marga__ has quit IRC | 23:24 | |
*** Marga_ has joined #openstack-ironic | 23:25 | |
SpamapS | it actually says it is 'iPXE 1.0.0+' | 23:27 |
SpamapS | the + is like, what, +mellanoxspecialsauce ? | 23:27 |
jroll | lol | 23:27 |
jroll | probably | 23:27 |
*** boris-42 has joined #openstack-ironic | 23:28 | |
SpamapS | same thing.. invalid argument | 23:28 |
*** Marga_ has quit IRC | 23:31 | |
*** Marga_ has joined #openstack-ironic | 23:32 | |
openstackgerrit | Michael Davies proposed openstack/python-ironicclient: Cache negotiated api microversion for server https://review.openstack.org/173674 | 23:35 |
TheJulia | SpamapS: hmmm | 23:36 |
TheJulia | SpamapS: Interesting...... can you packet capture everything for that mac address? Kind of agree with jroll, I tested on a moonshot without any issues, but I don't remember if it was chain loading off-hand. | 23:38 |
TheJulia | SpamapS: oh, this is the mellonox nics? | 23:39 |
TheJulia | SpamapS: want to possibly try ilo virtual media? :) | 23:40 |
SpamapS | TheJulia: I will try anything that might work. | 23:42 |
TheJulia | Looks like nobody in ironic-core has not +2+a'ed the ilo rev :) | 23:43 |
rloo_ | cinerama: thx. Yeah, I think we're going to go with 'bare metal' :) | 23:43 |
TheJulia | https://review.openstack.org/#/c/191218/ | 23:44 |
jroll | TheJulia: boom | 23:44 |
TheJulia | SpamapS: From there, it would take using our bleeding edge dynamic inventory code to deploy. I would still love a pcap of the exchange | 23:44 |
TheJulia | jroll: awesomeness | 23:44 |
jroll | I figure I should at least do one review today :P | 23:45 |
*** rloo_ has quit IRC | 23:45 | |
*** yuanying has joined #openstack-ironic | 23:45 | |
SpamapS | TheJulia: dump in 8 minutes... | 23:45 |
TheJulia | Merci | 23:45 |
TheJulia | SpamapS: I guess your using a csv file? | 23:47 |
*** chlong has joined #openstack-ironic | 23:47 | |
jroll | devananda: do you have time to look at https://review.openstack.org/#/c/185171/5/specs/liberty/feature-based-releases.rst | 23:48 |
jroll | tomorrow is a fine answer | 23:48 |
jroll | I had at least one question specifically for you there | 23:48 |
*** Haomeng has joined #openstack-ironic | 23:49 | |
SpamapS | TheJulia: I did use a csv file yes, the node is enrolled (there's just one) and claims to know that power is on/off | 23:49 |
*** absubram has quit IRC | 23:49 | |
*** Haomeng|2 has quit IRC | 23:52 | |
openstackgerrit | Merged openstack/bifrost: Enable agent_ilo driver https://review.openstack.org/191218 | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!