| *** piet has quit IRC | 00:00 | |
| *** rama_y has quit IRC | 00:01 | |
| *** sabeen has joined #openstack-ironic | 00:05 | |
| *** rbudden has quit IRC | 00:18 | |
| *** daemontool has quit IRC | 00:21 | |
| *** ChrisAusten has quit IRC | 00:24 | |
| *** Sukhdev has quit IRC | 00:29 | |
| *** hoangcx has joined #openstack-ironic | 00:38 | |
| *** mtanino has quit IRC | 00:53 | |
| *** garthb has quit IRC | 00:53 | |
| *** piet has joined #openstack-ironic | 00:57 | |
| *** tiendc has joined #openstack-ironic | 01:05 | |
| tiendc | good morning, folks | 01:06 |
|---|---|---|
| tiendc | lucasagomes vdrok jlvillal: hi, could you help to review https://review.openstack.org/#/c/328168/ https://review.openstack.org/#/c/293873/ | 01:12 |
| tiendc | lucasagomes vdrok jlvillal: they got some +2 from you, only minor changes remain | 01:13 |
| *** piet has quit IRC | 01:17 | |
| *** ijw has joined #openstack-ironic | 01:19 | |
| *** baoli has joined #openstack-ironic | 01:25 | |
| *** clenimar_ has quit IRC | 01:29 | |
| *** phuongnh has joined #openstack-ironic | 01:32 | |
| *** hshiina has joined #openstack-ironic | 01:38 | |
| *** rloo has quit IRC | 01:39 | |
| *** clenimar_ has joined #openstack-ironic | 01:41 | |
| *** baoli has quit IRC | 01:42 | |
| *** ijw has quit IRC | 01:49 | |
| *** ccamacho has joined #openstack-ironic | 01:53 | |
| *** ijw has joined #openstack-ironic | 01:54 | |
| *** ijw has quit IRC | 01:59 | |
| *** caoshufeng has joined #openstack-ironic | 02:18 | |
| *** jaybeale has joined #openstack-ironic | 02:19 | |
| *** joprovost has joined #openstack-ironic | 02:26 | |
| TheJulia | yeouch, bifrost with cirros on a CI run... [ 1230.758644] systemd[1]: Startup finished in 25.705s (kernel) + 38.269s (initrd) + 19min 26.631s (userspace) = 20min 30.607s. :( | 02:31 |
| caoshufeng | Good morning all. I need some suggestion with this bug report. https://bugs.launchpad.net/ironic/+bug/1602490 | 02:35 |
| openstack | Launchpad bug 1602490 in Ironic "inconsistent log of node_id and node_uuid when acquiring node lock" [Undecided,New] - Assigned to Cao ShuFeng (caosf-fnst) | 02:35 |
| caoshufeng | Should this bug be fixed? | 02:36 |
| caoshufeng | Or this bug is invalid? | 02:36 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Do Not Merge: Canery test for ironic networking changes https://review.openstack.org/330138 | 02:37 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Do Not Merge: another test for ironic networking https://review.openstack.org/330210 | 02:37 |
| TheJulia | And I'm worried bifrost's gate might be broken hence ^^^ :( | 02:38 |
| TheJulia | looking caoshufeng | 02:38 |
| caoshufeng | TheJulia, Thanks in advance | 02:39 |
| *** jaybeale has quit IRC | 02:41 | |
| *** ijw has joined #openstack-ironic | 02:41 | |
| *** vishwanathj has quit IRC | 02:41 | |
| *** ijw has quit IRC | 02:42 | |
| *** thrash|wknd has quit IRC | 02:42 | |
| *** vishwanathj has joined #openstack-ironic | 02:42 | |
| *** ijw has joined #openstack-ironic | 02:42 | |
| *** baoli has joined #openstack-ironic | 02:42 | |
| *** thrash has joined #openstack-ironic | 02:44 | |
| *** thrash has quit IRC | 02:44 | |
| *** thrash has joined #openstack-ironic | 02:44 | |
| *** bvandewa_ has joined #openstack-ironic | 02:44 | |
| *** amoralej|off has quit IRC | 02:46 | |
| *** amoralej has joined #openstack-ironic | 02:47 | |
| *** ijw has quit IRC | 02:47 | |
| *** bvandewa has quit IRC | 02:48 | |
| TheJulia | caoshufeng: oh, I see... the way the task_manager.acquire statement, at least with update_port, sets the port_obj.node_id instead of task.node.uuid | 02:48 |
| *** bvandewa_ has quit IRC | 02:48 | |
| TheJulia | caoshufeng: I think a more verbose example might help for anyone looking at the bug, but yes, that does seem like a bug | 02:49 |
| caoshufeng | TheJulia, yeah, I known it's sort of "trivalfix" if I try to close it. | 02:50 |
| caoshufeng | So I ask here in advance. | 02:51 |
| caoshufeng | TheJulia, I will commit a change to fix it,since not only myself treat it as a bug. | 02:52 |
| TheJulia | https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1630 | 02:53 |
| TheJulia | :) | 02:53 |
| TheJulia | I marked the bug as confirmed | 02:53 |
| TheJulia | and with that, I think I likely need to call it a night | 02:54 |
| TheJulia | Have a wonderful day caoshufeng | 02:55 |
| caoshufeng | Oh, yeah, Thanks. | 02:56 |
| caoshufeng | I will start the patch soon. | 02:56 |
| TheJulia | caoshufeng: you may just want to look for other places where that is done, espescially if you have more examples of such inconsistency :) | 02:57 |
| gbraad | TheJulia have a good one | 02:58 |
| caoshufeng | TheJulia, can I updata codes here: https://github.com/openstack/ironic/blob/master/ironic/conductor/task_manager.py#L198 | 02:58 |
| caoshufeng | 1. get the node_uuid from node_id 2. log the node_uuid | 02:59 |
| TheJulia | caoshufeng: not exactly.... https://github.com/openstack/ironic/blob/master/ironic/conductor/task_manager.py#L171 | 03:01 |
| caoshufeng | Yes, I will update this function. | 03:02 |
| TheJulia | it is a parameter, I think we should be honoring the input, it just seems like the input is wrong | 03:02 |
| TheJulia | I think. My brain is fried for the day | 03:02 |
| caoshufeng | TheJulia, so I will update this function :https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1630 | 03:03 |
| caoshufeng | And look for other places that passed "node_id" but not "node_uuid" | 03:04 |
| *** bvandewa has joined #openstack-ironic | 03:04 | |
| TheJulia | caoshufeng: Excellent, thank you | 03:05 |
| caoshufeng | TheJulia, Thanks very much for your guidance. | 03:05 |
| *** cdearborn has joined #openstack-ironic | 03:09 | |
| *** cdearborn has quit IRC | 03:14 | |
| *** vishwanathj has quit IRC | 03:17 | |
| *** vishwanathj has joined #openstack-ironic | 03:18 | |
| *** rama_y has joined #openstack-ironic | 03:29 | |
| *** joprovost has quit IRC | 03:43 | |
| *** Sukhdev has joined #openstack-ironic | 03:48 | |
| *** hshiina has quit IRC | 03:50 | |
| *** garthb has joined #openstack-ironic | 03:52 | |
| *** hshiina has joined #openstack-ironic | 04:01 | |
| *** vishwanathj has quit IRC | 04:02 | |
| *** links has joined #openstack-ironic | 04:03 | |
| *** ijw has joined #openstack-ironic | 04:11 | |
| *** baoli has quit IRC | 04:20 | |
| *** piet has joined #openstack-ironic | 04:20 | |
| *** sdake has joined #openstack-ironic | 04:23 | |
| *** yogi has joined #openstack-ironic | 04:35 | |
| *** Fdaisuke_ has joined #openstack-ironic | 04:39 | |
| *** Fdaisuke has quit IRC | 04:40 | |
| *** rama_y has quit IRC | 04:41 | |
| *** ijw has quit IRC | 04:45 | |
| *** piet has quit IRC | 04:47 | |
| *** ijw has joined #openstack-ironic | 04:49 | |
| *** sdake has quit IRC | 04:50 | |
| *** ijw has quit IRC | 04:53 | |
| *** amotoki has joined #openstack-ironic | 05:26 | |
| *** hparekh has quit IRC | 05:29 | |
| *** hparekh has joined #openstack-ironic | 05:43 | |
| *** ChubYann has quit IRC | 05:45 | |
| *** amotoki has quit IRC | 05:51 | |
| *** kalpase has joined #openstack-ironic | 05:53 | |
| *** amotoki has joined #openstack-ironic | 05:53 | |
| *** kalpase has quit IRC | 05:53 | |
| *** amotoki has quit IRC | 05:57 | |
| *** Goneri has joined #openstack-ironic | 05:57 | |
| *** edand has joined #openstack-ironic | 05:58 | |
| *** amotoki has joined #openstack-ironic | 05:59 | |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Fix Quick-start example syntax. https://review.openstack.org/340944 | 06:02 |
| *** Goneri has quit IRC | 06:02 | |
| *** Goneri has joined #openstack-ironic | 06:02 | |
| *** mjura has joined #openstack-ironic | 06:04 | |
| *** Sukhdev has quit IRC | 06:07 | |
| *** makowals has quit IRC | 06:07 | |
| *** mjura has quit IRC | 06:11 | |
| *** mjura has joined #openstack-ironic | 06:11 | |
| *** hshiina2 has joined #openstack-ironic | 06:21 | |
| *** hshiina has quit IRC | 06:23 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add nodes tagging support - API https://review.openstack.org/250478 | 06:28 |
| *** baoli has joined #openstack-ironic | 06:32 | |
| *** pcaruana has joined #openstack-ironic | 06:35 | |
| *** baoli has quit IRC | 06:36 | |
| *** makowals has joined #openstack-ironic | 06:41 | |
| *** fragatina has joined #openstack-ironic | 06:48 | |
| *** openstackgerrit has quit IRC | 06:48 | |
| *** amotoki has quit IRC | 06:48 | |
| *** openstackgerrit has joined #openstack-ironic | 06:48 | |
| *** fragatina has quit IRC | 06:49 | |
| *** fragatina has joined #openstack-ironic | 06:50 | |
| *** mkovacik__ has joined #openstack-ironic | 06:51 | |
| *** _milan_ has quit IRC | 06:53 | |
| *** rajinir has quit IRC | 06:55 | |
| *** garthb has quit IRC | 06:56 | |
| *** mkovacik__ has quit IRC | 06:56 | |
| *** amotoki has joined #openstack-ironic | 07:01 | |
| *** clenimar__ has joined #openstack-ironic | 07:03 | |
| *** clenimar_ has quit IRC | 07:06 | |
| *** tesseract- has joined #openstack-ironic | 07:10 | |
| *** rcernin has joined #openstack-ironic | 07:13 | |
| *** Romanenko_K has joined #openstack-ironic | 07:16 | |
| *** daemontool has joined #openstack-ironic | 07:17 | |
| *** athomas has joined #openstack-ironic | 07:22 | |
| *** _Fdaisuke_ has joined #openstack-ironic | 07:28 | |
| *** Fdaisuke_ has quit IRC | 07:29 | |
| *** daemontool_ has joined #openstack-ironic | 07:31 | |
| *** bvandewa has quit IRC | 07:32 | |
| *** moshele has joined #openstack-ironic | 07:34 | |
| *** daemontool has quit IRC | 07:34 | |
| *** ohamada has joined #openstack-ironic | 07:35 | |
| openstackgerrit | Andreas Jaeger proposed openstack/virtualbmc: Remove unused releasenote setup https://review.openstack.org/341333 | 07:36 |
| *** bvandewa has joined #openstack-ironic | 07:37 | |
| *** moshele has quit IRC | 07:40 | |
| *** amotoki has quit IRC | 07:40 | |
| *** moshele has joined #openstack-ironic | 07:40 | |
| *** ifarkas has joined #openstack-ironic | 07:44 | |
| vdrok | good morning ironic! | 07:46 |
| Romanenko_K | Morning! | 07:47 |
| *** amotoki has joined #openstack-ironic | 07:50 | |
| moshele | morning :) | 07:50 |
| *** zzzeek has quit IRC | 08:00 | |
| *** pcaruana has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:01 | |
| *** d0ugal has joined #openstack-ironic | 08:01 | |
| *** fragatina has quit IRC | 08:02 | |
| vdrok | morning Romanenko_K and moshele :) | 08:02 |
| *** jpich has joined #openstack-ironic | 08:10 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:11 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:11 |
| lucasagomes | morning all | 08:14 |
| *** pcaruana has joined #openstack-ironic | 08:14 | |
| *** jrist_ has joined #openstack-ironic | 08:14 | |
| aarefiev | morning Ironic! | 08:15 |
| ifarkas | morning all! | 08:16 |
| *** rbartal has joined #openstack-ironic | 08:16 | |
| alineb | morning! | 08:16 |
| *** mbound has joined #openstack-ironic | 08:16 | |
| *** mbound has quit IRC | 08:17 | |
| *** hkominos has joined #openstack-ironic | 08:19 | |
| *** MattMan has quit IRC | 08:19 | |
| hkominos | Good morning everybody | 08:19 |
| *** MattMan has joined #openstack-ironic | 08:19 | |
| *** hshiina2 has quit IRC | 08:21 | |
| *** hshiina has joined #openstack-ironic | 08:21 | |
| *** jtomasek__ has joined #openstack-ironic | 08:22 | |
| hkominos | goodmorning everyone | 08:25 |
| *** derekh has joined #openstack-ironic | 08:33 | |
| *** amotoki has quit IRC | 08:38 | |
| *** milan has joined #openstack-ironic | 08:41 | |
| milan | morning team! | 08:41 |
| milan | #pixiesay Morning, Team! -m flexing | 08:41 |
| PixieBoots | ᕙʕ⇀ᴥ⇀ʔᕗ: Morning, Team! | 08:41 |
| *** tiendc has quit IRC | 08:48 | |
| *** chlong has joined #openstack-ironic | 08:48 | |
| *** mgould|afk is now known as mgould | 08:49 | |
| mgould | morning Ironic! | 08:49 |
| *** dtantsur|afk is now known as dtantsur | 08:53 | |
| dtantsur | Morning Ironic, morning mgould, milan, alineb, ifarkas, hkominos, lucasagomes and everyone | 08:53 |
| *** hshiina has quit IRC | 08:55 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: Discover driver type and rotational speed https://review.openstack.org/341370 | 08:56 |
| mgould | morning dtantsur milan alineb lucasagomes ifarkas hkominos vdrok moshele aarefiev | 09:00 |
| vdrok | morning all :) | 09:00 |
| milan | mgould, vdrok, dtantsur good morning! :) | 09:01 |
| *** bvandewa has quit IRC | 09:02 | |
| lucasagomes | milan, mgould dtantsur hkominos alineb ifarkas vdrok aarefiev morning :-) | 09:02 |
| aarefiev | morning all! : | 09:03 |
| milan | lucasagomes, aarefiev good morning :) | 09:03 |
| *** sabeen has quit IRC | 09:03 | |
| alineb | morning dtantsur mgould lucasagomes | 09:05 |
| dtantsur | morning aarefiev | 09:05 |
| *** moshele has quit IRC | 09:14 | |
| *** moshele has joined #openstack-ironic | 09:15 | |
| *** mbound has joined #openstack-ironic | 09:17 | |
| *** bvandewa has joined #openstack-ironic | 09:17 | |
| *** pcaruana has quit IRC | 09:18 | |
| openstackgerrit | Yuuki Mori proposed openstack/ironic: Add keystone_auth descripton for Mitaka Release https://review.openstack.org/341383 | 09:21 |
| *** Goneri has quit IRC | 09:22 | |
| *** mbound has quit IRC | 09:22 | |
| *** e0ne has joined #openstack-ironic | 09:24 | |
| *** tiendc has joined #openstack-ironic | 09:30 | |
| *** amotoki has joined #openstack-ironic | 09:30 | |
| *** pcaruana has joined #openstack-ironic | 09:31 | |
| *** makowals has quit IRC | 09:34 | |
| *** yonglihe has quit IRC | 09:34 | |
| *** amotoki has quit IRC | 09:34 | |
| *** Goneri has joined #openstack-ironic | 09:36 | |
| *** makowals has joined #openstack-ironic | 09:38 | |
| *** makowals has quit IRC | 09:39 | |
| *** pcaruana has quit IRC | 09:48 | |
| *** pcaruana has joined #openstack-ironic | 09:48 | |
| *** pcaruana has quit IRC | 09:50 | |
| *** pcaruana has joined #openstack-ironic | 09:50 | |
| *** pcaruana has quit IRC | 09:51 | |
| *** e0ne has quit IRC | 09:53 | |
| *** fragatina has joined #openstack-ironic | 09:57 | |
| *** pcaruana has joined #openstack-ironic | 09:57 | |
| *** jrist_ has quit IRC | 10:01 | |
| *** e0ne has joined #openstack-ironic | 10:01 | |
| *** athomas has quit IRC | 10:08 | |
| *** bvandewa has quit IRC | 10:12 | |
| *** fragatina has quit IRC | 10:12 | |
| *** athomas has joined #openstack-ironic | 10:13 | |
| vsaienk0 | Morning aarefiev, dtantsur, vdrok, milan, alineb and all Ironic'ers! | 10:14 |
| milan | morning vsaienk0! :) | 10:15 |
| *** daemontool_ has quit IRC | 10:16 | |
| *** daemontool_ has joined #openstack-ironic | 10:21 | |
| *** radek_ has joined #openstack-ironic | 10:23 | |
| *** e0ne has quit IRC | 10:24 | |
| *** e0ne has joined #openstack-ironic | 10:25 | |
| *** tiendc has quit IRC | 10:26 | |
| jroll | hey, morning y'all | 10:29 |
| jroll | JayF: replied in gerrit to your question | 10:29 |
| dtantsur | morning jroll, vsaienk0 | 10:34 |
| lucasagomes | jroll, vsaienk0 hey there | 10:34 |
| *** moshele has quit IRC | 10:35 | |
| *** moshele has joined #openstack-ironic | 10:36 | |
| openstackgerrit | Yuuki Mori proposed openstack/ironic: Add keystone_auth descripton for Mitaka Release https://review.openstack.org/341383 | 10:39 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add multitenancy-related fields to port API object https://review.openstack.org/206244 | 10:40 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add 'neutron' network interface https://review.openstack.org/317393 | 10:40 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Expose node's network_interface field in API https://review.openstack.org/317392 | 10:40 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 10:40 |
| *** e0ne has quit IRC | 10:41 | |
| *** hoangcx has quit IRC | 10:44 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Expose node's network_interface field in API https://review.openstack.org/317392 | 10:45 |
| *** jrist has joined #openstack-ironic | 10:49 | |
| *** jrist has quit IRC | 10:49 | |
| *** jrist has joined #openstack-ironic | 10:49 | |
| mgould | morning vsaienk0 jroll | 10:50 |
| *** ccamacho is now known as ccamacho|lunch | 10:56 | |
| *** baoli has joined #openstack-ironic | 10:59 | |
| sambetts|afk | Morning all | 11:03 |
| *** sambetts|afk is now known as sambetts|catinbo | 11:03 | |
| *** sambetts|catinbo is now known as sambetts|cat | 11:03 | |
| jroll | catbetts | 11:04 |
| dtantsur | lol | 11:04 |
| dtantsur | morning sambetts|cat | 11:04 |
| sambetts|cat | hehe, I wish | 11:04 |
| sambetts|cat | schrodinger fitted ... | 11:04 |
| sambetts|cat | jroll: your up early again | 11:05 |
| sambetts|cat | o/ dtantsur | 11:05 |
| lucasagomes | sambetts|cat, here but not here or both, again? | 11:06 |
| sambetts|cat | yup | 11:06 |
| lucasagomes | :D | 11:06 |
| *** baoli has quit IRC | 11:06 | |
| jroll | sambetts|cat: meh, it's 7am already :P | 11:08 |
| *** edand has quit IRC | 11:11 | |
| lucasagomes | that sounds very early for /me heh | 11:11 |
| lucasagomes | tho sometimes I find mornings very productive, way less interruptions | 11:12 |
| lucasagomes | (works for very late too) | 11:12 |
| jroll | indeed, I love mornings | 11:12 |
| *** lucasagomes is now known as lucas-hungry | 11:13 | |
| * lucas-hungry will grab some bites, brb | 11:13 | |
| sambetts|cat | hehe /me is a night owl, my best work has been at 3 in the morning while topped up on way to much caffine | 11:14 |
| mgould | morning sambetts|cat | 11:14 |
| sambetts|cat | o/ mgould | 11:14 |
| * mgould is not a morning person either | 11:15 | |
| *** skramaja has quit IRC | 11:25 | |
| *** moshele has quit IRC | 11:28 | |
| *** moshele has joined #openstack-ironic | 11:32 | |
| *** edand has joined #openstack-ironic | 11:39 | |
| *** hkominos has quit IRC | 11:42 | |
| *** sdake has joined #openstack-ironic | 11:48 | |
| *** sdake_ has joined #openstack-ironic | 11:50 | |
| *** e0ne has joined #openstack-ironic | 11:51 | |
| *** jrist has quit IRC | 11:51 | |
| *** jrist has joined #openstack-ironic | 11:51 | |
| *** jrist has quit IRC | 11:51 | |
| *** jrist has joined #openstack-ironic | 11:51 | |
| *** sdake has quit IRC | 11:54 | |
| vdrok | dtantsur: milan btw I will be in brno this saturday, can meet you there if you're around :) | 11:59 |
| *** ccamacho|lunch is now known as ccamacho | 11:59 | |
| dtantsur | vdrok, I think we can :) not sure about my plans, but nothing for now | 11:59 |
| vdrok | dtantsur: okie, I'll pm you then | 12:00 |
| *** pece has joined #openstack-ironic | 12:01 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Tests for testing chassis-create command https://review.openstack.org/293634 | 12:02 |
| *** loki_ has joined #openstack-ironic | 12:05 | |
| TheJulia | vdrok: heh, I had a feeling you squashed the two revs together :) | 12:07 |
| vdrok | TheJulia: yup :) git commit --amend instead of git rebase --continue :) | 12:07 |
| *** lucas-hungry is now known as lucasagomes | 12:13 | |
| *** mbound has joined #openstack-ironic | 12:13 | |
| lucasagomes | vdrok, you should try to zebras (means ribs) while you are there | 12:16 |
| lucasagomes | don't miss it! | 12:16 |
| TheJulia | vdrok: well, it looked mostly good ;) | 12:16 |
| *** tshefi has joined #openstack-ironic | 12:16 | |
| vdrok | lucasagomes: :) will do | 12:16 |
| mgould | meet your protein-intake goals for the week in one meal... | 12:16 |
| * TheJulia is now intruiged | 12:17 | |
| lucasagomes | mgould, lol indeed | 12:17 |
| vdrok | lucasagomes: there should also be pork knee or something like that | 12:18 |
| vdrok | that looked really huge :) | 12:19 |
| mgould | vdrok: it is | 12:19 |
| lucasagomes | vdrok, yup! Tho I'm not sure if I've tried that yet | 12:19 |
| tshefi | Any one successfully use Ironic pxe_ssh driver with esxi/vcenter? | 12:20 |
| lucasagomes | tshefi, I'm about to asnwer ur email... but tl;dr I haven't tried esxi | 12:22 |
| tshefi | lucasagomes, thanks dude im almost ready to give up on this :( | 12:23 |
| lucasagomes | tshefi, one option, have you tried to use it with VirtualBMC ? | 12:23 |
| lucasagomes | tshefi, https://github.com/openstack/virtualbmc | 12:23 |
| tshefi | lucasagomes, no didn't know about this I'll check it out now, thanks | 12:24 |
| lucasagomes | tshefi, here's a demo:https://raw.githubusercontent.com/umago/virtualbmc/master/images/demo.gif | 12:24 |
| lucasagomes | tshefi, so basically it converts ipmi commands (like ipmitool) to libvirt | 12:24 |
| *** phuongnh has quit IRC | 12:24 | |
| lucasagomes | and libvirt supports esxi as per http://lbvirt.org/ | 12:24 |
| lucasagomes | so you can create a vbmc for the vms and then enroll the node in ironic using the default pxe_ipmitool driver | 12:25 |
| tshefi | lucasagomes, nice I'll go over it and try. thanks | 12:25 |
| lucasagomes | tshefi, cool | 12:26 |
| *** rbudden has joined #openstack-ironic | 12:27 | |
| tshefi | lucasagomes, one more thing i forgot to mention on the email if it helps, on esxi/vc I don't even see ssh attempts from ironic, only when I try ssh. | 12:27 |
| lucasagomes | tshefi, strange, I thought it was trying since the error in the email is "SSH connection cannot be established" | 12:29 |
| tshefi | lucasagomes, exactly what i was thinking till i figured lets look at esxi's access logs | 12:30 |
| tshefi | lucasagomes, which means it's probably not even opening it or failing before it actually attempts to ssh. | 12:31 |
| tshefi | lucasagomes, any way if you want to test this or look at it, ping me up any time. | 12:31 |
| *** clenimar__ has quit IRC | 12:32 | |
| lucasagomes | tshefi, will do | 12:32 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Tests for testing chassis-create command https://review.openstack.org/293634 | 12:33 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Tests for testing node-create command https://review.openstack.org/262055 | 12:33 |
| *** caoshufeng has quit IRC | 12:34 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Tests for testing node-create command https://review.openstack.org/262055 | 12:37 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Tests for testing chassis-create command https://review.openstack.org/293634 | 12:40 |
| thiagop | Good morning, Ironicers | 12:40 |
| *** rloo has joined #openstack-ironic | 12:41 | |
| lucasagomes | thiagop, hi there | 12:41 |
| TheJulia | good morning rloo | 12:42 |
| vdrok | morning thiagop and rloo | 12:42 |
| TheJulia | good morning thiagop | 12:42 |
| *** skramaja has joined #openstack-ironic | 12:42 | |
| thiagop | morning lucasagomes TheJulia vdrok rloo | 12:43 |
| milan | vdrok, awesome, let's have a ton of beer&bbq :D | 12:43 |
| rloo | morning TheJulia, vdrok, thiagop, lucasagomes, milan! | 12:44 |
| vdrok | milan: not sure about a ton of beer, but bbq sounds good :) | 12:44 |
| milan | good morning rloo :) | 12:44 |
| rloo | vdrok: think we can get all the network patches done today? :) | 12:44 |
| milan | vdrok, ack :) | 12:44 |
| vdrok | rloo: I have 3 more hours till pto so might be :D | 12:45 |
| rloo | vdrok: OH. when will you be back? | 12:45 |
| *** vmud213 has joined #openstack-ironic | 12:45 | |
| vdrok | on tuesday | 12:45 |
| rloo | vdrok: anyone going to take over your stuff in the mean time? | 12:45 |
| vmud213 | good morning all | 12:45 |
| vdrok | rloo: yup, I think vsaienk0 will be able to resolve stuff if needed | 12:46 |
| rloo | vdrok: awesome. If I hadn't met you both, I'd say you were the same person :) | 12:46 |
| vdrok | rloo: there won't be much to change anyway after the first one is in :) | 12:46 |
| vdrok | heh | 12:46 |
| vdrok | morning vmud213 | 12:47 |
| rloo | vdrok: have a great PTO! | 12:47 |
| vdrok | thx rloo :) | 12:47 |
| vmud213 | morning vdrok | 12:47 |
| *** sdake_ has quit IRC | 12:47 | |
| vmud213 | lucasagomes: one question related to image building with rhel7. | 12:48 |
| *** daemontool_ has quit IRC | 12:49 | |
| lucasagomes | vmud213, shoot (tho you may want to ask at #tripleo since they take care of building the images) | 12:49 |
| lucasagomes | rloo, morning | 12:49 |
| thiagop | morning vmud213 | 12:49 |
| *** sdake has joined #openstack-ironic | 12:49 | |
| vmud213 | lucasagomes: During the image building process when it is trying to execute grub2-mkinstall it fails with some error saying /usr/lib/grub/x86_64-efi/modinfo.sh doesn't exist | 12:49 |
| milan | dtantsur & vdrok's bbq is now in milan's calendar, ifarkas how about you and zebra? this Saturday in Brno? ;) | 12:49 |
| vmud213 | morning thiagop | 12:49 |
| milan | of course anyone who can manage let's meet here :) | 12:49 |
| ifarkas | milan, lol, that would be awesome | 12:50 |
| *** daemontool_ has joined #openstack-ironic | 12:50 | |
| vmud213 | lucasagomes: after googling ii got a link, https://bugzilla.redhat.com/show_bug.cgi?id=1101352 which says to install grub2-efi-modules | 12:50 |
| openstack | bugzilla.redhat.com bug 1101352 in grub2 "[UEFI] grub2-install grub2-install: error: modinfo.sh doesn't exist" [Unspecified,Closed: notabug] - Assigned to pjones | 12:50 |
| milan | ifarkas, cool :) let's meet in Brno then on Saturday! :D | 12:50 |
| vdrok | hope there will be no rain | 12:51 |
| vmud213 | llucasagomes:but i could not get it? Any clue? I remember i faced the same issue when fixing issue with fedora uefi local boot with partition images. | 12:51 |
| ifarkas | milan, ack, but not this Saturday :D | 12:51 |
| mgould | morning thiagop TheJulia rloo | 12:52 |
| milan | ifarkas, aaah, too bad, vdrok it's this Saturday, right? | 12:52 |
| rloo | hi mgould | 12:52 |
| vdrok | milan: yup :( | 12:52 |
| mgould | morning vmud213 | 12:52 |
| lucasagomes | vmud213, you couldn't install grub2-efi-modules? Or you installed and it failed anyway? | 12:52 |
| vmud213 | lucasagomes: I tried in tripleO related to redhat image building issues..But not much response..You are the one atleast i know are part of redhat.. | 12:53 |
| vmud213 | so poking here :) | 12:53 |
| *** jistr is now known as jistr|cowork | 12:53 | |
| sambetts|cat | vmud213: what call are you using? are you using the grub2 element or manaually listing the grub2 package ?? | 12:53 |
| milan | vdrok, ifarkas, OK there's enough BBQ for both this and next Saturday in Brno :) | 12:53 |
| vdrok | woohoo :) | 12:53 |
| vmud213 | Actually i am building it for vm which will include bootloader and grub2 | 12:53 |
| * vmud213 looking into thye elements listed | 12:54 | |
| sambetts|cat | what does your DIB command look like? | 12:54 |
| vmud213 | sambetts|cat: disk-image-create rhel7 vm dhcp-all-interfaces -o rhel7-dib-mkfs-uefi | 12:55 |
| vmud213 | lucasagomes,sambetts|cat:The element dependencies resolved to the following "rhel7 dib-python rpm-distro install-types install-bin redhat-common dib-run-parts rhel-common vm manifests base pkg-map yum cache-url source-repositories dib-init-system bootloader dhcp-all-interfaces package-installs" | 12:56 |
| sambetts|cat | hmm, so your not using the grub2 or localboot elements :/ | 12:56 |
| vmud213 | morning mgould | 12:56 |
| lucasagomes | vmud213, as sambetts|cat I think you need to add "grub2" to the list of elements there | 12:57 |
| sambetts|cat | not sure if you do if your using VM not baremetal | 12:57 |
| sambetts|cat | but the grub2 element does include a shim that fixes some problems with installing grub2 | 12:58 |
| lucasagomes | https://github.com/openstack/diskimage-builder/blob/7c6f91fe37f1cff4f486e8f20edf501445bc4d0f/elements/grub2/pkg-map#L16 | 12:58 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Add create command to ironic client https://review.openstack.org/328955 | 12:59 |
| vmud213 | lucasagomes: I have pasted the relevent log here http://paste.openstack.org/show/531698/ | 13:01 |
| vmud213 | lucasagomes,sambetts|cat: The packages are already included | 13:01 |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: Discover driver type, rotational speed, raid_level, controller name and location. https://review.openstack.org/341370 | 13:02 |
| vmud213 | shim only comes in picture if we are interested in secure boot | 13:02 |
| *** moshele has quit IRC | 13:02 | |
| *** moshele has joined #openstack-ironic | 13:03 | |
| *** radek_ has quit IRC | 13:08 | |
| vmud213 | lucasagomes,sambetts|cat: Will try including grub2 element and see how it goes..Thanks for the pointers.. | 13:09 |
| lucasagomes | vmud213, yeah I don't have much idea, I have to try as well | 13:10 |
| lucasagomes | and sorry cause I'm kinda looking at other stuff at the same time | 13:10 |
| *** links has quit IRC | 13:11 | |
| vmud213 | lucasagomes: no prob..will try... | 13:11 |
| rloo | hi milan, i just saw your email reply wrt network_interface. | 13:13 |
| milan | rloo, hi, wdyt? | 13:13 |
| rloo | milan: did anyone discuss with you? they met yesterday about it. | 13:13 |
| milan | rloo, nope | 13:14 |
| rloo | milan: i am not up to date on what they decided yesterday. | 13:14 |
| rloo | milan: but... | 13:14 |
| milan | I hope I wasn't too much out of line | 13:14 |
| rloo | milan: yes, you were very out of line :D ANd TOOO LATE :D but seriously, i think it is good that you replied. | 13:14 |
| milan | rloo, lol OK | 13:14 |
| milan | :D | 13:14 |
| *** baoli has joined #openstack-ironic | 13:14 | |
| * TheJulia has not looked at email yet today | 13:15 | |
| rloo | milan: give me a few minutes to see what they decided yesterday. unless sambetts|cat is around or someone else that was at that meeting. | 13:15 |
| * TheJulia opens email | 13:16 | |
| milan | rloo, thx | 13:16 |
| *** sdake has quit IRC | 13:16 | |
| *** adu has joined #openstack-ironic | 13:16 | |
| rloo | milan: this is the etherpad: https://etherpad.openstack.org/p/ironic_network_interface_discussion | 13:16 |
| sambetts|cat | milan: and this is the patch updating the driver comp spec with comments: https://review.openstack.org/#/c/341084/ | 13:17 |
| milan | rloo, sambetts|cat thanks | 13:18 |
| *** baoli_ has joined #openstack-ironic | 13:19 | |
| rloo | sambetts|cat: so you all actually decided that a hardware_type could not (or should not?) specify a default_network_interface? | 13:19 |
| *** chlong has quit IRC | 13:20 | |
| sambetts|cat | As I understand it yes | 13:20 |
| rloo | sambetts|cat: so 'should not' means we don't think they should but they can. 'could not' means we don't provide the ability to do it. | 13:20 |
| TheJulia | Yes, that ws the consensus | 13:20 |
| sambetts|cat | rloo: I'm pretty sure we went with could not | 13:21 |
| *** ametts has joined #openstack-ironic | 13:21 | |
| rloo | sambetts|cat: it sez "shouldn't". | 13:21 |
| TheJulia | anytime we approached should, we all started to get headaches due to the implications | 13:21 |
| rloo | sambetts|cat, TheJulia: sigh. why didn't someone put 'could not' or 'cannot'... now what. | 13:22 |
| *** baoli has quit IRC | 13:22 | |
| TheJulia | sambetts|cat: is your hardware case the case where it could be useful to allow it to be specified? | 13:22 |
| sambetts|cat | no, my hardware case is actual an ideal argument against having a default | 13:23 |
| TheJulia | rloo: weeds grow quickly in cut fields? :) | 13:24 |
| *** piet has joined #openstack-ironic | 13:24 | |
| TheJulia | sambetts|cat: okay, in that case I just think the notes need to be fixed | 13:24 |
| rloo | TheJulia: ha ha. | 13:24 |
| rloo | TheJulia, sambetts|cat: if I understand what you all decided, I really don't like the inconsistency between network_interface vs the rest of the interfaces, and it makes it look like it should not be part of the hardware type's interfaces :-( | 13:25 |
| * milan simplified network_interface in his mind to an attribute of the conductor rather than of the node | 13:25 | |
| rloo | TheJulia, sambetts|cat: I mean, with this proposal, why not just have a node.network_thing and a config default_network_thing? | 13:26 |
| sambetts|cat | because we still need hardware_types to be able list supported_network_things | 13:26 |
| sambetts|cat | IMO hardware_types are for listing supported things not things to use | 13:26 |
| rloo | sambetts|cat: makes me sad if we special case the network interface from the other interfaces. I wanted a generic/consistent model for them all. | 13:28 |
| sambetts|cat | rloo: well its not the only one either, volume interfaces is also a special cases, and is even not consistent with network interfaces | 13:30 |
| sambetts|cat | special case* | 13:30 |
| TheJulia | networks and storage are both very... different things that may have some influence from hardware, but ultimately rest upon the operator. It would kind of be the same case for power if someone had a transparent redfish to ipmi converter hidden in the network | 13:30 |
| TheJulia | sambetts|cat: I was actually thinking about that, and we could make it consistent, the operator still has to opt in by populating capabilities | 13:30 |
| TheJulia | unless inspector magically populates that, then... *headache* | 13:31 |
| rloo | sambetts|cat, TheJulia: I guess I don't like the 'cannot' restriction. Anyway, as long as we can get the network stuff in, i can think about the driver composition stuff later. | 13:31 |
| jroll | devil's advocate question: for network/volumes, what if we just hardcoded the default that should work in 'most' environments? | 13:31 |
| jroll | instead of leaving the default to the ops | 13:31 |
| sambetts|cat | we discussed making the noop interface that | 13:32 |
| jroll | I'm kind of tired of saying "making a decision is too hard, leave it to deployers" | 13:32 |
| jroll | hence our massive amount of config options | 13:32 |
| rloo | jroll: regardless of whether we put in a default (or suggested default), i think we'd need a config option for this particular case. | 13:33 |
| TheJulia | jroll: ops may have separate networks, so magically expecting consistency with a default would cause headaches for anyone who does not mach that environment or configuration exactly | 13:33 |
| rloo | i actually like config options, the more the merrier. | 13:33 |
| * jroll counts 378 configs in our sample file at the moment | 13:34 | |
| * dtantsur also likes more options | 13:34 | |
| rloo | jroll: true (well, assuming you can count), but not all those are from *our* code. | 13:34 |
| jroll | TheJulia: well, we can choose the most common or recommended | 13:34 |
| jroll | rloo: I understand | 13:34 |
| rloo | jroll: are the number of config options, something that ops folks complain about? | 13:35 |
| rloo | jroll: as long as the defaults are appropriate, they shouldn't have to even touch them. and if they have to touch them, then it is good that there are configs to tweak. | 13:35 |
| jroll | rloo: a common request is making deploying openstack easier; part of that is the number of config options | 13:35 |
| TheJulia | jroll: but then if the operator is in a different environment, what do they have to go through to change what the community thinks is a default? | 13:35 |
| dtantsur | I disagree that less options make deployment easier | 13:36 |
| jroll | TheJulia: add --network-interface foo to their node enrollment script | 13:36 |
| * rloo thinks of a good panel discussion at summit. more or fewer config options... | 13:36 | |
| dtantsur | we have to tune a surprising amount of things for tripleo to work sanely... | 13:36 |
| TheJulia | and if their environment is a completely different driver? | 13:36 |
| TheJulia | that is contrary to the default? | 13:36 |
| TheJulia | and there is no reason for the default to exist in their environment? | 13:36 |
| jroll | TheJulia: not sure what you're saying, I'm talking specifically about e.g. making the default 'flat' always | 13:37 |
| jroll | and overrideable via node.network_interface | 13:37 |
| dtantsur | for me more options is like more logs | 13:37 |
| TheJulia | jroll: that is how it is coded now for backwards compatability | 13:37 |
| sambetts|cat | TheJulia argument is my argument about removing all defaults from hardware_types | 13:37 |
| dtantsur | they don't get in way when everything is fine, and they help a lot when it's not | 13:38 |
| jroll | TheJulia: okay, so my question is, what's the issue with that which requires adding a default_network_interface config? | 13:38 |
| sambetts|cat | people deploying into a scenario where they want to use neutron for all created nodes and don't want to override it everytime | 13:39 |
| TheJulia | jroll: operator convenience as an override for the default flat setting at present. | 13:39 |
| jroll | okay, so the big question here is, is that convenience worth the inconsistency, right? rloo? | 13:39 |
| *** mbound_ has joined #openstack-ironic | 13:39 | |
| dtantsur | jroll, if we don't have the default, people using old API versions are out of luck | 13:40 |
| jroll | and a secondary question is, should we do it for all interfaces and remove the inconsistency, which it sounds like y'all decided 'no' | 13:40 |
| sambetts|cat | I say remove all defaults and make everything explict | 13:40 |
| sambetts|cat | implict things just lead to problems | 13:41 |
| jroll | dtantsur: people using the old API version can't add switchport data, they're out of luck anyway :) | 13:41 |
| dtantsur | jroll, but they also won't be able to switch from "flat" to "manual". so their nodes will require non-existing neutron | 13:41 |
| milan | rloo, sambetts|cat what if instead a default_X_interface it was a per-conductor.X_interface that is used when talking to X ("network") so hardware_type needn't specify this? I may miss the reason why hardware_type should verify network interface compatibility... | 13:42 |
| jroll | dtantsur: "manual"? | 13:42 |
| TheJulia | noop | 13:42 |
| jroll | is that the new noop | 13:42 |
| jroll | ok | 13:42 |
| dtantsur | ah | 13:42 |
| dtantsur | yeah, sorry, I was using my old suggestion | 13:43 |
| *** mbound has quit IRC | 13:43 | |
| *** trown|outtypewww is now known as trown | 13:43 | |
| dtantsur | so if "flat" won't be available or not usable for any reason, new nodes will end up effectively broken | 13:43 |
| dtantsur | s/won't/isn't/ | 13:43 |
| jroll | today it chooses noop or flat based on dhcp provider, so I think that will work for them | 13:43 |
| * TheJulia resists urge to make a joke about noop | 13:43 | |
| rloo | milan: cuz that adds a new/different *thing* and volume interface may be a similar issue so I think we'd like to have some uniform way to deal with them. | 13:43 |
| *** amotoki has joined #openstack-ironic | 13:43 | |
| dtantsur | jroll, this is an implicit default btw | 13:44 |
| dtantsur | jroll, if we say "remove defaults", we can't use dhcp providers for the same reason | 13:44 |
| jroll | dtantsur: didn't say it wasn't, just trying to understand everything here | 13:44 |
| sambetts|cat | milan: if we specify default_X_interface then it has to be validated against enabled hardware_types so that we know its a valid fall back for nodes created with any hardware_type | 13:44 |
| rloo | jroll: wrt convenience vs consistency. I think the *user* is our customer, so I go with trying to address the user's concerns first, before code consistency. | 13:45 |
| *** mgoddard_ has joined #openstack-ironic | 13:45 | |
| jroll | rloo: well, it's about API and configuration consistency, too, right? | 13:45 |
| dtantsur | milan, per-conductor settings are nearly never a good idea, cause a node can change conductors during its life cycle | 13:45 |
| milan | sambetts|cat, it makes sense to me for other interfaces but I've got the impression that the network_interface is different here; why would the hardware_type verify that network is provided through neutron | 13:46 |
| milan | or that it isn't | 13:46 |
| TheJulia | milan: also conductor take_over upon conductor failure would complicate matters with per-conductor config | 13:47 |
| rloo | jroll: yup. that's true. | 13:47 |
| jroll | milan: because cisco makes crazy weird servers :) | 13:47 |
| sambetts|cat | milan: huh? It doesn't we're not suggesting that, the hardware_type does no validation | 13:47 |
| sambetts|cat | hardware_type just lists supported things | 13:47 |
| sambetts|cat | conductors do validation that a default is supported by all hardware_types | 13:47 |
| rloo | jroll: which is why i want to allow for defaults/settings at every level, node, hwtype, global (config) | 13:47 |
| milan | sambetts|cat, precisely, so in supported_types you'd declare particular net? | 13:47 |
| milan | *network type? | 13:47 |
| sambetts|cat | yes, because I can make a network interface that is tied to my server, because I have magic nics | 13:48 |
| *** mgoddard has quit IRC | 13:48 | |
| jroll | rloo: right, okay got it | 13:48 |
| milan | sambetts|cat, OK that makes sense then | 13:48 |
| sambetts|cat | its not related to whether or not its neutron or not, its whther or not it interacts with my magic nics | 13:48 |
| jroll | so, if this is for user convenience, why doesn't for example default_power_interface make sense? | 13:49 |
| * milan had the impression this was rather about the switch than the nic | 13:49 | |
| sambetts|cat | jroll: default_power_interface in config or hardware_type? | 13:49 |
| jroll | sambetts|cat: config | 13:49 |
| jroll | I see in the etherpad: "default configs don't make sense for other interfaces (power, mgmt, etc)" | 13:49 |
| jroll | why? | 13:50 |
| sambetts|cat | jroll: how would that play with the hardware_type default for that interface? | 13:50 |
| jroll | sambetts|cat: that's unrelated to if it makes sense, right? | 13:50 |
| jroll | sambetts|cat: though, I would think it'd just override it | 13:50 |
| dtantsur | sambetts|cat, the same question about network interfaces. what if an hw type has the default? | 13:50 |
| rloo | sambetts|cat: it only "plays" with hardware_type default for that interface, if the hw-default is 'use the config value' | 13:51 |
| vdrok | hw-type default overriding config default should be ok | 13:51 |
| sambetts|cat | also specifying a default in the config means that all hardware_types must have an intersect between supported_x_interfce | 13:51 |
| jroll | let's slow down, I want to understand why default_power_interface doesn't make sense | 13:51 |
| *** amotoki has quit IRC | 13:51 | |
| sambetts|cat | ^ specifying a default in the config means that all hardware_types must have an intersect between supported_x_interfce | 13:51 |
| jroll | I'm not sure who said/wrote that, but it sounds like a convenience to me | 13:52 |
| jroll | sure | 13:52 |
| rloo | sambetts|cat: so that is what i don't understand. why does a config default mean that all hwtypes must have an intersect between supported_x_interface? | 13:52 |
| jroll | OR it means that we fail if a hardware type without the default is used, without specifying a different power interface | 13:52 |
| sambetts|cat | rloo: because otherwise Ironic would set an implementation unsupported by a hardware_tyoe | 13:53 |
| jroll | or throw a 400 | 13:53 |
| *** jcoufal has joined #openstack-ironic | 13:53 | |
| *** PollyZ has joined #openstack-ironic | 13:53 | |
| jroll | right? | 13:53 |
| jroll | we need a whiteboard :| | 13:54 |
| rloo | sambetts|cat: guess i don't understand. a hwtype has a list of supported x_interfaces. a hwtype can specify the default x_inteface. if default x_interface is in supported, all is ok. regardless of default_x-config value. | 13:54 |
| sambetts|cat | thats another solution we discussed, but that means that all nodes of particular hardware_type will always have to be overriden | 13:54 |
| TheJulia | I think it would be conductor failing to start territory | 13:54 |
| jroll | sambetts|cat: sure, that's fine | 13:54 |
| jroll | TheJulia: why? | 13:54 |
| *** mgoddard has joined #openstack-ironic | 13:55 | |
| TheJulia | I think validation was discussed at start-up of the conductor so if matches were not found based upon defined drivers, it seemed like in the dicussion that we wouldn't have a working conductor | 13:55 |
| *** mgoddard_ has quit IRC | 13:55 | |
| jroll | sambetts|cat: fwiw, I don't see a heterogenous deployment *using* this option at all | 13:55 |
| jroll | but take a deployment of identical supermicro boxes that wants to use pyghmi | 13:55 |
| sambetts|cat | rloo: I have an environment with 2 loaded hw_types, which don't support any of the same power interfaces, no default will work for this environemnt | 13:56 |
| jroll | pyghmi won't be the hw_ype default | 13:56 |
| jroll | sambetts|cat: if you're deploying that environment, you'd probably not set that default, right? | 13:56 |
| TheJulia | jroll: we need a really large white board | 13:56 |
| jroll | TheJulia: ikr | 13:56 |
| rloo | sambetts|cat: does 'default-anything' value mean that the whole universe has to work with default-anything? | 13:56 |
| TheJulia | jroll: and whisky | 13:56 |
| jroll | +1 | 13:57 |
| sambetts|cat | rloo: yes, if we fall back to that value if we don't get given another | 13:57 |
| sambetts|cat | for everything | 13:57 |
| *** amotoki has joined #openstack-ironic | 13:58 | |
| rloo | sambetts|cat: so *IF* a hwtype chooses to set it's default to the config setting, yes. but otherwise, I don't see why. | 13:58 |
| TheJulia | This compeltely seems like a conundrum we can solve later, and in order to come to a happy place for now, we can remove the default_network_interfaces setting in the network patchset chain, and just force users to define their network interface if not flat for Newton | 13:58 |
| TheJulia | And by later, I mean in-person | 13:58 |
| sambetts|cat | rloo: we said that config values will override hw_Type defaults | 13:58 |
| rloo | sambetts|cat: and if the user specifies a default that doesn't make sense, we have enough checks in place to error out. | 13:59 |
| loki_ | review please https://review.openstack.org/#/c/272658/ | 13:59 |
| rloo | sambetts|cat: OH. who said that? Well it wasn't in my proposal. i need to look at what you guys decided. | 13:59 |
| rloo | sambetts|cat: and i disagree. config values never? do they? override higher level settings. | 13:59 |
| *** vmud213 has quit IRC | 14:00 | |
| sambetts|cat | rloo: config value is a higher level setting than the hardware_types default, because the hardware_type default can not be changed | 14:00 |
| rloo | sambetts|cat: yikes. | 14:01 |
| jroll | I would think config would need to override hw_type, right? /me goes back to the ipmi case with default_power_interface=pyghmi | 14:01 |
| jroll | but the other way around is interesting | 14:02 |
| rloo | sambetts|cat: so you're saying that *if* a hwtype could specify a default interface value, and *if* there was a corresponding config default interface, the config default interface would override the hw default? | 14:02 |
| TheJulia | an override is intruiging | 14:02 |
| sambetts|cat | yes | 14:02 |
| rloo | sambetts|cat: that isn't how configs normally work, unless I guess, you call it 'an override'. | 14:02 |
| rloo | sambetts|cat: is this really the functionality that is being asked for? | 14:02 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Tests for testing port-create command https://review.openstack.org/291802 | 14:03 |
| sambetts|cat | well then default_X_interface in the config file doesn't make any ssense | 14:03 |
| jroll | I think the hierarchy here is node > config > hw type | 14:03 |
| jroll | because hw type is code | 14:03 |
| sambetts|cat | +! | 14:03 |
| sambetts|cat | +1 | 14:03 |
| *** amotoki has quit IRC | 14:03 | |
| rloo | i think the hierarchy here is node -> hwtype -> config. it is *only* if hwtype is unspecified (ie, specified to use config value), that config is used. | 14:04 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Add internal_info field to port https://review.openstack.org/341552 | 14:04 |
| *** wajdi has joined #openstack-ironic | 14:04 | |
| jroll | rloo: right, I guess I misunderstood that | 14:04 |
| rloo | did i ever mention that i didn't like hwtype being in code? but i think that was to enforce things. | 14:04 |
| *** yuriyz has quit IRC | 14:05 | |
| jroll | I shouldn't say I think that's the hierarchy, I should say that's how I understood the hierarchy being discussed | 14:05 |
| sambetts|cat | and this is why I hate hardware_types having defaults | 14:05 |
| rloo | jroll: *unless* we don't call it a default config, but an override config I guess. | 14:05 |
| jroll | right | 14:05 |
| rloo | so i guess i don't understand the functionality we are looking for | 14:05 |
| sambetts|cat | I'm going back to IMO hardware_types should specifc support implementations and not which implementation to use in anyway | 14:06 |
| sambetts|cat | specify* | 14:06 |
| rloo | are we saying that if a hwtype can specify default network='flat', that operators will want to override that and not via node.network_interface but via a network_interface_thingy config? | 14:06 |
| *** loki_ has quit IRC | 14:07 | |
| jroll | rloo: that's what I'm hearing, but not that operators will want to override that, but providing it as a convenience | 14:07 |
| rloo | or is sam saying, hwtype cannot specify any default interface, only supported; and the corresponding config specifies the default? | 14:07 |
| jroll | rloo: and sam would prefer what you just said | 14:07 |
| sambetts|cat | rloo: I don't even really want config options | 14:07 |
| rloo | jroll: then what's the use of a hwtype having a default? | 14:07 |
| TheJulia | rloo: essentially yes because it could all be neutron for them, or they only want to explicity opt in the flat nodes in the special clsuter over int he corner | 14:07 |
| jroll | oh | 14:07 |
| jroll | rloo: for when the operator does not want to specify the default, I guess? | 14:08 |
| rloo | although hwtype (at least for corresponding classic drivers) need to have defaults :) | 14:08 |
| jroll | idk, I wrote down the two cases I see here: https://etherpad.openstack.org/p/ironic_network_interface_discussion | 14:08 |
| jroll | or how I see this working, if we have a default_*_interface for everything | 14:08 |
| sambetts|cat | IMO you put a node into Enroll and then you have to update all the driver_info anyway to make it get through validation, so why not just make setting the interface explict per node | 14:08 |
| sambetts|cat | no defaults anywher | 14:09 |
| jroll | I also would like consistency, so I'm trying to understand why we need default_network_interface but default_anythingelse_interface doesn't make sense | 14:09 |
| rloo | sambetts|cat: cuz when i am testing, i am too lazy to get that command correct. | 14:09 |
| TheJulia | sambetts|cat: then we have no "explicitly known to work" base configurations | 14:09 |
| rloo | sambetts|cat: and it goes back to jroll saying we need to make it easy for operators/usage; pick sane defaults | 14:10 |
| jroll | what I would like to get to => a modern ironic deployment with e.g. opencompute hardware is just specifying --hardware-type ipmi and ipmi creds | 14:11 |
| jroll | and you get a relatively secure deployment, tenant isolation, etc | 14:11 |
| *** alex_xu has quit IRC | 14:11 | |
| jroll | more medium term because back-compat, but yeah | 14:11 |
| sambetts|cat | jroll: I would prefer --hardware_type opencompute --power-interface ipmi because then I know what I'm getting | 14:12 |
| *** joprovost has joined #openstack-ironic | 14:12 | |
| *** sdake has joined #openstack-ironic | 14:12 | |
| jroll | sambetts|cat: s/opencompute/generic whitebox/ | 14:12 |
| *** alex_xu has joined #openstack-ironic | 14:12 | |
| jroll | sambetts|cat: you'd know what you were getting because the hardware type docs would tell you that the default is ipmi :) | 14:13 |
| rloo | sambetts|cat: and/or by querying ironic, whenever we have an API for getting that info... | 14:13 |
| sambetts|cat | and if as a deployer I blcok ipmi in my environemtn and want to force nodes to use WOL how do I do? | 14:13 |
| jroll | sambetts|cat: --power-interface wol | 14:14 |
| rloo | sambetts|cat: is WOL a supported interface? if so, what jroll said ^^ | 14:14 |
| rloo | sambetts|cat: if it isn't supported, ... complain to dtantsur :) | 14:14 |
| jroll | fwiw, I don't believe that any production deployment of ironic will have a person sitting at a computer enrolling nodes one by one, but rather some sort of script or tool | 14:14 |
| sambetts|cat | jroll: but if I don't specify it then the node will be created with IPMI even though as a deployer I know it won't work | 14:14 |
| jroll | sambetts|cat: then why wouldn't you specify it? | 14:15 |
| rloo | sambetts|cat: so specify it. you are allowed to do that. | 14:15 |
| sambetts|cat | jroll: depends if the deployer is the user of Ironic or not right? They might not know? | 14:15 |
| sambetts|cat | case when deployer != operator | 14:16 |
| jroll | if the person onlining hardware doesn't know that IPMI won't work in the environment, there's a bigger problem somewhere | 14:16 |
| jroll | I don't believe that the person enrolling hardware is completely oblivious to the datacenter architecture | 14:16 |
| rloo | sambetts|cat: the person might not know what? | 14:16 |
| jroll | ever | 14:16 |
| jroll | that's just silly | 14:16 |
| sambetts|cat | then why not just remove deaults and make it explict ? | 14:16 |
| sambetts|cat | because if in my env I don't want IPMI then every node I enroll I have to override | 14:17 |
| rloo | sambetts|cat: that's like asking why not remove *all* defaults, even for config values and make the user/operator specify *everything* | 14:17 |
| jroll | because in most cases where someone is dpeloying generic whitebox hardware, they'll want to use ipmi | 14:17 |
| jroll | and that's what we would recommend | 14:17 |
| jroll | just like if someone uses ilo, we'd recommend ilo, they'd likely use ilo. why make them specify it for equality with the one operator out there that uses ipmitool with ilo hardware? | 14:18 |
| *** ChrisAusten has joined #openstack-ironic | 14:18 | |
| sambetts|cat | comes back to consistacy vs convience I guess | 14:18 |
| jroll | depends how you define consistency :) | 14:19 |
| jroll | in this case, consistent to me means: | 14:19 |
| jroll | every hardware type is documented, with the defaults | 14:19 |
| jroll | when you don't specify *_interface, the documented default is used | 14:19 |
| sambetts|cat | consistent means to me: If I run ironic node-create --hardware_type X that behaves the same, and if the default works in some clouds but not others than that is inconsistent | 14:21 |
| sambetts|cat | if we make it explict, then operators can disable certain interfaces, and if I make an ironic node-create call with an interface that is disabled then I know | 14:22 |
| sambetts|cat | its not going to work | 14:22 |
| sambetts|cat | and it'll fail | 14:22 |
| jroll | well | 14:22 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Add internal_info field to port https://review.openstack.org/341552 | 14:22 |
| jroll | disabling interfaces is a separate topic, right? | 14:23 |
| jlvillal | krtaylor, sambetts|cat: Any chance one of you could chair the QA meeting today? I am supposed to get my new work laptop 30 minutes before the meeting starts. So I don't think I will make it :( | 14:23 |
| jlvillal | krtaylor, sambetts|cat The appointment was made over a month ago, and I can't change the time :( | 14:23 |
| sambetts|cat | not really because specifying a default_X_interface automatically adds it to the enabled_interfaces list | 14:23 |
| sambetts|cat | jroll: ^ | 14:23 |
| mariojv | morning ironic | 14:23 |
| mariojv | is there a way to see summit abstracts related to ironic that have been submitted yet? | 14:24 |
| jroll | mariojv: not until submissions close | 14:24 |
| sambetts|cat | jroll: so as an operator I'm forced to enable certain interfaces even if I know it won't work in my environemt | 14:24 |
| jlvillal | mariojv, Yes. Just hack into the submission system :P | 14:24 |
| mariojv | thanks | 14:24 |
| jlvillal | mariojv, Otherwise, I don't think so :( | 14:24 |
| *** dtantsur is now known as dtantsur|brb | 14:25 | |
| jroll | sambetts|cat: grumble | 14:25 |
| rloo | sambetts|cat: not sure i understand. why would the operator enable an interface that doesn't work? wouldn't they want to specify the one that does work? | 14:25 |
| sambetts|cat | rloo: thats my whole point, default_X_interfaces take that out of the operators control | 14:25 |
| jroll | rloo: the default for a hw type is always enabled | 14:25 |
| *** mtanino has joined #openstack-ironic | 14:25 | |
| rloo | sambetts|cat, jroll: but that's why you just override it at the node level? | 14:26 |
| vdrok | sambetts|cat: they still can be overriden per node | 14:26 |
| *** sabeen has joined #openstack-ironic | 14:26 | |
| *** wajdi has quit IRC | 14:26 | |
| rloo | sambetts|cat: i mean, isn't that what you were advocating with your node-create command? | 14:26 |
| jroll | right, so sambetts|cat point is that there's a vector for creating a broken node, if you fail to override it | 14:27 |
| jroll | if the default does not work | 14:27 |
| *** wajdi has joined #openstack-ironic | 14:27 | |
| rloo | jroll: right. isn't that the way other thigns work or don't work? if the default doesn't work in your environment, you change it. | 14:27 |
| jroll | I honestly don't think that's a major problem, I believe that operators should/will write a script for enrolling nodes | 14:27 |
| rloo | i mean, regardless of default network/power/whatever. that's the nature of defaults. | 14:28 |
| jroll | and in this case that script would always pass the override | 14:28 |
| sambetts|cat | rloo: but you can't change the default for a hardware_tpye | 14:28 |
| jroll | rloo: you can't change the default interfaces for a hw type | 14:28 |
| sambetts|cat | its written in code | 14:28 |
| rloo | sambetts|cat: OH. that is a different problem than what we are trying to address (or what i was initially) | 14:29 |
| rloo | sambetts|cat: first, do we want to allow for that? | 14:29 |
| rloo | sambetts|cat: i am guessing you do. | 14:29 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Tests for testing chassis-create command https://review.openstack.org/293634 | 14:29 |
| rloo | sambetts|cat: so, after thinking about it, i am not sure we want to allow for that. | 14:30 |
| rloo | sambetts|cat: the whole idea of driver composition, is to allow the configuration/composition to be flexible. | 14:30 |
| rloo | sambetts|cat: so i think it makes sense that you 'change' the default on a per-node basis. | 14:31 |
| rloo | sambetts|cat: the default is a convenience thing. | 14:31 |
| *** sdake has quit IRC | 14:31 | |
| rloo | sambetts|cat: so if as you would like, there were no default, you'd have to specify it at the node level anyway. | 14:31 |
| rloo | sambetts|cat: or am i missing something? | 14:31 |
| *** sdake has joined #openstack-ironic | 14:32 | |
| jroll | rloo: right, sambetts|cat is saying the user should always need to specify, so they always know what they are getting | 14:32 |
| krtaylor | jlvillal, I'm not sure I'll be able to make it either :( but maybe mjturek1 can run the meeting if sambetts|cat can't | 14:33 |
| rloo | jroll, sambetts|cat: so I don't like the 'should always'. I mean, we can say they 'should always', if that's what you want to recommend. | 14:33 |
| rloo | jroll, sambetts|cat: BUT I would oppose it if you said they 'must' | 14:33 |
| jroll | rloo: right, 'must' is the right word here | 14:33 |
| rloo | jroll, sambetts|cat: what's the argument for 'must'? I see no reason to make people do that. | 14:34 |
| sambetts|cat | I think if you remove defaults that hardware_type supported_X_interfaces + enabled_X_interface should be able to gurantee that the node you are creating will work in the environment you are deploying into, defaults remove the control from the enabled_X_interfaces config option | 14:34 |
| jroll | rloo: so it's always consistent | 14:34 |
| jroll | rloo: I agree with you | 14:34 |
| jroll | rloo: the argument is an environment where the default will not work, and the person enrolling nodes is doing it by hand, and does not know this. | 14:35 |
| jroll | which I don't believe is a case that should ever exist | 14:35 |
| rloo | jroll: I don't actually understand that argument. | 14:35 |
| rloo | jroll: if the user doesn't know that some X implementation doesn't work, even if they were to be forced to specify it, they still wouldn't know it didn't work. | 14:36 |
| sambetts|cat | they wouldn't be able to create a node with that implementation though because the operator knowing it doesn't work will have removed it from the enabled_interfaces | 14:36 |
| rloo | jroll: if the X implementation is in the list of enabled/supported, how does it matter that there is or isn't a default? | 14:37 |
| rloo | OH. sambetts|cat, so your concern is that hw-default is *always* considered supported/enabled. | 14:37 |
| sambetts|cat | rloo: yes | 14:38 |
| jroll | yes | 14:38 |
| rloo | sambetts|cat, jroll: that might be a valid concern, honestly, i glossed over all that wrt the driver composition, although i wondered about it. | 14:39 |
| sambetts|cat | krtaylor, jlvillal: I can chair if neither of you are around | 14:39 |
| rloo | sambetts|cat, jroll: so honestly, i really want to get the network stuff done. i suspect we may have more conversations about the driver composition stuff. | 14:39 |
| rloo | sambetts|cat, jroll: can we capture this concern somewhere and try to address it with dtantsur|brb later? | 14:40 |
| jlvillal | sambetts|cat, Thank you! :) | 14:40 |
| sambetts|cat | rloo: right so I think the way we've done the network stuff right now is good to go and isn't blocked by this conversation | 14:40 |
| *** ChrisAusten has quit IRC | 14:40 | |
| *** amotoki has joined #openstack-ironic | 14:40 | |
| sambetts|cat | this is tied more heaviy to the driver comp stuff than anything else | 14:40 |
| rloo | sambetts|cat, jroll: I just want to know two things> 1. are network_interfaces part of the hwtype interfaces; and 2. do we have a story on how the default-interface-config fits into the driver composition stuff. | 14:40 |
| rloo | sambetts|cat: ok great. please bring this up later! | 14:41 |
| *** moshele has quit IRC | 14:41 | |
| rloo | sambetts|cat: cuz https://review.openstack.org/#/c/285852/ has 3 +2's and I'm going to approve it sometime today unless someone disagrees or someone else beats me to approving it | 14:42 |
| sambetts|cat | rloo: I think network_interfaces need to be tied to hardware_type because of cases like mine and oneview's where we have server NIC specific network_interfaces | 14:42 |
| *** vishwanathj has joined #openstack-ironic | 14:42 | |
| rloo | sambetts|cat: right. that makes sense. | 14:42 |
| sambetts|cat | rloo: I'm not sure where to track the concern about default interfaces | 14:43 |
| rloo | maybe in the RFE for that feature? | 14:43 |
| rloo | sambetts|cat: add a comment? and hope we remember? | 14:43 |
| sambetts|cat | Not sure what the timeline of the driver comp stuff is but maybe we need a whiteboard session at the summit about it | 14:44 |
| *** moshele has joined #openstack-ironic | 14:44 | |
| rloo | sambetts|cat: i would *really* like to get that merged in newton. | 14:44 |
| rloo | sambetts|cat: but not as much as i want the network stuff done :) | 14:44 |
| sambetts|cat | :-P | 14:44 |
| mjturek1 | krtaylor: I'd love to but today is the department picnic :( sorry | 14:44 |
| thiagop | not many people to the meeting though... | 14:45 |
| *** moshele has quit IRC | 14:46 | |
| *** moshele has joined #openstack-ironic | 14:58 | |
| *** ijw has joined #openstack-ironic | 14:59 | |
| *** ijw has quit IRC | 15:00 | |
| *** jistr|cowork is now known as jistr|mtg | 15:00 | |
| *** ijw has joined #openstack-ironic | 15:01 | |
| *** links has joined #openstack-ironic | 15:01 | |
| *** vishwanathj has quit IRC | 15:02 | |
| *** vishwanathj has joined #openstack-ironic | 15:02 | |
| *** vishwanathj has quit IRC | 15:03 | |
| *** vishwanathj has joined #openstack-ironic | 15:04 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Tests for testing chassis-create command https://review.openstack.org/293634 | 15:06 |
| *** dtantsur|brb is now known as dtantsur | 15:07 | |
| dtantsur | jroll, rloo, I'm back, could you give me a tl;dr? | 15:07 |
| rloo | dtantsur: ask sambetts|cat :) He is concerned that with the driver composition stuff, the hwtype's default_x_interface will always be considered enabled/supported, even if it isn't in the supported_x or enabled_x list. | 15:08 |
| jroll | dtantsur: in mtgs now :( | 15:09 |
| dtantsur | no worries | 15:09 |
| sambetts|cat | just writing up my concern as a comment on the RFE | 15:09 |
| rloo | dtantsur: so that if for whatever reason, that default_x_interface doesn't work in their environment, the operator has no way to disable/say-don't-use-it. | 15:09 |
| rloo | thx sambetts|cat | 15:09 |
| dtantsur | that's correct | 15:09 |
| dtantsur | by enabling the hardware type you enable the default interface. this is a limitation, but it's a useful limitations for the majority of cases | 15:10 |
| *** ametts has quit IRC | 15:10 | |
| dtantsur | and remember that we have all-or-nothing approach right now, so it's still an improvement :) | 15:10 |
| sambetts|cat | https://bugs.launchpad.net/ironic/+bug/1524745 | 15:10 |
| openstack | Launchpad bug 1524745 in Ironic "RFE: driver composition reform" [Wishlist,In progress] - Assigned to Dmitry Tantsur (divius) | 15:10 |
| rloo | dtantsur: i'll leave it for you to discuss with sambetts|cat. I'd be happy to discuss later (in a few weeks or whatever). right now, i want to get networking stuff reviewed. | 15:11 |
| sambetts|cat | I've left a comment there with a example, I hope it makes sense | 15:11 |
| TheJulia | networking ++ | 15:12 |
| *** edand has quit IRC | 15:12 | |
| dtantsur | sambetts|cat, it makes sense, but I'm fine with the behaviour you've described, even if it breaks some corner cases... | 15:12 |
| dtantsur | sambetts|cat, that's essentially why we introduce the default option for network/volume: because these cases are not rare for them | 15:13 |
| dtantsur | but e.g. for deploy, or power, or management... | 15:13 |
| dtantsur | but this is a good reminder that we should think twice when setting the defaults for hardware types | 15:13 |
| sambetts|cat | the reason this discussion came up is because we were discussing the consistancy of having different behaviours for different interfaces, and whether hardware_types should be able to specify a default for network_interface | 15:15 |
| *** tshefi has quit IRC | 15:15 | |
| dtantsur | fwiw I've always been fine with more user-friendly behavior at the expense of slight inconsistency | 15:16 |
| sambetts|cat | if hardware_types can specify a default for network_interface, then what is an edge case for the other interfaces is a much bigger deal for network_interface | 15:16 |
| *** vishwanathj has quit IRC | 15:17 | |
| dtantsur | I agree with having the default option for network and volume | 15:17 |
| *** vishwanathj has joined #openstack-ironic | 15:17 | |
| sambetts|cat | dtantsur: whats you stance of if hardware_types should be able to specify a default for network_interface? | 15:17 |
| sambetts|cat | your stance* | 15:17 |
| sambetts|cat | on | 15:17 |
| sambetts|cat | (its getting to that point in the day again)... | 15:17 |
| rloo | sambetts|cat: please, just don't cry, i don't think i can stand it ;) | 15:18 |
| dtantsur | sambetts|cat, I think it can, but I kind of think that the configuration option should have a bigger priority | 15:18 |
| *** moshele has quit IRC | 15:19 | |
| sambetts|cat | dtantsur: problem is that having 1 configuration option only works if all your hardware_types you load have at least 1 interface of that type that is the same | 15:20 |
| dtantsur | that's true, and I think we can live with it | 15:20 |
| *** Romanenko_K has quit IRC | 15:20 | |
| dtantsur | it will definitely prevent us from setting the vendor-specific thing as the default, unless all hw types support it | 15:20 |
| dtantsur | but yeah, the driver comp is not without limitations... | 15:21 |
| sambetts|cat | dtantsur: what if I have 2 hardware types one has supported_network_interfaces = [flat] and the other has supported_network_interfaces = [CiscoFlat] I can't set a sane default then? | 15:22 |
| dtantsur | then only "noop" is your common default (both must include it) | 15:22 |
| *** sabeen1 has joined #openstack-ironic | 15:22 | |
| dtantsur | which is sad, I agree, but how realistic is hardware not supporting our "flat" implementation at all? | 15:22 |
| sambetts|cat | do we then enforce all hardware_types support noop? | 15:23 |
| dtantsur | sambetts|cat, on code review - yes | 15:23 |
| lucasagomes | hi all, can someone take a quick look at https://review.openstack.org/#/c/338245/ ? (2+2 already) | 15:23 |
| *** ametts has joined #openstack-ironic | 15:23 | |
| sambetts|cat | dtantsur: its a possblity with hardwre that requires nics to be poofed out of thin air and has no "nics" to begin with | 15:23 |
| TheJulia | dtantsur: well, in some hardware, even in flat, the ports need to be plugged of sorts. An example is oneview kind of | 15:23 |
| *** sabeen has quit IRC | 15:23 | |
| dtantsur | oh, I see now | 15:24 |
| dtantsur | well, it looks like no default here. I guess in this case we should require an explicit network_interface | 15:24 |
| sambetts|cat | so you set the conf option to None, then if hardware_types have default it falls back to that which also might be broken in this environment | 15:26 |
| *** ifarkas has quit IRC | 15:26 | |
| *** jistr|mtg is now known as jistr | 15:27 | |
| dtantsur | it's also possible. I see the problems, I don't see the solution, however. Do you have practical ideas on fixing it? | 15:27 |
| *** pcaruana has quit IRC | 15:28 | |
| *** trown is now known as trown|lunchnlear | 15:29 | |
| *** trown|lunchnlear is now known as trown|lunchlearn | 15:30 | |
| sambetts|cat | My only solution that works in my head so far is remove all defaults, make hardware_types just list supported_X_interfaces, make interfaces explictly enabled by the deployer using enabled_X_interfaces, and make interfaces explictly picked by the operator during node enrollment | 15:30 |
| sambetts|cat | Another possiblity is that we remove the idea of a hardware_type have 1 default interface | 15:31 |
| sambetts|cat | make the support_X_interfaces prioritised, and then have Ironic take [0] from the list generated by interseting enabled_X_interfaces with supported_X_interfaces | 15:32 |
| sambetts|cat | as the default | 15:32 |
| *** sukhdev has joined #openstack-ironic | 15:32 | |
| sambetts|cat | although I've not though that one through completely | 15:32 |
| sambetts|cat | thought* | 15:32 |
| dtantsur | the first idea sacrifices basic usability and I'm not sure what you get in the end. You'll just have to update all interfaces explicitly,. | 15:33 |
| dtantsur | you can do it with the current proposal too | 15:33 |
| *** rcernin has quit IRC | 15:33 | |
| dtantsur | the second case is interesting, but it's very implicit, so it should be well-thought | 15:33 |
| *** rajinir has joined #openstack-ironic | 15:34 | |
| *** adu has quit IRC | 15:34 | |
| sambetts|cat | the first idea regains the deployers control of enabled_X_interfaces, so they can remove interfaces they don't want to/can't support and then if you try to create a node with a disable interface i'll fail | 15:34 |
| sambetts|cat | it'll* | 15:34 |
| dtantsur | yeah, but the API side is terrible: you'll have to list all (8? 9?) interfaces during enrollment | 15:35 |
| dtantsur | which is terrible already, and is completely impossible with old API versions | 15:35 |
| *** mjura has quit IRC | 15:35 | |
| *** mordred has quit IRC | 15:35 | |
| dtantsur | and again: how does it fix your case? | 15:36 |
| sambetts|cat | because a node-create with an interface not in enabled_X_interfaces will fail | 15:37 |
| sambetts|cat | so its impossible to create a broken node | 15:37 |
| sambetts|cat | unlike right now were by default I'd get a broken node | 15:37 |
| dtantsur | well ok, we can check that the default is in the supported list, and refuse to start if it's not | 15:38 |
| dtantsur | does it cover this problem? | 15:38 |
| *** jtomasek__ has quit IRC | 15:39 | |
| sambetts|cat | no, because that means I all my hardware_interfaces have to have at least one common interface | 15:39 |
| sambetts|cat | for each type | 15:39 |
| *** piet has quit IRC | 15:40 | |
| *** piet has joined #openstack-ironic | 15:40 | |
| dtantsur | then set both defaults to "None" and let node enrollment fail if the explicit value is not provided | 15:41 |
| sambetts|cat | I can't change the hardware_types default | 15:41 |
| *** jpich has quit IRC | 15:41 | |
| *** watanabe_isao has joined #openstack-ironic | 15:42 | |
| dtantsur | right, you only need to set the default in configuration to None | 15:43 |
| dtantsur | I mean, we have it now: some deploy interfaces require swift and will break if it's not deployed | 15:43 |
| sambetts|cat | then it falls back to my hardware_types default which doesn't work in this deploy | 15:43 |
| dtantsur | ok, I'm fine with it | 15:43 |
| sambetts|cat | so I end up with a broken node by default | 15:44 |
| dtantsur | this is something that happens now and will happen in the future | 15:44 |
| dtantsur | correct. try iscsi_ilo driver without swift - iirc it's the same situation | 15:44 |
| sambetts|cat | and this could be easily resovled if the deployer could just remove the hardware_type default from the enabled_X_interface list | 15:44 |
| sambetts|cat | but if we do that then what do we fall back to | 15:45 |
| dtantsur | correct, that's why we can't do that | 15:45 |
| sambetts|cat | so I say remove defaults, and make it so that you always have to be explict | 15:45 |
| dtantsur | also these is a vague situation with driver vendor passthru in this case | 15:45 |
| dtantsur | no | 15:45 |
| dtantsur | "we can't heal 2 people, so lets destroy the whole hospital" :) | 15:46 |
| *** jrist has quit IRC | 15:46 | |
| *** d0ugal has quit IRC | 15:46 | |
| sambetts|cat | we can't heal 2 people with this solution so lets heal both with a slight differnt one | 15:46 |
| dtantsur | we're moving in circles already | 15:47 |
| * TheJulia suspects the weeds grow fast with this topic | 15:47 | |
| dtantsur | I'm not buying making terrible UX in favor of a rare case, which exists now already and can be worked around | 15:47 |
| sambetts|cat | IMO you have to do a bunch of calls specific to the interfaces anyway to update the driver info, so I don't see why not just set the interface at the same time | 15:47 |
| dtantsur | sambetts|cat, please show me an example of how to do it with an old API version, to begin with :) | 15:48 |
| rloo | TheJulia: I think this is a 'lets get popcorn' one :) | 15:48 |
| sambetts|cat | haha | 15:48 |
| TheJulia | rloo: or beer, since I can't eat popcorn | 15:48 |
| rloo | TheJulia: I'm at https://review.openstack.org/#/c/206244/, getting there... | 15:49 |
| dtantsur | but seriously, barely anybody will like adding 7-8 mandatory arguments to node creation API just because we have a rare case to fix | 15:49 |
| rloo | TheJulia: didn't realize you can't have popcorn. so sad. | 15:49 |
| sambetts|cat | dtantsur: Im not a fan that interfaces are mandatory like that anyway IMO thats why we have enroll state.. so I can update the nodes async then make them avaialbe | 15:50 |
| TheJulia | rloo: That ones seems really strait forward to me. I stayed up till nearly midnight local looking at reviews | 15:50 |
| *** ametts has quit IRC | 15:50 | |
| TheJulia | s/ones/one/ | 15:50 |
| mariojv | win 20 | 15:50 |
| mariojv | sorry, mistype | 15:50 |
| rloo | thx TheJulia, for working on it. I went to sleep :) | 15:50 |
| TheJulia | mariojv: did we all win $20 ? ;) | 15:51 |
| *** Sukhdev_ has joined #openstack-ironic | 15:51 | |
| sambetts|cat | all /me has won is a headache | 15:51 |
| TheJulia | rloo: you may want to have a cup of coffee before you get to the documentation patch | 15:51 |
| dtantsur | sambetts|cat, I already envision people asking "why could not you create a default??" | 15:51 |
| dtantsur | :) | 15:51 |
| mariojv | sure, if you buy some cloud from us :P jk | 15:51 |
| dtantsur | sambetts|cat, but old API versions give me even more headache | 15:51 |
| rloo | TheJulia: oh, you mean I shouldn't review that one eh? :) | 15:52 |
| sambetts|cat | dtantsur: yeah :/ | 15:52 |
| TheJulia | rloo: possibly, I've asked for it to be broken out of the main install text body and there are a number of areas where more context is required. I think I left 19 comments on it last night | 15:52 |
| rloo | TheJulia: I'm not too worried about the doc one landing soon ;) | 15:53 |
| TheJulia | rloo: yeah, I think the doc might take another week or two | 15:53 |
| rloo | TheJulia: quick question for you. the port.pxe_enabled and port.local_link_connection fields. do you think of those as fields for multitenancy use (only)? | 15:53 |
| *** watanabe_isao has quit IRC | 15:54 | |
| TheJulia | I only think that for local_link_connection | 15:54 |
| TheJulia | pxe_enabled has been around for a while and was just masked until now | 15:54 |
| *** vishwanathj has quit IRC | 15:54 | |
| rloo | TheJulia: yeah. not sure i like seeing them described as multitenancy. will think about it. | 15:55 |
| *** vishwanathj has joined #openstack-ironic | 15:55 | |
| TheJulia | rloo: I kind of saw it falling under the unbrella since someone would very likely want to not dhcp on some of their network ports... or they can't potentially | 15:55 |
| TheJulia | at least when configuring their bare metal network connectivity | 15:56 |
| sambetts|cat | I think pxe_enabled is not multitenant specific I think the flat network interface case might take that into account at some point too, if people register more than 1 nic in Ironic for a node | 15:56 |
| TheJulia | sambetts|cat: I think it already does | 15:56 |
| sambetts|cat | cool :) | 15:56 |
| TheJulia | I remember the database migration was missed for it origionally, so all of a sudden we were not writing pxe configs | 15:56 |
| rloo | thx. i'll keep that in mind when reviewing the code. <strike>multitenancy</strike> :) | 15:56 |
| *** watanabe_isao has joined #openstack-ironic | 15:57 | |
| *** catintheroof has joined #openstack-ironic | 15:58 | |
| *** baoli_ has quit IRC | 15:59 | |
| *** amotoki has quit IRC | 15:59 | |
| *** rcernin has joined #openstack-ironic | 16:00 | |
| *** rpioso has joined #openstack-ironic | 16:00 | |
| *** davideagnello has quit IRC | 16:01 | |
| *** mgoddard_ has joined #openstack-ironic | 16:02 | |
| *** openstackgerrit has quit IRC | 16:03 | |
| *** openstackgerrit has joined #openstack-ironic | 16:03 | |
| *** bvandewa has joined #openstack-ironic | 16:04 | |
| *** tesseract- has quit IRC | 16:06 | |
| *** mgoddard has quit IRC | 16:06 | |
| vdrok | rloo: it's not multitenancy now - I've renamed it to advanced_net_fields iirc | 16:10 |
| vdrok | and /me leaves :) | 16:10 |
| vdrok | good night everyone! | 16:10 |
| rloo | vdrok: yes, i noticed that. except in at least one place (and the commit) :) | 16:10 |
| rloo | vdrok: bye, have a great time! | 16:11 |
| *** wajdi has quit IRC | 16:11 | |
| vdrok | rloo: whoops, that happens all the time :) | 16:11 |
| *** wajdi has joined #openstack-ironic | 16:11 | |
| rloo | vdrok: no worries. cosmetic stuff :) GO! | 16:11 |
| thiagop | night vdrok, have fun! | 16:11 |
| lucasagomes | dtantsur, on top of the discussion in the internal channel but now related to upstream | 16:11 |
| lucasagomes | dtantsur, what do you think about having a clean step to remove the disks metadatas rather than the whole data | 16:12 |
| *** tiendc has joined #openstack-ironic | 16:12 | |
| lucasagomes | dtantsur, for situations like ours where the data from previous tenants (since it's a installer) is not important and can be in the disk | 16:12 |
| lucasagomes | dtantsur, but the metadata can cause problems | 16:12 |
| dtantsur | lucasagomes, in theory we're supposed to remove disk metadata before deployment, maybe we didn't have it back in liberty? | 16:12 |
| dtantsur | oh, maybe we remove metadata only from one disk | 16:12 |
| lucasagomes | dtantsur, only for the disk we are deploying onto | 16:13 |
| lucasagomes | dtantsur, exactly not all disks | 16:13 |
| dtantsur | correct, correct, ignore me | 16:13 |
| lucasagomes | like cleaning would be suppose to do | 16:13 |
| dtantsur | I agree, a step to wipe metadata would be helpful. or should it be a flag for the existing step? | 16:13 |
| lucasagomes | dtantsur, could be as well | 16:15 |
| lucasagomes | but that's implementation detail | 16:15 |
| dtantsur | yeah. mind filing an RFE? | 16:15 |
| lucasagomes | dtantsur, will do, probably tomorrow cause I've stopped everything to look at that probelm | 16:15 |
| lucasagomes | I will finish some stuff and as a TODO tomorrow morning | 16:15 |
| lucasagomes | I will fill one out | 16:15 |
| *** joprovost1 has joined #openstack-ironic | 16:16 | |
| dtantsur | awesome, thanks! | 16:16 |
| *** joprovost has quit IRC | 16:17 | |
| *** joprovost1 is now known as joprovost | 16:17 | |
| openstackgerrit | Nisha Agarwal proposed openstack/proliantutils: Discover iscsi_boot and iscsi_iqn https://review.openstack.org/341653 | 16:18 |
| NobodyCam | good morning Ironers | 16:24 |
| *** sdake has quit IRC | 16:24 | |
| NobodyCam | :p Ironicers even | 16:24 |
| TheJulia | good morning NobodyCam | 16:24 |
| *** adu has joined #openstack-ironic | 16:25 | |
| NobodyCam | Morning TheJulia | 16:26 |
| dtantsur | morning NobodyCam | 16:26 |
| dtantsur | and good night everyone else for I call it a day :) | 16:26 |
| jroll | sambetts|cat: dtantsur: I still assert that instead of requiring enabled interfaces have something in common for all enabled hw types, the default_network_interface config can be set to any enabled interface, and if a node is created with a hw type that doesn't have that interface, the request fails | 16:26 |
| Sukhdev_ | devananda dtantsur sambetts|cat - are we having a voice call to get the ironic networking patches merged? | 16:27 |
| dtantsur | jroll, that's an option too | 16:27 |
| sambetts|cat | jroll: I'm working on a proposal that might resolve that part | 16:27 |
| dtantsur | Sukhdev_, not me, sorry :( | 16:27 |
| TheJulia | goodnight dtantsur | 16:27 |
| jroll | Sukhdev_: we had one yesterday to get over the hurdle we were facing | 16:27 |
| NobodyCam | morning and good night dtantsur | 16:27 |
| NobodyCam | morning sambetts|cat roll :) | 16:27 |
| Sukhdev_ | jroll : cool - and welcome back | 16:28 |
| NobodyCam | gah ... roll = roll :) | 16:28 |
| NobodyCam | what the | 16:28 |
| jroll | Sukhdev_: thanks | 16:28 |
| jroll | morning NobodyCam :) | 16:28 |
| *** dtantsur is now known as dtantsur|afk | 16:28 | |
| * jroll assumes NobodyCam j key is broken :P | 16:28 | |
| TheJulia | NobodyCam: broken j key? | 16:28 |
| TheJulia | heh | 16:28 |
| NobodyCam | jroll nope its the auto correct :p | 16:29 |
| jroll | lol | 16:29 |
| sambetts|cat | jroll: https://etherpad.openstack.org/p/ironic_network_interface_discussion can you look at my second solution @L101 and see what you think? | 16:29 |
| TheJulia | autocorrect is evil | 16:29 |
| TheJulia | and not the good kind of evil, it is the bad kind of evil | 16:29 |
| jroll | the evil kind of evil? | 16:30 |
| TheJulia | Well, there is the evil league of evil.... | 16:30 |
| jroll | TheJulia: autocorrect for pokemon: https://i.redditmedia.com/LID9qujoR1LN1oguTwIxNrrION1kmuugiGwoUmfmzUQ.jpg?w=432&s=5d4678ce9ab6043e46b5b58b533e17f0 | 16:30 |
| *** rbartal has quit IRC | 16:30 | |
| * sambetts|cat is still waiting for EU release of go... | 16:31 | |
| jroll | sambetts|cat: not a terrible idea | 16:31 |
| jroll | I obviously haven't thought it through fully | 16:31 |
| * TheJulia will just stick with ingress whens he is in the mood | 16:31 | |
| sambetts|cat | jroll: works pretty well for the network_interface case too, because a vendor might recomned neutron above flat, but if you remove neutron from enabled then it'll fall back to the next thing in the list | 16:32 |
| lucasagomes | NobodyCam, TheJulia morning | 16:32 |
| jroll | sambetts|cat: rightright | 16:33 |
| *** piet has quit IRC | 16:33 | |
| lucasagomes | I'm also calling it a day folks, have a great night all | 16:33 |
| jroll | wow, I'm good at typing too | 16:33 |
| lucasagomes | see you all tomorrow | 16:33 |
| *** lucasagomes is now known as lucas|afk | 16:33 | |
| NobodyCam | night lucas|afk | 16:33 |
| sambetts|cat | rloo, TheJulia, dtantsur|afk: https://etherpad.openstack.org/p/ironic_network_interface_discussion @L101 is a proposed solution to the defaults issue can you take a look through when you get a moment? | 16:33 |
| TheJulia | sure sambetts|cat | 16:34 |
| JayF | sambetts|cat: as an operator, is there a way for me to know before creating the node what network interface I'd get on a given hardware type? | 16:34 |
| jroll | ooo yeah, that is a bit implicit | 16:34 |
| JayF | sambetts|cat: The only reason I wouldn't like that is that it makes the behavior a little ... squishier. Although I guessed if I cared that much I would only enable one interface in my config? | 16:34 |
| sambetts|cat | JayF: we could provide an API to return the list of possible interfaces for that hardware_type | 16:35 |
| sambetts|cat | we're providing one to list the support interfaces anyway right? | 16:36 |
| jroll | http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/driver-composition-reform.html#rest-api-impact | 16:36 |
| jroll | also has default | 16:36 |
| JayF | the lists-of-defaults thing is just a strange interface, I agree it's smarter in general, but I'm wondering if it makes it less predictable | 16:37 |
| JayF | I don't have a strong opinion either way just asking the question | 16:37 |
| jroll | it would be list-of-preferred, I guess | 16:38 |
| sambetts|cat | list of supported interfaces in order of preference by the vendor | 16:38 |
| *** jaybeale has joined #openstack-ironic | 16:38 | |
| sambetts|cat | we must not forget the supported part :) | 16:39 |
| *** derekh has quit IRC | 16:39 | |
| TheJulia | sambetts|cat: it is an interesting idea, I'll have to simmer on it a little | 16:39 |
| JayF | With the supported interfaces bit, are we going to require that some drivers support all interfaces? | 16:39 |
| JayF | Like with neutron or flat, it's pretty well independent of the hardware, right? Why wouldn't every driver support those? | 16:40 |
| JayF | I feel like I'm missing something -- namely the case where a vendor would want to blacklist a "generic"-style network interface in their driver | 16:40 |
| jroll | because there is hardware that needs to do extra things like poof nics out of thin air :( | 16:40 |
| jroll | see: cisco, oneview | 16:41 |
| JayF | but how is that any different than requiring port metadata be provided for "normal" hardware with a neutron driver? | 16:41 |
| jroll | because it needs to be done on the fly in the network driver, apparently | 16:41 |
| JayF | like, Ironic doesn't guarantee the nics are sanely physically configured in any case, do we? | 16:41 |
| jroll | no | 16:42 |
| jroll | but we assume they exist if they are registered | 16:42 |
| TheJulia | So that is pretty much what I told the oneview folks that the neutron stuff can partially solve the issue if they are an ML2 driver and the MAC addresses are known, the problem is, they can be assigned from a pool afaik | 16:42 |
| jroll | idk what magic the cisco driver does, sambetts|cat is that open source? | 16:42 |
| sambetts|cat | not that I'm aware of | 16:42 |
| JayF | jroll: so why is the cisco/oneview case different from the "I have a server with no registered nics" case? | 16:43 |
| jroll | JayF: because in the latter we just fail the deployment | 16:43 |
| TheJulia | and in the case with oneview on a c7000 chassis, the port exists in the profile, but to dynamicly connect things even in flat since ironic is not the ultimate controller of the hardware, it still has to ask for the port to get attached to a specific network | 16:43 |
| jroll | in the former I guess it sets it up | 16:43 |
| jroll | but honestly I have no way to tell | 16:43 |
| JayF | So why is that not an acceptable answer for those other cases too? | 16:43 |
| sambetts|cat | JayF: because we can create a network interface that can poof nics out of thin air | 16:43 |
| JayF | Why add all this complexity to solve a case we can't fully solve anyway? | 16:43 |
| jroll | so this makes me wonder: do we do all of this work to handle an edge case | 16:44 |
| jroll | right | 16:44 |
| JayF | yeah that's what I'm getting at | 16:44 |
| jroll | an edge case for a proprietary out of tree driver* | 16:44 |
| JayF | and we aren't even fully handling the edge case | 16:44 |
| sambetts|cat | ? | 16:44 |
| sambetts|cat | https://review.openstack.org/#/c/327046/ | 16:45 |
| sambetts|cat | ^ this + the BM VlAN spec makes that case work just fine | 16:45 |
| JayF | sambetts|cat: basically asking the question: Why can't it be OK for those drivers to just fail a deployment if given a network interface that doens't provide full functionality, in the same way a "generic" node would fail a deployment if it didn't have any nics registered | 16:45 |
| jroll | JayF: oh, that's just a terrible user experience | 16:46 |
| sambetts|cat | we shouldn't even be able to create a broken node | 16:46 |
| sambetts|cat | like that | 16:46 |
| jroll | right | 16:46 |
| sambetts|cat | thats what the hardware_types are for | 16:46 |
| jroll | my bigger question, is why add this complexity for an edge case? | 16:46 |
| jroll | but I suspect the answer is, this becomes less of an edge case | 16:47 |
| JayF | jroll: how is that different than what I asked? | 16:47 |
| jroll | as more vendors build hardware that can do this | 16:47 |
| *** ijw has quit IRC | 16:47 | |
| jroll | JayF: my 'edge case' == 'hardware that has virtual NICs' | 16:47 |
| JayF | okay | 16:47 |
| jroll | not 'someone put the wrong network interface on a node' | 16:47 |
| jroll | I guess it's both cases combined, though | 16:47 |
| sambetts|cat | I think it solves the edge case, but it also makes it possible to make network_interfaces consistent with the other interface types in hardware_types | 16:48 |
| *** bvandewa has quit IRC | 16:49 | |
| sambetts|cat | which in the current solution network_interfaces are special because hardware_types can't define a default for them | 16:49 |
| JayF | jroll: yeah; that's what I'm getting at | 16:49 |
| JayF | jroll: and I'm OK if the virtual nic case is handled better, just wanted to boil it down to the most basic issue | 16:49 |
| jroll | sambetts|cat: I do worry about the implicitness, but it's likely fine | 16:49 |
| jroll | JayF: sure | 16:49 |
| jroll | JayF: that's why I've been hesitant to solve this edge case, it depends on two things | 16:50 |
| sambetts|cat | the defaults are kind of implict anyway, and obay the deplyers decision to not enable them | 16:50 |
| sambetts|cat | and don't obay * | 16:50 |
| jroll | JayF: 1) deployers are enrolling nodes by hand and might forget to add the non-default interface needed | 16:50 |
| jroll | JayF: 2) people enrolling nodes don't know that the default interface won't work in the deployment they're enrolling them to (different ironic ops vs deployers etc) | 16:51 |
| *** fragatina has joined #openstack-ironic | 16:52 | |
| *** Nisha_away has joined #openstack-ironic | 16:52 | |
| *** fragatina has quit IRC | 16:52 | |
| *** vishwanathj has quit IRC | 16:52 | |
| jroll | off topic: whoa. https://lwn.net/Articles/692638/ | 16:53 |
| *** fragatina has joined #openstack-ironic | 16:53 | |
| *** piet has joined #openstack-ironic | 16:53 | |
| *** vishwanathj has joined #openstack-ironic | 16:53 | |
| sambetts|cat | with network_interfaces many hardware_types are likely to intersect on which interfaces they support, but power interfaces not so much, and if a deployer doesn't want to supprt the default power interface for a particular hardware_type then the only way to disable it is to disable that hardwre_type | 16:54 |
| sambetts|cat | even if that hardware_type does support other power interfaces that deployer is willing to support | 16:54 |
| jroll | yeah, I've gathered as much | 16:55 |
| sambetts|cat | more for JayF's benfit than anything else :) | 16:55 |
| jroll | I just tend to think people working on a deployment won't be so far out of touch that someone will enroll a node with the default interface | 16:55 |
| sambetts|cat | but why even allow that in the first place | 16:55 |
| JayF | I honestly just dislike things that are implicit in the way the "intersecting preference lists" suggestions are | 16:56 |
| jroll | I think allowing that is better than requiring every interface be specified in an enroll command | 16:56 |
| jroll | I know we're looking at better options | 16:56 |
| *** sdake has joined #openstack-ironic | 16:56 | |
| jroll | I just think this is probably the least worst | 16:56 |
| JayF | I feel like an operator who doesn't understand Ironic's internals will get surprised by that behavior | 16:56 |
| JayF | but I don't feel strongly about it, just food for thought | 16:56 |
| *** edand has joined #openstack-ironic | 16:57 | |
| *** cdearborn has joined #openstack-ironic | 16:59 | |
| sambetts|cat | my brain feels like its been put in a blender after all this discussion | 17:00 |
| *** ijw has joined #openstack-ironic | 17:00 | |
| *** links has quit IRC | 17:01 | |
| sambetts|cat | I don't know what right or not anymore but at least this solution gives us the best of both worlds, and gurantees you get a "working" node by default | 17:01 |
| * jroll hands sambetts|cat a beer | 17:01 | |
| sambetts|cat | much needed, thank you | 17:02 |
| jroll | I have to miss the QA meeting today, sorry :( | 17:02 |
| sambetts|cat | tbh this is problem solving is a welcome relieve from the downstream deadline I've got atm :-P | 17:02 |
| sambetts|cat | jroll: ok :) | 17:02 |
| *** sambetts|cat is now known as sambetts | 17:03 | |
| *** edand has quit IRC | 17:04 | |
| *** Sukhdev_ has quit IRC | 17:05 | |
| *** sambetts is now known as sambetts|cat | 17:06 | |
| *** PollyZ has quit IRC | 17:06 | |
| *** baoli has joined #openstack-ironic | 17:07 | |
| *** penick has joined #openstack-ironic | 17:09 | |
| *** mgoddard_ has quit IRC | 17:09 | |
| *** mgoddard has joined #openstack-ironic | 17:09 | |
| *** bnemec has quit IRC | 17:10 | |
| *** ChrisAusten has joined #openstack-ironic | 17:13 | |
| *** bnemec has joined #openstack-ironic | 17:13 | |
| *** fragatina has quit IRC | 17:15 | |
| lazy_prince | anyone here form oneview team..? | 17:16 |
| TheJulia | thiagop | 17:16 |
| thiagop | Me | 17:17 |
| thiagop | :) | 17:17 |
| lazy_prince | so i am trying to use ironic-oneview cli.. | 17:17 |
| lazy_prince | do you know if there is a way to specify the output directory for genrc cmd..? | 17:18 |
| *** marios is now known as marios|gone | 17:19 | |
| thiagop | lazy_prince: I think the last question is the path where to save the genrc file | 17:19 |
| *** marios|gone is now known as marios | 17:19 | |
| lazy_prince | yes.. | 17:19 |
| lazy_prince | thiagop: and what is the default path.. | 17:20 |
| thiagop | lazy_prince: ~/ironic-oneview-cli.rc or something (it is shown between brackets) | 17:21 |
| lazy_prince | okay.. anyway to change that by some parameter..? | 17:21 |
| *** trown|lunchlearn is now known as trown | 17:23 | |
| *** PollyZ has joined #openstack-ironic | 17:24 | |
| lazy_prince | thiagop: sorry.. I get your response now.. | 17:24 |
| lazy_prince | thank you.. | 17:24 |
| thiagop | lazy_prince: np ;) | 17:25 |
| * mgould -> home; good night! | 17:25 | |
| lazy_prince | thiagop: so now that it is generated in a different folder, how do I specify to use it..? | 17:25 |
| *** mgould is now known as mgould|afk | 17:25 | |
| thiagop | just pass a different (full) path in the last step | 17:26 |
| thiagop | full == with filename included | 17:26 |
| lazy_prince | I did that.. but when I invoke node-create, what option i have to use to use this file.. | 17:27 |
| lazy_prince | say, I generated two rc files.. and i want to switch between then on need basis.. | 17:27 |
| *** PollyZ has quit IRC | 17:29 | |
| *** penick has quit IRC | 17:29 | |
| *** fellypefca has joined #openstack-ironic | 17:29 | |
| lazy_prince | for eg: I could have two oneview setups and each one of them may be used in different regions... so I will generate two rc files for each oneview. | 17:30 |
| lazy_prince | thiagop: let us say I need to register nodes for one of the oneview in respective region, how can i specify the rc file with node-create | 17:32 |
| thiagop | lazy_prince: have you 'sourced' the file? | 17:32 |
| thiagop | lazy_prince: like 'source ironic-oneview-cli.rc' then 'ironic-oneview-cli node-create' | 17:33 |
| lazy_prince | aha.. so dumb and I was was worried of how to specify it as a parameter.. | 17:34 |
| lazy_prince | thanks much... | 17:34 |
| *** yogi has quit IRC | 17:35 | |
| thiagop | lazy_prince: np, we thought of that to be as near as possible of the ironicclient usage | 17:35 |
| *** PollyZ has joined #openstack-ironic | 17:37 | |
| lazy_prince | thiagop: last question.. does it support v3 auth APIs..? I do not see Domain info being accepted... | 17:39 |
| *** ohamada has quit IRC | 17:39 | |
| thiagop | lazy_prince: I think it doesn't. Is Ironic already supporting v3? | 17:39 |
| lazy_prince | thiagop: as a client, I guess yes.. | 17:40 |
| lazy_prince | I mean cli client.. | 17:41 |
| *** trown is now known as trown|relocating | 17:41 | |
| *** PollyZ has quit IRC | 17:43 | |
| *** PollyZ has joined #openstack-ironic | 17:51 | |
| *** vishwanathj has quit IRC | 17:52 | |
| *** vishwanathj has joined #openstack-ironic | 17:53 | |
| *** sukhdev has quit IRC | 17:53 | |
| *** PollyZ has quit IRC | 17:56 | |
| *** ametts has joined #openstack-ironic | 18:00 | |
| *** rama_y has joined #openstack-ironic | 18:02 | |
| thiagop | sambetts|cat: rajinir I'm setting something on your speaker profile just to submit the panel, please update it later :) | 18:03 |
| rajinir | thiagop: sure thanks | 18:03 |
| *** PollyZ has joined #openstack-ironic | 18:05 | |
| sambetts|cat | thiagop: I didn't know you could change that | 18:08 |
| *** PollyZ has quit IRC | 18:09 | |
| thiagop | sambetts|cat: kinda strange indeed, but some fields were required | 18:09 |
| sambetts|cat | thiagop: have you got a link to the presentation? | 18:09 |
| *** pcaruana has joined #openstack-ironic | 18:09 | |
| thiagop | sambetts|cat: just for edition, I think once the selection starts, they'll send a link | 18:10 |
| thiagop | just to edit* | 18:11 |
| sambetts|cat | ah, its not appearing in the "Presentations Other Submitted with you as a speaker" section on my profie | 18:11 |
| sambetts|cat | thats why I wondered if you had got the right me | 18:11 |
| sambetts|cat | :-p | 18:11 |
| sambetts|cat | Does it have my blue logo? | 18:11 |
| thiagop | sambetts|cat: yep. Didn't you received an e-mail about the submission? | 18:12 |
| sambetts|cat | thiagop: I got an email saying to edited my profile that was all | 18:12 |
| thiagop | LOL | 18:12 |
| *** ccamacho has quit IRC | 18:15 | |
| sambetts|cat | I'm heading off | 18:16 |
| sambetts|cat | night all | 18:16 |
| *** sambetts|cat is now known as sambetts | 18:16 | |
| *** sambetts is now known as sambetts|afk | 18:16 | |
| thiagop | night sambetts|afk | 18:17 |
| *** mordred has joined #openstack-ironic | 18:19 | |
| *** vishwanathj has quit IRC | 18:22 | |
| *** ametts has quit IRC | 18:24 | |
| *** mbound_ has quit IRC | 18:26 | |
| *** ametts has joined #openstack-ironic | 18:28 | |
| *** mbound has joined #openstack-ironic | 18:28 | |
| *** mbound has quit IRC | 18:29 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic: Metric chassis, driver, node, and port API calls https://review.openstack.org/301923 | 18:31 |
| *** ChubYann has joined #openstack-ironic | 18:33 | |
| JayF | jroll: ^ updated, and it deps on the global-requirements patch | 18:34 |
| *** ccamacho has joined #openstack-ironic | 18:34 | |
| *** wajdi has quit IRC | 18:35 | |
| rloo | hear ye, hear ye, anyone interested in the networking patches. The next path has three +2's. Does anyone want to look? If not (I'll wait 10 minutes), I'll +A. https://review.openstack.org/#/c/285852/ | 18:36 |
| *** trown|relocating is now known as trown | 18:38 | |
| *** Sukhdev has joined #openstack-ironic | 18:42 | |
| *** PollyZ has joined #openstack-ironic | 18:45 | |
| *** PollyZ has quit IRC | 18:50 | |
| * TheJulia sees a +A and feels like dancing... except the work is not done | 18:52 | |
| *** PollyZ has joined #openstack-ironic | 18:52 | |
| TheJulia | Next patch in the networking series, has 2x +2s, one +1. https://review.openstack.org/#/c/317393 If you haven't reviewed it yet, might be a good time to review it now. | 18:53 |
| rloo | TheJulia: do you have a good answer for Yuriy's question in that patch? | 18:53 |
| rloo | TheJulia: I was thinking it was cuz cleaning is 'flaky' | 18:53 |
| TheJulia | I _think_ it is because we explicitly call cleaning and we may need to retry it | 18:53 |
| rloo | TheJulia: cuz the cleaning didn't finish, right? | 18:54 |
| TheJulia | We likely need a remove_all_ports thing or something | 18:54 |
| TheJulia | yup | 18:54 |
| *** tiendc has quit IRC | 18:54 | |
| TheJulia | or cleaning had to be restarted due to some unknown reasons | 18:54 |
| *** baoli has quit IRC | 18:54 | |
| rloo | TheJulia: the 'flaky' bit :) | 18:55 |
| TheJulia | yeah..... I think we fixed part of the flaky bit though :) | 18:55 |
| rloo | TheJulia: but provisioning should be more certain... Although I wonder if we should also roll back? What/Can we get into a similar state with provisioning? | 18:55 |
| TheJulia | truthfully, really, we need a "nuke all my ports" or something method, but after these are landed | 18:55 |
| rloo | TheJulia: yeah. So I am fine if we also land that patch. Anything we do can be added later. | 18:56 |
| rloo | TheJulia: but maybe we can wait for one of the v* to reply? dunno. | 18:56 |
| *** ijw has quit IRC | 18:56 | |
| TheJulia | rloo: if we can't get that patch in nova, we will need something to ports that are attached that don't matter or that are invalid based on the state machine in order to properly attach networking | 18:56 |
| TheJulia | rloo: I was thinking of waiting until they could look at it again in the morning, I just would also love other eyes on it, looking at who has reviewed it | 18:56 |
| rloo | TheJulia: yup, can wait til tomorrow. vdrok is away til tues but vsaienk0 I think will look. | 18:57 |
| rloo | TheJulia: one a day is good :) | 18:57 |
| TheJulia | thiagop: How about a nice neutron driver before you leave for the day? *ducks* | 18:57 |
| * TheJulia is kidding, because if memory serves it is something like 5pm there | 18:58 | |
| *** sdake has quit IRC | 18:58 | |
| rloo | TheJulia: ha ha. Maybe if you promise to review his patch(es) :) | 18:58 |
| *** mordred has quit IRC | 18:59 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Fix iLO drivers temporarily for local_gb https://review.openstack.org/341752 | 19:00 |
| *** baoli has joined #openstack-ironic | 19:01 | |
| TheJulia | I commented in the rev with our context since Yuriy will surely see early tomorrow morning | 19:01 |
| rloo | thx TheJulia. Good thinking! | 19:02 |
| jroll | ]/b 117 | 19:02 |
| jroll | oooops. | 19:02 |
| *** trown is now known as trown|mtg | 19:03 | |
| *** bvandewa has joined #openstack-ironic | 19:07 | |
| *** moshele has joined #openstack-ironic | 19:08 | |
| *** ChrisAusten has quit IRC | 19:12 | |
| *** causten_ has joined #openstack-ironic | 19:12 | |
| thiagop | TheJulia: missing context | 19:16 |
| * thiagop reads | 19:16 | |
| TheJulia | :) | 19:17 |
| thiagop | TheJulia: 4pm in fact | 19:18 |
| TheJulia | oh... I guess it was DST when I thought it was 2 hours | 19:18 |
| thiagop | TheJulia: "Winter is here" </endquote> | 19:19 |
| *** bvandewa has quit IRC | 19:19 | |
| thiagop | #pixiesay quotes_count += 1 | 19:19 |
| PixieBoots | ʕ•͡ᴥ•ʔ: quotes_count += 1 | 19:19 |
| TheJulia | thiagop: "Summer is here, but winter is coming." | 19:21 |
| *** sdake has joined #openstack-ironic | 19:22 | |
| *** nicodemos has quit IRC | 19:26 | |
| *** bvandewa has joined #openstack-ironic | 19:28 | |
| *** nicodemos has joined #openstack-ironic | 19:29 | |
| *** mbound has joined #openstack-ironic | 19:30 | |
| *** dprince has joined #openstack-ironic | 19:32 | |
| *** fellypefca has quit IRC | 19:34 | |
| *** mbound has quit IRC | 19:35 | |
| *** pcaruana has quit IRC | 19:35 | |
| thiagop | https://review.openstack.org/#/c/285852/ | 19:36 |
| thiagop | ops | 19:36 |
| thiagop | bad paste | 19:36 |
| *** moshele has quit IRC | 19:39 | |
| *** e0ne has quit IRC | 19:43 | |
| *** moshele has joined #openstack-ironic | 19:47 | |
| jroll | this is the last patch to unblock iLO CI, if someone has a cycle or two to review https://review.openstack.org/#/c/264590/29 | 19:48 |
| devananda | jroll: spec is merged, could you drop your -2 on https://review.openstack.org/#/c/325599/ ? | 19:49 |
| jroll | devananda: done | 19:49 |
| devananda | I'm picking up work on that code again today | 19:49 |
| devananda | tyvm | 19:49 |
| jroll | np, ty! | 19:49 |
| * devananda reviews the ilo devstack patch | 19:53 | |
| *** bvandewa has quit IRC | 19:53 | |
| *** vishwanathj has joined #openstack-ironic | 19:54 | |
| *** sdake has quit IRC | 19:57 | |
| *** moshele has quit IRC | 19:58 | |
| *** catintheroof has quit IRC | 20:07 | |
| *** bvandewa has joined #openstack-ironic | 20:14 | |
| *** Nisha_away has quit IRC | 20:15 | |
| watanabe_isao | rajinir, ping | 20:16 |
| *** trown|mtg is now known as trown | 20:16 | |
| thiagop | watanabe_isao: have you received an email notification about the submission of the panel? | 20:19 |
| thiagop | mjturek1: ^ | 20:19 |
| *** trown is now known as trown|outtypewww | 20:21 | |
| devananda | jroll: I'm reviewing the network patches now -- there are 5 or 6 in a chain, including the API one with my procedural -2 on it | 20:21 |
| devananda | jroll: wdyt of landing aaallll of that now? I see some +2's from you, but not on all of them yet | 20:22 |
| watanabe_isao | thiagop, yes. A mail said I was ad to the panel. | 20:23 |
| *** ijw has joined #openstack-ironic | 20:24 | |
| thiagop | watanabe_isao: wonderful, thanks | 20:25 |
| openstackgerrit | Merged openstack/ironic: Add network interface to base driver class https://review.openstack.org/285852 | 20:25 |
| openstackgerrit | Merged openstack/ironic: Allow to use network interfaces in devstack https://review.openstack.org/293520 | 20:26 |
| *** ijw_ has joined #openstack-ironic | 20:27 | |
| *** nicodemos has left #openstack-ironic | 20:27 | |
| *** nicodemos has joined #openstack-ironic | 20:27 | |
| *** piet has quit IRC | 20:28 | |
| *** ijw has quit IRC | 20:28 | |
| *** ijw_ has quit IRC | 20:30 | |
| *** ijw has joined #openstack-ironic | 20:30 | |
| *** athomas has quit IRC | 20:31 | |
| mjturek1 | thiagop: sorry! yes I recieved the notification, thank you! | 20:31 |
| *** bvandewa has quit IRC | 20:34 | |
| *** clenimar__ has joined #openstack-ironic | 20:35 | |
| *** joprovost has quit IRC | 20:35 | |
| *** bvandewa has joined #openstack-ironic | 20:35 | |
| *** PollyZ_ has joined #openstack-ironic | 20:36 | |
| *** PollyZ has quit IRC | 20:37 | |
| thiagop | mjturek1: great! | 20:38 |
| *** daemontool_ has quit IRC | 20:41 | |
| *** clenimar__ has quit IRC | 20:42 | |
| *** clenimar_ has joined #openstack-ironic | 20:42 | |
| *** david-lyle has joined #openstack-ironic | 20:43 | |
| *** piet has joined #openstack-ironic | 20:45 | |
| *** piet has quit IRC | 20:46 | |
| *** piet has joined #openstack-ironic | 20:46 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Upadte devstack section of quickstart to use agent_ssh https://review.openstack.org/341801 | 20:47 |
| *** david-lyle has quit IRC | 20:47 | |
| *** david-lyle has joined #openstack-ironic | 20:49 | |
| JayF | NobodyCam: AFAIK we aren't deprecating pxe_ssh? | 20:52 |
| JayF | NobodyCam: it's been reimplented to work with IPA doing the iscsi exposures | 20:53 |
| jroll | devananda: I have no opposition to landing all the things :) | 20:53 |
| NobodyCam | I thought we were going to move it to the staging repo | 20:53 |
| JayF | the *_ssh drivers are going to staging repo if/when we move our gate to *_ipmitool | 20:54 |
| JayF | which is when we'd change devstack to use a *_ipmitool driver w/vbmc instead of *_ssh | 20:54 |
| jroll | devananda: I just got back home from a quick appt and was planning to review some of it, but if I don't need to that makes me happy :) | 20:54 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Upadte devstack section of quickstart to use agent_ssh https://review.openstack.org/341801 | 20:54 |
| JayF | devananda: ^ same question I just posed to nobodycam | 20:55 |
| TheJulia | same rev even | 20:55 |
| devananda | JayF: I just edited a typo in there | 20:55 |
| devananda | no opinion on the actual deprecation of pxe_ssh | 20:55 |
| TheJulia | have we officially deprecated it yet? | 20:56 |
| TheJulia | it being the ssh drivers? | 20:56 |
| thiagop | devananda: s/Upadte/Update/ | 20:56 |
| JayF | We can't until we have the gate running on *_ipmitool drivers. | 20:56 |
| JayF | We haven't changed those yet because the pass rate, as graphed, isn't as high as our *_ssh drivers | 20:56 |
| JayF | I would be +1 to a change making the ipmitool + vbmc the devstack default and what our docs reccomend, though | 20:56 |
| NobodyCam | devananda: thank you :) | 20:56 |
| TheJulia | I thoughtahh yeah, stil two ssh jobs | 20:56 |
| *** e0ne has joined #openstack-ironic | 20:57 | |
| betherly | yay ironic-ui new docs finally building so they will be all up to date :D | 20:59 |
| NobodyCam | awesome betherly :) | 20:59 |
| devananda | betherly: o/^5 | 21:00 |
| *** ijw has quit IRC | 21:01 | |
| betherly | if anyone wants to try out the docs and the ui and give any user feedback it is always always appreciated so we can make sure that everything works as you would expect and require :D | 21:01 |
| betherly | add and delete node functionalities have merged and work. its the edit node stuff that is still WIP so that could be frustrating but i promise its coming soon!! | 21:02 |
| jroll | NobodyCam: left another note - tl;dr pxe_* is iscsi, agent_* is http, both use IPA, neither is more capable just different | 21:03 |
| *** adu has quit IRC | 21:04 | |
| NobodyCam | ack :) | 21:05 |
| *** e0ne has quit IRC | 21:06 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [neutron] https://review.openstack.org/304838 | 21:06 |
| *** ijw has joined #openstack-ironic | 21:10 | |
| rajinir | watanabe_isao: hi | 21:11 |
| *** adu has joined #openstack-ironic | 21:13 | |
| *** dprince has quit IRC | 21:21 | |
| *** vishwanathj has quit IRC | 21:25 | |
| watanabe_isao | rajinir, sorry. Nothing at all, now. :) Moving to the company... | 21:25 |
| *** watanabe_isao has quit IRC | 21:25 | |
| *** vishwanathj has joined #openstack-ironic | 21:25 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [DEFAULT] https://review.openstack.org/309070 | 21:30 |
| openstackgerrit | Devananda van der Veen proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 21:32 |
| devananda | TheJulia: I've added the depends-on ^ | 21:32 |
| TheJulia | Merci | 21:32 |
| *** rpioso has quit IRC | 21:32 | |
| devananda | incidentally, I also -1'd the thing it depends on | 21:32 |
| *** baoli has quit IRC | 21:33 | |
| TheJulia | devananda: I saw | 21:33 |
| devananda | that seems to be my only real objection in that series | 21:33 |
| devananda | though I'd like to see rloo's and my comments on 206244 fixed | 21:34 |
| rloo | devananda: yeah, i'd be fine with that too. I was being nice :) | 21:35 |
| devananda | vdrok, vsaienk0: I know it's late for you both, but just checking if you're around & have a moment to discuss 317392? | 21:36 |
| *** cdearborn has quit IRC | 21:37 | |
| *** aNupoisc has joined #openstack-ironic | 21:39 | |
| openstackgerrit | Merged openstack/ironic: Add support for building ISO for deploy ramdisk https://review.openstack.org/264590 | 21:39 |
| openstackgerrit | Merged openstack/ironic: Config variable to configure [glance] section https://review.openstack.org/266803 | 21:39 |
| *** daemontool_ has joined #openstack-ironic | 21:40 | |
| TheJulia | devananda: I think this is the driver https://review.openstack.org/#/c/188370/20/specs/approved/driver-composition-reform.rst@198 What if we just take whatever when the post occurs? Validation still has to occur, updating still has to have a check based on the driver composition spec. | 21:40 |
| *** vishwanathj has quit IRC | 21:41 | |
| devananda | TheJulia: exactly | 21:41 |
| devananda | I just posted a longer comment on that review, too | 21:41 |
| TheJulia | awesome | 21:41 |
| *** vishwanathj has joined #openstack-ironic | 21:41 | |
| jroll | devananda: "I'm fairly convinced the RPC call get_enabled_network_interfaces will need to be removed later -- this should be visible from the database, much like the current list of enabled drivers is determined by querying the database." that is the plan | 21:42 |
| devananda | jroll: cool. I didn't see any notes inline about that | 21:42 |
| jroll | devananda: I should say, that's what the driver comp spec outlines | 21:43 |
| jroll | and we're treating this as part of that | 21:43 |
| devananda | it outlines adding the RPC call? | 21:43 |
| * devananda must have forgotten that detail | 21:43 | |
| jroll | it outlines putting enabled interfaces in the db | 21:43 |
| devananda | right | 21:43 |
| devananda | that I knew | 21:43 |
| jroll | okay cool | 21:43 |
| *** rcernin has quit IRC | 21:43 | |
| devananda | my objection is adding an RPC call into POST | 21:43 |
| jroll | yep, I agree | 21:44 |
| devananda | :) | 21:44 |
| devananda | the three patches before that LGTM | 21:44 |
| TheJulia | agreed, just posted another comment to it | 21:44 |
| jroll | just making sure you know that this being visible from the db is the goal for the driver comp stuff | 21:44 |
| devananda | I'm going through the grenade logs now | 21:44 |
| jroll | and we plan to refactor some of this to fit this better | 21:44 |
| devananda | jroll: yea. I was pretty sure that was the plan | 21:44 |
| devananda | thanks for confirming :) | 21:44 |
| jroll | :) | 21:44 |
| thiagop | I'm calling it a day | 21:45 |
| thiagop | see you tomorrow | 21:45 |
| devananda | thiagop: g'night! | 21:45 |
| jroll | night thiagop | 21:45 |
| TheJulia | ditto, I think it is time to call it a day | 21:45 |
| *** thiagop has quit IRC | 21:45 | |
| TheJulia | goodnight everyone | 21:45 |
| jroll | \o TheJulia | 21:45 |
| jroll | devananda: did you want my eyes on those or not +W for some other reason? | 21:46 |
| devananda | http://logs.openstack.org/44/206244/116/check/gate-grenade-dsvm-ironic/634ba53/logs/new/screen-ir-cond.txt.gz?level=INFO#_2016-07-13_11_59_00_840 | 21:47 |
| devananda | jroll: I hav enot +W'd yet because I hae ve an upgrade concern | 21:47 |
| devananda | http://logs.openstack.org/44/206244/116/check/gate-grenade-dsvm-ironic/634ba53/logs/etc/ironic/ironic.conf.txt.gz | 21:47 |
| jroll | k | 21:47 |
| devananda | [neutron] | 21:47 |
| devananda | cleaning_network_uuid = b7a97c12-24cd-499b-8530-9f4b6f5ed8de | 21:47 |
| devananda | y | 21:47 |
| devananda | the grenade 'new' env includes this in the ironic.conf | 21:47 |
| devananda | and the 'new' ironic-conductor logs are, indeed, doing all the wonderful network magic | 21:48 |
| devananda | even before hte API changes are included | 21:48 |
| devananda | that actually concerns me | 21:48 |
| devananda | *even before the API or devstack changes have landed | 21:48 |
| *** Goneri has quit IRC | 21:48 | |
| jroll | devananda: wait, which magic concerns you? | 21:49 |
| devananda | hm | 21:49 |
| devananda | I'm still digging -- pardon my thinking outloud | 21:49 |
| jroll | no worries | 21:49 |
| jroll | trying to help you walk through it :) | 21:49 |
| devananda | I guess we're setting [neutron] cleaning_network_id all the time now ? | 21:49 |
| jroll | we always have been | 21:49 |
| jroll | I believe | 21:50 |
| devananda | it is not actually required though | 21:50 |
| TheJulia | it is not required for the current state afaik | 21:50 |
| jroll | yeah, see an unrelated change http://logs.openstack.org/90/264590/29/check/gate-tempest-dsvm-ironic-agent_ssh/6beab66/logs/etc/ironic/ironic.conf.txt.gz | 21:50 |
| TheJulia | unless the neutron tenant networkd river is selected | 21:50 |
| jroll | eh? | 21:50 |
| jroll | it's totally required to do cleaning today, before this work lands | 21:50 |
| jroll | ironic manages the cleaning ports today | 21:51 |
| devananda | I had an issue in my dev lab with these patches a couple months ago. "noop" network broke w/o a cleaning_net_id | 21:51 |
| jroll | it does not use the nova ports for cleaning | 21:51 |
| devananda | jroll: only in some cases | 21:51 |
| jroll | devananda: in current trunk, it manages cleaning ports when using neutron for dhcp | 21:51 |
| devananda | jroll: right. I think I didn't realize that because it wasn't logging it >_< | 21:51 |
| TheJulia | devananda: that has been fixed | 21:51 |
| jroll | yeah, maybe | 21:51 |
| devananda | TheJulia: thus my surprise :) | 21:52 |
| jroll | we don't test noop dhcp driver, so we won't be able to test the upgrade in that mode | 21:52 |
| devananda | so, I see it is not some bad magic in the network patches -- it's just bette rlogging :) | 21:52 |
| jroll | hehe | 21:52 |
| TheJulia | indeed() | 21:52 |
| rloo | devananda, TheJulia, jroll: fyi, vdrok is on PTO, he'll be back on Tues. In the meantime, he said vsaienk0 would baby the network patches. | 21:53 |
| devananda | rloo: ah, ty | 21:54 |
| devananda | unless ya'll speak up now, I'm going to land the first three of those patches -- I think they're good enough() and they've been passing tests and grenade | 21:54 |
| devananda | and the second API patch is separate enough now that we can land it later | 21:54 |
| rloo | go for it devananda!!! | 21:55 |
| jroll | devananda: please do | 21:56 |
| rloo | devananda: didn't you -1 the third one though? | 21:57 |
| devananda | rloo: I did, with nits. | 21:57 |
| devananda | suitale to a follow on, as we often do | 21:57 |
| *** vishwanathj has quit IRC | 21:57 | |
| rloo | devananda: i thought if nits only, you do +, not -1. Anyway, I don't really care. Will just be amused if you can +A/-1. | 21:58 |
| rloo | devananda: darn! :) | 21:58 |
| devananda | rloo: you're correct. I -1'd while still going through the patches ... and just changed my -1 to a +2. call me fickle if you will | 21:59 |
| rloo | devananda: ha ha. | 21:59 |
| JayF | you're going to get dinged by reviewstats for a disagreement! | 22:00 |
| JayF | :P | 22:00 |
| *** sinh has quit IRC | 22:02 | |
| *** adu has quit IRC | 22:02 | |
| devananda | LOL | 22:03 |
| *** jcoufal has quit IRC | 22:04 | |
| devananda | ok - those three are in the gate now .... | 22:04 |
| * devananda crosses fingers | 22:04 | |
| jroll | \o/ | 22:04 |
| JayF | nice! | 22:04 |
| devananda | time to move the car & find a late lunch .... bbiab | 22:05 |
| jroll | anyone have a reason not to move py35 jobs to voting? | 22:15 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add power state change notifications https://review.openstack.org/321865 | 22:15 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: Add notification base classes and docs https://review.openstack.org/298461 | 22:15 |
| *** sinh has joined #openstack-ironic | 22:15 | |
| JayF | jroll: I'd sat go fo rit | 22:16 |
| jroll | JayF: I just realized this means going through all projects we own >.> | 22:18 |
| jroll | which is fine | 22:18 |
| jroll | just took on more work than I thought :P | 22:18 |
| JayF | jroll: going through all the projects we own? | 22:19 |
| jroll | JayF: yeah, IPA, ironic-lib, etc | 22:20 |
| jroll | make sure the jobs are massing | 22:20 |
| jroll | passing. | 22:20 |
| JayF | gotcha | 22:20 |
| *** causten_ has quit IRC | 22:20 | |
| *** ametts has quit IRC | 22:20 | |
| *** garthb has joined #openstack-ironic | 22:22 | |
| betherly | jroll: updated https://wiki.openstack.org/wiki/Ironic#Projects to add the ironic-ui | 22:27 |
| betherly | let me know if you want me to edit it further | 22:27 |
| jroll | JayF: if you're curious, they were all passing on all of our projects that support py3 at all, here's the patch https://review.openstack.org/341830 | 22:28 |
| JayF | I'll review it | 22:28 |
| jroll | betherly: we should really use git.o.o for everything on that page, but that isn't your problem, I'll fix it. LGTM :) | 22:28 |
| jroll | betherly: also, what're you doing here, it's borderline late for me let alone you | 22:28 |
| betherly | jroll: woops sorry :/ just knew i said id update it * months ago and have completely failed to. let me know if you want me to submit it through gerrit somehow | 22:29 |
| *** jrist has joined #openstack-ironic | 22:29 | |
| * betherly is not sure how though so would appreciate jroll insight | 22:29 | |
| jroll | betherly: no, I mean the code links should be to git.openstack.org instead of github | 22:29 |
| jroll | fixing that now though | 22:29 |
| jroll | thank you for the updates :) | 22:30 |
| betherly | jroll: dont worry im not up stupid o'clock. im in CA this week so its only 15:30 :) | 22:30 |
| jroll | aha! | 22:30 |
| jroll | enjoy the best coast, then | 22:30 |
| betherly | what was stupid o'clock was my body waking me up at 3:30am local time today and yesterday | 22:30 |
| betherly | facepalm | 22:30 |
| jroll | hehe | 22:31 |
| *** ElCoyote_ has left #openstack-ironic | 22:35 | |
| *** jrist has quit IRC | 22:46 | |
| JayF | https://review.openstack.org/#/c/301923/ has 1x+1, 1x+2, is an easy review, and the requirement bump it depends on is in the gate | 22:48 |
| JayF | someone should do me a solid and land it :D | 22:48 |
| *** milan has quit IRC | 22:52 | |
| *** PollyZ_ has quit IRC | 22:58 | |
| *** mordred has joined #openstack-ironic | 23:01 | |
| *** sabeen1 has quit IRC | 23:01 | |
| *** adu has joined #openstack-ironic | 23:03 | |
| *** ionutbalutoiu has quit IRC | 23:05 | |
| *** bradjones has quit IRC | 23:06 | |
| *** bradjones has joined #openstack-ironic | 23:07 | |
| *** bradjones has quit IRC | 23:07 | |
| *** bradjones has joined #openstack-ironic | 23:07 | |
| *** ionutbalutoiu has joined #openstack-ironic | 23:08 | |
| *** piet has quit IRC | 23:10 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Upadte devstack section of quickstart to use agent_ipmitool https://review.openstack.org/341801 | 23:15 |
| NobodyCam | JayF: like that ^^^^^ | 23:15 |
| NobodyCam | ?? | 23:15 |
| NobodyCam | :p | 23:15 |
| NobodyCam | lol | 23:16 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Update devstack section of quickstart to use agent_ipmitool https://review.openstack.org/341801 | 23:16 |
| JayF | NobodyCam: +2'd | 23:17 |
| NobodyCam | :) | 23:17 |
| JayF | NobodyCam: wanna take a look at 301923 ^ I linked earlier | 23:17 |
| NobodyCam | sure | 23:18 |
| *** piet has joined #openstack-ironic | 23:21 | |
| *** rbudden has quit IRC | 23:22 | |
| jroll | NobodyCam: quick nits there | 23:23 |
| jroll | I can fix them and land if you want though | 23:23 |
| *** ijw has quit IRC | 23:24 | |
| JayF | jroll: your project-config change got V-1 and I commented a -1 on it | 23:24 |
| jroll | JayF: lame | 23:24 |
| JayF | jroll: you have to also modify the names of the actual jobs, not just the list in zuul layout | 23:24 |
| jroll | whaaaaat | 23:24 |
| JayF | yep | 23:24 |
| jroll | ugh I hate jjb | 23:25 |
| NobodyCam | JayF: I kinda wise that had a release note just because its adding a new thing | 23:25 |
| NobodyCam | jroll: let me take a quick look | 23:25 |
| JayF | NobodyCam: that's exceedingly valid, especially since it adds config options | 23:25 |
| JayF | NobodyCam: please -1 it with that comment | 23:25 |
| jroll | thanks for copying out the errors for me JayF | 23:25 |
| jroll | NobodyCam: good catch on reno | 23:25 |
| *** adu has quit IRC | 23:26 | |
| *** sdake has joined #openstack-ironic | 23:26 | |
| JayF | I should probably also regen config file for that too, honestly | 23:26 |
| JayF | and make sure it's actually pulling in those option | 23:26 |
| JayF | *options | 23:26 |
| jroll | hrm, ag ironic-python35 gives me nothing, so where were the old ones defined? :/ | 23:27 |
| NobodyCam | JayF: oh that would be good too. thou I did not comment about it | 23:27 |
| jroll | oh I see it, lame | 23:28 |
| JayF | yeah; it's really obnoxious how it's laid out | 23:28 |
| *** sdake__ has joined #openstack-ironic | 23:28 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Update devstack section of quickstart to use agent_ipmitool https://review.openstack.org/341801 | 23:29 |
| NobodyCam | jroll: nits addressed :) | 23:29 |
| jroll | NobodyCam: +2 | 23:29 |
| jroll | thanks | 23:29 |
| JayF | +2a | 23:30 |
| jroll | I might leave that til morning for the euro opinion | 23:30 |
| jroll | or not | 23:30 |
| jroll | lol | 23:30 |
| JayF | I mean, I can if we need/want to? | 23:30 |
| JayF | I can unland it, lol | 23:30 |
| NobodyCam | lol | 23:30 |
| NobodyCam | :) | 23:30 |
| JayF | seems like an obvious/minor change | 23:30 |
| jroll | yeah the s/pxe/agent/ is the weird part | 23:30 |
| JayF | Patch Set 5: -Workflow | 23:31 |
| JayF | Going to let someone less invested in the agent land this :P | 23:31 |
| NobodyCam | :) +++ | 23:31 |
| *** sdake has quit IRC | 23:32 | |
| *** ijw has joined #openstack-ironic | 23:33 | |
| *** piet has quit IRC | 23:33 | |
| * devananda eagerly watches the network patches finish the gate | 23:33 | |
| *** piet has joined #openstack-ironic | 23:33 | |
| jroll | sorry, power failure | 23:34 |
| * jroll should stop working and go shopping for a UPS | 23:34 | |
| jroll | thanks JayF | 23:34 |
| jroll | I'll land tomorrow if nobody nacks it | 23:34 |
| devananda | jroll: UPS's are great | 23:35 |
| jroll | indeed | 23:35 |
| NobodyCam | ++ on UPS's | 23:35 |
| jroll | I've been lazy | 23:35 |
| devananda | I've had my wireless router, NAS, etc, on UPS for years. | 23:35 |
| jroll | also didn't have many power problems in CA, this is the second blip in a month here | 23:35 |
| devananda | it's fun whenthe power goes out at home, I'm on the road, and I can still VPN in to my testbed :) | 23:36 |
| NobodyCam | devananda: have you replaced the battery on the ups? | 23:36 |
| devananda | NobodyCam: nope | 23:36 |
| NobodyCam | :p | 23:36 |
| *** sdake__ has quit IRC | 23:36 | |
| devananda | I think it's 7yr old now? ... still working well enough. | 23:36 |
| *** jaybeale has quit IRC | 23:36 | |
| devananda | the systems I have attached draw maybe 30W ... it's designed for 400W ;) | 23:37 |
| NobodyCam | mine go about every three to four years | 23:37 |
| NobodyCam | ahh | 23:37 |
| *** yuanying_off is now known as yuanying | 23:37 | |
| openstackgerrit | Merged openstack/ironic: Add 'neutron' network interface https://review.openstack.org/317393 | 23:40 |
| devananda | \o/ | 23:41 |
| *** watanabe_isao has joined #openstack-ironic | 23:41 | |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Update devstack section of quickstart to use agent_ipmitool https://review.openstack.org/341801 | 23:41 |
| NobodyCam | fixed pep8 on the link | 23:42 |
| openstackgerrit | Merged openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 23:42 |
| openstackgerrit | Merged openstack/ironic: Add multitenancy-related fields to port API object https://review.openstack.org/206244 | 23:42 |
| harlowja | rloo lucas|afk https://review.openstack.org/341858 | 23:43 |
| devananda | \o/ \o/ /o/ \o\ \o/ \o/ | 23:43 |
| NobodyCam | hehehe | 23:43 |
| devananda | only took 12 months for that patch to merge ;) | 23:44 |
| *** mtanino has quit IRC | 23:44 | |
| devananda | Sukhdev: we've made progress ^^^ | 23:44 |
| NobodyCam | \o/ did we start this journey at the Paris summit? | 23:45 |
| NobodyCam | did == didn't | 23:45 |
| *** adu has joined #openstack-ironic | 23:50 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/341859 | 23:51 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/341860 | 23:51 |
| *** adu has quit IRC | 23:55 | |
| *** Sukhdev has quit IRC | 23:58 | |
| *** Sukhdev has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
