| *** meghal1 has joined #openstack-ironic | 00:02 | |
| *** meghal has quit IRC | 00:02 | |
| *** davideagnello has quit IRC | 00:04 | |
| *** krotscheck1 has joined #openstack-ironic | 00:06 | |
| *** krotscheck1 has quit IRC | 00:08 | |
| *** dims has quit IRC | 00:10 | |
| *** praneshp has quit IRC | 00:12 | |
| *** meghal1 has quit IRC | 00:15 | |
| *** mtanino_ has joined #openstack-ironic | 00:22 | |
| *** mtanino has quit IRC | 00:24 | |
| *** raildo-afk is now known as raildo | 00:25 | |
| *** raddaoui has quit IRC | 00:30 | |
| *** meghal has joined #openstack-ironic | 00:36 | |
| *** meghal has quit IRC | 00:41 | |
| *** absubram has quit IRC | 00:41 | |
| *** devananda has quit IRC | 00:47 | |
| *** meghal has joined #openstack-ironic | 00:50 | |
| openstackgerrit | Yuiko Takada proposed openstack/ironic-python-agent: Add 'vendor' to inspect data https://review.openstack.org/274545 | 00:55 |
|---|---|---|
| *** jamielennox|away is now known as jamielennox | 00:55 | |
| *** rbudden has quit IRC | 01:02 | |
| *** hoangcx has joined #openstack-ironic | 01:03 | |
| *** krotscheck1 has joined #openstack-ironic | 01:05 | |
| *** piet has quit IRC | 01:12 | |
| *** krotscheck1 has quit IRC | 01:13 | |
| *** piet has joined #openstack-ironic | 01:16 | |
| *** jaybeale has joined #openstack-ironic | 01:18 | |
| *** krotscheck1 has joined #openstack-ironic | 01:19 | |
| *** meghal has left #openstack-ironic | 01:20 | |
| *** jaybeale has quit IRC | 01:23 | |
| *** jamielennox is now known as jamielennox|away | 01:23 | |
| *** mtanino_ has quit IRC | 01:27 | |
| *** shu-mutou has joined #openstack-ironic | 01:29 | |
| *** penick has quit IRC | 01:30 | |
| *** shuquan has joined #openstack-ironic | 01:32 | |
| *** garthb has quit IRC | 01:34 | |
| *** krotscheck1 has quit IRC | 01:40 | |
| *** Sukhdev has quit IRC | 01:41 | |
| *** izaakk has quit IRC | 01:53 | |
| *** raildo is now known as raildo-afk | 01:59 | |
| *** krotscheck1 has joined #openstack-ironic | 01:59 | |
| *** shuquan has quit IRC | 02:00 | |
| *** krotscheck1 has quit IRC | 02:01 | |
| *** Sukhdev has joined #openstack-ironic | 02:04 | |
| *** Sukhdev has quit IRC | 02:05 | |
| *** piet has quit IRC | 02:05 | |
| *** alexpilotti has joined #openstack-ironic | 02:06 | |
| *** lazy_prince has joined #openstack-ironic | 02:07 | |
| *** krotscheck1 has joined #openstack-ironic | 02:07 | |
| *** alexpilotti has quit IRC | 02:10 | |
| *** dims has joined #openstack-ironic | 02:11 | |
| *** Marga_ has quit IRC | 02:20 | |
| *** Marga_ has joined #openstack-ironic | 02:22 | |
| *** jaybeale has joined #openstack-ironic | 02:23 | |
| *** krotscheck1 has quit IRC | 02:26 | |
| *** tiendc has joined #openstack-ironic | 02:26 | |
| *** jaybeale has quit IRC | 02:26 | |
| *** coolsvap|away is now known as coolsvap | 02:26 | |
| *** Marga_ has quit IRC | 02:27 | |
| *** krotscheck1 has joined #openstack-ironic | 02:28 | |
| *** krotscheck1 has quit IRC | 02:31 | |
| *** krotscheck1 has joined #openstack-ironic | 02:33 | |
| *** alexpilotti has joined #openstack-ironic | 02:42 | |
| *** jamielennox|away is now known as jamielennox | 02:44 | |
| *** alexpilotti has quit IRC | 02:46 | |
| *** coolsvap is now known as coolsvap|away | 02:59 | |
| *** ijw has quit IRC | 02:59 | |
| *** lazy_prince has quit IRC | 03:04 | |
| *** krotscheck1 has quit IRC | 03:06 | |
| *** lazy_prince has joined #openstack-ironic | 03:07 | |
| *** piet has joined #openstack-ironic | 03:09 | |
| *** krotscheck1 has joined #openstack-ironic | 03:10 | |
| *** amotoki has quit IRC | 03:10 | |
| *** jaybeale has joined #openstack-ironic | 03:10 | |
| *** lazy_prince has quit IRC | 03:12 | |
| *** jaybeale has quit IRC | 03:15 | |
| *** links has joined #openstack-ironic | 03:15 | |
| *** alexpilotti has joined #openstack-ironic | 03:17 | |
| *** krotscheck1 has quit IRC | 03:18 | |
| *** shuquan has joined #openstack-ironic | 03:19 | |
| *** shuquan has quit IRC | 03:20 | |
| *** krotscheck1 has joined #openstack-ironic | 03:21 | |
| *** yuanying_ has joined #openstack-ironic | 03:21 | |
| *** alexpilotti has quit IRC | 03:22 | |
| *** spandhe has quit IRC | 03:22 | |
| *** yuanying has quit IRC | 03:24 | |
| *** spandhe has joined #openstack-ironic | 03:25 | |
| *** krotscheck1 has quit IRC | 03:26 | |
| *** krotscheck1 has joined #openstack-ironic | 03:30 | |
| *** krotscheck1 has quit IRC | 03:32 | |
| *** krotscheck1 has joined #openstack-ironic | 03:33 | |
| *** shuquan has joined #openstack-ironic | 03:35 | |
| *** amotoki has joined #openstack-ironic | 03:35 | |
| *** spandhe has quit IRC | 03:37 | |
| *** jaybeale has joined #openstack-ironic | 03:39 | |
| *** harshs has quit IRC | 03:39 | |
| *** yuanying has joined #openstack-ironic | 03:41 | |
| *** Marga_ has joined #openstack-ironic | 03:42 | |
| *** Marga_ has quit IRC | 03:42 | |
| *** krotscheck1 has quit IRC | 03:43 | |
| *** achanda has joined #openstack-ironic | 03:43 | |
| *** jrist has joined #openstack-ironic | 03:43 | |
| *** Marga_ has joined #openstack-ironic | 03:43 | |
| *** yuanying_ has quit IRC | 03:44 | |
| *** rloo has quit IRC | 03:45 | |
| *** Marga_ has quit IRC | 03:45 | |
| *** Marga_ has joined #openstack-ironic | 03:46 | |
| *** Nisha_away has joined #openstack-ironic | 03:50 | |
| *** jamielennox has left #openstack-ironic | 03:50 | |
| *** shuquan has quit IRC | 03:52 | |
| *** dims has quit IRC | 03:53 | |
| *** jaybeale has quit IRC | 03:55 | |
| *** amotoki has quit IRC | 04:01 | |
| *** achanda has quit IRC | 04:04 | |
| *** yuanying has quit IRC | 04:05 | |
| *** yuanying has joined #openstack-ironic | 04:06 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection https://review.openstack.org/201904 | 04:06 |
| *** coolsvap|away is now known as coolsvap | 04:07 | |
| *** yuanying_ has joined #openstack-ironic | 04:07 | |
| *** yuanying has quit IRC | 04:07 | |
| *** achanda has joined #openstack-ironic | 04:11 | |
| *** vishwanathj has quit IRC | 04:16 | |
| *** Marga_ has quit IRC | 04:25 | |
| *** Marga_ has joined #openstack-ironic | 04:25 | |
| *** shuquan has joined #openstack-ironic | 04:33 | |
| *** spandhe has joined #openstack-ironic | 04:34 | |
| *** spandhe_ has joined #openstack-ironic | 04:37 | |
| *** spandhe has quit IRC | 04:39 | |
| *** spandhe_ is now known as spandhe | 04:39 | |
| *** tiendc has quit IRC | 04:47 | |
| *** shuquan has quit IRC | 04:58 | |
| *** harshs has joined #openstack-ironic | 05:04 | |
| *** amotoki has joined #openstack-ironic | 05:05 | |
| *** harshs_ has joined #openstack-ironic | 05:10 | |
| *** deray has joined #openstack-ironic | 05:10 | |
| *** harshs has quit IRC | 05:11 | |
| *** harshs_ is now known as harshs | 05:11 | |
| *** Sukhdev has joined #openstack-ironic | 05:17 | |
| *** vinm213 has joined #openstack-ironic | 05:19 | |
| *** alexpilotti has joined #openstack-ironic | 05:24 | |
| *** lazy_prince has joined #openstack-ironic | 05:26 | |
| *** alexpilotti has quit IRC | 05:28 | |
| *** jaybeale has joined #openstack-ironic | 05:31 | |
| *** jaybeale has quit IRC | 05:32 | |
| *** harshs has quit IRC | 05:43 | |
| *** piet has quit IRC | 05:44 | |
| *** piet has joined #openstack-ironic | 05:45 | |
| *** harshs has joined #openstack-ironic | 05:45 | |
| *** kalpase has joined #openstack-ironic | 05:48 | |
| *** hoangcx has quit IRC | 05:52 | |
| *** Marga_ has quit IRC | 06:00 | |
| *** piet has quit IRC | 06:11 | |
| *** harshs has quit IRC | 06:13 | |
| *** ChubYann has quit IRC | 06:13 | |
| *** rcernin has joined #openstack-ironic | 06:22 | |
| *** bkumar has joined #openstack-ironic | 06:23 | |
| *** boris-42 has quit IRC | 06:23 | |
| *** Marga_ has joined #openstack-ironic | 06:30 | |
| openstackgerrit | Kan proposed openstack/ironic: Enable removing name when updating node https://review.openstack.org/267327 | 06:38 |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic-python-agent: add new portal_port argument into iscsi.start_iscsi_target api https://review.openstack.org/272420 | 06:46 |
| *** rcernin has quit IRC | 06:46 | |
| *** itamarl has joined #openstack-ironic | 06:48 | |
| *** amotoki_ has joined #openstack-ironic | 06:51 | |
| *** amotoki has quit IRC | 07:03 | |
| *** keedya has joined #openstack-ironic | 07:04 | |
| *** ijw has joined #openstack-ironic | 07:08 | |
| *** hoangcx has joined #openstack-ironic | 07:10 | |
| *** spandhe has quit IRC | 07:12 | |
| *** spandhe has joined #openstack-ironic | 07:13 | |
| *** ukalifon has joined #openstack-ironic | 07:13 | |
| *** coolsvap is now known as coolsvap|away | 07:16 | |
| *** spandhe has quit IRC | 07:22 | |
| *** spandhe has joined #openstack-ironic | 07:23 | |
| *** mkovacik has quit IRC | 07:23 | |
| *** keedya has quit IRC | 07:26 | |
| *** ianw has quit IRC | 07:29 | |
| *** ianw has joined #openstack-ironic | 07:30 | |
| *** spandhe_ has joined #openstack-ironic | 07:34 | |
| *** spandhe has quit IRC | 07:36 | |
| *** spandhe_ is now known as spandhe | 07:36 | |
| *** Sukhdev has quit IRC | 07:38 | |
| openstackgerrit | Vaishnavi proposed openstack/proliantutils: Add supported boot modes to server capabilities. https://review.openstack.org/275053 | 07:39 |
| *** achanda has quit IRC | 07:40 | |
| *** lekha is now known as lekha|away | 07:40 | |
| openstackgerrit | Vaishnavi proposed openstack/proliantutils: Add supported boot modes to server capabilities. https://review.openstack.org/275053 | 07:41 |
| *** ijw has quit IRC | 07:42 | |
| *** achanda has joined #openstack-ironic | 07:43 | |
| *** spandhe has quit IRC | 07:44 | |
| *** spandhe has joined #openstack-ironic | 07:45 | |
| *** tiendc has joined #openstack-ironic | 07:48 | |
| *** achanda has quit IRC | 07:55 | |
| *** amotoki_ is now known as amotoki | 08:03 | |
| *** spandhe_ has joined #openstack-ironic | 08:08 | |
| *** spandhe has quit IRC | 08:10 | |
| *** spandhe_ is now known as spandhe | 08:10 | |
| *** jistr has joined #openstack-ironic | 08:11 | |
| *** boris-42 has joined #openstack-ironic | 08:17 | |
| *** spandhe has quit IRC | 08:23 | |
| *** Marga_ has quit IRC | 08:24 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection https://review.openstack.org/201904 | 08:27 |
| *** athomas has joined #openstack-ironic | 08:31 | |
| *** e0ne has joined #openstack-ironic | 08:38 | |
| openstackgerrit | Nisha Agarwal proposed openstack/python-ironicclient: Add CLI support for RAID configuration https://review.openstack.org/226234 | 08:44 |
| *** moshele has joined #openstack-ironic | 08:46 | |
| *** kalpase has quit IRC | 08:48 | |
| *** kalpase has joined #openstack-ironic | 08:48 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add documentation for RAID https://review.openstack.org/226330 | 08:57 |
| *** ndipanov has joined #openstack-ironic | 08:58 | |
| *** ifarkas has joined #openstack-ironic | 08:58 | |
| *** ndipanov has quit IRC | 08:59 | |
| *** mbound has joined #openstack-ironic | 09:02 | |
| *** dprince has joined #openstack-ironic | 09:03 | |
| *** deray has quit IRC | 09:05 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Fix delete vif_port_id for sync cleaning https://review.openstack.org/275080 | 09:14 |
| lucasagomes | morning all | 09:19 |
| *** achanda has joined #openstack-ironic | 09:25 | |
| *** jcoufal has joined #openstack-ironic | 09:28 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Restart consoles on conductor startup https://review.openstack.org/257252 | 09:28 |
| *** deray has joined #openstack-ironic | 09:29 | |
| *** mgoddard has joined #openstack-ironic | 09:30 | |
| *** phil_231 has joined #openstack-ironic | 09:30 | |
| *** achanda has quit IRC | 09:30 | |
| *** krotscheck1 has joined #openstack-ironic | 09:37 | |
| vdrok | good morning ironic and lucasagomes | 09:40 |
| *** amotoki has quit IRC | 09:40 | |
| openstackgerrit | Merged openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/274919 | 09:43 |
| *** Nisha_away has quit IRC | 09:48 | |
| *** mbound has quit IRC | 09:52 | |
| *** mbound has joined #openstack-ironic | 09:52 | |
| *** mbound_ has joined #openstack-ironic | 09:55 | |
| *** mbound has quit IRC | 09:55 | |
| *** amotoki has joined #openstack-ironic | 09:55 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Switch to accepting keystone session objects in client constructor https://review.openstack.org/271255 | 10:05 |
| *** amotoki has quit IRC | 10:10 | |
| *** dprince has quit IRC | 10:12 | |
| mrda | hey lucasagomes | 10:13 |
| lucasagomes | mrda, hey there! Good evening :-) | 10:13 |
| mrda | lucasagomes: Came back on just to ping you :) | 10:13 |
| *** deray has quit IRC | 10:14 | |
| *** yuikotakada has quit IRC | 10:14 | |
| mrda | lucasagomes: so do you know much about selecting which disk to install on when using a RAID array and IPA? | 10:14 |
| mrda | lucasagomes: I've tried using the wwn/serial hint, but so far to no avail. Someone suggested you might be an expert with such things :) | 10:15 |
| lucasagomes | mrda, oh hah right... yeah, this is complicated, for similar reasons I added a patch supporting "name" for device hints | 10:15 |
| lucasagomes | mrda, for raid you can set a volume name | 10:15 |
| * lucasagomes looks at the patch | 10:16 | |
| lucasagomes | mrda, https://review.openstack.org/#/c/269639/ | 10:16 |
| lucasagomes | not merged yet :-( | 10:16 |
| *** openstackgerrit has quit IRC | 10:17 | |
| *** pas-ha has joined #openstack-ironic | 10:17 | |
| mrda | lucasagomes: so this is pre-deploy, in the setup of a hardware raid, to specify a name that ipa can later find and use? | 10:18 |
| *** openstackgerrit has joined #openstack-ironic | 10:18 | |
| lucasagomes | mrda, yeah | 10:18 |
| lucasagomes | or if you are using ironic to create the RAID it will support you adding both as part of the request | 10:18 |
| lucasagomes | you can say that device will be the root device and also specify the volume name for it | 10:19 |
| mrda | hmmm, so you'd do this in the BMC? (or in my case the DRAC of my Dell R720) | 10:19 |
| openstackgerrit | Merged openstack/ironic-inspector: Set timeout in gate to 10 minutes https://review.openstack.org/273027 | 10:21 |
| mrda | lucasagomes: ^^^ | 10:21 |
| lucasagomes | mrda, AFAIK (pinging ifarkas here that worked on RAID for drac) yes, you can do RAID out of band with drac and you can specify the "virtual disk name" | 10:21 |
| mrda | cool, that's a big help as I've been a little stuch here :) | 10:22 |
| mrda | *stuck | 10:22 |
| ifarkas | lucasagomes, mrda, the DRAC RAID patches were never merged because they are using vendor-passthru instead of the RAID interface | 10:23 |
| ifarkas | I am planning to update the patch once manual cleaning landed | 10:23 |
| lucasagomes | ifarkas, right, but it's possible to set a "virtual disk name" as part of the request to name the RAID device it will create correct? | 10:23 |
| lucasagomes | (I mean, it's something supported by DRAC) | 10:24 |
| ifarkas | lucasagomes, I think so, lemme check | 10:24 |
| lucasagomes | thanks | 10:24 |
| *** amotoki has joined #openstack-ironic | 10:24 | |
| * mrda would only have to patch ipa with 269639 himself, and build a new disk image, and then it all should work \o/ | 10:25 | |
| lucasagomes | mrda, https://review.openstack.org/#/c/176319/10/ironic/drivers/modules/drac/raid.py (L99) | 10:25 |
| lucasagomes | ifarkas, ^ | 10:25 |
| mrda | cool | 10:25 |
| ifarkas | lucasagomes, mrda, yeah, I also found it in the spec for the interface: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ironic-generic-raid-interface.html | 10:26 |
| mrda | huh | 10:26 |
| mrda | I hadn't read that spec | 10:27 |
| lucasagomes | mrda, the support root device hints for name still needs to be merged :-( I will update the patch in IPA | 10:27 |
| * mrda wondered how people were handling RAID without this kind of feature. Must be a few downstream patches around the place. | 10:27 | |
| lucasagomes | ifarkas, thanks a lot | 10:27 |
| lucasagomes | mrda, maybe some RAID devices does have a unique serial/wwn and drac do not | 10:28 |
| lucasagomes | not sure | 10:28 |
| lucasagomes | but yeah, when root device was introduced it didn't have RAID in mind :-/ | 10:28 |
| ifarkas | mrda, I used to separate the creation of root raid volume vs the rest and saving the root device id with inspector in-between | 10:29 |
| mrda | So even the serial isn't working, I presume because it's the serial of a physical drive, when its a property of a virtual drive you want | 10:29 |
| mrda | but I don't know much about this stuff, so I shouldn't speculate | 10:29 |
| *** deray has joined #openstack-ironic | 10:29 | |
| *** yuikotakada has joined #openstack-ironic | 10:29 | |
| mrda | thanks ifarkas and lucasagomes | 10:30 |
| lucasagomes | yeah, and I think it may be diff from diff vendors too... the name tho should work more generically | 10:30 |
| lucasagomes | mrda, will try to ping people to review it after updating the patches | 10:30 |
| mrda | cool, thanks | 10:30 |
| *** amotoki_ has joined #openstack-ironic | 10:30 | |
| *** amotoki has quit IRC | 10:31 | |
| lucasagomes | ty! | 10:32 |
| *** hoangcx has quit IRC | 10:32 | |
| * mrda wanders back into his evening... | 10:32 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Extend root device hints to support device name https://review.openstack.org/269629 | 10:32 |
| lucasagomes | mrda, have a g'night! | 10:33 |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic: Updated documentation to recommend ipmitool version https://review.openstack.org/274547 | 10:36 |
| *** jcoufal has quit IRC | 10:39 | |
| *** amotoki_ has quit IRC | 10:42 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Enable doc8 style checker for *.rst files https://review.openstack.org/272988 | 10:45 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix doc8's "duplicated target names" (D000) error https://review.openstack.org/273044 | 10:45 |
| *** amotoki has joined #openstack-ironic | 10:45 | |
| lucasagomes | rebased after a merge conflict ^ | 10:46 |
| *** yuikotak_ has joined #openstack-ironic | 10:49 | |
| *** yuikotakada has quit IRC | 10:49 | |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/274918 | 10:49 |
| *** electrofelix has joined #openstack-ironic | 10:53 | |
| *** dtantsur|afk is now known as dtantsur | 10:54 | |
| dtantsur | Morning Ironic | 10:54 |
| openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support https://review.openstack.org/264257 | 10:55 |
| vdrok | morning dtantsur | 10:59 |
| pas-ha | morning all | 11:00 |
| openstackgerrit | Merged openstack/ironic-inspector: Use new introspection data save command in exercise.sh https://review.openstack.org/272507 | 11:01 |
| *** dims_ has joined #openstack-ironic | 11:02 | |
| aarefiev | good morning all! | 11:04 |
| *** mgould has joined #openstack-ironic | 11:08 | |
| dtantsur | morning vdrok, pas-ha, aarefiev | 11:08 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions https://review.openstack.org/236982 | 11:10 |
| mgould | morning everyone | 11:10 |
| *** sambetts has joined #openstack-ironic | 11:10 | |
| sambetts | Morning all o/ | 11:10 |
| lucasagomes | morning pas-ha dtantsur mgould aarefiev sambetts | 11:10 |
| dtantsur | morning sambetts, lucasagomes, mgould | 11:10 |
| sambetts | o/ dtantsur lucasagomes | 11:11 |
| dtantsur | gate is back, keep hacking \o/ | 11:11 |
| sambetts | \o/ whoop! Need to update the whiteboard then | 11:11 |
| aarefiev | morning dtantsur sambetts mgould lucasagomes :) | 11:11 |
| mgould | morning dtantsur lucasagomes sambetts aarefiev! | 11:12 |
| mgould | dtantsur, woohoo! | 11:12 |
| *** alexpilotti has joined #openstack-ironic | 11:13 | |
| *** deray has quit IRC | 11:14 | |
| *** alexpilotti has quit IRC | 11:15 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix agent driver proxy nits https://review.openstack.org/274599 | 11:15 |
| vdrok | morning mgould sambetts | 11:16 |
| sambetts | o/ vdrok | 11:17 |
| mgould | vdrok, morning! | 11:17 |
| *** alexpilotti has joined #openstack-ironic | 11:18 | |
| *** jistr has quit IRC | 11:22 | |
| *** achanda has joined #openstack-ironic | 11:29 | |
| *** amotoki has quit IRC | 11:30 | |
| *** _degorenko|afk is now known as degorenko | 11:30 | |
| openstackgerrit | Merged openstack/ironic: Migrate Tempest tests into Ironic tree https://review.openstack.org/253982 | 11:33 |
| *** achanda has quit IRC | 11:34 | |
| openstackgerrit | Merged openstack/ironic: Remove hard-coded DEPLOYWAIT timeout from Baremetal Scenario https://review.openstack.org/269249 | 11:40 |
| *** NobodyCa1 has joined #openstack-ironic | 11:48 | |
| *** NobodyCa1 has quit IRC | 11:53 | |
| *** lazy_prince has quit IRC | 12:04 | |
| *** tiendc has quit IRC | 12:08 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Enable doc8 style checker for *.rst files https://review.openstack.org/272988 | 12:08 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix doc8's "duplicated target names" (D000) error https://review.openstack.org/273044 | 12:08 |
| openstackgerrit | Merged openstack/ironic: Use node uuid in some exception log https://review.openstack.org/272945 | 12:10 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Enable Keystone v3 endpoints for Inspector https://review.openstack.org/274720 | 12:11 |
| *** raildo-afk is now known as raildo | 12:12 | |
| sinval | morning folks | 12:14 |
| sambetts | o/ sinval | 12:14 |
| *** smoriya_ has quit IRC | 12:15 | |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic: Updated with more details on boot mode capability https://review.openstack.org/274618 | 12:20 |
| *** amotoki has joined #openstack-ironic | 12:20 | |
| *** thiagop_ has joined #openstack-ironic | 12:25 | |
| *** bnemec has joined #openstack-ironic | 12:26 | |
| *** thiagop has quit IRC | 12:26 | |
| lucasagomes | dtantsur, re doc8 identing was my first attempt https://review.openstack.org/#/c/272988/1/doc/source/drivers/amt.rst | 12:29 |
| lucasagomes | rloo had some concerns about it | 12:29 |
| *** thiagop_ is now known as thiagop | 12:30 | |
| thiagop | morning Ironicers | 12:30 |
| dtantsur | lucasagomes, I don't get rloo's concerns, this is the correct way to deal with code blocks.. | 12:31 |
| lucasagomes | dtantsur, right, so I just left w/o a box around it to be the same with what we currently have | 12:32 |
| dtantsur | lucasagomes, not sure why we should keep the wrong syntax there... it's clear to me that initial intent was to have this box | 12:33 |
| lucasagomes | yeah, I will add it back... hoping we don't keep going back and forward with it (it's a nit) | 12:34 |
| *** links has quit IRC | 12:36 | |
| *** agordeev has quit IRC | 12:36 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Enable doc8 style checker for *.rst files https://review.openstack.org/272988 | 12:39 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix doc8's "duplicated target names" (D000) error https://review.openstack.org/273044 | 12:39 |
| *** links has joined #openstack-ironic | 12:39 | |
| *** agordeev has joined #openstack-ironic | 12:39 | |
| * lucasagomes goes grab some lunch | 12:39 | |
| *** lucasagomes is now known as lucas-hungry | 12:39 | |
| sambetts | dtantsur: https://review.openstack.org/#/c/274720/ should I migrate to use sessions in this patch as palov is suggesting or shall we cover that in another patch? This patch was really to combat the potential issues surrounding the devstack change to v3 as the default, WDYT? | 12:39 |
| sambetts | dtantsur: I don't even know if our gate was broken by that change or not :/ | 12:40 |
| dtantsur | sambetts, I think we can do these things separately | 12:40 |
| sambetts | ok, I've responded as such | 12:42 |
| *** boris-42 has quit IRC | 12:43 | |
| *** deva_ has joined #openstack-ironic | 12:49 | |
| openstackgerrit | Vaishnavi proposed openstack/proliantutils: Add supported boot modes to server capabilities https://review.openstack.org/275053 | 12:50 |
| *** baoli has joined #openstack-ironic | 12:53 | |
| *** mkovacik has joined #openstack-ironic | 12:55 | |
| *** trown|outttypeww is now known as trown | 12:57 | |
| *** deva_ has quit IRC | 12:58 | |
| *** NobodyCa1 has joined #openstack-ironic | 12:59 | |
| *** anteaya has joined #openstack-ironic | 12:59 | |
| *** logan- has quit IRC | 13:00 | |
| *** vinm213 has quit IRC | 13:00 | |
| *** amotoki has quit IRC | 13:00 | |
| *** logan- has joined #openstack-ironic | 13:00 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Fix spawn error hook in "continue_node_clean" RPC method https://review.openstack.org/273594 | 13:02 |
| *** mkovacik has quit IRC | 13:05 | |
| *** anteaya has quit IRC | 13:06 | |
| *** NobodyCa1 has quit IRC | 13:06 | |
| TheJulia | Good morning! | 13:15 |
| *** amotoki has joined #openstack-ironic | 13:16 | |
| sambetts | o/ TheJulia | 13:16 |
| mgould | TheJulia, morning! | 13:16 |
| krotscheck1 | TheJulia: HELLO THERE, ARE YOUHOW COFFEE | 13:18 |
| *** thiagop has quit IRC | 13:18 | |
| krotscheck1 | eep. Sorry. Inside voice. | 13:19 |
| *** davidlenwell has quit IRC | 13:24 | |
| *** thiagop has joined #openstack-ironic | 13:24 | |
| *** Marga_ has joined #openstack-ironic | 13:25 | |
| *** davidlenwell has joined #openstack-ironic | 13:29 | |
| TheJulia | krotscheck1: coffee is beginning to flow through the tissues and the world is slowly becoming a brighter place! | 13:32 |
| *** achanda has joined #openstack-ironic | 13:33 | |
| openstackgerrit | Merged openstack/ironic-inspector: Add new conditions: matches and contains https://review.openstack.org/272612 | 13:36 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Enable Ironic Inspector for Cisco Drivers https://review.openstack.org/275207 | 13:37 |
| *** achanda has quit IRC | 13:38 | |
| *** bkumar has quit IRC | 13:39 | |
| *** thiagop has quit IRC | 13:39 | |
| *** keedya has joined #openstack-ironic | 13:40 | |
| *** thiagop has joined #openstack-ironic | 13:41 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Clean up documentation on introspection rules conditions https://review.openstack.org/275212 | 13:43 |
| *** lucas-hungry is now known as lucasagomes | 13:50 | |
| *** krtaylor has quit IRC | 13:51 | |
| *** keedya has quit IRC | 13:51 | |
| vdrok | morning sinval thiagop TheJulia and krotscheck1 | 13:53 |
| *** Marga__ has joined #openstack-ironic | 13:53 | |
| sinval | vdrok o/ | 13:54 |
| thiagop | vdrok: o/ | 13:57 |
| *** Marga_ has quit IRC | 13:57 | |
| *** phil_231 has quit IRC | 13:57 | |
| *** rloo has joined #openstack-ironic | 13:58 | |
| *** jcoufal has joined #openstack-ironic | 14:00 | |
| *** baoli has quit IRC | 14:00 | |
| *** baoli has joined #openstack-ironic | 14:01 | |
| *** rbudden has joined #openstack-ironic | 14:03 | |
| *** krtaylor has joined #openstack-ironic | 14:05 | |
| lucasagomes | TheJulia, krotscheck1 sinval thiagop morning :-) | 14:06 |
| *** alexpilotti has quit IRC | 14:09 | |
| sinval | lucasagomes, o/ | 14:11 |
| *** cdearborn has joined #openstack-ironic | 14:15 | |
| *** lazy_prince has joined #openstack-ironic | 14:15 | |
| *** piet has joined #openstack-ironic | 14:16 | |
| *** achanda has joined #openstack-ironic | 14:18 | |
| dtantsur | jroll, wdyt about https://review.openstack.org/#/c/183599/ ? | 14:23 |
| *** alexpilotti has joined #openstack-ironic | 14:24 | |
| thiagop | lucasagomes: o/ | 14:25 |
| *** mkovacik has joined #openstack-ironic | 14:27 | |
| *** achanda has quit IRC | 14:27 | |
| *** alexpilotti has quit IRC | 14:27 | |
| *** alexpilotti has joined #openstack-ironic | 14:28 | |
| jroll | dtantsur: I totally agree with that | 14:29 |
| *** piet has quit IRC | 14:29 | |
| jroll | I don't remember who wanted to keep them | 14:29 |
| *** mkovacik has quit IRC | 14:30 | |
| dtantsur | hopefully not me :D | 14:30 |
| lucasagomes | it was an old decision I believe | 14:30 |
| *** moshele has quit IRC | 14:30 | |
| *** jistr has joined #openstack-ironic | 14:31 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Fix spawn error hook in "continue_node_clean" RPC method https://review.openstack.org/273594 | 14:31 |
| lucasagomes | honestly, I don't mind... it's easy to troubleshoot by interactively dealing with the API | 14:31 |
| jroll | lucasagomes: nah, we talked about it a few weeks ago and people wanted to keep it | 14:31 |
| lucasagomes | yeah | 14:31 |
| jroll | but yeah my vote is and has been to remove it | 14:31 |
| lucasagomes | we now have a config option for that right? | 14:31 |
| jroll | yep | 14:32 |
| lucasagomes | fair enuff | 14:32 |
| jroll | but I think the spirit of this spec is to remove it | 14:32 |
| lucasagomes | yeah, those are the API guidelines | 14:33 |
| lucasagomes | it's not a rule but yeah, it's good if we can follow it | 14:33 |
| *** moshele has joined #openstack-ironic | 14:34 | |
| *** piet has joined #openstack-ironic | 14:35 | |
| lucasagomes | jroll, btw, there's a reno file for this already https://review.openstack.org/#/c/269639/ (the last file) | 14:36 |
| *** kalpase has quit IRC | 14:37 | |
| jroll | lucasagomes: wow, wtf | 14:38 |
| * jroll chugs his coffee | 14:38 | |
| jroll | +2 now | 14:38 |
| lucasagomes | ++ coffee | 14:39 |
| lucasagomes | ty! | 14:39 |
| NobodyCam | good morning Ironicers | 14:40 |
| thiagop | good morning mr NobodyCam | 14:41 |
| sinval | NobodyCam, o/ | 14:41 |
| *** absubram has joined #openstack-ironic | 14:41 | |
| lucasagomes | NobodyCam, morning | 14:42 |
| *** absubram_ has joined #openstack-ironic | 14:42 | |
| vdrok | morning jroll and NobodyCam | 14:42 |
| NobodyCam | morning thiagop sinval dtantsur lucasagomes jroll sambetts jlvillal rloo mgould vdrok TheJulia | 14:43 |
| NobodyCam | :) | 14:43 |
| sambetts | o/ NobodyCam | 14:43 |
| NobodyCam | o/ | 14:43 |
| jroll | hey hey | 14:43 |
| rloo | hi and morning NobodyCam, vdrok, jroll, lucasagomes, sambetts, thiagop, sinval, dtantsur and everyone else | 14:46 |
| sambetts | o/ rloo | 14:46 |
| lucasagomes | rloo, hello! good morning | 14:46 |
| jroll | hi rloo :) | 14:46 |
| thiagop | rloo: o/ | 14:46 |
| *** absubram has quit IRC | 14:46 | |
| *** absubram_ is now known as absubram | 14:46 | |
| *** keedya has joined #openstack-ironic | 14:47 | |
| mgould | NobodyCam, morning! | 14:48 |
| NobodyCam | :) | 14:49 |
| *** ronald has joined #openstack-ironic | 14:53 | |
| *** ronald is now known as rbradfor_phone | 14:56 | |
| openstackgerrit | Merged openstack/ironic: Enable doc8 style checker for *.rst files https://review.openstack.org/272988 | 14:56 |
| dtantsur | morning jroll, NobodyCam, thiagop, rloo | 14:56 |
| NobodyCam | morning dy | 14:56 |
| thiagop | dtantsur: o/ | 14:56 |
| NobodyCam | gah | 14:57 |
| NobodyCam | dtantsur: :) morning | 14:57 |
| TheJulia | good morning | 15:01 |
| vdrok | morning rloo :) | 15:02 |
| *** krtaylor has quit IRC | 15:03 | |
| NobodyCam | :) | 15:05 |
| *** piet has quit IRC | 15:08 | |
| *** mbound_ has quit IRC | 15:08 | |
| *** rajinir has joined #openstack-ironic | 15:11 | |
| *** links has quit IRC | 15:11 | |
| rloo | hi vdrok! | 15:14 |
| dtantsur | jroll, wdyt about explicitly documenting the inventory format in http://docs.openstack.org/developer/ironic-python-agent? | 15:14 |
| dtantsur | then I could link to it from inspector docs | 15:15 |
| *** krtaylor has joined #openstack-ironic | 15:15 | |
| jroll | dtantsur: I don't mind | 15:16 |
| dtantsur | ok | 15:16 |
| *** baoli has quit IRC | 15:17 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add db api layer for CRUD operations on node tags https://review.openstack.org/232598 | 15:21 |
| *** mbound has joined #openstack-ironic | 15:23 | |
| *** amotoki has quit IRC | 15:24 | |
| lucasagomes | jroll, deva(...) re from last meeting (sorry i missed it, I was flying back from brussels) tar images | 15:26 |
| lucasagomes | jroll, I think this is something we may want to support, even tho there's some discussion whether glance should or not support "tar" as a container type | 15:26 |
| lucasagomes | it already does support OVF | 15:26 |
| lucasagomes | http://docs.openstack.org/developer/glance/formats.html#container-format | 15:26 |
| lucasagomes | which is a tar | 15:26 |
| lucasagomes | and cloud images are distributed as ovf as well | 15:27 |
| lucasagomes | e.g http://uec-images.ubuntu.com/releases/14.04/release/ubuntu-14.04-server-cloudimg-amd64.ovf | 15:27 |
| lucasagomes | OVA* | 15:27 |
| *** mtanino has joined #openstack-ironic | 15:27 | |
| jroll | lucasagomes: did you read the conversation in this channel yesterday? | 15:28 |
| jroll | after the meeting | 15:28 |
| lucasagomes | nop only the meeting one | 15:28 |
| lucasagomes | gotta check eavesdrop | 15:28 |
| lucasagomes | oh no actually I did mgould pointed me to that | 15:29 |
| *** ukalifon has quit IRC | 15:30 | |
| * lucasagomes thought that was the meeting | 15:30 | |
| lucasagomes | anyway... just pointing out that OVA is a valid format for images and is basically a tar... | 15:32 |
| jroll | lucasagomes: the concern is about IPA knowing about filesystems and stuff, and having to flush that all through | 15:33 |
| lucasagomes | jroll, yeah probably that should be passed as a parameter, whether from flavor or glance image metadata | 15:34 |
| jroll | lucasagomes: sure, it would bring a lot of complexity though, and I'm not sure it's worth it | 15:34 |
| lucasagomes | or even an ironic config (e.g driver_info) | 15:34 |
| lucasagomes | sure, yeah I don't plan to add support for it in the near future | 15:34 |
| lucasagomes | just saying, the use case is valid | 15:35 |
| jroll | yeah, I think it's more complexity, both for us and for the user, than it's worth | 15:35 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Document hardware inventory sent to lookup and inspection https://review.openstack.org/275272 | 15:35 |
| lucasagomes | right, yeah... I mean, ironic-lib can pretty much do it already | 15:36 |
| lucasagomes | we can partiton the disk and create filesystems | 15:36 |
| lucasagomes | we just need to learn to untar the content in the root partition instead of dd it | 15:36 |
| lucasagomes | just saying because there's plenty of system's that uses tar-ish deployments (anaconda can do it wiht tar payload, google cloud does it too) | 15:37 |
| lucasagomes | and so on... | 15:37 |
| *** baoli has joined #openstack-ironic | 15:38 | |
| rloo | lucasagomes: i didn't realize that google cloud supported that too | 15:38 |
| *** piet has joined #openstack-ironic | 15:39 | |
| rloo | jroll: how much more complexity would there be? | 15:39 |
| jroll | well, lucasagomes yeah, I'm not strictly opposed to it down the road, but right now I am | 15:39 |
| lucasagomes | rloo, yeah, AFAIK they do tar ball deployment internally as well (but I know that because of a google college) | 15:39 |
| lucasagomes | jroll, sure | 15:39 |
| lucasagomes | just wanted to weight in the conversation | 15:39 |
| jroll | rloo: how does the user tell ironic what FS they want? | 15:40 |
| lucasagomes | since I wasn't here | 15:40 |
| jroll | the nova user, to be clear | 15:40 |
| rloo | jroll: dunno, how does the nova user tell ironic anything? | 15:40 |
| lucasagomes | jroll, IMO it can be the same way a user says BIOS or UEFI? Flavor? | 15:40 |
| jroll | also all the code to make filesystems, etc | 15:40 |
| jroll | lucasagomes: as a user, I can't create a flavor | 15:40 |
| jroll | so now the operator has to supply x times each flavor, where x is every filesystem ever | 15:41 |
| lucasagomes | right, they probably have to be pre-created | 15:41 |
| jroll | which is terrible imo | 15:41 |
| lucasagomes | why the user has to choose the filesystem, clouds can offer <some> available fs types | 15:41 |
| rloo | jroll: the user that wants this is yahoo and yes, yahoo already has many many flavours. I don't think adding a few more makes a diff. | 15:41 |
| jroll | because the use case is "some of my users want to use xfs or ext3 or ext4 or btrfs or..." | 15:42 |
| jroll | and then there's all the code to handle managing these filesystems etc | 15:42 |
| jroll | anyway, let's take this to the spec because I don't want to rehash this whole thing | 15:42 |
| lucasagomes | jroll, that's seems valid... I mean, some fs works better depending on the workload the user wants for that node | 15:42 |
| jroll | right, so they make an image with that fs | 15:43 |
| lucasagomes | fwiw we allow choosing the ephemeral disk fs (not the user but a config) | 15:43 |
| lucasagomes | what we can do is to allows mkfs.<FS> operators make sure the modules are installed | 15:43 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Restart consoles on conductor startup https://review.openstack.org/257252 | 15:43 |
| lucasagomes | jroll, yeah we can always argue both side | 15:43 |
| jroll | here's my real opposition | 15:43 |
| lucasagomes | but anyway... spec is good | 15:43 |
| jroll | we have a HUGE amount of features people want to add | 15:44 |
| jroll | and we say "seems valid, let's add it" | 15:44 |
| jroll | which isn't in itself bad, but we let all these things distract us from actually getting anything done | 15:44 |
| lucasagomes | yes, it's an opensource software | 15:44 |
| lucasagomes | noawdays people run linux on light bulb | 15:44 |
| lucasagomes | that's the beauty of it | 15:44 |
| rloo | as lucasagomes sez, it is opensource | 15:44 |
| jroll | right, but while we realistically have roughly... 6? 8? people that work on this full time, we don't have enough time to entertain these features as they come | 15:45 |
| rloo | the spec process isn't meant to block due to other things needing to get done, is it? | 15:45 |
| *** raddaoui has joined #openstack-ironic | 15:45 | |
| *** NobodyCa1 has joined #openstack-ironic | 15:45 | |
| jroll | no | 15:45 |
| lucasagomes | jroll, sure... just saying that we should not -2 ideas. Ofc we may not have time and that is not a priority for the current core team which has to focus on specify things | 15:46 |
| rloo | i think you're being unfair here. we should review specs based on their merit. whether they are prioritized wrt getting the feature landed is a diff story | 15:46 |
| lucasagomes | but ideas should *always* be open | 15:46 |
| lucasagomes | if one wants to work on that | 15:46 |
| *** chaustin has joined #openstack-ironic | 15:46 | |
| lucasagomes | yes they should be able to | 15:46 |
| jroll | I'm not saying that we should -2 based on that | 15:46 |
| jroll | but this is a pretty narrow use case IMO, easily solved by just converting the tarball to a qcow2 | 15:46 |
| lucasagomes | sure, that's one way | 15:47 |
| *** baoli has quit IRC | 15:47 | |
| jroll | so I don't think the work is worth the benefit | 15:47 |
| lucasagomes | but you agree that have 1 tarball instead of <many> images for diff filesystem is a thing | 15:47 |
| lucasagomes | plus network bandwidth | 15:47 |
| lucasagomes | I mean, just saying, people do it already | 15:47 |
| lucasagomes | there are pros/cons as anything else | 15:47 |
| lucasagomes | threfore I think it's a valid case | 15:48 |
| lucasagomes | sorry I just started this converstation to point out that I think it's a valid case | 15:48 |
| jroll | sure, and I don't think the pros outweigh the cons here | 15:48 |
| *** baoli has joined #openstack-ironic | 15:48 | |
| jroll | yeah I understand | 15:48 |
| jroll | and I do appreciate it | 15:48 |
| lucasagomes | don't wanna discuss the technicals about it | 15:48 |
| rloo | btw I don't believe that the use case can just be solved by converting the tarball to qcow2 | 15:49 |
| *** lazy_prince has quit IRC | 15:49 | |
| *** lazy_prince has joined #openstack-ironic | 15:50 | |
| *** NobodyCa1 has quit IRC | 15:51 | |
| lucasagomes | rloo, let's bring it to the spec | 15:51 |
| lucasagomes | (sorry for starting it) | 15:51 |
| lucasagomes | rloo, info in the channel get lost anyway | 15:51 |
| rloo | lucasagomes: i want the author to bring it to the spec :) but thx for voicing your opinion! | 15:51 |
| lucasagomes | rloo, ++ yeah just wanted to compensate my absence yesterday | 15:52 |
| jroll | yes, thank you both for the chat | 15:52 |
| rloo | lucasagomes: ++ | 15:52 |
| jroll | sorry for the frustration that leaked | 15:52 |
| rloo | jroll: understandable. I'm also frustrated at the slow progress etc. | 15:53 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Register the keystone service and endpoint in the devstack plugin https://review.openstack.org/275286 | 15:54 |
| *** dims has joined #openstack-ironic | 15:54 | |
| *** dims_ has quit IRC | 15:54 | |
| *** lazy_prince has quit IRC | 15:55 | |
| *** lazy_prince has joined #openstack-ironic | 15:56 | |
| *** mgarza has joined #openstack-ironic | 15:57 | |
| *** garthb has joined #openstack-ironic | 15:59 | |
| *** devananda has joined #openstack-ironic | 16:00 | |
| *** david-lyle has joined #openstack-ironic | 16:02 | |
| *** itamarl has quit IRC | 16:04 | |
| *** garthb has quit IRC | 16:04 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Set node last_error in TaskManager https://review.openstack.org/273992 | 16:07 |
| *** chaustin has quit IRC | 16:08 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Enable Keystone v3 endpoints for Inspector https://review.openstack.org/274720 | 16:11 |
| *** chaustin has joined #openstack-ironic | 16:12 | |
| *** mkovacik has joined #openstack-ironic | 16:12 | |
| *** lazy_prince has quit IRC | 16:20 | |
| *** garthb has joined #openstack-ironic | 16:32 | |
| vdrok | jlvillal, lucasagomes, rloo could you please look once again at https://review.openstack.org/#/c/254295/ and https://review.openstack.org/#/c/254296/? They have a bunch of +2s now and the gate is green at last | 16:36 |
| rloo | vdrok: looking... | 16:37 |
| vdrok | thanks rloo :) | 16:39 |
| rloo | vdrok: I +A'd 254295. | 16:39 |
| rloo | vdrok: I'll look at 254296 in a few minutes maybe. gotta get something else done first. | 16:40 |
| vdrok | sure, thanks for this | 16:40 |
| *** prometheanfire has left #openstack-ironic | 16:41 | |
| *** izaakk has joined #openstack-ironic | 16:42 | |
| *** harshs has joined #openstack-ironic | 16:42 | |
| *** ianw has quit IRC | 16:43 | |
| *** chaustin has quit IRC | 16:44 | |
| *** intr1nsic has quit IRC | 16:44 | |
| *** konetzed has quit IRC | 16:44 | |
| *** mgoddard_ has joined #openstack-ironic | 16:45 | |
| *** chaustin has joined #openstack-ironic | 16:45 | |
| *** chaustin has quit IRC | 16:45 | |
| *** dims_ has joined #openstack-ironic | 16:47 | |
| *** chaustin has joined #openstack-ironic | 16:48 | |
| *** mgoddard has quit IRC | 16:48 | |
| *** chaustin has quit IRC | 16:48 | |
| *** harshs has quit IRC | 16:48 | |
| *** chaustin has joined #openstack-ironic | 16:49 | |
| *** dims has quit IRC | 16:49 | |
| *** bnemec has quit IRC | 16:53 | |
| *** jistr has quit IRC | 16:53 | |
| *** absubram has quit IRC | 16:55 | |
| lucasagomes | vdrok, will take a look | 16:55 |
| *** e0ne has quit IRC | 16:55 | |
| vdrok | lucasagomes, thank you | 16:56 |
| *** mbound has quit IRC | 16:56 | |
| *** achanda has joined #openstack-ironic | 16:57 | |
| *** baoli has quit IRC | 16:59 | |
| *** baoli has joined #openstack-ironic | 17:00 | |
| lucasagomes | vdrok, re https://review.openstack.org/#/c/254296 there's a bunch of comments there | 17:02 |
| lucasagomes | sure we need a following patch for that? Maybe just squash it onto that patch? | 17:02 |
| *** raddaoui has quit IRC | 17:02 | |
| * lucasagomes would +2 it anyway.... | 17:02 | |
| lucasagomes | in any case, rloo's asking you to update the commit message it will reset the votes | 17:03 |
| vdrok | lucasagomes, yep, I guess rloo wants the commit message to change, so I guess I'll squash them | 17:03 |
| lucasagomes | vdrok, yeah | 17:03 |
| lucasagomes | sounds sane | 17:03 |
| lucasagomes | vdrok, just ping me when you update it and I'll gladly add my +2 back | 17:03 |
| rloo | oh. wait. did you want to squash? I can wait. | 17:03 |
| *** mgoddard_ has quit IRC | 17:03 | |
| vdrok | lucasagomes, will do :) | 17:03 |
| *** mgoddard has joined #openstack-ironic | 17:04 | |
| rloo | vdrok: ? | 17:04 |
| rloo | vdrok: but i didn't look yet at the followon. and i added a comment about validating. | 17:04 |
| vdrok | rloo, hm, ok, then it's better to leave it as is :) | 17:05 |
| rloo | vdrok: i just commented on the followon. | 17:05 |
| vdrok | as for the validation, there was a discussion about this a while ago | 17:05 |
| *** Nisha has joined #openstack-ironic | 17:06 | |
| vdrok | i'll try to find this | 17:06 |
| *** achanda has quit IRC | 17:06 | |
| vdrok | rloo, I think the reason to not to do the checks in validate was that no exception is raised | 17:08 |
| vdrok | rloo, as image download may work well without proxies too | 17:08 |
| rloo | vdrok: so you want to continue even if the proxy info isn't specified correctly in driver_info? how will the user know? | 17:09 |
| vdrok | there will be warnings logged | 17:09 |
| rloo | vdrok: isn't that the whole idea of the validate()? to validate the driver_info stuff? | 17:09 |
| *** chaustin has quit IRC | 17:10 | |
| *** mbound has joined #openstack-ironic | 17:10 | |
| *** chaustin has joined #openstack-ironic | 17:10 | |
| rloo | vdrok: anyway, that's why it is a bit questionable. even if we added a vendorinterface.validate(), at that point, i'm not sure we want to stop the deployment. | 17:11 |
| *** rbudden has quit IRC | 17:11 | |
| rloo | vdrok: which is why i was wondering if the deployinterface.validate() should be validating that stuff and not deploy if it is incorrect. | 17:11 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Switch to Futurist library for periodic tasks https://review.openstack.org/264720 | 17:12 |
| *** raddaoui has joined #openstack-ironic | 17:12 | |
| vdrok | rloo, my intent was that we should try to continue | 17:12 |
| vdrok | and if some parameters are wrong we just don't pass them to agent | 17:12 |
| rloo | vdrok: "only logging" doesn't seem like the right thing to do cuz the user may not look at the logs and won't know that they aren't using a proxy. | 17:13 |
| dtantsur | see you tomorrow | 17:13 |
| rloo | bye dtantsur | 17:13 |
| *** dtantsur is now known as dtantsur|afk | 17:13 | |
| *** ifarkas has quit IRC | 17:14 | |
| vdrok | rloo, the problem is that we cannot properly validate proxies from conductor anyway | 17:15 |
| vdrok | as proxy server may not be available from conductor | 17:15 |
| rloo | vdrok: but we are doing some validation of the strings/values specified for those parameters | 17:16 |
| rloo | vdrok: just that the validation isn't being done in a validate() but when the values are being used | 17:16 |
| *** baoli has quit IRC | 17:16 | |
| rloo | vdrok: I am not asking you to write code to ping/make sure the URLs are alive and kicking | 17:16 |
| *** vinm213 has joined #openstack-ironic | 17:16 | |
| vdrok | rloo, yes, I understand that :) ok, I think it makes sense to move this validation to validate | 17:17 |
| *** degorenko is now known as _degorenko|afk | 17:17 | |
| rloo | vdrok: do you want to do that in the followon patch? | 17:17 |
| vdrok | rloo, I'll do everything in that patch I guess | 17:18 |
| vdrok | as other stuff is mostly clarifications/moving code around | 17:18 |
| vdrok | will update shortly | 17:18 |
| *** moshele has quit IRC | 17:18 | |
| rloo | vdrok: in 254296, not the followon? ok with me, ping me when it is updated. thx. | 17:18 |
| vdrok | rloo, yep :) | 17:19 |
| *** chaustin has quit IRC | 17:19 | |
| *** sambetts has quit IRC | 17:20 | |
| *** baoli has joined #openstack-ironic | 17:22 | |
| *** rbradfor_phone has quit IRC | 17:24 | |
| *** bkumar has joined #openstack-ironic | 17:25 | |
| *** bkumar has quit IRC | 17:26 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Extend root device hints to support device name https://review.openstack.org/269639 | 17:29 |
| *** mgarza has quit IRC | 17:30 | |
| openstackgerrit | Merged openstack/ironic: Add ability to cache swift temporary URLs https://review.openstack.org/254295 | 17:30 |
| *** openstackgerrit has quit IRC | 17:32 | |
| *** mgarza_ has joined #openstack-ironic | 17:33 | |
| *** openstackgerrit has joined #openstack-ironic | 17:33 | |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic: Updated iLO documentation for boot mode capability https://review.openstack.org/274618 | 17:33 |
| vdrok | rloo, after some second thought, there might be a problem with moving it to validate. an example: image_http_proxy is valid, image_https_proxy is not. image_no_proxy is invalid too, but for the image that is going to be used, no_proxy won't be used, as e.g. it does not have an URL where the image is hosted. Should we fail in this case? | 17:34 |
| vdrok | and the image link is generated only after agent_deploy.prepare and put to instance_info['image_url'] | 17:35 |
| vdrok | so in validate it might not be known is the image http or https | 17:35 |
| vdrok | rloo, or did you mean to move it to validate but not to change from logging to raising exceptions | 17:37 |
| *** mbound has quit IRC | 17:42 | |
| lucasagomes | folks I will call it a day (gotta do some groceries yet, pos-fosdem) | 17:49 |
| lucasagomes | vdrok, is it ok if I review that patch tomorrow morning? | 17:49 |
| TheJulia | goodnight lucasagomes | 17:49 |
| vdrok | lucasagomes, yeah sure, good night! | 17:49 |
| NobodyCam | night lucasagomes | 17:49 |
| lucasagomes | ack | 17:50 |
| lucasagomes | night y'all! Talk soon | 17:50 |
| thiagop | see ya lucasagomes | 17:50 |
| *** vinm213 has quit IRC | 17:51 | |
| *** krotscheck1 has quit IRC | 17:51 | |
| mgould | lucasagomes, good night! | 17:56 |
| *** rbudden has joined #openstack-ironic | 17:58 | |
| rloo | vdrok: argh. too many questions. i don't remember what i meant. sec. | 18:00 |
| rloo | vdrok: wrt https://review.openstack.org/#/c/254296/10/ironic/drivers/modules/agent.py | 18:03 |
| rloo | vdrok: lines 416-427 | 18:04 |
| rloo | vdrok: that validation. there is no image link checking there. | 18:04 |
| rloo | vdrok: oh, actually you have more checking up to line 446. | 18:04 |
| vdrok | rloo, yes, requests will use what is appropriate wrt currently requested image | 18:05 |
| vdrok | rloo, it will happen in agent | 18:05 |
| vdrok | rloo, the question is, should we fail if some parameters are incorrect | 18:05 |
| rloo | vdrok: I think we should fail at/before deploy time. | 18:06 |
| rloo | vdrok: failing when the vendor interface method is being invoked is too late to be failing | 18:06 |
| rloo | vdrok: at least, that's what i think. | 18:06 |
| openstackgerrit | Merged openstack/ironic: devstack 'cleanup-node' script should delete OVS bridges https://review.openstack.org/263508 | 18:07 |
| vdrok | rloo, but then we need to know the actual image url, and fail only if the parameters that will be used for download are incorrect | 18:07 |
| rloo | vdrok: if we fail, we want to do it so the user knows. | 18:07 |
| vdrok | e.g. if image_http_proxy will not be used, we should not fail if it is incorrect? | 18:07 |
| rloo | vdrok: you can do some validating w/o knowing the image url, right? | 18:07 |
| rloo | vdrok: where in that agent.py file, does it validate against the actual image url? i think i am missing something. | 18:08 |
| vdrok | rloo, so an example above | 18:09 |
| *** davideagnello has joined #openstack-ironic | 18:09 | |
| vdrok | we have an image, generated temp url will be http, but we dont know this yet | 18:09 |
| vdrok | and we have image_http_proxy and image_https_proxy set | 18:10 |
| vdrok | image_https_proxy is incorrect | 18:10 |
| *** athomas has quit IRC | 18:10 | |
| vdrok | rloo, should we fail in this case? | 18:10 |
| rloo | vdrok: so i think it doesn't matter about the actual image. if the user set a value for image_https_proxy and it is incorrect, we should fail. | 18:10 |
| vdrok | image_https_proxy is incorrect but will not be used | 18:11 |
| openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/268450 | 18:11 |
| *** athomas has joined #openstack-ironic | 18:11 | |
| vdrok | aha | 18:11 |
| rloo | vdrok: these are user-settable values, so if they set it incorrectly... they should know. | 18:11 |
| *** athomas has quit IRC | 18:12 | |
| vdrok | rloo, yes, here is the place my understanding was different. because if we would fail when only parameters that are being used are incorrect, there would be too many checks | 18:12 |
| vdrok | rloo, I'll change the validation appropriately, thanks | 18:12 |
| rloo | vdrok: ahh. ok. yeah, i mean 'simple' checks/validations like we do with other driver_info parameters/values. | 18:13 |
| *** intr1nsic has joined #openstack-ironic | 18:13 | |
| rloo | vdrok: i think in most of the code, the validate() will call some parse_driver_info() method that actually does the checking. | 18:13 |
| *** anteaya has joined #openstack-ironic | 18:13 | |
| rloo | vdrok: or something like that. it has been awhile since i looked at that code. | 18:14 |
| vdrok | rloo, yes, and I guess I'll put it in agent deploy, as stuff like device hints is validated there currently | 18:14 |
| vdrok | *agent deploy validate | 18:14 |
| rloo | vdrok: ah, ok, that makes sense! | 18:14 |
| *** ianw has joined #openstack-ironic | 18:15 | |
| *** jcoufal has quit IRC | 18:16 | |
| *** harshs has joined #openstack-ironic | 18:17 | |
| *** achanda has joined #openstack-ironic | 18:19 | |
| *** konetzed has joined #openstack-ironic | 18:19 | |
| rloo | jroll, devananda: wrt tempest plugin, is this the next patch that we want -- it turns it on: https://review.openstack.org/#/c/265311/ | 18:20 |
| *** chaustin has joined #openstack-ironic | 18:21 | |
| rloo | jroll, devananda: there's also this patch, is that something we want in soon too? https://review.openstack.org/#/c/260358 | 18:21 |
| *** chaustin has quit IRC | 18:25 | |
| *** chaustin has joined #openstack-ironic | 18:25 | |
| *** baoli has quit IRC | 18:25 | |
| *** ChubYann has joined #openstack-ironic | 18:30 | |
| * mgould -> home; good night everyone! | 18:34 | |
| *** mgould has quit IRC | 18:34 | |
| NobodyCam | night mgould | 18:35 |
| *** trown is now known as trown|lunch | 18:38 | |
| *** mkovacik has quit IRC | 18:40 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow to define the number of test vms to be created https://review.openstack.org/275371 | 18:40 |
| *** raddaoui has quit IRC | 18:44 | |
| *** absubram has joined #openstack-ironic | 18:46 | |
| *** Saulo_Aislan has joined #openstack-ironic | 18:48 | |
| *** Saulo_Aislan has quit IRC | 18:49 | |
| *** Saulo-Aislan has joined #openstack-ironic | 18:50 | |
| *** garthb_ has joined #openstack-ironic | 18:52 | |
| *** garthb has quit IRC | 18:52 | |
| *** Sukhdev has joined #openstack-ironic | 18:55 | |
| Saulo-Aislan | Hi, I am testinghttp://specs.openstack.org/openstack/ironic-specs/specs/kilo-implemented/new-ironic-state-machine.html the state changes I could make Deploying starting from setanto AVAILABLE state ACTIVE state without the requirement of image, flavor only with a FAKE drive. Can someone explain to me what happened? | 18:56 |
| jroll | rloo: yes and kinda yeah | 18:57 |
| *** garthb_ has quit IRC | 18:57 | |
| rloo | jroll: ok, after you update the first i'll review, and i'll look at the second today | 18:57 |
| jroll | rloo: ok, thanks | 18:58 |
| rloo | Saulo-Aislan: are you testing with latest ironic code? | 18:58 |
| rloo | Saulo-Aislan: that spec could be out of date. would this diagram be more useful? http://docs.openstack.org/developer/ironic/dev/states.html | 18:59 |
| rloo | Saulo-Aislan: if you are using the fake driver, that driver doesn't do anything cuz it is fake. | 19:00 |
| *** garthb_ has joined #openstack-ironic | 19:00 | |
| *** electrofelix has quit IRC | 19:00 | |
| *** chaustin has quit IRC | 19:01 | |
| *** harshs has quit IRC | 19:01 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add proxy related parameters to agent driver https://review.openstack.org/254296 | 19:02 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add documentation for proxies usage with IPA https://review.openstack.org/250878 | 19:02 |
| *** chaustin has joined #openstack-ironic | 19:03 | |
| Saulo-Aislan | rloo Yes i using driver fake, but even though the fake I would not give a nova-boot to be able to do Deploying | 19:03 |
| *** whydidyoustealmy has joined #openstack-ironic | 19:03 | |
| *** raddaoui has joined #openstack-ironic | 19:03 | |
| rloo | Saulo-Aislan: sorry, in meeting now | 19:03 |
| Saulo-Aislan | rloo: thanks | 19:04 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add proxy related parameters to agent driver https://review.openstack.org/254296 | 19:04 |
| *** crinkle_ has joined #openstack-ironic | 19:05 | |
| *** lintan_ has joined #openstack-ironic | 19:06 | |
| *** crinkle has quit IRC | 19:06 | |
| vdrok | rloo, updated according to your suggestions, thanks for thorough review | 19:07 |
| *** crinkle_ is now known as crinkle | 19:07 | |
| vdrok | good night everyone! | 19:08 |
| *** vishwanathj has joined #openstack-ironic | 19:10 | |
| thiagop | night vdrok | 19:10 |
| NobodyCam | night vdrok | 19:10 |
| *** bcrochet has joined #openstack-ironic | 19:11 | |
| *** bcrochet has quit IRC | 19:11 | |
| *** bcrochet has joined #openstack-ironic | 19:11 | |
| *** lucasagomes_ has joined #openstack-ironic | 19:11 | |
| *** mari0jv has joined #openstack-ironic | 19:11 | |
| *** Shalmazar has joined #openstack-ironic | 19:11 | |
| *** yuriyz has quit IRC | 19:12 | |
| *** barra204 has quit IRC | 19:12 | |
| *** lucasagomes has quit IRC | 19:12 | |
| *** thrash has quit IRC | 19:12 | |
| *** lintan has quit IRC | 19:12 | |
| *** kbyrne has quit IRC | 19:12 | |
| *** mordred has quit IRC | 19:12 | |
| *** gabriel has quit IRC | 19:12 | |
| *** _degorenko|afk has quit IRC | 19:12 | |
| *** mariojv has quit IRC | 19:12 | |
| *** Madasi has quit IRC | 19:12 | |
| *** thingee has quit IRC | 19:12 | |
| *** Shalmazar is now known as Madasi | 19:12 | |
| *** bcrochet is now known as thrash | 19:12 | |
| *** penick has joined #openstack-ironic | 19:12 | |
| *** kbyrne has joined #openstack-ironic | 19:15 | |
| *** mordred has joined #openstack-ironic | 19:15 | |
| *** harshs has joined #openstack-ironic | 19:15 | |
| *** yuriyz has joined #openstack-ironic | 19:16 | |
| *** chaustin has quit IRC | 19:16 | |
| *** gabriel has joined #openstack-ironic | 19:16 | |
| *** chaustin has joined #openstack-ironic | 19:18 | |
| *** _degorenko|afk has joined #openstack-ironic | 19:18 | |
| *** chaustin has quit IRC | 19:18 | |
| *** boris-42 has joined #openstack-ironic | 19:18 | |
| *** chaustin has joined #openstack-ironic | 19:21 | |
| *** chaustin has quit IRC | 19:22 | |
| *** chaustin has joined #openstack-ironic | 19:22 | |
| *** ijw has joined #openstack-ironic | 19:23 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Log warning if ipmi_username/ipmi_password missing https://review.openstack.org/273891 | 19:25 |
| *** thingee has joined #openstack-ironic | 19:26 | |
| devananda | jlvillal: re ^, one approach to avoid it being a breaking change would be a data migration that updates ipmi user/pass to be '' where it is not currently set for that driver | 19:27 |
| jlvillal | devananda: I was also thinking maybe a config option that would allow empty username/password. And then default it to False. | 19:28 |
| jlvillal | devananda: Then those crazy users could set it to true if they like. And have a log message that tells them not allowed unless they set parameter | 19:28 |
| jlvillal | But I was thinking that would be a future patch :) | 19:29 |
| * jlvillal has to go to lunch now... | 19:29 | |
| devananda | config option for that -1 | 19:29 |
| devananda | enjoy lunch :) | 19:29 |
| NobodyCam | mmmm food | 19:29 |
| NobodyCam | :p | 19:30 |
| *** NobodyCa1 has joined #openstack-ironic | 19:30 | |
| *** deva_ has joined #openstack-ironic | 19:30 | |
| *** mgarza_ has quit IRC | 19:31 | |
| *** chaustin has quit IRC | 19:33 | |
| *** trown|lunch is now known as trown | 19:33 | |
| *** NobodyCa1 has quit IRC | 19:35 | |
| *** deva_ has quit IRC | 19:36 | |
| *** moshele has joined #openstack-ironic | 19:36 | |
| *** chaustin has joined #openstack-ironic | 19:36 | |
| * TheJulia rolls around the idea of a config option in her head | 19:37 | |
| rloo | jroll: Can I assume that you will edit renos at the end? update/delete/ whatever? eg, first bullet in upgrade: http://docs.openstack.org/releasenotes/ironic/current-series.html#upgrade-notes | 19:39 |
| jroll | rloo: yeah, we'll clean them up | 19:40 |
| jroll | if you're bored, feel free to fix it now though :P | 19:40 |
| *** [1]cdearborn has joined #openstack-ironic | 19:40 | |
| rloo | jroll: ok. i am never bored cuz we always seem to have high priority bugs. | 19:40 |
| jroll | yep | 19:40 |
| rloo | jroll: not to mention 'add', 'adds', 'updated' tenses. and patches where i don't think we should have renos. but i leave that up to you to fix or not! | 19:41 |
| jroll | rloo: yeah, I can touch it up before release | 19:41 |
| rloo | jroll: thx. btw, i noticed that the ironicclient was missing some renos in that previous release but we'll do better now :) | 19:43 |
| jroll | rloo: yeah, :( | 19:43 |
| *** jaybeale has joined #openstack-ironic | 19:50 | |
| thiagop | jroll: how can I discover the order of the patches in the network isolation "chain"? | 19:51 |
| jroll | thiagop: the box in the top right goes from first at the bottom to last at the top | 19:52 |
| jroll | on any of those patches | 19:52 |
| *** whydidyoustealmy is now known as barra204 | 19:53 | |
| *** Sukhdev has quit IRC | 19:54 | |
| *** cdearborn has quit IRC | 19:55 | |
| thiagop | jroll: ty | 19:58 |
| jroll | np | 19:58 |
| *** harshs has quit IRC | 19:59 | |
| *** harshs has joined #openstack-ironic | 19:59 | |
| *** Marga__ has quit IRC | 20:00 | |
| *** Nisha has quit IRC | 20:01 | |
| *** Sukhdev has joined #openstack-ironic | 20:02 | |
| *** Sukhdev has quit IRC | 20:03 | |
| *** Sukhdev has joined #openstack-ironic | 20:04 | |
| *** achanda has quit IRC | 20:11 | |
| *** thrash is now known as thrash|bbl | 20:13 | |
| *** mkovacik has joined #openstack-ironic | 20:17 | |
| *** piet has quit IRC | 20:28 | |
| jlvillal | devananda, TheJulia: I responded to your comments on the logging. Basically the logging messages are reflecting how the code operates. Thanks for the reviews :) | 20:30 |
| *** piet has joined #openstack-ironic | 20:32 | |
| mrda | Morning Ironic | 20:33 |
| *** chaustin has quit IRC | 20:34 | |
| jlvillal | mrda, Morning | 20:34 |
| * jlvillal walks back to work.... | 20:34 | |
| mrda | o/ | 20:34 |
| *** chaustin has joined #openstack-ironic | 20:34 | |
| thiagop | morning mrda | 20:35 |
| *** chaustin has quit IRC | 20:36 | |
| *** chaustin has joined #openstack-ironic | 20:36 | |
| mrda | o/ | 20:36 |
| *** moshele has quit IRC | 20:41 | |
| *** ijw has quit IRC | 20:51 | |
| *** Sukhdev has quit IRC | 20:52 | |
| *** achanda has joined #openstack-ironic | 20:52 | |
| *** ijw has joined #openstack-ironic | 20:52 | |
| TheJulia | jlvillal: thank you | 20:54 |
| *** raildo is now known as raildo-afk | 20:54 | |
| *** ijw has quit IRC | 20:57 | |
| *** garthb_ has quit IRC | 20:57 | |
| *** garthb__ has joined #openstack-ironic | 20:57 | |
| thiagop | folks, I'm calling it a day | 20:58 |
| thiagop | see ya | 20:58 |
| TheJulia | goodnight | 20:58 |
| *** thiagop has quit IRC | 21:02 | |
| *** raddaoui has quit IRC | 21:12 | |
| *** NobodyCa1 has joined #openstack-ironic | 21:20 | |
| *** raddaoui has joined #openstack-ironic | 21:21 | |
| *** Marga_ has joined #openstack-ironic | 21:22 | |
| *** ubuntu has joined #openstack-ironic | 21:23 | |
| *** ubuntu is now known as Guest40848 | 21:23 | |
| *** NobodyCa1 has quit IRC | 21:25 | |
| *** chaustin has quit IRC | 21:34 | |
| *** rbudden has quit IRC | 21:35 | |
| *** chaustin has joined #openstack-ironic | 21:37 | |
| *** baoli has joined #openstack-ironic | 21:38 | |
| *** vishwana_ has joined #openstack-ironic | 21:46 | |
| *** ijw has joined #openstack-ironic | 21:47 | |
| *** vishwanathj has quit IRC | 21:48 | |
| *** achanda has quit IRC | 21:48 | |
| *** vishwana_ has quit IRC | 21:49 | |
| *** thrash|bbl is now known as thrash | 21:50 | |
| *** ijw has quit IRC | 21:54 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add proxy related parameters to agent driver https://review.openstack.org/254296 | 21:56 |
| *** achanda has joined #openstack-ironic | 21:59 | |
| *** Sukhdev has joined #openstack-ironic | 22:01 | |
| *** trown is now known as trown|outttypeww | 22:02 | |
| jlvillal | jroll: Do you have time to help out again with a cloud-config question? | 22:04 |
| jlvillal | gmmaha and I are finally getting back to trying to debug the issue we were seeing | 22:04 |
| jlvillal | We have the /var/log/cloud-init.log here: http://paste.openstack.org/show/485777/ | 22:05 |
| jlvillal | Does anyone have an example of a working /var/log/cloud-init.log? Trying to figure out why ours isn't working... | 22:10 |
| jlvillal | It doesn't seem to be mounting the config-drive. | 22:10 |
| *** piet has quit IRC | 22:10 | |
| *** raddaoui has quit IRC | 22:11 | |
| *** chaustin has quit IRC | 22:19 | |
| TheJulia | hmmm | 22:21 |
| TheJulia | how was the image built? | 22:22 |
| *** chaustin has joined #openstack-ironic | 22:22 | |
| jlvillal | TheJulia: We think we didn't set DIB_CLOUD_INIT_DATASOURCES | 22:23 |
| jlvillal | TheJulia: We are setting it now. And want to rebuild our dib image | 22:23 |
| *** piet has joined #openstack-ironic | 22:24 | |
| TheJulia | yeah, that would help :) | 22:26 |
| TheJulia | looks like its defaulting out to none | 22:26 |
| jroll | jlvillal: I'm around now if you still need a hand | 22:26 |
| TheJulia | that happens to be default if simple-init is engaged, but it sounds like your trying to get cloud-init to work | 22:27 |
| jlvillal | TheJulia: A hint on the simple ansible command to rebuild the dib image would be nice :) If we can just do that one thing, if possible. | 22:27 |
| *** achanda has quit IRC | 22:30 | |
| *** e0ne has joined #openstack-ironic | 22:31 | |
| TheJulia | jlvillal: a simple command cannot be provided without greater context into exactly what your doing. Although, I typically find myself just manually new images with diskimage-builder. | 22:31 |
| jlvillal | TheJulia: Ah okay. Makes sense. We were trying to figure out if we could do: ansible-playbook something something something :) | 22:32 |
| *** chaustin has quit IRC | 22:35 | |
| TheJulia | jlvillal: ahh, some things like that exist, but... it seems most advanced folk just prefer to execute dib directly | 22:36 |
| * jlvillal is not feeling advanced ;) | 22:36 | |
| * TheJulia kicks self for what will ultimately be lots of unit test changes | 22:36 | |
| TheJulia | | last_error | Error while attempting to adopt node | | 22:37 |
| TheJulia | | | 88a98621-6bb0-49e5-86d5-60b4c958e70e: global name 'context' is not | | 22:37 |
| TheJulia | | | defined <------ I think this is where I go get tequila and start drinking it | 22:37 |
| rloo | TheJulia: apparently, adoptions are a hard process :) | 22:41 |
| *** e0ne has quit IRC | 22:43 | |
| TheJulia | rloo: oh, when i realized manage doesn't actually validate anything really... I started putting validation code in... and well... yeah... *grump* | 22:43 |
| TheJulia | rloo: and yes, adoptions are hard processes, in all universes | 22:44 |
| *** [1]cdearborn has quit IRC | 22:44 | |
| rloo | TheJulia: nice to see that you're working on it | 22:44 |
| *** [1]cdearborn has joined #openstack-ironic | 22:45 | |
| *** rbudden has joined #openstack-ironic | 22:46 | |
| TheJulia | rloo: it is a high priority for me. I was hoping to take my laptop to the local bar tonight, order dinner, and begin writing the doc for it | 22:47 |
| rloo | TheJulia: wow! | 22:48 |
| *** thrash is now known as thrash|g0ne | 22:48 | |
| TheJulia | rloo: that might not happen... then again... it might be fix the unit tests I wrote, and then.... begin writing doc | 22:49 |
| TheJulia | and... it worked | 22:51 |
| *** ijw has joined #openstack-ironic | 22:51 | |
| * TheJulia dances | 22:51 | |
| * rloo cheers for TheJulia | 22:51 | |
| *** piet has quit IRC | 22:53 | |
| *** piet has joined #openstack-ironic | 22:53 | |
| TheJulia | rloo: I've basically got it calling power and boot validations before firing off prepare and takeover which if things are missing from what they need based on the config, things will blow up and the node ends up in adoption failed | 22:54 |
| rloo | TheJulia: sweet! | 22:55 |
| *** ijw has quit IRC | 22:55 | |
| *** rloo is now known as rloo_away | 22:56 | |
| TheJulia | sadly, I don't have every single driver, so I can't test them all, but ipmi seems to work like a champ :) | 22:56 |
| *** lucasagomes_ has quit IRC | 22:56 | |
| *** lucasagomes has joined #openstack-ironic | 23:03 | |
| TheJulia | oh, and disable setting the next boot device by default... that might be a bug on _do_takeover | 23:05 |
| *** e0ne has joined #openstack-ironic | 23:19 | |
| *** baoli has quit IRC | 23:20 | |
| *** MattMan has quit IRC | 23:25 | |
| *** MattMan has joined #openstack-ironic | 23:25 | |
| *** e0ne has quit IRC | 23:28 | |
| *** lucasagomes has quit IRC | 23:35 | |
| gmmaha | TheJulia: if you dont mind, had a quick Q.. i am using ubuntu cloud image 14.04 and it has cloud-init but config drive isnt working. thats the logs jlvillal shared a while back | 23:39 |
| gmmaha | does DIB_CLOUD_INIT_DATASOURCES have any effect on cloud-init or is that an option for just simple-init? | 23:39 |
| gmmaha | duh, it sys cloud init.. :) | 23:40 |
| TheJulia | simple-init basically turns off cloud-init by setting the datasources to none | 23:40 |
| *** lucasagomes has joined #openstack-ironic | 23:40 | |
| gmmaha | we want to configure network interface and the hostname and thats not happening.. hence the hunt to see what will let us do that on a ubuntu image via configdrive | 23:40 |
| TheJulia | gmmaha: have you seen https://github.com/openstack/diskimage-builder/tree/master/elements/cloud-init-datasources#environment-variables ? | 23:41 |
| TheJulia | gmmaha: and your making custom configdrive filess? | 23:42 |
| gmmaha | TheJulia: yes, we just set that to value "ConfigDrive" and re-created the dib image.. | 23:42 |
| gmmaha | TheJulia: at this point, no.. its just the basic network settings (metadata.json) file that bifrost creates | 23:42 |
| gmmaha | nothing special or different | 23:42 |
| jlvillal | TheJulia: Basically our simple goal is to setup the network and hostname. We are trying to figure out how we should do that. | 23:42 |
| TheJulia | okay | 23:42 |
| TheJulia | so... my immediate thought is to validate that your metadata.json file parses properly | 23:44 |
| TheJulia | and what I mean by that is that it is valid json | 23:44 |
| TheJulia | cloud-init, if memory serves, is horrible at actuaally logging when there is a problem | 23:44 |
| TheJulia | with the data it parses | 23:44 |
| *** Sukhdev has quit IRC | 23:45 | |
| TheJulia | so, best thing to do is to crack open one of your config drive files that were generated, do your best ducky impression, and go from there :) | 23:45 |
| jlvillal | TheJulia: Thanks. Did: | 23:47 |
| jlvillal | >>> fd = open('openstack/latest/meta_data.json') | 23:47 |
| jlvillal | >>> json.load(fd) | 23:47 |
| jlvillal | And success | 23:47 |
| jlvillal | We didn't see the config-drive being mounted though. | 23:48 |
| TheJulia | burn a new image with dev_user element (look it up first) and examine the deployed machine? | 23:48 |
| jlvillal | If using simple-init shoudl it mount it? | 23:48 |
| TheJulia | yeah, it hides the actual mount operation | 23:48 |
| TheJulia | and then unmounts it | 23:48 |
| jlvillal | Nothing in the log file?? | 23:49 |
| TheJulia | simple-init will mount it, then if memory serves unmount it | 23:49 |
| TheJulia | yup, nothing in the log | 23:49 |
| TheJulia | it is actually looking for a disk label config-2 | 23:49 |
| *** Sukhdev has joined #openstack-ironic | 23:49 | |
| jlvillal | :( That makes debugging a little hard. | 23:49 |
| TheJulia | yeah, devuser and dhcp-all-interfaces can be very helpful | 23:50 |
| jlvillal | TheJulia: So /var/log/cloud-init.log is the log file we should look at. I hope. | 23:50 |
| TheJulia | jlvillal: yes, also /var/lib/cloud folder and... see if a blkid lists config-2 | 23:51 |
| jlvillal | TheJulia: yes we do have the 'config-2'. | 23:51 |
| TheJulia | jlvillal: also, out of curiousity, if your willing to share, your dib command line and env vars, I might see something obvious | 23:51 |
| jlvillal | TheJulia: This worked for us: mount /dev/disk/by-label/config-2 /mnt/config | 23:51 |
| *** ijw has joined #openstack-ironic | 23:51 | |
| jlvillal | gmmaha: is looking... | 23:52 |
| TheJulia | so if you clean out the lock files, I think you can just run it by hand, but it will replace the ssh host key | 23:52 |
| * TheJulia takes mr. corgi out since the $bar_of_food_and_beverage shoudl begin to empty out soon | 23:53 | |
| jlvillal | :) | 23:53 |
| NobodyCam | :p | 23:53 |
| *** boris-42 has quit IRC | 23:53 | |
| *** ijw has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!