| *** baoli has quit IRC | 00:00 | |
| *** wajdi has quit IRC | 00:02 | |
| *** wajdi has joined #openstack-ironic | 00:03 | |
| *** sdake has joined #openstack-ironic | 00:16 | |
| *** rbudden has quit IRC | 00:16 | |
| *** hoangcx2 has quit IRC | 00:38 | |
| *** hoangcx has joined #openstack-ironic | 00:50 | |
| *** adnavare has joined #openstack-ironic | 01:01 | |
| *** aNupoisc has joined #openstack-ironic | 01:01 | |
| *** sdake_ has joined #openstack-ironic | 01:06 | |
| *** sdake has quit IRC | 01:10 | |
| *** sdake_ has quit IRC | 01:19 | |
| *** sdake has joined #openstack-ironic | 01:22 | |
| *** baoli has joined #openstack-ironic | 01:28 | |
| *** moshele has joined #openstack-ironic | 01:37 | |
| *** zhangyufei has joined #openstack-ironic | 01:42 | |
| *** zhangyufei has left #openstack-ironic | 01:44 | |
| openstackgerrit | Merged openstack/ironic-inspector: Use upper constraints for all jobs in tox.ini https://review.openstack.org/358564 | 01:53 |
|---|---|---|
| *** zhangjl has joined #openstack-ironic | 02:00 | |
| *** mtanin___ has quit IRC | 02:07 | |
| *** moshele has quit IRC | 02:07 | |
| *** yuanying has quit IRC | 02:10 | |
| *** yuikotakadamori has joined #openstack-ironic | 02:13 | |
| *** yuikotakadamori has quit IRC | 02:14 | |
| *** baoli has quit IRC | 02:15 | |
| Madasi | quit | 02:19 |
| Madasi | misfire, sorry | 02:20 |
| *** wajdi has quit IRC | 02:32 | |
| *** wajdi has joined #openstack-ironic | 03:24 | |
| *** wajdi has quit IRC | 03:31 | |
| *** Goneri has quit IRC | 03:31 | |
| *** vikrant has joined #openstack-ironic | 03:34 | |
| *** vikrant is now known as vikrant|brb | 03:34 | |
| *** vikrant|brb is now known as vikrant | 03:47 | |
| *** rloo has quit IRC | 03:49 | |
| *** vmud213 has joined #openstack-ironic | 03:53 | |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic: Merge imports in code https://review.openstack.org/357597 | 03:58 |
| *** yuanying has joined #openstack-ironic | 03:59 | |
| *** adnavare has quit IRC | 04:04 | |
| *** aNupoisc has quit IRC | 04:04 | |
| *** Jaison has joined #openstack-ironic | 04:15 | |
| *** jhooker has quit IRC | 04:16 | |
| *** jhooker has joined #openstack-ironic | 04:17 | |
| *** Jaison has quit IRC | 04:24 | |
| *** jraju has joined #openstack-ironic | 04:26 | |
| *** baoli has joined #openstack-ironic | 04:27 | |
| *** sabeen has quit IRC | 04:29 | |
| *** baoli has quit IRC | 04:31 | |
| *** vmud213 has quit IRC | 04:37 | |
| *** jtomasek has quit IRC | 04:42 | |
| *** hoangcx has quit IRC | 04:44 | |
| *** hoangcx has joined #openstack-ironic | 04:48 | |
| *** fragatina has quit IRC | 04:59 | |
| *** Sukhdev has joined #openstack-ironic | 05:05 | |
| *** Sukhdev has quit IRC | 05:07 | |
| *** Sukhdev has joined #openstack-ironic | 05:07 | |
| openstackgerrit | Merged openstack/ironic: Centralize config options - [iscsi] https://review.openstack.org/309615 | 05:08 |
| *** jaosorior has joined #openstack-ironic | 05:10 | |
| *** vishwanathj has quit IRC | 05:17 | |
| *** vmud213 has joined #openstack-ironic | 05:18 | |
| *** sdake_ has joined #openstack-ironic | 05:21 | |
| *** moshele has joined #openstack-ironic | 05:24 | |
| *** sdake has quit IRC | 05:24 | |
| *** vmud213 has quit IRC | 05:24 | |
| *** vmud213 has joined #openstack-ironic | 05:30 | |
| *** syed_ has quit IRC | 05:32 | |
| *** sdake_ has quit IRC | 05:41 | |
| *** Sukhdev has quit IRC | 05:48 | |
| *** vishwanathj has joined #openstack-ironic | 05:55 | |
| *** hoangcx has quit IRC | 06:04 | |
| *** fragatina has joined #openstack-ironic | 06:04 | |
| *** fragatina has quit IRC | 06:05 | |
| *** vsaienko has joined #openstack-ironic | 06:06 | |
| *** fragatina has joined #openstack-ironic | 06:07 | |
| *** fragatina has quit IRC | 06:07 | |
| *** fragatina has joined #openstack-ironic | 06:09 | |
| *** vsaienko has quit IRC | 06:12 | |
| *** vsaienko has joined #openstack-ironic | 06:12 | |
| *** fragatina has quit IRC | 06:14 | |
| *** pcaruana has joined #openstack-ironic | 06:14 | |
| *** vsaienko has quit IRC | 06:20 | |
| *** vishwanathj is now known as vishwanathj_zzz | 06:24 | |
| *** hoangcx has joined #openstack-ironic | 06:25 | |
| *** vsaienko has joined #openstack-ironic | 06:27 | |
| openstackgerrit | Ankit Kumar proposed openstack/ironic: Fix iLO drivers temporarily for local_gb https://review.openstack.org/341752 | 06:31 |
| *** wajdi has joined #openstack-ironic | 06:32 | |
| *** vsaienko has quit IRC | 06:34 | |
| *** hoangcx has quit IRC | 06:36 | |
| *** vsaienko has joined #openstack-ironic | 06:36 | |
| *** mjura has joined #openstack-ironic | 06:59 | |
| openstackgerrit | Rodion Promyshlennikov proposed openstack/python-ironicclient: Add baremetal port delete command to OSC plugin https://review.openstack.org/346075 | 07:02 |
| *** vsaienko has quit IRC | 07:12 | |
| *** milan has joined #openstack-ironic | 07:12 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for port-related commands https://review.openstack.org/259354 | 07:13 |
| *** Lord_Phoenix has joined #openstack-ironic | 07:17 | |
| *** tesseract- has joined #openstack-ironic | 07:18 | |
| *** vsaienko has joined #openstack-ironic | 07:18 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with port. https://review.openstack.org/259354 | 07:24 |
| *** ohamada has joined #openstack-ironic | 07:26 | |
| *** jpich has joined #openstack-ironic | 07:27 | |
| *** vsaienko has quit IRC | 07:33 | |
| *** vsaienko has joined #openstack-ironic | 07:33 | |
| mrda | hey, so our gate jobs that use virtualbmc - do they PXE boot QEMU images? Or are we deploying another way? | 07:36 |
| *** ifarkas_afk is now known as ifarkas | 07:42 | |
| *** eus has joined #openstack-ironic | 07:43 | |
| *** eus has quit IRC | 07:44 | |
| *** moshele has left #openstack-ironic | 07:44 | |
| *** moshele has joined #openstack-ironic | 07:44 | |
| *** e0ne has joined #openstack-ironic | 07:45 | |
| *** vsaienko has quit IRC | 07:47 | |
| *** mpsairam has quit IRC | 07:48 | |
| *** vsaienko has joined #openstack-ironic | 07:48 | |
| *** vsaienko has quit IRC | 07:56 | |
| *** rcernin has quit IRC | 07:56 | |
| *** hoangcx has joined #openstack-ironic | 07:57 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** vsaienko has joined #openstack-ironic | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:01 | |
| *** openstackgerrit has quit IRC | 08:03 | |
| *** openstackgerrit has joined #openstack-ironic | 08:04 | |
| *** yuanying has quit IRC | 08:05 | |
| *** jtomasek has joined #openstack-ironic | 08:07 | |
| *** vmud213 has quit IRC | 08:12 | |
| *** Goneri has joined #openstack-ironic | 08:13 | |
| *** athomas has joined #openstack-ironic | 08:14 | |
| *** vsaienko has quit IRC | 08:15 | |
| vdrok | morning ironic, good evening mrda | 08:15 |
| *** lucas-dinner is now known as lucasagomes | 08:15 | |
| vdrok | mrda: I think the process is the same as for ssh drivers | 08:15 |
| lucasagomes | morning | 08:15 |
| vdrok | morning lucasagomes | 08:15 |
| *** derekh has joined #openstack-ironic | 08:18 | |
| *** ohamada_ has joined #openstack-ironic | 08:20 | |
| *** ohamada has quit IRC | 08:20 | |
| *** fragatina has joined #openstack-ironic | 08:22 | |
| mrda | thanks vdrok | 08:25 |
| *** rbartal has quit IRC | 08:26 | |
| *** rbartal has joined #openstack-ironic | 08:26 | |
| *** vmud213 has joined #openstack-ironic | 08:30 | |
| vsaienk01 | Morning vdrok, lucasagomes, sambetts and all Ironickers! | 08:34 |
| lucasagomes | vsaienk01, hi there | 08:34 |
| lucasagomes | mrda, evening :D | 08:34 |
| *** vmud213 has quit IRC | 08:36 | |
| pas-ha | morning Ironicers :) | 08:36 |
| *** yogi has joined #openstack-ironic | 08:37 | |
| *** vmud213 has joined #openstack-ironic | 08:37 | |
| mrda | hey lucasagomes! | 08:37 |
| mrda | lucasagomes: So I'd like to use virtualbmc to PXE boot QEMU instances. Am I crazy? | 08:38 |
| lucasagomes | mrda, maybe ? heh jk, vbmc you can use to power on/off the instances and change the boot device | 08:39 |
| lucasagomes | so it may fit your case yeah | 08:39 |
| * mrda wonders whether anyone has used agent_ipmitool with virtualbmc | 08:40 | |
| mrda | and got it working... | 08:41 |
| *** ifarkas has quit IRC | 08:41 | |
| mrda | so our gate jobs just use ssh to deploy then? | 08:41 |
| * mrda hasn't looked at that | 08:41 | |
| *** ifarkas has joined #openstack-ironic | 08:42 | |
| lucasagomes | mrda, there are jobs in gate for agent_ and pxe_ with vbmc | 08:42 |
| lucasagomes | mrda, gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-nv | 08:43 |
| mrda | oh, ok. So at least for some subset of configurations, vbmc can pxe book. Thanks! I just need to perservere to get it working | 08:44 |
| mrda | (and read what others have done in that gate job) | 08:44 |
| mrda | Thanks yuou, much appreciated! | 08:44 |
| lucasagomes | mrda, yw :D | 08:44 |
| openstackgerrit | hubian proposed openstack/ironic-specs: input a spec for Ironic XClarity Power driver https://review.openstack.org/359050 | 08:50 |
| *** vmud213 has quit IRC | 08:54 | |
| *** vmud213 has joined #openstack-ironic | 08:55 | |
| *** electrofelix has joined #openstack-ironic | 09:00 | |
| *** ifarkas_ has joined #openstack-ironic | 09:02 | |
| openstackgerrit | Jacek Tomasiak proposed openstack/ironic: Removed unneeded vlan settings from neutron config https://review.openstack.org/354685 | 09:05 |
| sambetts|afk | Morning all | 09:14 |
| *** sambetts|afk is now known as sambetts | 09:14 | |
| *** d0ugal has quit IRC | 09:15 | |
| *** d0ugal has joined #openstack-ironic | 09:16 | |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Switch to admin_token auth for standalone mode https://review.openstack.org/359061 | 09:16 |
| sambetts | vsaienk01: ping | 09:16 |
| *** milan has quit IRC | 09:17 | |
| vdrok | pas-ha: sambetts morning | 09:17 |
| sambetts | o/ vdrok | 09:17 |
| *** aarefiev_ is now known as aarefiev | 09:19 | |
| aarefiev | morning | 09:20 |
| vdrok | morning aarefiev :) | 09:20 |
| aarefiev | hey vdrok | 09:20 |
| openstackgerrit | Rodion Promyshlennikov proposed openstack/python-ironicclient: Add baremetal port list command to OSC plugin https://review.openstack.org/346722 | 09:20 |
| *** hoangcx has left #openstack-ironic | 09:21 | |
| *** vsaienko has joined #openstack-ironic | 09:22 | |
| openstackgerrit | hubian proposed openstack/ironic-specs: add missed sections and subsections https://review.openstack.org/359067 | 09:30 |
| *** Goneri has quit IRC | 09:30 | |
| *** Goneri has joined #openstack-ironic | 09:33 | |
| *** athomas has quit IRC | 09:34 | |
| *** tangchen has quit IRC | 09:36 | |
| *** tangchen has joined #openstack-ironic | 09:37 | |
| openstackgerrit | Rodion Promyshlennikov proposed openstack/python-ironicclient: Verify JSON response of chassis commands https://review.openstack.org/336596 | 09:38 |
| *** lucasagomes is now known as lucas-afk | 09:40 | |
| *** vsaienko has quit IRC | 09:40 | |
| *** athomas has joined #openstack-ironic | 09:40 | |
| *** amotoki has joined #openstack-ironic | 09:40 | |
| *** dtantsur|afk is now known as dtantsur | 09:42 | |
| dtantsur | Morning Ironic | 09:42 |
| sambetts | Hey dtantsur | 09:42 |
| *** vsaienko has joined #openstack-ironic | 09:46 | |
| *** vsaienko has quit IRC | 09:47 | |
| openstackgerrit | hubian proposed openstack/ironic-specs: fix https://review.openstack.org/359075 | 09:51 |
| *** tariqne has joined #openstack-ironic | 09:51 | |
| *** amotoki has quit IRC | 09:51 | |
| *** ifarkas has quit IRC | 09:54 | |
| *** ifarkas_ is now known as ifarkas | 09:54 | |
| aarefiev | dtantsur: hey, looks like we need to backport https://review.openstack.org/#/c/339457/ to mitaka | 09:59 |
| dtantsur | aarefiev, we can't, it requires g-r bump... | 09:59 |
| aarefiev | dtantsur, ahh looking on your patch http://logs.openstack.org/73/358673/1/check/gate-ironic-inspector-python34-db/1601a72/console.html.gz | 10:00 |
| dtantsur | we probably need to start the wide conversation | 10:00 |
| dtantsur | maybe we need to use u-c, lemme try | 10:00 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Make tox respect upper-constraints.txt https://review.openstack.org/359083 | 10:02 |
| dtantsur | aarefiev, let us see if this helps ^^^ | 10:02 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Switch to admin_token auth for standalone mode https://review.openstack.org/359061 | 10:02 |
| vdrok | morning dtantsur | 10:02 |
| dtantsur | hey vdrok | 10:02 |
| aarefiev | dtantsur: stevedore===1.12.0 in u-c | 10:03 |
| dtantsur | and which one do we need? | 10:04 |
| aarefiev | looks like 1.15.0 | 10:04 |
| dtantsur | no, I mean, which is minimal that used to work? | 10:04 |
| dtantsur | (in case you know) | 10:04 |
| aarefiev | I'm not sure but 15 is minimal I guess | 10:06 |
| aarefiev | lets see what jenkins says | 10:06 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Switch to admin_token auth for standalone mode https://review.openstack.org/359061 | 10:08 |
| vsaienk01 | morning dtantsur! | 10:12 |
| *** vmud213 has quit IRC | 10:12 | |
| dtantsur | morning vsaienk01 | 10:13 |
| *** vmud213 has joined #openstack-ironic | 10:13 | |
| *** ccamacho is now known as ccamacho|afk | 10:15 | |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic-inspector: Add PCI devices plugin to inspector https://review.openstack.org/352295 | 10:16 |
| aarefiev | dtantsur: works, yeah that make sense, versions before 1.15.0 should work | 10:27 |
| dtantsur | lucas-afk, when you're no longer afk, please check if https://review.openstack.org/359099 makes sense to you | 10:31 |
| *** vsaienk01 has left #openstack-ironic | 10:35 | |
| *** vsaienk01 has joined #openstack-ironic | 10:36 | |
| *** vsaienko2 has joined #openstack-ironic | 10:37 | |
| *** vsaienko2 has left #openstack-ironic | 10:37 | |
| *** vsaienk01 has left #openstack-ironic | 10:37 | |
| *** vmud213 has quit IRC | 10:41 | |
| lazy_prince | lucas-afk: dtantsur: can we merge https://review.openstack.org/#/c/354186/ given that https://review.openstack.org/#/c/354608/ has already merged.. | 10:41 |
| *** amotoki has joined #openstack-ironic | 10:42 | |
| openstackgerrit | avnish proposed openstack/ironic-inspector: Fix tempest.conf generation https://review.openstack.org/359109 | 10:43 |
| *** vmud213 has joined #openstack-ironic | 10:47 | |
| lucas-afk | dtantsur, will do, I'm almost setup now | 10:47 |
| openstackgerrit | avnish proposed openstack/ironic-inspector: Fix tempest.conf generation https://review.openstack.org/359109 | 10:52 |
| *** amotoki has quit IRC | 10:53 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Adds node boot device & passthu OSC commands https://review.openstack.org/357400 | 10:57 |
| *** milan has joined #openstack-ironic | 11:04 | |
| dtantsur | lucas-afk, let's see if it passes CI first | 11:04 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Make tox respect upper-constraints.txt https://review.openstack.org/359083 | 11:08 |
| *** ccamacho|afk is now known as ccamacho | 11:09 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Make tox respect upper-constraints.txt https://review.openstack.org/359083 | 11:10 |
| dtantsur | aarefiev, now it must be correct ^^^ | 11:11 |
| *** mjura_ has joined #openstack-ironic | 11:14 | |
| aarefiev | dtantsur: looks like it can resolve branches, because mitaka version was installed | 11:17 |
| dtantsur | aarefiev, zuul can, but locally it won't work | 11:17 |
| aarefiev | yep, just didn't know that | 11:18 |
| *** dprince has joined #openstack-ironic | 11:29 | |
| aarefiev | dtantsur: I'm wondering should we do this for client as well | 11:29 |
| dtantsur | likely | 11:29 |
| dtantsur | sambetts, if you have some time, could you please check inspector-related patch https://review.openstack.org/#/c/357963/ | 11:33 |
| *** tariqne has quit IRC | 11:33 | |
| openstackgerrit | Merged openstack/ironic-inspector: Make tox respect upper-constraints.txt https://review.openstack.org/359083 | 11:35 |
| * dtantsur backports to liberty now | 11:36 | |
| *** jtomasek has quit IRC | 11:37 | |
| *** moshele has quit IRC | 11:38 | |
| *** bfournie has quit IRC | 11:40 | |
| *** moshele has joined #openstack-ironic | 11:41 | |
| *** ccamacho is now known as ccamacho|lunch | 11:41 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Make tox respect upper-constraints.txt https://review.openstack.org/359148 | 11:42 |
| dtantsur | aarefiev, liberty ^^^ | 11:42 |
| *** amotoki has joined #openstack-ironic | 11:42 | |
| *** rcernin has joined #openstack-ironic | 11:43 | |
| *** vmud213 has quit IRC | 11:48 | |
| *** jtomasek has joined #openstack-ironic | 11:49 | |
| *** vmud213 has joined #openstack-ironic | 11:49 | |
| *** jaosorior has quit IRC | 11:50 | |
| *** jaosorior has joined #openstack-ironic | 11:51 | |
| *** wajdi has quit IRC | 11:55 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fix invalid chassis/nodes/chassis_uuid API url https://review.openstack.org/316149 | 11:55 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Fix all possible node subresources routing collisions https://review.openstack.org/315766 | 11:55 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Prevent URL collisions with sub-controllers: nodes/ports https://review.openstack.org/314514 | 11:55 |
| *** vmud213 has quit IRC | 11:55 | |
| *** wajdi has joined #openstack-ironic | 11:55 | |
| *** Goneri has quit IRC | 11:57 | |
| jaosorior | dtantsur: hey, can you take a look at https://review.openstack.org/#/c/358457/ ? I still need to fix py3X issues. But would still be nice to get some feedback | 12:00 |
| dtantsur | jaosorior, sure, will get to it today | 12:00 |
| jaosorior | dtantsur: thanks | 12:00 |
| jaosorior | devananda, JayF: Some days ago I came with the issue about not being able to rebuild the ironic-python-agent image due to support agreements. Well, that's an approach that might help in this use-case ^^ | 12:02 |
| *** Goneri has joined #openstack-ironic | 12:04 | |
| openstackgerrit | Merged openstack/ironic-inspector: Make tox respect upper-constraints.txt https://review.openstack.org/359148 | 12:07 |
| *** rbudden has joined #openstack-ironic | 12:09 | |
| *** phschwartz has quit IRC | 12:14 | |
| *** zhangjl has quit IRC | 12:15 | |
| *** rcernin has quit IRC | 12:23 | |
| *** mjura_ has quit IRC | 12:23 | |
| *** MattMan has quit IRC | 12:26 | |
| *** MattMan has joined #openstack-ironic | 12:26 | |
| *** rcernin has joined #openstack-ironic | 12:28 | |
| *** cdearborn has joined #openstack-ironic | 12:29 | |
| *** jcoufal has joined #openstack-ironic | 12:32 | |
| *** moshele has quit IRC | 12:33 | |
| wznoinsk | hi all | 12:36 |
| wznoinsk | could someone have a look at https://bugs.launchpad.net/ironic-inspector/+bug/1616057 and comment | 12:37 |
| openstack | Launchpad bug 1616057 in Ironic Inspector ""Unknown column 'nodes.finished_at' in 'where clause'"" [Undecided,New] - Assigned to Waldemar Znoinski (wznoinsk) | 12:37 |
| *** phschwartz has joined #openstack-ironic | 12:38 | |
| *** bfournie has joined #openstack-ironic | 12:39 | |
| *** vikrant has quit IRC | 12:41 | |
| *** moshele has joined #openstack-ironic | 12:43 | |
| *** causten has joined #openstack-ironic | 12:43 | |
| xavierr | good morning Ironic :) | 12:45 |
| *** dtantsur is now known as dtantsur|mtg | 12:50 | |
| *** baoli has joined #openstack-ironic | 12:54 | |
| *** chlong has quit IRC | 12:56 | |
| *** vishwanathj_zzz is now known as vishwanathj | 12:57 | |
| *** wajdi has quit IRC | 13:00 | |
| devananda | g'morning, folks | 13:00 |
| dtantsur|mtg | morning devananda | 13:01 |
| *** devananda is now known as devananda|OSE | 13:01 | |
| *** causten has quit IRC | 13:02 | |
| *** rcernin has quit IRC | 13:03 | |
| *** lucas-afk is now known as lucas-hungry | 13:12 | |
| *** chlong has joined #openstack-ironic | 13:12 | |
| *** ccamacho|lunch is now known as ccamacho | 13:21 | |
| openstackgerrit | Merged openstack/python-ironic-inspector-client: Use constraints for all the things https://review.openstack.org/318693 | 13:23 |
| sambetts | mjturek1: ping | 13:27 |
| mjturek1 | hey Sam, just reading the comments now | 13:27 |
| mjturek1 | what's up | 13:27 |
| sambetts | mjturek1: just wanted to make sure you see the comments on the portgroups patch, we need to decide how we want to handle portgroups in terms of if they are enforced or not | 13:28 |
| mjturek1 | sambetts: sorry on a call, I had assumed it was the way you said in case b (in your last comment). If so I would think your suggestion would be the right way to go. vdrok, have you seen the comments? | 13:33 |
| TheJulia | be afraid, I am now licensed to operate vehicles >26,000 pounds or ~15,000 kg :) | 13:35 |
| TheJulia | And Good morning everyone! | 13:35 |
| devananda|OSE | TheJulia: congrats! and good morning :) | 13:35 |
| *** sdake has joined #openstack-ironic | 13:36 | |
| *** sdake_ has joined #openstack-ironic | 13:37 | |
| vdrok | morning xavierr devananda|OSE TheJulia and mjturek1 | 13:37 |
| mjturek1 | morning :) | 13:37 |
| vdrok | sambetts: mjturek1 so https://review.openstack.org/#/c/357222/2 is the way to go for now? not https://review.openstack.org/#/c/206163/? | 13:38 |
| *** jcoufal_ has joined #openstack-ironic | 13:38 | |
| vdrok | in case of nova, we'll then fail if there are any portgroups defined, if we're adding vif_port_ids manually, that means we kind of know what we do, why forbid it? | 13:40 |
| *** [1]cdearborn has joined #openstack-ironic | 13:40 | |
| *** dprince has quit IRC | 13:40 | |
| *** jraju has quit IRC | 13:40 | |
| *** jcoufal has quit IRC | 13:40 | |
| mjturek1 | vdrok: so portgroups in ironic are representing what's possible rather than what's enforced? | 13:40 |
| *** sdake has quit IRC | 13:41 | |
| *** dprince has joined #openstack-ironic | 13:41 | |
| vdrok | mjturek1: I thought so, lemme grab vasyl here too | 13:41 |
| mjturek1 | cool, thanks :) | 13:42 |
| *** causten has joined #openstack-ironic | 13:42 | |
| *** vsaienk0 has joined #openstack-ironic | 13:43 | |
| vdrok | vsaienk0: so wdyt regarding the last sambetts' comment on the portgroups patch? | 13:43 |
| sambetts | vdrok: 357222 was created because we can't merge the full nova support for portgroups in Newton | 13:43 |
| mjturek1 | right | 13:43 |
| sambetts | vdrok: it will be superceded by adding the full support | 13:43 |
| vdrok | yep, gotcha | 13:44 |
| sambetts | vdrok: the question is how do we handle portgroups when Ironic supports them but nova doesn't | 13:44 |
| *** thiagop has joined #openstack-ironic | 13:44 | |
| thiagop | Good morning, Ironicers! | 13:44 |
| vdrok | in the patch adding the support we filter out the ports that have portgroup set | 13:44 |
| thiagop | Good morning, Ironicers! | 13:44 |
| sambetts | vdrok: yes, so portgroups will always be used | 13:45 |
| vdrok | but it does not handle the case when portgroup mac equals to one of the ports mac | 13:45 |
| *** mgoddard_ has joined #openstack-ironic | 13:45 | |
| sambetts | so the options are do we enforce that if a node has portgroups, then you can't deploy it through nova | 13:46 |
| sambetts | or do we fall back to using the nics as single nics | 13:46 |
| vsaienk0 | I think that vif_port id should be set on portgroup from Nova, there is no need to prohibit setting vift_port_id on port that is member of portgoup | 13:46 |
| *** sabeen has joined #openstack-ironic | 13:46 | |
| vsaienk0 | sambetts: If we just prohibit setting vif_port_id on port that is member of portgroup we will have a failed deployment | 13:47 |
| sambetts | but whether or not the nic will works as a single nic depends on the infrastrucutre | 13:47 |
| vsaienk0 | nova should be portgroup aware, and it should know when exactly it is working with portgroups | 13:47 |
| sambetts | vsaienk0: nova can't be portgroup aware in newton that is the problem | 13:48 |
| sambetts | vsaienk0: because we want to merge the code in Ironic | 13:48 |
| *** Lord_Phoenix has quit IRC | 13:48 | |
| vsaienk0 | sambetts: have you checked my demo related to static portgroups? | 13:49 |
| *** mgoddard has quit IRC | 13:49 | |
| vdrok | sambetts: hm, I think I still don't get it, the problem is not in ironic patch but in nova's - all the changes should go there anyway I think | 13:49 |
| vdrok | and in ironic we still allow to set vif_port_id on port in a portgroup because we don't know the infrastructure | 13:50 |
| vdrok | and it may work | 13:50 |
| vdrok | do I get that correctly? :) | 13:50 |
| sambetts | vdrok, vsaienk0: The problem is this: we want to merge the code to support portgroups in Ironic, but we can't merge the code to support portgrups in nova in Newton, so Ironic will have them, and be able to use them, but Nova won't | 13:50 |
| vsaienk0 | sambetts: even if all node ports are memebers of portgroups we shouldn't block using this node for instances without portgroups | 13:50 |
| sambetts | vsaienk0: but what if that nodes portgroups can't fall back to working indivisually? | 13:51 |
| vsaienk0 | sambetts: what is the benefit of landing current portgroup code? | 13:51 |
| mjturek1 | vsaienk0: it'll work with stand alone ironic | 13:51 |
| sambetts | and it speeds up the development for the next cycle, because we only have to focus on the Nova part | 13:52 |
| vsaienk0 | sambetts: if portgroup can't fallback to stand alone mode, we should have a dedicated nic for provisioning | 13:52 |
| vsaienk0 | sambetts: in my understanding portgroup interfaces should fall-back to single interface mode if on another side is non portgroup aware device | 13:53 |
| *** cdearborn has quit IRC | 13:53 | |
| sambetts | vsaienk0: that depends on the switch | 13:53 |
| vsaienk0 | sambetts: I think that depends on portgroup configuration, which is static at the moment | 13:53 |
| vdrok | sambetts: I think it's OK to fail for now, but not if the portgroup just exists, maybe let's check that there is at least one port not associated with portgroup | 13:54 |
| vsaienk0 | mjturek1:have you checked my demo related to portgroups, and how they work at the moment without any changes to Ironic? The same approach will work for standalone case | 13:54 |
| mjturek1 | vsaienk0: I did not! I'll definitely check it out | 13:54 |
| sambetts | vdrok: we can't without making nova portgroup aware and filtering them | 13:54 |
| vsaienk0 | sambetts: so I think we should clarify first if portgroup interfaces should fall-back or not | 13:55 |
| mjturek1 | vsaienk0: sambetts: is there a case where you can't fall back? | 13:55 |
| *** jhooker has quit IRC | 13:56 | |
| sambetts | mjturek1: in our switches its a setting, but when I spoke to people at the summit there are switches that can't | 13:56 |
| vsaienk0 | sambetts: my opinion is that we should always fall-back to single interface mode | 13:56 |
| mjturek1 | vsaienk0: but sambetts just pointed to cases where that won't work | 13:56 |
| *** jhooker has joined #openstack-ironic | 13:56 | |
| vsaienk0 | sambetts: so we want to implement feature for specific hardware? :) | 13:57 |
| vsaienk0 | not sure that it is a community way | 13:57 |
| sambetts | no, we want to support all hardware, which means lowest common denominator | 13:57 |
| *** dprince has quit IRC | 13:58 | |
| vsaienk0 | so supporting such hardware provisioning should always be done via single nic | 13:58 |
| *** dprince has joined #openstack-ironic | 13:58 | |
| sambetts | sure, I can have 3 nics, 1 single, and 2 in a bond, and what happens if someone does nova boot --nic <uuid> --nic <uuid> | 13:58 |
| sambetts | then current nova thinks that it has 3 ports, when in reality it only has 1 that will work in single mode | 13:59 |
| vdrok | ok I think I get it now | 13:59 |
| vsaienk0 | sambetts: I meant that wen ironic creates a port in provisioning network it should be always stand-alone port | 13:59 |
| sambetts | vsaienk0: which is marked by pxe_enabled | 14:00 |
| vsaienk0 | sambetts: so if some vendor doesn't support dynamic protgroups in theirs ML2 driver, why he can't just configure portgroups on the fly? | 14:00 |
| vsaienk0 | sambbets: sorry I meant doesn't support fallback to single interface mode | 14:00 |
| *** anush_ has joined #openstack-ironic | 14:00 | |
| *** fragatina has quit IRC | 14:01 | |
| sambetts | I guess it depends how configurable the switch is from the ml2 driver, unfortunatly I don't know the ins and outs of all switches | 14:02 |
| *** pcaruana has quit IRC | 14:02 | |
| vdrok | sambetts: what if we'll filter out the ports that have portgroup_uuid set on the api side if api microversion is < 1.23? | 14:03 |
| vsaienk0 | sambetts: if the switch is LACP capable it should support fall-back. I just want to make sure we are trying to fix real use-case... | 14:03 |
| *** syed_ has joined #openstack-ironic | 14:03 | |
| vsaienk0 | sambetts: another option why not just create a bootstrap that works with portgroup? | 14:05 |
| sambetts | vsaienk0: ? | 14:05 |
| TheJulia | I've seen switches advertised as LACP capable assume that port carrier being up means that the bond/grouping is working | 14:05 |
| vsaienk0 | if switch doesn't support fallback, user may create a ramdisk with portgroup support right? | 14:06 |
| TheJulia | I think the driver behind the feeling of need is similar experiences | 14:06 |
| sambetts | vsaienk0: pxe doesn't support bonding in any case | 14:06 |
| TheJulia | vsaienk0: user or operator | 14:06 |
| TheJulia | ? | 14:06 |
| vsaienk0 | sambetts: ah, you are right... | 14:06 |
| sambetts | vdrok: I thought about that but then it doesn't just hide them from nova it hides ports from all calls to list-ports | 14:06 |
| sambetts | I'm happy to allow fall back if we know we can support fall back in all cases, and if we're happy with all nodes in Newton continuing to run single nics even if configured with portgroups | 14:07 |
| vdrok | sambetts: yeah, that's a bit strange if you've created a port and then someone added it to a portgroup and it disappeared, but then if we document it properly maybe that's fine | 14:07 |
| vdrok | otoh, why not just bump the version on the nova side? is it considered too risky? | 14:08 |
| sambetts | thats what the patch that mjturek1 put up does, and that patch lists portgroups then fails the deploy if a node has portgroups | 14:09 |
| mjturek1 | vdrok: we thought it was strange to bump the API version just to show that a feature is unsupported | 14:09 |
| sambetts | also this ^ | 14:09 |
| vsaienk0 | sambetts: might be rise the question at openstack-dev about vendors who doesn't support a fall-back, and check if it is possible to configure portgroups dynamically by theirs ML2? | 14:10 |
| vdrok | well, if we'll filter out the ports associated with portgroups it would make a bit more sense | 14:10 |
| *** lucas-hungry is now known as lucasagomes | 14:10 | |
| vdrok | it will be not 'just to show that a feature is unsupported' but 'to make it work' | 14:10 |
| vdrok | in all cases I think | 14:10 |
| vsaienk0 | sambetts: I don't think it is a good idea to fail deployment, we should pick rather right node and right port | 14:11 |
| vsaienk0 | it may be done via flavor capabilities filtering | 14:11 |
| sambetts | vsaienk0: if we fail the deploy nova will reschedule until it lands on a node that works | 14:11 |
| vdrok | vsaienk0: I think it will be retried anyway | 14:11 |
| vdrok | yep | 14:11 |
| mjturek1 | sambetts: will retrying round robin to another available node? | 14:11 |
| sambetts | yup | 14:12 |
| mjturek1 | cool | 14:12 |
| vsaienk0 | sambetts: right but by default it is allowed only 3 times to reschedule failed build | 14:12 |
| vsaienk0 | and there is no guarantee that during this 3 times or higher right node will be picked | 14:12 |
| sambetts | vsaienk0: if an operator has configured Ironic with portgroups knowing that nova doesn't support it then thats configuration error right/ | 14:12 |
| sambetts | ? | 14:13 |
| sambetts | unless they are planning on using ironic in standalone and with nova at the same time | 14:13 |
| vsaienk0 | sambetts: it doesn't look like cloud case, it means that user is allowed to build instances with portgroups only | 14:13 |
| vsaienk0 | what if administrator enrolled half nodes with portgroups and rest nodes without portgroups? | 14:14 |
| sambetts | vsaienk0: well thats why we added the support on the Ironic side because it was meant to be operator enforced and not a user option | 14:14 |
| vsaienk0 | I think the goal is to support mixed environment | 14:15 |
| sambetts | vsaienk0: but if their nova deployment doesn't support it then they've configuration Ironic wrong | 14:15 |
| vsaienk0 | sambetts: why not filter nodes based on capabilities? | 14:15 |
| mjturek1 | vsaienk0: so if nova doesn't support portgroups, only schedule on nodes that don't have them? | 14:16 |
| vsaienk0 | if user request instance without portgroups it useses flavor that indicates nova to pick node without portgroups | 14:16 |
| vsaienk0 | and vice-verse, when user wants to spawn instance with portgroups it choses another flavor | 14:17 |
| *** pcaruana has joined #openstack-ironic | 14:17 | |
| sambetts | but there is no case for having a flavor that doesn't say don't use portgroups because nova can't handle it | 14:17 |
| sambetts | even if I ask for it | 14:17 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add tempest tests for node Adoption https://review.openstack.org/344975 | 14:18 |
| *** rama_y has joined #openstack-ironic | 14:18 | |
| mjturek1 | sambetts: right :( | 14:19 |
| *** rajinir has joined #openstack-ironic | 14:19 | |
| vsaienk0 | sambetts: another option is to wait when dynamic portgroups is supported | 14:20 |
| vdrok | vsaienk0: you mean not merging portgroups at all until that happens? | 14:20 |
| sambetts | vsaienk0: right, the other option is to not merge the code on the Ironic side, but Sukhdev wanted it landed | 14:20 |
| sambetts | the right way to do it, would be to get both the nova part and the Ironic part merged in the same cycle | 14:21 |
| vsaienk0 | sambetts: as far I know Arista supports lacp fallback :) | 14:21 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Documentation update https://review.openstack.org/356788 | 14:22 |
| vsaienk0 | the only one use-case that is not supported without landing portgroups code is MLAG. Not sure if Arista ML2 supports it | 14:22 |
| vdrok | sambetts: mjturek1 I think I'm OK with a version bump in nova if there is a need to merge the ironic-side changes earlier. but I'd prefer it not to only do the pg-list but to filter out the ports associated with portgroup | 14:23 |
| vsaienk0 | sambetts: so what do you think if we document that lacp fallback is known limitiation | 14:23 |
| vdrok | or that ^ | 14:23 |
| sambetts | vdrok: that then means that nodes that have all portgroups but support fallback won't get provisioned onto | 14:24 |
| vdrok | huh, then the only way I see is the doc | 14:25 |
| sambetts | vsaienk0: I still would like to find out from jroll, his opinion of whether portgroups should be enforced, but the reason we implmented it on the Ironic side and not as a user facing option is because we wanted it enforced by the operator | 14:25 |
| *** wajdi has joined #openstack-ironic | 14:26 | |
| sambetts | e.g. if as a datacenter manager I want this link configured and used as a bond then I configure it as such in Ironic, else I would just not configure it | 14:26 |
| sambetts | as a portgroup | 14:26 |
| *** dtantsur|mtg is now known as dtantsur | 14:26 | |
| *** adam_g has quit IRC | 14:29 | |
| *** jaosorior is now known as jaosorior_away | 14:32 | |
| vsaienk0 | sambetts: adding support of vendors that doesn't support lacp fallback at the moment adds a requirement to have a dedicated nic for provisioning. | 14:34 |
| vsaienk0 | Do we want to add this requirement to all clouds even if infra supports lacp fallback? | 14:34 |
| *** rloo has joined #openstack-ironic | 14:34 | |
| *** adam_g has joined #openstack-ironic | 14:34 | |
| *** adam_g has quit IRC | 14:34 | |
| *** adam_g has joined #openstack-ironic | 14:34 | |
| *** sdake_ has quit IRC | 14:35 | |
| *** davidlenwell has quit IRC | 14:36 | |
| *** davidlenwell has joined #openstack-ironic | 14:37 | |
| sambetts | thats not how I see it, if you have infra that supports LACP fallback then you can set one of your ports in the portgroup to the pxe_enabled port | 14:38 |
| *** sdake has joined #openstack-ironic | 14:41 | |
| *** fragatina has joined #openstack-ironic | 14:43 | |
| *** mjturek_phone has joined #openstack-ironic | 14:48 | |
| vsaienk0 | sambetts: so we should allow to set vif_port_id on pxe_enabled ports that are members of portgroups right? | 14:48 |
| vsaienk0 | sambetts: because I thought that you propose to block setting vif_port_id for any port that is member of portgoup | 14:48 |
| sambetts | vsaienk0: if we say that having pxe_enabled implys that the port can also be used as a single port then yes | 14:49 |
| sambetts | the other possiblity is to add a flag to ports that specfically states they can be used as both | 14:51 |
| sambetts | by both I mean in a bond or as a single | 14:51 |
| *** mjturek_phone has quit IRC | 14:51 | |
| sambetts | or we put a flag on the portgroup, e.g. supports_fallback | 14:52 |
| sambetts | then use that to decide | 14:52 |
| *** Goneri has quit IRC | 14:52 | |
| vsaienk0 | sambetts: I think that pxe_enabled flag is good for that | 14:52 |
| sambetts | and if someone tries to set vif_port_id on a port from a portgroup that doesn't support fallback then we fail | 14:52 |
| sambetts | vsaienk0: I don't like using the pxe_enabled flag for that because the pxe_enabled flag is used to decide which port to use for provisioning, and if you have multiple bonds that support fall back you might not want all of them to be choosen as the pxe port | 14:54 |
| sambetts | but you might want all of the ports to be used as tenant ports | 14:54 |
| vsaienk0 | sambetts: so might be it should be a portgroup option? fallback_support for example which indicates that all ports from this portgroup may be used for stand-alone mode? | 14:57 |
| sambetts | yeah thats what I think maybe best | 14:58 |
| sambetts | then we don't have to guess, the operator can tell us, and if it does support it or the ML2 driver can dynamically delete/recreate the bond on the switch then we can use the ports as individual | 14:59 |
| vsaienk0 | sambetts: it makes sense to me | 14:59 |
| sambetts | and if the portgroup a port belongs to doesn't support fallback then we say you can't use that port by itself | 14:59 |
| *** vmud213 has joined #openstack-ironic | 15:00 | |
| openstackgerrit | Vladyslav Drok proposed openstack/bifrost: Test admin_token keystoneauth loader https://review.openstack.org/359266 | 15:01 |
| *** mgoddard_ has quit IRC | 15:01 | |
| *** mgoddard has joined #openstack-ironic | 15:01 | |
| vsaienk0 | sambetts: yeah this approach looks reasonable. Because previously I thought that we just blocking setting vif_port_id for any port that is a member of portgroup... | 15:02 |
| *** rpioso has joined #openstack-ironic | 15:03 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Support names for {cleaning,provisioning}_network https://review.openstack.org/357983 | 15:06 |
| *** Goneri has joined #openstack-ironic | 15:07 | |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic: Merge imports in code https://review.openstack.org/357597 | 15:07 |
| *** jtomasek has quit IRC | 15:08 | |
| vsaienk0 | dtantsur: neutron network name is not unique, not sure that it is possible to support network name in the config | 15:09 |
| dtantsur | vsaienk0, we can fail, if it's not unique | 15:09 |
| *** _vsaienko has joined #openstack-ironic | 15:10 | |
| vsaienk0 | dtantsur: so we should prohibit users to create a network with ironic provision network name? | 15:10 |
| dtantsur | or not set names in this case | 15:11 |
| * dtantsur would avoid duplicate names, but sigh.. | 15:11 | |
| vsaienk0 | hm... So if someone from users created a network with the same name, it will lead to deployment failures? | 15:11 |
| dtantsur | I guess so | 15:11 |
| mariojv | yuriyz__: vdrok: vsaienk0: thanks for the reviews on https://review.openstack.org/#/c/298461/. have 2 +2s now, only needs workflow :) | 15:12 |
| dtantsur | there are always opportunities to screw up inter-process interaction | 15:12 |
| vsaienk0 | so what is the benefit having ability to identify network by name? | 15:12 |
| dtantsur | vsaienk0, 1. user-friendly, 2. installers-friendly | 15:12 |
| vsaienk0 | right, but there is no guarantee that it will always work | 15:13 |
| dtantsur | the latter is because you can't create a network before openstack is installed, unless the service allows you to provide UUID | 15:13 |
| dtantsur | and ironic soon will fail to start, if we don't provide an UUID (in some cases it already fails) | 15:14 |
| vsaienk0 | yeah it is a restriction, but in the same time we cant create database while db server is not configured :) | 15:14 |
| dtantsur | true, and now you require to provide an internal table ID from an uncreated database | 15:15 |
| dtantsur | (let's not play with analogies too much) | 15:15 |
| *** yuriyz__ has left #openstack-ironic | 15:15 | |
| *** mjura has quit IRC | 15:15 | |
| mariojv | JayF: i know you had a previous +2 on this: https://review.openstack.org/#/c/298461/ not much has changed since then, if you feel like landing something today | 15:15 |
| *** yuriyz has joined #openstack-ironic | 15:15 | |
| mariojv | JayF: i also updated the RFE you commented on, thanks for that https://bugs.launchpad.net/ironic/+bug/1611137 | 15:16 |
| openstack | Launchpad bug 1611137 in Ironic "[RFE] Allow specifying a maximum time allowed per clean step" [Undecided,New] - Assigned to Mario Villaplana (mario-villaplana-j) | 15:16 |
| mariojv | sambetts: was patch set 12 on https://review.openstack.org/#/c/316149/11..12 just a rebase? | 15:17 |
| sambetts | mariojv: more or less, it also changes the chassis_uuid parameter I added to parent_chassis_uuid to be more inline with the comments from the other patches | 15:19 |
| mariojv | thanks | 15:19 |
| vsaienk0 | dtantsur: by setting provision network name, we introducing a case when user may block of Ironic work by creating a network with the same name, but its a risk that administrator takes when configures ironic in such way.... | 15:21 |
| *** ifarkas is now known as ifarkas_afk | 15:21 | |
| mariojv | since this is a bug fix, i don't think a version bump is needed | 15:22 |
| * dtantsur grumbles at people who don't make names unique... | 15:22 | |
| *** _vsaienko has quit IRC | 15:22 | |
| *** baoli has quit IRC | 15:22 | |
| *** baoli has joined #openstack-ironic | 15:23 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Remove node update from conductor change_node_power_state() https://review.openstack.org/359287 | 15:23 |
| sambetts | vsaienk0, dtantsur: we should only search for a network with that name within a specfic tenant, and that tenant should be controlled by the admin, and if they create two networks with the same name then it's their fault | 15:24 |
| dtantsur | sambetts, fair, we use tenant "service", right? so we'll only see shared networks | 15:25 |
| vsaienk0 | sambetts: yeah, I just thought about it | 15:25 |
| *** davidlenwell has quit IRC | 15:27 | |
| *** _vsaienko has joined #openstack-ironic | 15:29 | |
| *** dprince has quit IRC | 15:30 | |
| JayF | What do folks think? Should https://bugs.launchpad.net/ironic/+bug/1611137 need a spec? | 15:32 |
| openstack | Launchpad bug 1611137 in Ironic "[RFE] Allow specifying a maximum time allowed per clean step" [Undecided,New] - Assigned to Mario Villaplana (mario-villaplana-j) | 15:32 |
| JayF | I'm leaning towards no, but I want more opinions before I approve it | 15:32 |
| *** dprince has joined #openstack-ironic | 15:32 | |
| JayF | jlvillal: tell Varun I said hello | 15:34 |
| jlvillal | JayF: I will. He is here in Oregon this week. vgadiraj ^^^ | 15:34 |
| JayF | yeah; jim and I have been mentoring him some w/osic | 15:35 |
| dtantsur | JayF, +1 for no spec | 15:35 |
| vdrok | morning JayF and jlvillal | 15:35 |
| vdrok | sambetts: JayF vsaienk0 please take a look if you have some time, have not updated them for a while - https://review.openstack.org/#/q/topic:ipa-partition-images | 15:36 |
| vdrok | changed a bit toda | 15:36 |
| vdrok | today | 15:36 |
| *** rbrud has joined #openstack-ironic | 15:36 | |
| JayF | it's open in my browser, will look in a bit | 15:37 |
| vdrok | thanks | 15:37 |
| JayF | Anyone wanna look at notifications before I land it? https://review.openstack.org/#/c/298461/27 +2s from me/vdrok/yuriy | 15:37 |
| *** davidlenwell has joined #openstack-ironic | 15:40 | |
| sambetts | JayF: is there any can were timeing out a clean step could result in a brick? | 15:42 |
| *** _vsaienko has quit IRC | 15:45 | |
| *** tesseract- has quit IRC | 15:46 | |
| JayF | sambetts: depending on operator action afterwards, sure | 15:46 |
| JayF | sambetts: if I have a node upgrading firmware, time it out too early, then do things after it times out | 15:46 |
| *** _vsaienko has joined #openstack-ironic | 15:47 | |
| JayF | sambetts: but if you know a firmware upgrade takes X seconds, timeout is X*3, and it goes off ... machine is probably already bricked :) | 15:47 |
| sambetts | JayF: true true, so its with great caution that we set these timeouts then | 15:48 |
| sambetts | JayF: also will it still be possible to have clean steps that don't have a timeout? | 15:48 |
| * sambetts has very little expereince playing with Ironic's clean steps | 15:49 | |
| JayF | hmm, idk if that's laid out in the rfe; but I'd imagine so | 15:51 |
| JayF | mariojv: ^ can you update the RFE to indicate no timeout specified means no timeout enforced? | 15:51 |
| mariojv | yes | 15:51 |
| jlvillal | dtantsur: Did you want to land this? https://review.openstack.org/353124 To make IPA work in stable/liberty. | 15:51 |
| mariojv | JayF: sambetts: done | 15:52 |
| mariojv | second paragraph from bottom | 15:52 |
| *** mmedvede has quit IRC | 15:53 | |
| *** _vsaienko has quit IRC | 15:53 | |
| vdrok | do we have a deprecation policy for clients like we do for services? Or is it just "if we remove something, major version bumps"? | 15:54 |
| dtantsur | jlvillal, done | 15:55 |
| rloo | vdrok: my assumption was that it followed the same deprecation policy as for ironic | 15:55 |
| dtantsur | vdrok, deprecation period, major version bump, release notes. yes, everything the same. | 15:55 |
| vdrok | rloo: https://governance.openstack.org/reference/tags/assert_follows-standard-deprecation.html | 15:55 |
| vdrok | hmm, OK, thanks | 15:55 |
| vdrok | that page lists only projects, not clients tho | 15:56 |
| *** mmedvede_ is now known as mmedvede | 15:56 | |
| sambetts | JayF: +1 for no spec on that RFE either, I think all my questions about it are covered there | 15:56 |
| *** pcaruana has quit IRC | 15:56 | |
| rloo | vdrok: is that the ironic project, which includes ironic, ironic-lib, ipa, etc, or is that ironic repo? | 15:56 |
| vdrok | rloo: https://governance.openstack.org/reference/projects/ironic.html#project-ironic | 15:57 |
| rloo | vdrok: i think that includes the clients | 15:57 |
| vdrok | rloo: here only ironic itself has assert:follows-standard-deprecation | 15:57 |
| *** Sukhdev has joined #openstack-ironic | 15:57 | |
| rloo | vdrok: oh. | 15:57 |
| rloo | well, we can discuss in monday meeting then :) | 15:57 |
| vdrok | that's why I asked, I'm not sure, but I supposed it should be similar for clients | 15:57 |
| vdrok | :) | 15:57 |
| *** _vsaienko has joined #openstack-ironic | 15:58 | |
| rloo | vdrok: i think we are (I am) assuming that the client follows that. | 15:58 |
| *** [1]cdearborn has quit IRC | 15:58 | |
| vdrok | rloo: I'll add it to the agenda just to be sure | 15:58 |
| rloo | vdrok: although i do wonder if clients have to provide a longer deprecation period. anyway, the sooner we start the deprecation, the better :) | 15:58 |
| *** [1]cdearborn has joined #openstack-ironic | 15:59 | |
| vdrok | oh btw good morning rloo :) | 16:00 |
| rloo | hi vdrok! | 16:00 |
| *** jaosorior_away is now known as jaosorior | 16:00 | |
| *** ohamada_ has quit IRC | 16:01 | |
| JayF | https://review.openstack.org/#/c/356788 Deva's doc update can land today if someone wants to give it a final review and +2W | 16:03 |
| jlvillal | dtantsur: Thanks | 16:03 |
| JayF | mariojv: ^ I'm not landing your notifications, because it conflicts with this | 16:03 |
| mariojv | i'll rebase whenever it lands today | 16:04 |
| *** _vsaienko has quit IRC | 16:05 | |
| *** Sukhdev has quit IRC | 16:08 | |
| *** michauds has joined #openstack-ironic | 16:08 | |
| *** pcaruana has joined #openstack-ironic | 16:09 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Add node validate OSC command https://review.openstack.org/359322 | 16:16 |
| *** jpich has quit IRC | 16:16 | |
| *** rama_y has quit IRC | 16:18 | |
| *** jaosorior has quit IRC | 16:18 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Add match_root_device_hints() to the utils.py module https://review.openstack.org/358000 | 16:22 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Extend parse_root_device_hints to support operators https://review.openstack.org/348953 | 16:22 |
| lucasagomes | rloo, ^ that should deal with whitespaces | 16:22 |
| *** vmud213 has quit IRC | 16:22 | |
| NobodyCam | Good Morning Ironic'ers :) | 16:28 |
| mariojv | morning NobodyCam | 16:28 |
| NobodyCam | morning qariojv :) | 16:29 |
| NobodyCam | gah | 16:29 |
| NobodyCam | mariojv: even | 16:29 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Add match_root_device_hints() to the utils.py module https://review.openstack.org/358000 | 16:29 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-lib: Extend parse_root_device_hints to support operators https://review.openstack.org/348953 | 16:29 |
| lucasagomes | NobodyCam, morning | 16:30 |
| NobodyCam | morning lucasagomes :) | 16:30 |
| rloo | lucasagomes: good. are they ready to be reviewed? | 16:31 |
| NobodyCam | morning rloo :) | 16:31 |
| lucasagomes | rloo, yup | 16:31 |
| rloo | lucasagomes: on my list for today then. i guess we probably want to release ironic-lib soon. | 16:32 |
| dtantsur | preferably tomorrow | 16:32 |
| lucasagomes | rloo, I had to add a function to normalize the hint which are string types | 16:32 |
| lucasagomes | not the most elegant thing to do, but solves the whitespace problem | 16:32 |
| lucasagomes | by normalize I mean, I use urllib.quote() to replace special characters with %XX syntax | 16:33 |
| NobodyCam | morning dtantsur :) | 16:34 |
| dtantsur | morning NobodyCam rloo | 16:34 |
| rloo | hi dtantsur, morning NobodyCam | 16:34 |
| rloo | lucasagomes: going to try to review in the next 1-2 hours | 16:35 |
| NobodyCam | :) | 16:35 |
| jlvillal | Good morning NobodyCam rloo dtantsur lucasagomes mariojv JayF vdrok sambetts and anyone else I missed :) | 16:35 |
| JayF | o/ | 16:36 |
| NobodyCam | morning jlvillal :) | 16:36 |
| rloo | morning jlvillal! | 16:36 |
| NobodyCam | morning JayF | 16:36 |
| lucasagomes | JayF, do you have more pixie boots stickers? If so can you bring one to barcelona? | 16:37 |
| lucasagomes | JayF, I got a new laptop and I don't have any left T.T | 16:37 |
| JayF | lucasagomes: if I am personally brought to barcelona | 16:37 |
| JayF | lucasagomes: I will bring my stickers | 16:37 |
| lucasagomes | JayF, cool | 16:37 |
| lucasagomes | cheers | 16:37 |
| JayF | lucasagomes: given it's unsure I'll get to go, might wanna give me an address in PM | 16:37 |
| JayF | lucasagomes: and I'll just mail it out | 16:37 |
| dtantsur | morning jlvillal, JayF as well | 16:37 |
| *** jraju has joined #openstack-ironic | 16:38 | |
| lucasagomes | JayF, that works too :-) | 16:38 |
| jlvillal | lucasagomes: I got a new naked laptop too :( | 16:38 |
| jlvillal | Which reminds me I need to put the sticker JayF sent me on my laptop | 16:39 |
| vdrok | morning NobodyCam | 16:39 |
| NobodyCam | morning vdrok :) | 16:39 |
| *** aNupoisc has joined #openstack-ironic | 16:39 | |
| *** jraju has quit IRC | 16:39 | |
| lucasagomes | jlvillal, nice one! Which laptop did you get | 16:40 |
| *** aNupoisc has quit IRC | 16:40 | |
| jlvillal | lucasagomes: Well there weren't a lot of choice from IT :( But I went with a Lenovo again. T460P 16GB RAM / 480 GB SSD | 16:41 |
| *** ntpttr___ has joined #openstack-ironic | 16:41 | |
| jlvillal | lucasagomes: I kind of miss my Lenovo Carbon X1 | 16:41 |
| *** aNupoisc has joined #openstack-ironic | 16:41 | |
| jlvillal | vdrok: Happy Ukraine Independence Day :) I assume you get tomorrow off. | 16:41 |
| dtantsur | T460s here, nice stuff (as soon as you make Linux work on it's new shiny chipset) | 16:41 |
| vdrok | jlvillal: yup, thanks :) | 16:41 |
| *** ipukha has quit IRC | 16:41 | |
| jlvillal | dtantsur: :) | 16:42 |
| lucasagomes | jlvillal, oh nice, I got a t460s | 16:42 |
| lucasagomes | jlvillal, they had the X1 to choose from but this t460s is an upgraded version with 20G RAM (the carbon was 8G) | 16:43 |
| jlvillal | lucasagomes: I think that looks nicer. The P model is kind of thick | 16:43 |
| jlvillal | lucasagomes: Sweet 20GB :) I guess I can upgrade mine to 32GB, but I would need my manager to approve buying some memory. | 16:44 |
| jlvillal | No upgrading the Carbon | 16:44 |
| dtantsur | lucasagomes, sweet! when I was ordering, they didn't have an upgraded version yet | 16:44 |
| dtantsur | so e.g. I had to approve buying a serious SSD separately | 16:45 |
| lucasagomes | jlvillal, yeah | 16:45 |
| lucasagomes | dtantsur, ah nice one | 16:46 |
| lucasagomes | dtantsur, to be fair, mine took AGES to arrive | 16:46 |
| lucasagomes | ~50 days since I opened the ticket | 16:46 |
| dtantsur | lol | 16:46 |
| lucasagomes | no kidding... something happened with the delivery and it had to return to germany and be sent again | 16:47 |
| lucasagomes | was a nightmare | 16:47 |
| lucasagomes | glad it's here now :D | 16:47 |
| *** e0ne has quit IRC | 16:52 | |
| *** penick has joined #openstack-ironic | 16:55 | |
| *** rama_y has joined #openstack-ironic | 16:56 | |
| *** mtanino has joined #openstack-ironic | 16:57 | |
| *** derekh has quit IRC | 16:59 | |
| dtantsur | see you tomorrow everyone | 17:01 |
| *** dtantsur is now known as dtantsur|afk | 17:01 | |
| lucasagomes | dtantsur|afk, have a good night | 17:01 |
| lucasagomes | see you | 17:01 |
| jlvillal | dtantsur|afk: ciao | 17:02 |
| NobodyCam | night dtantsur|afk | 17:02 |
| lucasagomes | I'm calling it a day as well | 17:07 |
| lucasagomes | have a great evening all, see you all tomorrow | 17:07 |
| *** fragatina has quit IRC | 17:08 | |
| *** ibmchas has joined #openstack-ironic | 17:08 | |
| *** lucasagomes is now known as lucas-afk | 17:08 | |
| *** sean-k-mooney has joined #openstack-ironic | 17:08 | |
| *** sambetts is now known as sambetts|afk | 17:09 | |
| *** mtanin___ has joined #openstack-ironic | 17:10 | |
| *** mtanino has quit IRC | 17:12 | |
| *** ibmchas has quit IRC | 17:13 | |
| *** yogi has quit IRC | 17:15 | |
| *** jrist has quit IRC | 17:18 | |
| sdake | hey folks | 17:21 |
| sdake | quick q | 17:21 |
| sdake | sean-k-mooney has implemented biforst n a container for integation with kolla | 17:22 |
| sdake | the work is 99% the way done - ready for merge from my pov | 17:22 |
| sdake | we have a small bug in that sshd is not enabled in the built image | 17:22 |
| sdake | is there some trick to do that? | 17:22 |
| *** moshele has quit IRC | 17:23 | |
| *** fragatina has joined #openstack-ironic | 17:24 | |
| jlvillal | sdake: TheJulia or cinerama might be the best info on that. | 17:26 |
| sdake | jlvillal roger thanks | 17:27 |
| TheJulia | sdake: what is the built image? | 17:27 |
| sdake | sean-k-mooney is doing the work here - so I'm not certain | 17:27 |
| sdake | sean-k-mooney is around tho | 17:27 |
| TheJulia | (pointing to code might also be useful for context) | 17:27 |
| sdake | let me go ping him over here ;) | 17:27 |
| sean-k-mooney | TheJulia: i was using the default debian image | 17:28 |
| sdake | TheJulia - do you realize how excited I am to have bifrost integration with kolla | 17:29 |
| sdake | ++++++++++++++++++++++ | 17:29 |
| sean-k-mooney | TheJulia: when i was testing in vms with the agent_ssh driver i never hit this issue only when using the agent_ipmitool driver. | 17:29 |
| TheJulia | sean-k-mooney: so where exactly are things going sideways then? After deployment? during deployment? What state is the node listed in ironic, since different drivers appear to have different results as you've indicated. | 17:31 |
| sean-k-mooney | TheJulia: after the node is deployed and in an active state in ironic i am not able to ssh into the node | 17:31 |
| sdake | and systemctl enable corrects it | 17:32 |
| TheJulia | can you ping it? | 17:32 |
| sdake | systmctl enable sshd | 17:32 |
| sdake | which means everything is good tog o minus one link somewhere in the filesystem related to systemd | 17:32 |
| TheJulia | which means it is a diskimage-builder issue then | 17:33 |
| sdake | TheJulia agree 100% | 17:33 |
| sean-k-mooney | i think was able too but i dont have the default image currently deployed. | 17:33 |
| TheJulia | sdake: when started, do the logs indicate that sshd is generating keys? | 17:34 |
| sdake | TheJulia wher ei struggle here is if its a config option to ironic or a pure dib bug | 17:34 |
| sean-k-mooney | i built a customised image with the dev_user element to take a look at what was going on. | 17:34 |
| sdake | TheJulia don't know - haven't tried it - supposed to eo n pto at m:) | 17:34 |
| TheJulia | sdake: :) | 17:35 |
| sdake | sean-k-mooney TheJulia just asked a solid question that needs an answer tho - if the keys arent there sshd may fail to start | 17:35 |
| TheJulia | Or if something built in might be wiping out the keys in a race | 17:35 |
| sean-k-mooney | TheJulia: i think they are generated but since i dont have the default image deployed current i cant check. | 17:36 |
| sean-k-mooney | TheJulia: i will need to redeploy with the default seeting instead of the custim images i built for debug | 17:36 |
| sdake | sean-k-mooney your doing defualt image + dev_user right? | 17:36 |
| TheJulia | It sounds like a pure dib bug, I just suspect we highlight it on a user built image | 17:36 |
| sean-k-mooney | that is whay i tried first but i hav ebuilt centos and ubuntu image since then whitch cloud init enabled. i need to get back to a standard image first | 17:37 |
| sean-k-mooney | sdake: TheJulia i can build the default image + dev_user so i can connect over serial to debug and see. | 17:38 |
| *** cdearborn has joined #openstack-ironic | 17:38 | |
| *** e0ne has joined #openstack-ironic | 17:39 | |
| *** milan has quit IRC | 17:41 | |
| TheJulia | sean-k-mooney: that would be best. I'm curious if 1) cloud-init ran since I believe that should be simple-init based, 2) if the system logs indicate anything regarding sshd 3) if upon sshd being enabled via `systemctl enable sshd` keys are generated. | 17:41 |
| sdake | TheJulia sean-k-mooney is in the uk - so he is probalby signing off soon | 17:41 |
| sdake | rather ireland | 17:41 |
| TheJulia | we should also take this discussion to the diskimage-builder room, but I need to lookup that channel name :) | 17:41 |
| sdake | TheJulia that wfm | 17:42 |
| TheJulia | Crazy thought.... I wonder if installing sshd on debian does not actually cause it to be enabled..... | 17:42 |
| sdake | i have meeting in 15 mins tho - so wont be able to contribute much | 17:42 |
| sean-k-mooney | TheJulia: yes initially it was using simple init . i was trying to get cloud-init to work to see if i could build in a could init script to activate teh service. | 17:42 |
| sdake | TheJulia ya that would be my guess as well | 17:42 |
| * wznoinsk thinks sean-k-mooney is not signing off soon ;) | 17:43 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks https://review.openstack.org/358855 | 17:43 |
| sean-k-mooney | hehe i tend to work late | 17:43 |
| mat128 | jlvillal: I have to run for a minute, but ^ | 17:43 |
| * jlvillal mat128 thanks | 17:44 | |
| * TheJulia looks through email and realizes that just asking the irc server what channels greghaynes is in might be easiest | 17:45 | |
| *** jcoufal_ has quit IRC | 17:45 | |
| TheJulia | doh! | 17:45 |
| TheJulia | greghaynes: cinerama mentioned there was now a dib channel, any hints would be awesome :) | 17:45 |
| *** jrist has joined #openstack-ironic | 17:46 | |
| sean-k-mooney | so in my centos7 image interestingly enough sshd is started and it works but i chage quite a few things when i built the image | 17:46 |
| greghaynes | TheJulia: #openstack-dib :) | 17:47 |
| TheJulia | sean-k-mooney: lets take this discussion over to #openstack-dib since changing the image really means that yes, this seems to be a problem related to initialization | 17:48 |
| TheJulia | greghaynes: thank you! :) | 17:48 |
| sean-k-mooney | sure i can do that but i think it would be better for me to get back to a know good state and start the converstation from there. | 17:49 |
| *** ibmchas has joined #openstack-ironic | 17:50 | |
| sean-k-mooney | as far as i can tell ironic was doing everyting right but it was just the dib image that was built by the bifrost playbook that had an issue. | 17:50 |
| TheJulia | Agreed, yeah, I suspect it is an unexpcted behavior, looking up some debian specific details at the moment | 17:51 |
| sean-k-mooney | it could also be user error on my part so i would like to rule that out before takeing more of your time. | 17:51 |
| *** [1]cdearborn has quit IRC | 17:52 | |
| *** fragatina has quit IRC | 17:53 | |
| *** zhugaoxiao has joined #openstack-ironic | 17:53 | |
| *** fragatina has joined #openstack-ironic | 17:53 | |
| sdake | sean-k-mooney sounds like you hae enough to work with for the moment | 17:54 |
| sdake | sean-k-mooney to determine if its really a dib problem or something else | 17:54 |
| *** ibmchas has quit IRC | 17:54 | |
| sean-k-mooney | sdake: yep the image that i had working i built manually with i think this command "DIB_DEV_USER_PASSWORD=tester DIB_DEV_USER_USERNAME=tester DIB_DEV_USER_SHELL=/bin/bash DIB_DEV_USER_PWDLESS_SUDO=Yes disk-image-create -o bm_image centos7 vm simple-init selinux-permissive dhcp-all-interfaces growroot enable-serial-console devuser baremetal" | 17:55 |
| *** e0ne has quit IRC | 17:55 | |
| *** Administrator__ has quit IRC | 17:55 | |
| sean-k-mooney | ill rebuilt the default image with jsut dev_user to confirm what is actully going on. | 17:55 |
| sean-k-mooney | thanks for everyones help. | 17:56 |
| TheJulia | Okay, I'm in the #openstack-dib room now as well. | 17:56 |
| sean-k-mooney | thanks ill keep you posted if i recreate the bug on a clean deploy with the default image +dev_user and 2 what the state of the services are on the server. | 17:59 |
| mat128 | jlvillal: back | 17:59 |
| *** baoli has quit IRC | 17:59 | |
| jlvillal | mat128: Just left some comments. | 18:00 |
| jlvillal | mat128: going to lunch now. | 18:00 |
| mat128 | jlvillal: alright, bon appétit :) | 18:00 |
| *** e0ne has joined #openstack-ironic | 18:00 | |
| *** baoli has joined #openstack-ironic | 18:03 | |
| *** jcoufal has joined #openstack-ironic | 18:06 | |
| rpioso | jroll: Is there a v2 API meeting today? | 18:06 |
| TheJulia | sean-k-mooney: Awesome, let us know. I don't think I've ever really reproduced what your encountering, but I've seen some others hit similar issues and this really seems like it is the first time that I've heard starting the service fixes the issue | 18:07 |
| TheJulia | rpioso: I believe jroll is out today | 18:07 |
| rpioso | TheJulia: I'll take that as a no :-) Thank you! | 18:07 |
| NobodyCam | rpioso: he'll be back Thursday | 18:08 |
| rpioso | NobodyCam: ty | 18:08 |
| rajinir | Booting ubuntu image gives an exec format error. Any idea? https://usercontent.irccloud-cdn.com/file/evazFApM/ExecFormatError | 18:08 |
| *** ibmchas has joined #openstack-ironic | 18:11 | |
| *** cdearborn has quit IRC | 18:12 | |
| *** moshele has joined #openstack-ironic | 18:12 | |
| *** cdearborn has joined #openstack-ironic | 18:13 | |
| *** athomas has quit IRC | 18:14 | |
| *** ibmchas has quit IRC | 18:15 | |
| *** jcoufal has quit IRC | 18:17 | |
| TheJulia | rajinir: I take it that the image written out has a boot loader ? | 18:20 |
| *** electrofelix has quit IRC | 18:21 | |
| *** Sukhdev has joined #openstack-ironic | 18:22 | |
| rajinir | May be I'm missing the ironic properties on the image 'kerner-id' and 'ramdisk-id"? | 18:22 |
| rajinir | TheJulia: | 18:23 |
| TheJulia | rajinir: Well, the true question is, are you intending to always network boot, or are you wanting to local boot? | 18:25 |
| *** jcoufal has joined #openstack-ironic | 18:26 | |
| rajinir | TheJulia: I'm running CI and made the default image to use Ubuntu. The environment was setup by devstack gate. | 18:28 |
| TheJulia | rajinir: Ahh, that makes more sense, that explains network booting then. Out of curiosity how are you getting the ubuntu image? | 18:30 |
| mariojv | anyone want to land this? https://review.openstack.org/#/c/356788/ it's blocking other patches from +w | 18:31 |
| *** ibmchas has joined #openstack-ironic | 18:32 | |
| TheJulia | mariojv: looking | 18:32 |
| mariojv | thanks TheJulia | 18:32 |
| TheJulia | heh, cascading documentation change | 18:33 |
| TheJulia | of sorts | 18:33 |
| rajinir | TheJulia: Downloaded trusty-server-cloudimg-amd64-disk1.img from ubuntu. And using that static image in the configs | 18:34 |
| rajinir | https://www.irccloud.com/pastebin/ZOXGiq5y/ubuntuImageconfig | 18:34 |
| *** ibmchas has quit IRC | 18:36 | |
| *** dtantsur|afk has quit IRC | 18:41 | |
| aNupoisc | Hi Guys, I am trying to boot an instance but my node after fetching ipa image and going into power-on power state goes into wait-call-back state. I don't see any error in i-cond service. Any idea for this error? | 18:42 |
| aNupoisc | I can see "ERROR neutron.plugins.ml2.drivers.openvswitch.agent.openflow.native.ofswitch [-] Switch connection timeout" in neutron-agent service | 18:43 |
| aNupoisc | But i am not sure is this because even q-agt service did not heard back from some other service. | 18:43 |
| rajinir | sambetts: I see u are using ubuntu image for CI, where did you download from? | 18:46 |
| rajinir | sambetts: Does it work? | 18:46 |
| TheJulia | rajinir: so the stock ubuntu image lacks a bootloader.... | 18:47 |
| TheJulia | rajinir: Most build an image with diskimage-builder such that the image has a boot loader | 18:47 |
| rajinir | TheJulia: Oh I see | 18:47 |
| TheJulia | rajinir: you can set the node configuration such that it network boots, but for CI testing, afaik, we expect ipxe to be able to launch the local boot loader from the disk | 18:48 |
| rajinir | TheJulia: Do you have a lnk that might have instructions? | 18:48 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Allow suppressing ramdisk logs collection https://review.openstack.org/359381 | 18:49 |
| TheJulia | rajinir: It _should_ be as simple as installing diskimage-builder and running the command, an example is located in the readme for ubuntu trusty | 18:49 |
| rajinir | TheJulia: Cool, thanks will try | 18:50 |
| TheJulia | rajinir: In short, diskimage-builder has a concept of elements to include, in the example "vm" is an element that loads a boot loader into the image, and ubuntu defines what image to use. | 18:51 |
| TheJulia | mariojv: fyi, I +2+A'ed the doc update | 18:54 |
| rajinir | TheJulia: Thank you | 18:54 |
| TheJulia | aNupoisc: what your describing sounds like IPA is booting, but it is not able to call back to the API? Is that correct | 18:56 |
| aNupoisc | TheJulia: Yes. IPA image is being supplied to the node and then i think it is waiting for ipa to come back with a request on the image | 18:59 |
| TheJulia | aNupoisc: confirmed via the console? | 19:00 |
| aNupoisc | TheJulia: This is my i-cond logs http://pastebin.com/A4XtTwE8 | 19:01 |
| mariojv | thank you, i'll rebase my patches when that is merged | 19:02 |
| *** milan has joined #openstack-ironic | 19:03 | |
| aNupoisc | TheJulia: Also i see "Fetching necessary kernel and ramdisk for node 03b2a839-f4fa-42ce-8a0f-f09fc2be4edc" | 19:04 |
| *** penick has quit IRC | 19:05 | |
| TheJulia | aNupoisc: what driver are you using? | 19:08 |
| *** ibmchas has joined #openstack-ironic | 19:13 | |
| *** rbartal has quit IRC | 19:16 | |
| aNupoisc | agent_ipmitool | 19:17 |
| aNupoisc | TheJulia: agent_ipmitool | 19:17 |
| aNupoisc | and openvswitch agent for neutron | 19:17 |
| *** ibmchas has quit IRC | 19:18 | |
| *** jtomasek has joined #openstack-ironic | 19:18 | |
| aNupoisc | TheJulia: you meant ironic driver right? | 19:19 |
| *** jtomasek has quit IRC | 19:20 | |
| *** jtomasek_ has joined #openstack-ironic | 19:20 | |
| *** pcaruana has quit IRC | 19:20 | |
| TheJulia | aNupoisc: Okay, so your not using any sort of virtual media then. The conductor may download the IPA image, but what we need to see is if the hardware is booting on the node with IPA. Based on your error from neutron, it kind of sounds like your switch configuration might be failing and thus the node may never be booting nor be able to call back the the | 19:23 |
| TheJulia | ironic api | 19:23 |
| aNupoisc | TheJulia: I am using devstack and I can see fake drivers in local.conf. Here is my local.conf | 19:26 |
| aNupoisc | TheJulia: may be i did not mentioned about my environment | 19:26 |
| openstackgerrit | Merged openstack/ironic-python-agent: Use constraints for all the things https://review.openstack.org/353124 | 19:28 |
| aNupoisc | TheJulia: local.conf http://pastebin.com/ujf02abR | 19:29 |
| TheJulia | aNupoisc: So are you attempting to deploy real bare metal nodes, or test virtual machines in devstack | 19:31 |
| TheJulia | ? | 19:31 |
| aNupoisc | TheJulia: virtual machines in devstack | 19:31 |
| aNupoisc | TheJulia: I have just single real machine | 19:32 |
| *** ibmchas has joined #openstack-ironic | 19:34 | |
| *** ricardoas has quit IRC | 19:34 | |
| *** ibmchas_ has joined #openstack-ironic | 19:36 | |
| openstackgerrit | Merged openstack/ironic: Move console documentation to separate file https://review.openstack.org/358844 | 19:38 |
| *** ibmchas has quit IRC | 19:38 | |
| *** moshele has quit IRC | 19:43 | |
| TheJulia | aNupoisc: Do you see anything in the baremetal console log then? | 19:45 |
| JayF | mariojv: notifications merged | 19:46 |
| JayF | apparently removing a W isn't enough to take it outta the queue, even if done immediately | 19:46 |
| JayF | I'm going to rebase devananda's doc update | 19:46 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Documentation update https://review.openstack.org/356788 | 19:49 |
| JayF | NobodyCam: TheJulia: https://review.openstack.org/#/c/356788 notifications knocked this outta the queue, I rebased the patch and restored my +2, would love to try and get this landed again | 19:50 |
| TheJulia | JayF: Done | 19:51 |
| NobodyCam | :) | 19:51 |
| aNupoisc | TheJulia: so you mean in /var/log/syslog? | 19:53 |
| aNupoisc | TheJulia: Logs still confuses me, many different logs. :) | 19:55 |
| aNupoisc | THeJulia: or you mean upstart/console.log | 19:56 |
| TheJulia | aNupoisc: Looks like you'll need your local.conf to indicate if the vm should be logged https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L896 | 19:58 |
| aNupoisc | TheJulia: oh that I have as IRONIC_VM_LOG_DIR=$HOME/ironic-bm-logs | 19:59 |
| aNupoisc | in local.conf | 19:59 |
| aNupoisc | THeJulia: let me go there and check | 19:59 |
| *** amotoki has quit IRC | 20:00 | |
| aNupoisc | TheJulia: this is my virtualbmc log http://pastebin.com/Xyz7bCGn and this is console log http://pastebin.com/3QKtCmXs | 20:06 |
| aNupoisc | TheJulia: seems it is not able to find network device | 20:06 |
| aNupoisc | Interesting | 20:06 |
| *** e0ne has quit IRC | 20:12 | |
| TheJulia | aNupoisc: so no ip address was available or networking between the VM and devstack is not working? | 20:15 |
| openstackgerrit | Ruby Loo proposed openstack/ironic-specs: Changes to some OSC commands https://review.openstack.org/357505 | 20:19 |
| *** e0ne has joined #openstack-ironic | 20:25 | |
| *** penick has joined #openstack-ironic | 20:25 | |
| openstackgerrit | John L. Villalovos proposed openstack/python-ironicclient: Update hacking test-requirement https://review.openstack.org/359411 | 20:27 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Update hacking test-requirement https://review.openstack.org/359412 | 20:27 |
| *** causten has quit IRC | 20:27 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Update hacking test-requirement https://review.openstack.org/359414 | 20:27 |
| *** [1]cdearborn has joined #openstack-ironic | 20:29 | |
| *** NobodyCam has quit IRC | 20:29 | |
| *** _vsaienko has joined #openstack-ironic | 20:29 | |
| openstackgerrit | Merged openstack/ironic: Documentation update https://review.openstack.org/356788 | 20:29 |
| *** NobodyCam has joined #openstack-ironic | 20:30 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks https://review.openstack.org/358855 | 20:30 |
| *** causten has joined #openstack-ironic | 20:30 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks https://review.openstack.org/358855 | 20:31 |
| *** aswadr_ has quit IRC | 20:31 | |
| *** e0ne has quit IRC | 20:37 | |
| *** cdearborn has quit IRC | 20:42 | |
| *** _vsaienko has quit IRC | 20:46 | |
| *** _vsaienko has joined #openstack-ironic | 20:47 | |
| *** priteau has joined #openstack-ironic | 20:47 | |
| *** adnavare has joined #openstack-ironic | 20:47 | |
| adnavare | TheJulia: strange it was working few days back. | 20:48 |
| *** aNupoisc has quit IRC | 20:48 | |
| adnavare | TheJulia: do i need to check some conf file to make sure networking options are set properly? | 20:51 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: [WIP] Break devstack when using pxe deploy with ipv6 https://review.openstack.org/359422 | 20:56 |
| TheJulia | adnavare: I would check your local.conf file compared to what we have published in the documentation. I don't fire up devstack very often but really haven't had an issue | 20:56 |
| adnavare | TheJulia: I am surprised as last week it worked. I will also check if anything from my side caused some problem. But at least i know now what is going wrong. | 20:58 |
| *** amotoki has joined #openstack-ironic | 21:01 | |
| TheJulia | adnavare: indeed, and that is the key. :) | 21:01 |
| *** jcoufal has quit IRC | 21:03 | |
| *** amotoki has quit IRC | 21:06 | |
| adnavare | TheJulia: Yes cool :) | 21:06 |
| *** wajdi_ has joined #openstack-ironic | 21:19 | |
| *** bfournie has quit IRC | 21:21 | |
| _vsaienko | jroll, jlvillal: I've just realized that we using partitioned image which is not supported by multitenant case on multinode job. I've update configuration according to voting multitenant job https://review.openstack.org/#/c/359431 once you have a time please review. | 21:22 |
| *** wajdi has quit IRC | 21:22 | |
| jlvillal | _vsaienko: Thanks! Will look now | 21:22 |
| *** wajdi_ has quit IRC | 21:24 | |
| *** dprince has quit IRC | 21:26 | |
| *** Goneri has quit IRC | 21:43 | |
| *** sdake has quit IRC | 21:45 | |
| *** sdake has joined #openstack-ironic | 21:45 | |
| *** sdake has joined #openstack-ironic | 21:46 | |
| *** thiagop has quit IRC | 21:47 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 21:50 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add notification base classes and docs https://review.openstack.org/298461 | 21:50 |
| mariojv | JayF: vdrok: yuriyz: notification patch did not merge despite 3 +2s because of the merge conflict with the docs update. looks like that made it in first, so i rebased above ^ | 21:50 |
| mariojv | somehow jenkins commented merge succeeded, but it rolled it back i guess | 21:51 |
| anteaya | jroll: or anyone else, have you ever seen a user story regarding ironic? | 21:53 |
| anteaya | the swg is asking about user stories as they assume they find their way to ptls | 21:54 |
| anteaya | I'm trying to give them a reality check on that | 21:54 |
| anteaya | so thought I would ask | 21:54 |
| anteaya | anyone in channel seen a user story? | 21:54 |
| anteaya | mariojv: jenkins rolled back a merge? | 21:56 |
| anteaya | mariojv: can you take the url for the patch in question to infra and ask about that please? | 21:57 |
| mariojv | anteaya: i might be wrong about it, workflow was removed really quickly | 21:57 |
| anteaya | yeah, that is why I ask you to take the url of the patch to infra so we can examine it | 21:57 |
| anteaya | either code has merged or it hasn't | 21:58 |
| mariojv | ok - it was totally intentional though | 21:58 |
| anteaya | it doesn't unmerge | 21:58 |
| anteaya | what was intentional? | 21:58 |
| mariojv | workflow being taken away | 21:58 |
| mariojv | we wanted the docs update merged before the patch set that had its workflow removed | 21:58 |
| anteaya | can you share a url please? | 21:59 |
| mariojv | https://review.openstack.org/#/c/298461 anteaya | 21:59 |
| mariojv | anteaya: this was the patch that merged before it https://review.openstack.org/#/c/356788/ | 22:00 |
| mariojv | i do believe there was a jenkins comment on that first one, but it's no longer there. JayF thought it had merged too | 22:00 |
| mariojv | fwiw the repo is fine, that code is unmerged (as it appears to be) | 22:01 |
| anteaya | the patch never merged | 22:02 |
| anteaya | workflow +1 was set | 22:02 |
| *** amotoki has joined #openstack-ironic | 22:02 | |
| anteaya | gate jobs started as you can see but clicking toggle ci | 22:02 |
| anteaya | Starting gate jobs. http://status.openstack.org/zuul/ | 22:02 |
| anteaya | then workflow +1 was unset | 22:03 |
| anteaya | so gate jobs passed and the merge attempt failed | 22:03 |
| anteaya | working as expected | 22:03 |
| mariojv | ok | 22:03 |
| mariojv | thanks | 22:04 |
| *** priteau has quit IRC | 22:05 | |
| _vsaienko | jlvillal, TheJulia: I've update commit message, and fixed job name in zuul/layout not all tests are passed, please review one more time :) | 22:06 |
| _vsaienko | https://review.openstack.org/#/c/359431 | 22:06 |
| jlvillal | _vsaienko: +1 from me :) Thank you. | 22:07 |
| *** amotoki has quit IRC | 22:07 | |
| *** causten has quit IRC | 22:07 | |
| *** trandles has joined #openstack-ironic | 22:08 | |
| anteaya | mariojv: welcome | 22:08 |
| *** notmorgan is now known as morganfainberg | 22:09 | |
| *** morganfainberg is now known as morgan | 22:09 | |
| *** Mr_T has joined #openstack-ironic | 22:10 | |
| *** morgan is now known as notmorgan | 22:11 | |
| *** baoli has quit IRC | 22:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/359457 | 22:17 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/359458 | 22:17 |
| *** sabeen has quit IRC | 22:24 | |
| rloo | anteaya: wrt user story, this is the only one I'm aware of: https://specs.openstack.org/openstack/openstack-user-stories/user-stories/proposed/baremetal-service.html | 22:27 |
| rloo | anteaya: jroll is on vacation, he'll be back Thurs | 22:27 |
| anteaya | rloo: thank you | 22:28 |
| rloo | anteaya: yw :) | 22:29 |
| anteaya | rloo: has that user story ever been discussed in ironic? | 22:29 |
| rloo | anteaya: not that I know of. but that doesn't mean much; i don't see all the irc conversations. jroll is aware of it. and probably some other ironic folks are aware of it too. | 22:30 |
| anteaya | okay thank you | 22:31 |
| rloo | anteaya: so no 'official' discussion of it anyway | 22:31 |
| anteaya | right that was my point to the scientific working group | 22:31 |
| anteaya | i'm encouraging them to attend ironic meetings | 22:31 |
| anteaya | and bring their use cases to the ironic channel | 22:31 |
| * trandles wandered in from the scientific working group ;) | 22:32 | |
| anteaya | trandles: welcome | 22:32 |
| rloo | anteaya: ah. i was wondering what swg was. i think at least one person on the swg was the author or helped author that spec | 22:32 |
| anteaya | trandles: thank you for being here | 22:32 |
| trandles | anteaya: thanks for having me | 22:32 |
| anteaya | rloo: sorry the scientific working group | 22:32 |
| anteaya | rloo: meet trandles | 22:32 |
| anteaya | trandles: this is rloo | 22:32 |
| trandles | o/ | 22:32 |
| rloo | hi trandles, welcome to ironic. where irony is our game :) | 22:32 |
| anteaya | rloo: I shouldn't have used the acronym | 22:32 |
| rloo | anteaya: no worries. | 22:33 |
| anteaya | trandles: I'm on the infra team, I just move around different places | 22:33 |
| anteaya | trandles: friendly group in here | 22:33 |
| trandles | gotcha | 22:33 |
| anteaya | do share what you are working on if you feel like it | 22:33 |
| anteaya | and how you are using ironic, if you are alreayd | 22:33 |
| anteaya | already | 22:33 |
| trandles | I _really_ hate to do this to you but I have to run in 15 minutes. :( But I'm very happy to come back tomorrow and share. | 22:34 |
| anteaya | rloo: I can find out if anyone on the scientific working group knows the authors of that spec | 22:34 |
| anteaya | trandles: ironic is always open | 22:34 |
| rloo | no worries trandles. your mileage will vary depending on who is around :) | 22:34 |
| anteaya | rloo: might not be here, but others will | 22:34 |
| trandles | rloo: should I look out for you in particular? | 22:35 |
| rloo | trandles: no, never look for me | 22:35 |
| trandles | lol | 22:35 |
| rloo | trandles: jroll, anyone else, just not me :) | 22:35 |
| trandles | kk | 22:35 |
| rloo | trandles: earlier in the day, there are more folks around. | 22:35 |
| anteaya | trandles: thanks for speaking up | 22:35 |
| *** michauds has quit IRC | 22:36 | |
| anteaya | rloo: I see zero names I know from the scientific working group on this patch: https://review.openstack.org/#/c/289627/ | 22:37 |
| anteaya | I do see names from the project working group | 22:37 |
| rloo | anteaya: oh, maybe i am wrong. i thought Leong was on the scientific work group. | 22:37 |
| rloo | anteaya: maybe he is on the project work group. or in neither and i'm wrong. | 22:38 |
| *** trandles has left #openstack-ironic | 22:38 | |
| anteaya | perhaps that person is | 22:40 |
| anteaya | I have asked the chairs of the working group if they know the authors of the patch | 22:40 |
| anteaya | it is possible I am wrong | 22:40 |
| anteaya | but in any case | 22:41 |
| anteaya | trandles found their way here and spoke up | 22:41 |
| anteaya | which was my point | 22:41 |
| anteaya | so yay, success | 22:41 |
| anteaya | I'm fine with user stories | 22:41 |
| anteaya | their existance should not prevent operators from spending time in channel and speaking about their needs | 22:41 |
| rloo | anteaya: mission accomplished :) | 22:41 |
| anteaya | :) | 22:41 |
| *** ccamacho has quit IRC | 22:42 | |
| *** sdake has quit IRC | 22:44 | |
| *** sdake has joined #openstack-ironic | 22:44 | |
| *** ibmchas_ has quit IRC | 22:49 | |
| *** chlong has quit IRC | 23:03 | |
| TheJulia | _vsaienko: Thank you! | 23:12 |
| *** mpsairam has joined #openstack-ironic | 23:15 | |
| *** sabeen has joined #openstack-ironic | 23:19 | |
| *** rpioso has quit IRC | 23:25 | |
| *** yuanying has joined #openstack-ironic | 23:30 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add tempest tests for node Adoption https://review.openstack.org/344975 | 23:31 |
| *** _vsaienko has quit IRC | 23:34 | |
| *** rama_y has quit IRC | 23:39 | |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic-python-agent: Enforce upper-constraints when building ramdisks https://review.openstack.org/358855 | 23:49 |
| mat128 | jlvillal: if you,re still there, going for a walk but pushed this so it gets more exposure ^ | 23:49 |
| TheJulia | mat128: interesting, I'll try and take a look in the morning. That makes me wonder about the dib based ipa images too. | 23:51 |
| *** anush_ has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!