| *** openstackstatus has quit IRC | 00:12 | |
| *** openstackstatus has joined #openstack-ironic | 00:14 | |
| *** ChanServ sets mode: +v openstackstatus | 00:14 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 00:23 | |
| *** AlexeyAbashkin has quit IRC | 00:27 | |
| *** alineb has quit IRC | 00:41 | |
| *** akhilaki_ has joined #openstack-ironic | 00:42 | |
| *** akhilaki_ has joined #openstack-ironic | 00:43 | |
| *** d0ugal_ has joined #openstack-ironic | 00:48 | |
| rpioso | jroll: http://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/devstack-vm-gate.sh#n445 | 01:00 |
|---|---|---|
| *** rpioso is now known as rpioso|afk | 01:01 | |
| *** akhilaki_ has quit IRC | 01:02 | |
| *** openstackgerrit has joined #openstack-ironic | 01:03 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic stable/ocata: DNM/WIP: Testing https://review.openstack.org/546464 | 01:03 |
| *** tiendc has joined #openstack-ironic | 01:07 | |
| *** alineb has joined #openstack-ironic | 01:11 | |
| *** tiendc has quit IRC | 01:17 | |
| *** tiendc has joined #openstack-ironic | 01:19 | |
| openstackgerrit | Merged openstack/ironic stable/pike: Clean nodes stuck in CLEANING state when ir-cond restarts https://review.openstack.org/546083 | 01:26 |
| *** hshiina has joined #openstack-ironic | 01:28 | |
| *** rcernin has quit IRC | 01:36 | |
| *** rcernin has joined #openstack-ironic | 01:36 | |
| *** rcernin has quit IRC | 01:42 | |
| *** rcernin has joined #openstack-ironic | 01:43 | |
| *** sapd has joined #openstack-ironic | 01:51 | |
| *** rcernin has quit IRC | 01:53 | |
| *** rcernin has joined #openstack-ironic | 01:53 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Set 'initrd' to 'rescue_ramdisk' for rescue with iPXE https://review.openstack.org/545186 | 01:55 |
| *** khyr0n has quit IRC | 01:58 | |
| *** phuongnh has joined #openstack-ironic | 02:21 | |
| *** khyr0n has joined #openstack-ironic | 02:27 | |
| *** harlowja_ has quit IRC | 02:28 | |
| *** links has joined #openstack-ironic | 02:49 | |
| *** akhilaki has joined #openstack-ironic | 03:13 | |
| *** stendulker has joined #openstack-ironic | 03:22 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 03:23 | |
| *** stendulker has quit IRC | 03:23 | |
| *** AlexeyAbashkin has quit IRC | 03:27 | |
| *** stendulker has joined #openstack-ironic | 03:54 | |
| *** vsaienk0 has joined #openstack-ironic | 03:58 | |
| *** khyr0n has quit IRC | 03:59 | |
| *** vsaienk0 has quit IRC | 04:09 | |
| *** d0ugal__ has joined #openstack-ironic | 04:12 | |
| *** d0ugal_ has quit IRC | 04:15 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 04:22 | |
| *** tiendc has quit IRC | 04:23 | |
| *** phuongnh has quit IRC | 04:23 | |
| *** phuongnh has joined #openstack-ironic | 04:23 | |
| *** tiendc has joined #openstack-ironic | 04:23 | |
| *** AlexeyAbashkin has quit IRC | 04:26 | |
| *** fragatina has joined #openstack-ironic | 05:10 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Update iLO documentation for deprecating classical drivers https://review.openstack.org/542593 | 05:14 |
| *** khyr0n has joined #openstack-ironic | 05:15 | |
| *** khyr0n has quit IRC | 05:16 | |
| *** khyr0n has joined #openstack-ironic | 05:17 | |
| *** etingof has quit IRC | 05:19 | |
| *** etingof has joined #openstack-ironic | 05:27 | |
| *** vsaienk0 has joined #openstack-ironic | 05:28 | |
| *** dpawar has joined #openstack-ironic | 05:30 | |
| *** vsaienk0 has quit IRC | 05:38 | |
| *** marios has joined #openstack-ironic | 06:03 | |
| *** pmannidi has quit IRC | 06:04 | |
| *** d0ugal__ has quit IRC | 06:27 | |
| *** d0ugal__ has joined #openstack-ironic | 06:32 | |
| *** charcol-v2 has joined #openstack-ironic | 06:40 | |
| *** charcol-v2 has quit IRC | 06:46 | |
| *** hjensas has quit IRC | 06:51 | |
| *** d0ugal__ has quit IRC | 06:52 | |
| *** mjura has joined #openstack-ironic | 07:04 | |
| *** vsaienk0 has joined #openstack-ironic | 07:06 | |
| *** khyr0n has quit IRC | 07:09 | |
| *** yangyape_ has quit IRC | 07:11 | |
| *** yangyapeng has joined #openstack-ironic | 07:12 | |
| *** yangyapeng has quit IRC | 07:17 | |
| *** rcernin has quit IRC | 07:21 | |
| *** parlos has joined #openstack-ironic | 07:23 | |
| *** fragatina has quit IRC | 07:41 | |
| *** hoonetorg has quit IRC | 07:46 | |
| *** pcaruana has joined #openstack-ironic | 07:47 | |
| *** fragatina has joined #openstack-ironic | 07:50 | |
| *** e0ne has joined #openstack-ironic | 07:52 | |
| *** fragatina has quit IRC | 07:54 | |
| *** hjensas has joined #openstack-ironic | 07:55 | |
| *** d0ugal has joined #openstack-ironic | 07:55 | |
| *** d0ugal has quit IRC | 07:55 | |
| *** d0ugal has joined #openstack-ironic | 07:55 | |
| *** hoonetorg has joined #openstack-ironic | 08:00 | |
| *** d0ugal has quit IRC | 08:00 | |
| *** d0ugal has joined #openstack-ironic | 08:01 | |
| *** MattMan has quit IRC | 08:03 | |
| *** akhilaki has quit IRC | 08:03 | |
| *** MattMan has joined #openstack-ironic | 08:03 | |
| *** e0ne has quit IRC | 08:04 | |
| *** d0ugal has quit IRC | 08:06 | |
| *** fragatina has joined #openstack-ironic | 08:06 | |
| *** fragatina has quit IRC | 08:08 | |
| *** fragatina has joined #openstack-ironic | 08:09 | |
| *** vabada has quit IRC | 08:11 | |
| *** vabada has joined #openstack-ironic | 08:12 | |
| *** fragatina has quit IRC | 08:13 | |
| *** d0ugal has joined #openstack-ironic | 08:15 | |
| *** e0ne has joined #openstack-ironic | 08:16 | |
| *** sapd_ has joined #openstack-ironic | 08:17 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 08:22 | |
| *** athomas has joined #openstack-ironic | 08:27 | |
| *** d0ugal has quit IRC | 08:29 | |
| *** tesseract has joined #openstack-ironic | 08:34 | |
| *** d0ugal has joined #openstack-ironic | 08:34 | |
| *** d0ugal has quit IRC | 08:39 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Set 'initrd' to 'rescue_ramdisk' for rescue with iPXE https://review.openstack.org/545186 | 08:39 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic master: Update iLO documentation for deprecating classical drivers https://review.openstack.org/542593 | 08:40 |
| *** e0ne has quit IRC | 08:47 | |
| *** d0ugal has joined #openstack-ironic | 08:48 | |
| *** d0ugal has quit IRC | 08:48 | |
| *** d0ugal has joined #openstack-ironic | 08:48 | |
| *** d0ugal has quit IRC | 08:58 | |
| *** jtomasek has joined #openstack-ironic | 09:01 | |
| *** vsaienk0 has quit IRC | 09:06 | |
| *** d0ugal has joined #openstack-ironic | 09:07 | |
| *** hamzy_ has joined #openstack-ironic | 09:17 | |
| *** hamzy has quit IRC | 09:19 | |
| *** mgoddard_ has joined #openstack-ironic | 09:19 | |
| *** derekh has joined #openstack-ironic | 09:39 | |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/python-ironic-inspector-client master: Clean imports in code https://review.openstack.org/546550 | 09:39 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode tempest tests https://review.openstack.org/528699 | 09:42 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/queens: Increase the instance_info column size to LONGTEXT on MySQL/MariaDB https://review.openstack.org/546551 | 09:45 |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-tempest-plugin master: Rescue mode standalone tests https://review.openstack.org/538119 | 09:47 |
| *** csatari has joined #openstack-ironic | 09:58 | |
| *** dtantsur|afk is now known as dtantsur | 10:02 | |
| dtantsur | morning ironic | 10:02 |
| *** pcaruana has quit IRC | 10:05 | |
| *** d0ugal_ has joined #openstack-ironic | 10:06 | |
| *** d0ugal has quit IRC | 10:06 | |
| *** hjensas has quit IRC | 10:08 | |
| etingof | o/ | 10:11 |
| *** d0ugal_ has quit IRC | 10:11 | |
| *** d0ugal_ has joined #openstack-ironic | 10:13 | |
| *** d0ugal__ has joined #openstack-ironic | 10:17 | |
| *** d0ugal_ has quit IRC | 10:18 | |
| *** pcaruana has joined #openstack-ironic | 10:20 | |
| *** e0ne has joined #openstack-ironic | 10:21 | |
| * dtantsur can finally propose meaningful names for the S release \o/ | 10:24 | |
| * dtantsur really likes OpenStack Spree though :) | 10:26 | |
| *** parlos has quit IRC | 10:26 | |
| *** jaganathan has quit IRC | 10:27 | |
| *** hjensas has joined #openstack-ironic | 10:29 | |
| *** priteau has joined #openstack-ironic | 10:34 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: DNM experiment re-using devstack-tempest job https://review.openstack.org/520167 | 10:38 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: DNM experiment re-using devstack-tempest job https://review.openstack.org/520167 | 10:44 |
| etingof | dtantsur, Schönefeld/berg ;) | 10:46 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: DNM experiment re-using devstack-tempest job https://review.openstack.org/520167 | 10:46 |
| dtantsur | etingof: I think Schoneberg is there | 10:47 |
| * etingof expects quite some UnicodeDecodeError's ;) | 10:48 | |
| *** d0ugal__ has quit IRC | 10:54 | |
| *** d0ugal has joined #openstack-ironic | 10:54 | |
| *** d0ugal has quit IRC | 10:54 | |
| *** d0ugal has joined #openstack-ironic | 10:54 | |
| *** phuongnh has quit IRC | 11:04 | |
| *** manjeets has quit IRC | 11:05 | |
| *** manjeets has joined #openstack-ironic | 11:06 | |
| *** tiendc has quit IRC | 11:12 | |
| *** stendulker has quit IRC | 11:12 | |
| *** parlos has joined #openstack-ironic | 11:14 | |
| *** pcaruana has quit IRC | 11:19 | |
| *** d0ugal has quit IRC | 11:23 | |
| *** etingof has quit IRC | 11:23 | |
| *** etingof has joined #openstack-ironic | 11:28 | |
| *** d0ugal has joined #openstack-ironic | 11:30 | |
| *** pcaruana has joined #openstack-ironic | 11:33 | |
| dtantsur | etingof: haha. well, they only accept ASCII | 12:03 |
| dtantsur | otherwise imagine the actual spelling of OpenStack Ocata (IIRC) ;) | 12:04 |
| *** d0ugal has quit IRC | 12:06 | |
| *** parlos has quit IRC | 12:07 | |
| *** d0ugal has joined #openstack-ironic | 12:10 | |
| *** parlos has joined #openstack-ironic | 12:13 | |
| *** mgagne has quit IRC | 12:19 | |
| *** d0ugal has quit IRC | 12:20 | |
| *** d0ugal has joined #openstack-ironic | 12:22 | |
| *** dprince has joined #openstack-ironic | 12:23 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Refresh port before storing VIF ID for cleaning https://review.openstack.org/546584 | 12:31 |
| *** d0ugal has quit IRC | 12:34 | |
| *** mgagne has joined #openstack-ironic | 12:36 | |
| *** mgagne is now known as Guest20946 | 12:36 | |
| *** milan_ has joined #openstack-ironic | 12:39 | |
| *** persia has quit IRC | 12:46 | |
| *** persia has joined #openstack-ironic | 12:47 | |
| *** melwitt has quit IRC | 12:47 | |
| *** melwitt has joined #openstack-ironic | 12:52 | |
| *** weshay_PTO is now known as weshay | 13:04 | |
| *** rbartal has joined #openstack-ironic | 13:11 | |
| *** d0ugal has joined #openstack-ironic | 13:13 | |
| *** d0ugal_ has joined #openstack-ironic | 13:19 | |
| *** d0ugal has quit IRC | 13:20 | |
| *** ricardoas has joined #openstack-ironic | 13:20 | |
| *** d0ugal_ has quit IRC | 13:25 | |
| *** hjensas has quit IRC | 13:27 | |
| hshiina | hi, it seems that vif detach without a lock causes another issue: https://bugs.launchpad.net/ironic/+bug/1750785 | 13:36 |
| openstack | Launchpad bug 1750785 in Ironic "Detached VIF may be restored during node cleaning" [Undecided,In progress] - Assigned to Hironori Shiina (shiina-hironori) | 13:36 |
| hshiina | i wounder if we should revert the lock removal or fix this issue for queens. | 13:36 |
| dtantsur | hshiina: hi! oh.. cannot we land and backport your patch above though? | 13:37 |
| *** d0ugal_ has joined #openstack-ironic | 13:38 | |
| *** dpawar has quit IRC | 13:38 | |
| hshiina | dtantsur: i think we can land my patch to mitigate this issue. | 13:39 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Rework logic handling reserved orphaned nodes in the conductor https://review.openstack.org/546273 | 13:41 |
| dtantsur | hshiina: anyway, thanks for raising this. I'll check with other folks later today to see what they think. | 13:41 |
| dtantsur | jroll, TheJulia, ^^^ | 13:42 |
| hshiina | dtantsur: thanks, i'm afraid i have to go home soon | 13:46 |
| dtantsur | hshiina: sure, have a nice evening | 13:46 |
| hshiina | dtantsur: thank you, good evening | 13:46 |
| *** d0ugal_ has quit IRC | 13:48 | |
| *** d0ugal has joined #openstack-ironic | 13:48 | |
| *** d0ugal has quit IRC | 13:48 | |
| *** d0ugal has joined #openstack-ironic | 13:48 | |
| *** hshiina is now known as hshiina|afk | 13:50 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Rework logic handling reserved orphaned nodes in the conductor https://review.openstack.org/546273 | 13:55 |
| *** d0ugal has quit IRC | 13:58 | |
| *** d0ugal has joined #openstack-ironic | 13:59 | |
| *** cdearborn has joined #openstack-ironic | 14:01 | |
| *** parlos has quit IRC | 14:01 | |
| *** parlos has joined #openstack-ironic | 14:04 | |
| *** d0ugal has quit IRC | 14:04 | |
| TheJulia | Good morning everyone | 14:10 |
| dtantsur | morning TheJulia | 14:10 |
| *** Goneri has joined #openstack-ironic | 14:12 | |
| *** cdearborn has quit IRC | 14:12 | |
| TheJulia | hshiina|afk: ugh, the vif should never get restored out of cleaning... I guess the port is saving :\ | 14:13 |
| TheJulia | dtantsur: +1 to Spree | 14:14 |
| *** etingof has quit IRC | 14:14 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Rework logic handling reserved orphaned nodes in the conductor https://review.openstack.org/546273 | 14:21 |
| *** d0ugal has joined #openstack-ironic | 14:22 | |
| *** hshiina|afk has quit IRC | 14:27 | |
| *** links has quit IRC | 14:29 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Support nested objects and object lists in as_dict https://review.openstack.org/545830 | 14:29 |
| *** mjturek has joined #openstack-ironic | 14:31 | |
| jroll | morning y'all | 14:36 |
| * jroll looks at vif thing | 14:36 | |
| dtantsur | it sounds like hshiina's patch is just a work around, and still a racy one | 14:37 |
| *** dmellado has quit IRC | 14:37 | |
| dtantsur | I wonder if we should get back to locking.. | 14:38 |
| dtantsur | jroll: this is also ready for review: https://review.openstack.org/546273 | 14:39 |
| patchbot | patch 546273 - ironic - Rework logic handling reserved orphaned nodes in t... | 14:39 |
| * jroll will look | 14:40 | |
| *** dmellado has joined #openstack-ironic | 14:42 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Register traits on nodes in devstack https://review.openstack.org/545370 | 14:48 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Validate instance_info.traits against node traits https://review.openstack.org/543461 | 14:48 |
| TheJulia | dtantsur: we really just need to remove the vif upon teardown, we just can't have it each and every different way imho | 14:49 |
| dtantsur | maybe.. what's the reason we don't remove it automatically? | 14:49 |
| *** d0ugal has quit IRC | 14:49 | |
| TheJulia | resistance | 14:49 |
| TheJulia | desire for pet management | 14:49 |
| TheJulia | assertion nova should remove, but we are super slow | 14:50 |
| *** baha has joined #openstack-ironic | 14:50 | |
| TheJulia | we being super slow meaning all of the processes we have to walk through to the point where nova could possibly complete the removal of the vif record | 14:50 |
| dtantsur | what's the problem with pet management though? | 14:51 |
| TheJulia | In so as much as nova's use model doesn't support it. The contract we established with nova was that nova was to add the vifs, and then remove the vifs via ironic. The pet management desire is for the vif to live to the machine after removal from active state | 14:52 |
| dtantsur | ugh | 14:52 |
| jroll | yeah, I still don't buy that argument | 14:53 |
| TheJulia | nor do I, I think we should nuke it | 14:53 |
| jroll | if you're going to need the vif on the same machine again, add it again | 14:53 |
| TheJulia | ++ | 14:53 |
| TheJulia | there is also rebuild which could be useful for a number of situations where a machine is desired to be rebuilt. We keep the vifs for that | 14:53 |
| jroll | indeed | 14:54 |
| TheJulia | the only middle ground is gutting and re-doing locking | 14:56 |
| *** fragatina has joined #openstack-ironic | 14:56 | |
| jroll | which we need to do, just isn't a short term thing :) | 14:57 |
| TheJulia | yeah, it is a very long term thing :( | 14:57 |
| *** d0ugal has joined #openstack-ironic | 14:57 | |
| TheJulia | because we would have to plan that across a release boundry and have dual locking behavior potentially | 14:58 |
| *** etingof has joined #openstack-ironic | 14:59 | |
| jroll | so fun | 14:59 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Prevent overwriting of last_error on cleaning failures https://review.openstack.org/546656 | 15:00 |
| *** crushil_ has joined #openstack-ironic | 15:00 | |
| *** crushil has joined #openstack-ironic | 15:00 | |
| TheJulia | EINSUFFCENTLIQUOR | 15:00 |
| *** crushil_ has quit IRC | 15:00 | |
| TheJulia | Anyway, packing up this laptop because I need to head to $new_house | 15:01 |
| jroll | woot | 15:01 |
| jroll | see you later \o | 15:01 |
| *** edleafe has quit IRC | 15:12 | |
| *** edleafe has joined #openstack-ironic | 15:12 | |
| *** rloo has joined #openstack-ironic | 15:13 | |
| rloo | good morning to all ironickers in the universe. This means jroll, dtantsur, TheJulia! | 15:19 |
| dtantsur | morning rloo | 15:19 |
| rloo | derekh: btw, thx for booking the pub for our PTG festivities! | 15:20 |
| derekh | rloo: no prob at all | 15:20 |
| dtantsur | derekh++ | 15:20 |
| rloo | derekh: :) | 15:22 |
| *** yangyapeng has joined #openstack-ironic | 15:32 | |
| *** deadnull has joined #openstack-ironic | 15:35 | |
| openstackgerrit | Merged openstack/ironic stable/queens: Increase the instance_info column size to LONGTEXT on MySQL/MariaDB https://review.openstack.org/546551 | 15:45 |
| rloo | dtantsur and others: i was reviewing the VNC graphical console spec: https://review.openstack.org/#/c/306074/ | 15:46 |
| patchbot | patch 306074 - ironic-specs - VNC Graphical console | 15:46 |
| rloo | dtantsur: do you know whether we need to have both serial & graphical consoles for a node, at the same time? Or only one console max, at any time? | 15:47 |
| dtantsur | rloo: both, these are separate interfaces with potentially separate purposes | 15:47 |
| rloo | dtantsur: so you see it as a new interface, not a ConsoleInterface? | 15:48 |
| dtantsur | rloo: this is pretty much what we all agreed on, no? | 15:48 |
| dtantsur | this has been discussed twice with the same outcome :) | 15:48 |
| rloo | dtantsur: i don't recall. i can look at past etherpads to see. i am asking cuz the existing spec isn't clear to me. | 15:48 |
| rloo | dtantsur: it makes sense for there to be two separate interfaces. maybe i am misunderstanding the current spec. sigh. i'll look at past etherpads. we need to get these done sooner :-( | 15:49 |
| *** rbartal has quit IRC | 15:53 | |
| dtantsur | I'm not sure what we would get by mixing console and graphical interfaces, to be honest. less additions to API - yes, but these are really not the same things, and probably won't even conflict | 15:53 |
| dtantsur | ah, yeah, I was also about to bring console logging, but Tim beat me to it | 15:54 |
| *** r-daneel has joined #openstack-ironic | 15:54 | |
| rloo | dtantsur: i saw two ways of implementing; easiest is a new eg GraphicalConsoleinterface. Harder but maybe? simpler, is existing ConsoleInterface supporting both serial and graphical, and we'd have to change some internal stuff to allow more than one of them per node (which is probably do-able and maybe useful for future stuff but ugh). | 15:56 |
| rloo | dtantsur: i think there is a spec for console logging but not sure. sooo many specs... :-( | 15:56 |
| dtantsur | it may be dangerous to assume that console and graphical access it using the same tech | 15:57 |
| dtantsur | e.g. we can have a vendor VNC but generic ipmitool sol | 15:57 |
| dtantsur | we can solve it with inheritance, but it may get ugly soon | 15:57 |
| dtantsur | also: I've been doing the driver composition so that we can have MOAR INTERFACES \o/ | 15:57 |
| rloo | dtantsur: right, that's the ugh part. i'm not sure exactly what it would entail, which means it could take longer to do | 15:58 |
| rloo | dtantsur: OH. i thought we didn't want to add more interfaces! :) | 15:58 |
| *** r-daneel_ has joined #openstack-ironic | 16:01 | |
| *** r-daneel has quit IRC | 16:02 | |
| *** r-daneel_ is now known as r-daneel | 16:02 | |
| *** zer0c00l has quit IRC | 16:04 | |
| *** crushil has quit IRC | 16:06 | |
| *** crushil has joined #openstack-ironic | 16:06 | |
| *** manjeets has quit IRC | 16:11 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [WIP] Prevent overwriting of last_error on cleaning failures https://review.openstack.org/546656 | 16:11 |
| dtantsur | rloo: with the driver composition in place it's much less of a pain | 16:13 |
| rloo | dtantsur: true! If we're ok with an explosion of interfaces, corresponding configs, etc :) but why not! | 16:13 |
| *** pcaruana has quit IRC | 16:16 | |
| dtantsur | rloo: as small review, if you don't mind https://review.openstack.org/545285 | 16:19 |
| patchbot | patch 545285 - ironic-inspector - Switch documentation to hardware types | 16:19 |
| *** openstackgerrit has quit IRC | 16:19 | |
| rloo | dtantsur: sure, in a few minutes; i'm going back in history right now, about vnc consoles :) | 16:19 |
| dtantsur | ack | 16:19 |
| rloo | dtantsur: ok, going to take a break from that cuz yeah, we talked about it in newton summit, apr 2016 :-( | 16:24 |
| dtantsur | time flies.. | 16:24 |
| *** dprince has quit IRC | 16:24 | |
| rloo | dtantsur: https://etherpad.openstack.org/p/ironic-rocky-ptg-graphical-console. i was going to copy/paste the stuff from the past ... | 16:25 |
| rloo | dtantsur: i might still do it... i think it is good/bad/eye opener/<add your adjective here> | 16:25 |
| dtantsur | yep, thanks for keeping an eye on it! | 16:29 |
| *** AlexeyAbashkin has quit IRC | 16:35 | |
| *** parlos has quit IRC | 16:36 | |
| *** dprince has joined #openstack-ironic | 16:37 | |
| *** r-daneel_ has joined #openstack-ironic | 16:43 | |
| *** r-daneel has quit IRC | 16:44 | |
| *** r-daneel_ is now known as r-daneel | 16:44 | |
| dtantsur | rloo: another urgent thing for queens: https://bugs.launchpad.net/ironic/+bug/1750785 and https://review.openstack.org/546584 | 16:53 |
| openstack | Launchpad bug 1750785 in Ironic "Detached VIF may be restored during node cleaning" [High,In progress] - Assigned to Hironori Shiina (shiina-hironori) | 16:53 |
| patchbot | patch 546584 - ironic - Refresh port before storing VIF ID for cleaning | 16:53 |
| dtantsur | what do you think about the work around? | 16:53 |
| rloo | dtantsur: two patches? | 16:53 |
| dtantsur | rloo: now, I'm just linking both the bug and the patch | 16:54 |
| rloo | dtantsur: oh, one is a bug | 16:54 |
| rloo | dtantsur: let me look... | 16:54 |
| *** khyr0n has joined #openstack-ironic | 16:56 | |
| *** khyr0n has quit IRC | 17:07 | |
| NobodyCam | Good Morning Ironic'ers | 17:08 |
| rloo | dtantsur: what about reverting the vif detach w/o locking thing? | 17:10 |
| dtantsur | rloo: I'm kind of leaning towards it as well.. but I'm not sure what we'll regress on /cc TheJulia | 17:11 |
| rloo | dtantsur: what are the ramifications if the tenant_vif_port_id is accidentally restored? | 17:11 |
| *** athomas has quit IRC | 17:11 | |
| dtantsur | rloo: it will mean restoring the VIF if I get it right | 17:11 |
| rloo | dtantsur: right, which means... ?? we cannot attach another vif? | 17:12 |
| dtantsur | rloo: yep, I guess this port is "occupied" | 17:12 |
| rloo | dtantsur: that would be bad then. since eg, this is in cleaning, the node is getting ready to be used again. by another tenant or whatever. | 17:13 |
| dtantsur | correct | 17:13 |
| rloo | dtantsur: i feel like we should revert | 17:14 |
| dtantsur | yeah, it may be the correct thing to do | 17:15 |
| dtantsur | rloo: wanna propose the revert? | 17:16 |
| rloo | dtantsur: the only other thing we can do is to change that patch to eg add some 'locking' when modifying a port but i'm not sure it is worth it and/or if we'll get it right... | 17:16 |
| rloo | dtantsur: no. i don't want to be blamed. | 17:16 |
| *** openstackgerrit has joined #openstack-ironic | 17:16 | |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Register traits on nodes in devstack https://review.openstack.org/545370 | 17:16 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Validate instance_info.traits against node traits https://review.openstack.org/543461 | 17:16 |
| peitolm | is there a common reason why cleaning would fail on a node with '[FAILED] Failed to start Switch Root.' | 17:16 |
| * dtantsur looks at THAT patch | 17:16 | |
| dtantsur | peitolm: not that I'm aware of. is it a coreos image? | 17:17 |
| peitolm | for cleaning? | 17:18 |
| TheJulia | rloo: dtantsur I'm good with reverting the locking change, but that reintroduces the detachment race with Nova which may still prevent reuse. We're just pushing the issue around, and I think the right thing is just rip the vif out upon teardown | 17:20 |
| * TheJulia returns to trying to get intertubes working so all the lolcats can be streamed to the house | 17:20 | |
| dtantsur | TheJulia: I'd do both: revert and rip the VIF | 17:20 |
| dtantsur | lolcats! | 17:20 |
| dtantsur | okay, I'll propose the revert, as it seems quite urgent | 17:20 |
| rloo | thx dtantsur, TheJulia! | 17:20 |
| dtantsur | then we can rip the vif on teardown, quickly merge it and let people complain | 17:21 |
| peitolm | dtantsur: Fedora 26 (cloud edition) | 17:21 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Revert "Don't try to lock for vif detach" https://review.openstack.org/546705 | 17:22 |
| TheJulia | dtantsur: thanks <3 | 17:22 |
| dtantsur | TheJulia, rloo ^^^ | 17:22 |
| dtantsur | hmm, what should I do with the release note? | 17:22 |
| dtantsur | the change was released, so I guess I need to keep the old one and add a new about the revert, wdyt? | 17:22 |
| peitolm | "Failed to switch root: Specified switch root path '/sysroot' does not seem to be an OS tree. os-release file is missing." | 17:22 |
| dtantsur | peitolm: so, Fedora Atomic? are you using it as a whole-disk image? I think it won't work as a partition image. | 17:23 |
| rloo | dtantsur: i think we can edit release notes in stable/queens. if we backport this, won't it delete the release note? | 17:23 |
| dtantsur | rloo: it will | 17:24 |
| dtantsur | ideally, we should remove the release note on master, but keep it (and add a new one) on stable | 17:24 |
| peitolm | dtantsur: how would i check, i'd expect it as a whole-disk image | 17:24 |
| dtantsur | peitolm: it depends on how you added it to glance (skipping kernel/ramdisk properties) | 17:24 |
| peitolm | hmm | 17:25 |
| rloo | dtantsur: oh. if we want to do that, i think we can remove the release note from the cherry-picked patch, so that it isn't deleted. and maybe in the same cherry-picked patch, add a new release note? or edit the existing one? | 17:25 |
| peitolm | can i get that info from glance, or do i need to try and find it in shell history? | 17:25 |
| *** e0ne has quit IRC | 17:25 | |
| dtantsur | rloo: yep, about to do it | 17:25 |
| peitolm | hmm, vm vs baremetal | 17:30 |
| peitolm | I think i have a whole disk image configured | 17:31 |
| *** marios has quit IRC | 17:32 | |
| *** baha has quit IRC | 17:32 | |
| *** etingof has quit IRC | 17:33 | |
| jroll | rloo | dtantsur: no. i don't want to be blamed. <- if anyone ever blames someone personally for any code change, I will... dunno. but I'll be mad. that shouldn't be a thing. | 17:33 |
| dtantsur | ++ | 17:33 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Revert "Don't try to lock for vif detach" https://review.openstack.org/546705 | 17:36 |
| dtantsur | TheJulia, rloo ^^ sorry, this should be correct now | 17:36 |
| peitolm | looks like a bug with fedora26 | 17:38 |
| rloo | jroll: heh. i think i wanted to get agreement from the folks that originally wanted that patch. I don't like undoing things that others put in, w/o discussing with them. | 17:38 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Stop running check-releasenotes in `tox -e releasenotes` https://review.openstack.org/546709 | 17:43 |
| dtantsur | just got a WUT moment with ^^^ | 17:43 |
| *** openstackgerrit has quit IRC | 17:48 | |
| *** milan_ has quit IRC | 17:51 | |
| *** mgoddard_ has quit IRC | 17:53 | |
| *** baha has joined #openstack-ironic | 17:57 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 17:57 | |
| *** Nasir has joined #openstack-ironic | 17:57 | |
| peitolm | dtantsur: switching the deploy images to coreOS seems to have worked, will have more of a play later | 17:58 |
| *** derekh has quit IRC | 18:00 | |
| *** AlexeyAbashkin has quit IRC | 18:01 | |
| Nasir | Hi, I'm confused about the openstack instructions on setting/configuring the ironic serial console using socat... I'm on Pike, and using Ironic with Nova, trying to configure the serial console that can be accessible through horizon | 18:02 |
| Nasir | Following these instructions, https://docs.openstack.org/ironic/pike/admin/console.html | 18:02 |
| Nasir | I get this error msg , 2018-02-21 17:59:38.346 26329 ERROR ironic.conductor.base_manager [req-799eac22-d8c4-41b2-9886-c75113fec9ac - - - - -] Failed to register hardware types. Could not find the following items in the 'ironic.hardware.interfaces.console' entrypoint: ipmitool_socat.: DriverNotFoundInEntrypoint: Could not find the following items in | 18:02 |
| Nasir | the 'ironic.hardware.interfaces.console' entrypoint: ipmitool_socat. | 18:02 |
| Nasir | when i set the "enabled_console_interfaces = ipmitool_socat,no-console" in ironic.conf, enabled_hardware_types = ipmi, enabled_drivers = pxe_ipmitool | 18:03 |
| Nasir | Should the "enabled_console_interface' be "pxe_ipmitool_socat, or ipmitool_socat ? Thanks for the help | 18:04 |
| dtantsur | Nasir: hi, you're confusing enabled_drivers and enabled_console_interfaces | 18:06 |
| dtantsur | these are for different types of drivers | 18:06 |
| dtantsur | the guide you're looking at refers to adding pxe_ipmitool_socat and/or agent_ipmitool_socat to enabled_drivers without touching enabled_hardware_types or enabled_console_interfaces | 18:06 |
| Nasir | dtantsur: What is the correct procedure then ... I want to be able to see the baremetal console from horizon, when a boot an ironic instance from Nova | 18:12 |
| *** crushil has quit IRC | 18:13 | |
| dtantsur | Nasir: add one or both of these drivers to enabled_drivers and use them for your nodes | 18:13 |
| *** crushil has joined #openstack-ironic | 18:13 | |
| dtantsur | Nasir: though the version from master will also work, and it may be much clearer: https://docs.openstack.org/ironic/latest/admin/console.html | 18:14 |
| *** openstackgerrit has joined #openstack-ironic | 18:14 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/queens: Revert "Don't try to lock for vif detach" https://review.openstack.org/546719 | 18:14 |
| dtantsur | TheJulia, rloo, cherry-pick, when/if the master version merges ^^^ | 18:14 |
| * dtantsur has to go now | 18:14 | |
| *** dtantsur is now known as dtantsur|afk | 18:15 | |
| rloo | dtantsur: thx. and ciao for now :) | 18:15 |
| dtantsur|afk | o/ | 18:15 |
| *** hjensas has joined #openstack-ironic | 18:15 | |
| Nasir | dtantsur: so my enabled_drivers is set to "pxe_ipmitool" right now... Should it now be enabled_drivers=pxe_ipmitool,pxe_ipmitool_socat ? | 18:15 |
| dtantsur|afk | Nasir: that's one of the options, yes | 18:15 |
| * dtantsur|afk goes now | 18:15 | |
| rloo | Nasir: you'll also need to update your node.driver to be 'pxe_ipmitool_socat' | 18:16 |
| Nasir | dtantsur: and i should leave enabled_console_interfaces commented out , the way it is by default in the conf file ? | 18:16 |
| Nasir | dtantsur|afk: and i should leave enabled_console_interfaces commented out , the way it is by default in the conf file ? | 18:17 |
| rloo | Nasir: it doesn't matter, you can leave the enabled*interfaces alone, set or not set. you're not using them. | 18:17 |
| *** tesseract has quit IRC | 18:25 | |
| *** dsneddon is now known as dsneddon_pto | 18:25 | |
| *** mjura has quit IRC | 18:28 | |
| *** r-daneel_ has joined #openstack-ironic | 18:31 | |
| *** r-daneel has quit IRC | 18:32 | |
| *** r-daneel_ is now known as r-daneel | 18:32 | |
| *** rpioso|afk is now known as rpioso | 18:34 | |
| rpioso | Good morning | 18:34 |
| *** edleafe- has joined #openstack-ironic | 18:34 | |
| *** Nasir has quit IRC | 18:45 | |
| jroll | TheJulia: have you thought about what to discuss with nova in the hour we have? I was chatting with jaypipes and he recommended bringing up the disk partitioning thing | 18:54 |
| jroll | I was also thinking we should look at the big picture of the interactions with our virt driver, and figure out how we really want to do this | 18:55 |
| *** e0ne has joined #openstack-ironic | 18:59 | |
| *** edleafe| has joined #openstack-ironic | 18:59 | |
| *** akhilaki has joined #openstack-ironic | 19:00 | |
| rloo | jroll, TheJulia: good question. i am guessing that we ought to discuss/agree on ironic API version negotiation. (see L247 of https://etherpad.openstack.org/p/IronicWhiteBoard) | 19:00 |
| *** e0ne has quit IRC | 19:00 | |
| jroll | ++ | 19:01 |
| rloo | jroll: and hopefully the traits/deploy steps work will help and we won't have to discuss with them, how the user can send RAID config info to ironic, when booting a node :) | 19:02 |
| *** edleafe- has quit IRC | 19:02 | |
| rloo | jroll: i mean, at least not discuss with them at THIS ptg :D | 19:02 |
| jroll | rloo: I think we'll still need it as an option - and if both ironic and nova can agree to the same schema at the same time, we can make good progress this cycle | 19:02 |
| jroll | without having to re-do things in ironic when it turns out nova doesn't like it, or vice versa | 19:03 |
| rloo | jroll: are we ready to discuss that with them next week? | 19:03 |
| rloo | jroll: we only have 60 min of their precious time! | 19:03 |
| jroll | rloo: I hope so, I have a proposal up, we'll see how the ironic discussion goes I guess | 19:04 |
| * jroll thinks we aren't decisive enough, hopes we can improve on that | 19:04 | |
| rloo | jroll: oh, i can decide. decide not to do anything. wheeee... | 19:04 |
| jroll | :/ | 19:04 |
| * rloo thinks we don't decide sometimes, because the (background) info needed for making decisions, hasn't been presented | 19:05 | |
| * rloo speaks for herself only | 19:05 | |
| jroll | I'm not saying all indecisiveness is bad or unwarranted, | 19:06 |
| rloo | jroll: yeah, i know what you mean. but also, even after deciding, things lag so long, that it is like we never discussed it. so we do it again. | 19:06 |
| jroll | maybe just that we're too afraid to experiment? or do anything without 100% of the details being clear? dunno | 19:06 |
| * jroll gets excited to retro | 19:06 | |
| jroll | yes | 19:06 |
| jroll | completely agree | 19:06 |
| rloo | jroll: take a look at the graphical console stuff, i've provided links to the discussions i remember: https://etherpad.openstack.org/p/ironic-rocky-ptg-graphical-console | 19:07 |
| jroll | yep | 19:07 |
| rloo | jroll: i guess we incrementally have gotten farther with it, but ... | 19:07 |
| jroll | I think for a while we would begin discussing things too early | 19:07 |
| jroll | or in other words, discuss things we knew we didn't have time for, for a long time | 19:08 |
| * jroll realizes we probably have some of those topics again | 19:08 | |
| rloo | jroll: yup. but i thought people wanted to do those discussions early. to socialize the idea. or whatever. | 19:09 |
| jroll | rloo: yeah, can't blame them | 19:09 |
| rloo | jroll: anyway. i was wondering which specs/things people should be reading/getting up to speed on, before PTG. | 19:09 |
| rloo | TheJulia: ^^ hopefully there are links to things on the etherpads... | 19:09 |
| jroll | rloo: yeah, I need to look as well, I haven't yet | 19:10 |
| *** khyr0n has joined #openstack-ironic | 19:12 | |
| *** Goneri has quit IRC | 19:20 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 19:21 | |
| *** AlexeyAbashkin has quit IRC | 19:25 | |
| *** akhilaki has quit IRC | 19:33 | |
| *** akhilaki has joined #openstack-ironic | 19:34 | |
| rloo | TheJulia: if you have a minute to revert this: https://review.openstack.org/#/c/546705/ | 19:37 |
| patchbot | patch 546705 - ironic - Revert "Don't try to lock for vif detach" | 19:37 |
| *** e0ne has joined #openstack-ironic | 19:51 | |
| *** fragatina has quit IRC | 19:53 | |
| *** akhilaki has quit IRC | 19:56 | |
| *** peitolm has quit IRC | 20:09 | |
| *** Goneri has joined #openstack-ironic | 20:22 | |
| *** crushil has quit IRC | 20:23 | |
| *** crushil has joined #openstack-ironic | 20:24 | |
| *** crushil has quit IRC | 20:25 | |
| *** crushil has joined #openstack-ironic | 20:26 | |
| *** edleafe| has quit IRC | 20:26 | |
| *** edleafe- has joined #openstack-ironic | 20:28 | |
| *** edleafe- has quit IRC | 20:31 | |
| *** devananda has joined #openstack-ironic | 20:32 | |
| *** crushil has quit IRC | 20:45 | |
| *** akhilaki has joined #openstack-ironic | 20:50 | |
| *** priteau has quit IRC | 20:52 | |
| *** dprince has quit IRC | 21:02 | |
| *** openstackgerrit has quit IRC | 21:03 | |
| *** dprince has joined #openstack-ironic | 21:04 | |
| *** crushil has joined #openstack-ironic | 21:09 | |
| *** marst has joined #openstack-ironic | 21:12 | |
| *** dprince has quit IRC | 21:13 | |
| *** openstackgerrit has joined #openstack-ironic | 21:21 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Imported Translations from Zanata https://review.openstack.org/545661 | 21:21 |
| *** khyr0n has quit IRC | 21:24 | |
| *** Nasir has joined #openstack-ironic | 21:25 | |
| *** olivierbourdon38 has quit IRC | 21:26 | |
| *** Nasir has quit IRC | 21:28 | |
| *** e0ne has quit IRC | 21:46 | |
| TheJulia | \o/ I have internets again | 21:49 |
| TheJulia | rloo: revert the revert? | 21:49 |
| rloo | TheJulia: heh, that's the revert :) | 21:49 |
| * TheJulia is all confused now | 21:50 | |
| TheJulia | But, I have working intertubes full of the lolcats | 21:50 |
| rloo | TheJulia: ok, i just saw my original request. I mean, approve the revert of the original patch :) | 21:50 |
| *** shoaibwr has joined #openstack-ironic | 21:50 | |
| TheJulia | vdrok approved it, checking if it has been cherrypicked | 21:50 |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/queens: Revert "Don't try to lock for vif detach" https://review.openstack.org/546719 | 21:51 |
| rloo | TheJulia: oh, yeah, just noticed that he approved it. | 21:51 |
| rloo | TheJulia: dmitry cherry-picked it already. sec. | 21:51 |
| TheJulia | oh, hmm | 21:51 |
| rloo | TheJulia: https://review.openstack.org/#/c/546719/ | 21:51 |
| patchbot | patch 546719 - ironic (stable/queens) - Revert "Don't try to lock for vif detach" | 21:51 |
| TheJulia | yup, same one, it just updated it | 21:51 |
| rloo | TheJulia: oh, cool. nice. smart. as it should do! :) | 21:52 |
| *** jtomasek has quit IRC | 21:52 | |
| rloo | TheJulia: oh. except that it updated so we need to wait for zuul. heh. | 21:52 |
| *** fragatina has joined #openstack-ironic | 21:54 | |
| *** shoaibwr has quit IRC | 21:57 | |
| *** baha has quit IRC | 21:59 | |
| *** dmellado has quit IRC | 22:00 | |
| * TheJulia retrieves beer | 22:01 | |
| TheJulia | rloo: eh, if the tests pass it can merge | 22:02 |
| rloo | TheJulia: yup. they (the stable/queens patch) already passed once; the update is causing the tests to run again :) | 22:03 |
| rloo | TheJulia: did you see jroll's question about nova-ironici ptg meeting, and what we are going to discuss there? ^^ (I suspect you have) | 22:03 |
| TheJulia | I have, it was already on my mind | 22:03 |
| jroll | :) | 22:04 |
| jroll | TheJulia: fwiw, I'm tracking topics in the nova etherpad, in an ironic section | 22:04 |
| TheJulia | jroll: thanks | 22:04 |
| *** crushil has quit IRC | 22:05 | |
| * TheJulia ponders going out into the garage and beginning the bed assembly of doom | 22:05 | |
| *** marst has left #openstack-ironic | 22:06 | |
| rloo | TheJulia: would you like this backported? https://review.openstack.org/#/c/544750/ :) | 22:07 |
| patchbot | patch 544750 - ironic - Disable .pyc files for grenade multinode | 22:07 |
| rloo | jroll: thx! | 22:07 |
| TheJulia | rloo: it won't make much of a difference, it only helps on moderately more stable builds where no binary objects have been touched | 22:07 |
| TheJulia | err | 22:07 |
| TheJulia | yeah, that | 22:08 |
| rloo | TheJulia: so it won't help the multinode grenade job in queens? aren't those stable builds? | 22:08 |
| TheJulia | tl;dr there is no fix short of client version negotiation handling in our nova virt library and upgrading nova, or totally retooling grenade | 22:08 |
| rloo | TheJulia: oh yeah. but in the meantime... ? | 22:09 |
| TheJulia | no fix, maybe we can negotiate a minimal backportable nova virt driver change to provide sanity again while we're at the ptg, but I suspect based on prior discussions there is little desire to do that from the nova team | 22:10 |
| TheJulia | s/do/permit/ | 22:10 |
| jroll | TheJulia: I mean... the nova negotiation thing was just a different hack around it. it isn't a permanent fix | 22:10 |
| jroll | what happens when the nova upgrade makes the previous ironic start segfaulting for the same reason? :) | 22:11 |
| TheJulia | well, not really, the guideline is only a mixed stack, it doesn't say that we have to run an old version of nova, we've only boxed ourselves into that artificially | 22:11 |
| jroll | sure | 22:11 |
| TheJulia | jroll: ironic gets upgraded and is running with current libraries and thus shared object files under the hood | 22:11 |
| jroll | if the tag is the goal, there's lots of simpler ways to test that | 22:12 |
| TheJulia | The tag is the goal, except there has also been resistance to retooling our own job because we could still operate with a mixed stack if we leverage a pin but fire up a newer api | 22:12 |
| jroll | we went with what we have today because it's extensive testing around a realistic-ish upgrade procedure, with a clear upgrade procedure documented for ops | 22:13 |
| TheJulia | Slight warning: I've had one beer, i'm exhausted from running errands as $new_homeower today... so my filters are gone | 22:13 |
| * jroll doesn't give a damn about the tag, tbh, I want a good upgrade experience for ops | 22:14 | |
| jroll | heh, feel free to hold off on this conversation until later / next week :) | 22:14 |
| TheJulia | Agreed, but we chose to leverage grenade for that, and grenade simply does not support what we _need_ | 22:14 |
| TheJulia | So grenade shoe-horns us into an unrealistic scenario | 22:15 |
| TheJulia | at least binary dependency wise | 22:15 |
| jroll | just like the rest of our CI, yes | 22:15 |
| TheJulia | in which case, the operator is going to be broken hard regardless and will have to upgrade | 22:15 |
| TheJulia | yeah | 22:15 |
| TheJulia | the other conundrum is the tag requirements state a full stack, so we can't go minimalist | 22:16 |
| jroll | openstack does a ton of work to provide co-installability, it's silly and I hate it | 22:16 |
| TheJulia | except across versions | 22:16 |
| jroll | well, there's never been a problem with mixed versions before | 22:17 |
| TheJulia | There has to be a better way that makes everyone happy | 22:17 |
| *** akhilaki has quit IRC | 22:17 | |
| TheJulia | I kind of disagree, but only because this is third cycle that upon the end of cycle grenade has been the bane of our existence | 22:18 |
| jroll | idk if this fell through the wrong cracks or what, but I'm surprised that more people aren't caring about this problem | 22:18 |
| jroll | I mean, rolling upgrade testing is a pain yes | 22:18 |
| jroll | but have dependencies been an issue with that job in the past? I don't recall that | 22:18 |
| TheJulia | we did have a very similar unexplained issue that would have just gone away if we upgraded nova last cycle | 22:19 |
| TheJulia | I don't remember what it was at this point | 22:19 |
| TheJulia | but it is how we got into the discussion of being able to support version negotiation in the client | 22:19 |
| TheJulia | in terms of handling more than just a single static version pin | 22:20 |
| jroll | gotcha | 22:20 |
| jroll | don't get me wrong, I think that's a useful thing | 22:20 |
| TheJulia | if we could just upgrade nova, the world would be a million times happier | 22:20 |
| jroll | I also think a relatively strict upgrade procedure for real operators is a useful thing | 22:20 |
| jroll | s/upgrade nova/run in virtualenvs/ | 22:20 |
| jroll | nothing else would need to change :) | 22:20 |
| TheJulia | yeah, which I happened to find the flag for that, it is broken trying to install nova | 22:20 |
| jroll | heh. | 22:21 |
| TheJulia | at least, nova + grenade | 22:21 |
| * TheJulia stares at the fireplace with the awesome housewarming gift that BadCub and NobodyCam gifted to Summer and myself... and sips beer | 22:24 | |
| NobodyCam | :) mmmmm beer | 22:24 |
| BadCub | ❤️❤️ | 22:24 |
| jroll | nice | 22:24 |
| jroll | you should go enjoy that :) | 22:24 |
| rloo | jroll, TheJulia: it would be good to discuss rolling upgrades at the PTG with infra folks. i know there is that ffu? upgrade session on Mon or Tues but they don't discuss infra-related, or they haven't in the past. | 22:29 |
| rloo | I think TheJulia is off starting today? so don't want to bug her until next week :) | 22:29 |
| jroll | rloo: that's about skipping versions during upgrades, completely unrelated to CI or rolling upgrades | 22:29 |
| rloo | jroll: right, but they were talking baby steps. oh yeah, they were cold upgrades. you're right. | 22:30 |
| rloo | it is all a blur... | 22:30 |
| persia | My feeling is that the FFU crew is close to consensus on how to do things, and that it might almost be time to take that as instruction for what to do to infra to imitate their plans. | 22:30 |
| rloo | persia: that would be good! guess we'll find out next week... | 22:31 |
| NobodyCam | hey hey persia long time no see. hope all is well and good with you. | 22:31 |
| persia | Yep. | 22:31 |
| *** rcernin has joined #openstack-ironic | 22:32 | |
| persia | NobodyCam: Your hopes are satisfied. Would mine be as well? | 22:32 |
| NobodyCam | They would! | 22:32 |
| NobodyCam | :) | 22:32 |
| persia | Excellent :) | 22:32 |
| *** r-daneel has quit IRC | 22:40 | |
| rloo | vdrok, jlvillal: if either of you are around and have time to review, looks like folks would like this fixed/backported: https://review.openstack.org/#/c/545830/ | 22:53 |
| patchbot | patch 545830 - ironic - Support nested objects and object lists in as_dict | 22:53 |
| openstackgerrit | Merged openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/545543 | 23:08 |
| *** shekar has quit IRC | 23:08 | |
| openstackgerrit | Merged openstack/python-ironicclient master: Updated from global requirements https://review.openstack.org/545561 | 23:08 |
| *** shekar has joined #openstack-ironic | 23:08 | |
| openstackgerrit | Merged openstack/ironic master: Updated from global requirements https://review.openstack.org/545542 | 23:14 |
| *** Goneri has quit IRC | 23:29 | |
| *** hshiina has joined #openstack-ironic | 23:41 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!