*** k_mouza has joined #openstack-ironic | 00:57 | |
*** k_mouza has quit IRC | 01:05 | |
*** root has joined #openstack-ironic | 01:15 | |
*** k_mouza has joined #openstack-ironic | 01:15 | |
*** root is now known as Guest21137 | 01:15 | |
*** Guest21137 is now known as jerrywang1 | 01:15 | |
*** k_mouza has quit IRC | 01:19 | |
*** k_mouza has joined #openstack-ironic | 01:56 | |
*** k_mouza has quit IRC | 02:00 | |
*** rcernin has joined #openstack-ironic | 02:27 | |
*** jdandrea has quit IRC | 02:40 | |
*** jerrywang1 has quit IRC | 03:10 | |
*** root has joined #openstack-ironic | 03:15 | |
*** root is now known as jerrywang1 | 03:16 | |
*** rcernin has quit IRC | 03:31 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Fix ipxe interface to perform ipxe boot without ipxe_enabled enabled https://review.opendev.org/697634 | 03:50 |
---|---|---|
*** mkrai has joined #openstack-ironic | 03:55 | |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Use correct command for Python virtualenv https://review.opendev.org/704854 | 04:57 |
*** yedongcan has joined #openstack-ironic | 05:23 | |
*** yedongcan has quit IRC | 05:40 | |
*** yedongcan has joined #openstack-ironic | 05:41 | |
*** ricolin has quit IRC | 06:24 | |
*** HagunKim has quit IRC | 06:32 | |
*** Lucas_Gray has joined #openstack-ironic | 06:44 | |
*** kaifeng has joined #openstack-ironic | 07:04 | |
*** dtantsur|afk is now known as dtantsur | 07:20 | |
dtantsur | morning ironic | 07:20 |
*** kaifeng has quit IRC | 07:33 | |
*** kaifeng has joined #openstack-ironic | 07:35 | |
kaifeng | morning dtantsur | 07:36 |
*** hwoarang has quit IRC | 07:36 | |
*** hwoarang has joined #openstack-ironic | 07:37 | |
*** pcaruana has joined #openstack-ironic | 07:43 | |
*** iurygregory has joined #openstack-ironic | 07:51 | |
iurygregory | good morning Ironic o/ | 07:51 |
kaifeng | hi iurygregory o/ | 07:54 |
*** yedongcan has quit IRC | 07:57 | |
*** Lucas_Gray has quit IRC | 07:58 | |
*** yedongcan has joined #openstack-ironic | 07:59 | |
iurygregory | kaifeng, o/ | 07:59 |
*** Lucas_Gray has joined #openstack-ironic | 08:03 | |
arne_wiebalck | Good morning, ironic! | 08:05 |
*** rpittau|afk is now known as rpittau | 08:05 | |
rpittau | good morning ironic! o/ | 08:05 |
kaifeng | hi arne_wiebalck, rpittau o/ | 08:10 |
rpittau | hey kaifeng :) | 08:10 |
arne_wiebalck | gm kaifeng o/ | 08:10 |
*** tesseract has joined #openstack-ironic | 08:14 | |
kaifeng | is there some wrong with ironic ci? I am seeing unit test failing after rebasing which is ok before. | 08:15 |
rpittau | kaifeng: which tests ? or do you have a log? | 08:16 |
kaifeng | this patch: https://review.opendev.org/#/c/697634/ | 08:16 |
patchbot | patch 697634 - ironic - Fix ipxe interface to perform ipxe boot without ip... - 5 patch sets | 08:16 |
rpittau | could it be a change in a package? I see lower-constraints is passing | 08:19 |
kaifeng | maybe, the failed tests seemed unrelated with the change, while the change is rebased on the latest master | 08:21 |
*** yedongcan has quit IRC | 08:21 | |
rpittau | yeah, I don't see any patch that could have caused that | 08:22 |
*** yedongcan has joined #openstack-ironic | 08:23 | |
*** khansa has joined #openstack-ironic | 08:27 | |
kaifeng | jsonpatch is suspecious | 08:30 |
rpittau | yeah, I'm testing with a precedent version | 08:36 |
rpittau | I saw it was updated recently in upper-constraints | 08:36 |
*** sziviani has quit IRC | 08:37 | |
kaifeng | i have to reestablish a whole working environment at home :( | 08:38 |
*** sziviani has joined #openstack-ironic | 08:38 | |
rpittau | kaifeng: I confirm the suspect, downgrading ONLY jsonpatch makes everything OK | 08:38 |
kaifeng | rpittau: thanks, maybe we need to cap upper version | 08:41 |
rpittau | kaifeng: probably, need to understand what happened, 1.25 is latest and it was released 5 days ago, 1.24 is 7 months old :/ | 08:42 |
*** sziviani has quit IRC | 08:42 | |
kaifeng | if it is a breaking change, it would not make sense to bump in the global requirement i think. | 08:43 |
*** sziviani has joined #openstack-ironic | 08:45 | |
*** ociuhandu has joined #openstack-ironic | 08:46 | |
iurygregory | morning rpittau arne_wiebalck o/ | 08:47 |
rpittau | kaifeng: orrr maybe now it's working as intended https://github.com/stefankoegl/python-json-patch/blob/master/jsonpatch.py#L77 | 08:47 |
rpittau | hey iurygregory :) | 08:47 |
arne_wiebalck | hey iurygregory o/ | 08:47 |
iurygregory | we can fix in our side or should we talk with requirements? | 08:48 |
rpittau | ok I see what changed | 08:50 |
rpittau | let me verify that | 08:50 |
rpittau | we might need to fix on our side | 08:50 |
*** ociuhandu has quit IRC | 08:50 | |
rpittau | this is the breaking change, which seems to actually make sense https://github.com/stefankoegl/python-json-patch/commit/b3726f3a8bdcdf0f0841e078228014de8477b0ec | 08:51 |
*** dougsz has joined #openstack-ironic | 08:53 | |
rpittau | we'll have to adjust the tests and probably raise the jsonpatch version | 08:53 |
*** lucasagomes has joined #openstack-ironic | 08:55 | |
rpittau | my laptop needs coffee | 08:57 |
*** ociuhandu has joined #openstack-ironic | 09:02 | |
kaifeng | for a list the change makes sense, it use another jsonpointer stuff so i didn't follow though | 09:03 |
*** yolanda has joined #openstack-ironic | 09:03 | |
rpittau | kaifeng: the point is that now instead of raising a python error, it's intercepted by JsonPatchConflict | 09:04 |
kaifeng | it sounds we capture that then everything will be ok? | 09:05 |
*** ociuhandu has quit IRC | 09:07 | |
*** k_mouza has joined #openstack-ironic | 09:15 | |
*** dougsz has quit IRC | 09:18 | |
*** dougsz has joined #openstack-ironic | 09:21 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Fix jsonpatch related tests https://review.opendev.org/705395 | 09:22 |
rpittau | ^ this should help, we might need to raise jsonpatch version in lowerconstraints though | 09:23 |
*** Lucas_Gray has quit IRC | 09:23 | |
kaifeng | rpittau: ack, thanks! bump lower version to 1.25 seems aggressive, maybe we can avoid that by adjusting our tests. | 09:28 |
rpittau | kaifeng: I agree, although it doesn't change a lot between 1.24 and 1.25, except for the breaking thing :D | 09:29 |
rpittau | let's see if that fixes the upper-contraints test and then we can think about adjusting the tests | 09:30 |
kaifeng | hmm, the lowerest version is 1.16, far from 1.24, and it's a valid change won't be reverted so we can't simply exclude | 09:33 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Fix jsonpatch related tests https://review.opendev.org/705395 | 09:34 |
*** derekh has joined #openstack-ironic | 09:39 | |
*** Lucas_Gray has joined #openstack-ironic | 09:39 | |
*** hjensas has joined #openstack-ironic | 09:47 | |
*** yedongcan has quit IRC | 09:51 | |
*** alexmcleod has joined #openstack-ironic | 09:52 | |
*** yedongcan has joined #openstack-ironic | 09:52 | |
*** lucasagomes_ has joined #openstack-ironic | 09:55 | |
*** lucasagomes has quit IRC | 09:55 | |
*** lucasagomes_ is now known as lucasagomes | 09:56 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Workaround for Python 2 installation https://review.opendev.org/705060 | 10:07 |
*** Lucas_Gray has quit IRC | 10:09 | |
iurygregory | rpittau, it seems that the "errors" from the ovs-vsctl are normal .-. | 10:11 |
iurygregory | according to neutron ppl | 10:11 |
rpittau | iurygregory: well, they're not normal, the operations need to be executed in a different way to avoid the error anyway | 10:15 |
rpittau | or betterm to avoit to generate the error and as a consequence fail the script | 10:16 |
rpittau | so what they suggest, executing the commands in one line instead of sequence of commands, will prevent the failure of the script, not the failure of the command | 10:18 |
rpittau | as I said before, we need to modify the code in setup_vxlan_network | 10:18 |
*** ociuhandu has joined #openstack-ironic | 10:18 | |
rpittau | https://opendev.org/openstack/ironic/src/branch/master/devstack/lib/ironic#L2892 | 10:18 |
*** dougsz has quit IRC | 10:19 | |
openstackgerrit | Merged openstack/bifrost master: Drop python 2.7 support and testing https://review.opendev.org/695800 | 10:20 |
* iurygregory asked on neutron channel XD | 10:20 | |
rpittau | iurygregory: check what slaweq said | 10:20 |
*** yedongcan has quit IRC | 10:21 | |
*** ociuhandu has quit IRC | 10:23 | |
*** dougsz has joined #openstack-ironic | 10:27 | |
*** lucasagomes has quit IRC | 10:27 | |
*** khansa has quit IRC | 10:27 | |
*** khansa has joined #openstack-ironic | 10:28 | |
*** lucasagomes has joined #openstack-ironic | 10:40 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: WIP - Native zuulv3 grenade multinode multitenant https://review.opendev.org/705030 | 10:43 |
* iurygregory prays for the CI jobs | 10:46 | |
*** ociuhandu has joined #openstack-ironic | 10:47 | |
*** Lucas_Gray has joined #openstack-ironic | 10:59 | |
*** Lucas_Gray has quit IRC | 11:09 | |
*** Lucas_Gray has joined #openstack-ironic | 11:10 | |
*** tkajinam has joined #openstack-ironic | 11:28 | |
*** Lucas_Gray has quit IRC | 11:31 | |
*** Lucas_Gray has joined #openstack-ironic | 11:41 | |
*** rpittau is now known as rpittau|bbl | 11:42 | |
*** ociuhandu has quit IRC | 11:44 | |
*** ociuhandu has joined #openstack-ironic | 11:45 | |
*** ociuhandu has quit IRC | 11:48 | |
*** ociuhandu has joined #openstack-ironic | 11:48 | |
*** ociuhandu has quit IRC | 11:51 | |
*** Lucas_Gray has quit IRC | 12:05 | |
*** ociuhandu has joined #openstack-ironic | 12:09 | |
*** Lucas_Gray has joined #openstack-ironic | 12:11 | |
*** ociuhandu has quit IRC | 12:15 | |
*** ociuhandu has joined #openstack-ironic | 12:16 | |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Run only one ovs-vsctl in setup_vxlan_network https://review.opendev.org/705430 | 12:23 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic stable/train: Run only one ovs-vsctl in setup_vxlan_network https://review.opendev.org/705431 | 12:24 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: WIP - Native zuulv3 grenade multinode multitenant https://review.opendev.org/705030 | 12:29 |
*** ociuhandu has quit IRC | 12:54 | |
*** ociuhandu has joined #openstack-ironic | 12:54 | |
*** bfournie has quit IRC | 12:57 | |
*** rh-jelabarre has joined #openstack-ironic | 12:58 | |
*** ociuhandu has quit IRC | 13:00 | |
*** hjensas has quit IRC | 13:16 | |
*** derekh has quit IRC | 13:24 | |
*** hjensas has joined #openstack-ironic | 13:27 | |
*** rpittau|bbl is now known as rpittau | 13:34 | |
*** rloo has joined #openstack-ironic | 13:38 | |
*** Goneri has joined #openstack-ironic | 13:40 | |
* iurygregory is sad, move the function to one command didn't help =( | 13:49 | |
*** bfournie has joined #openstack-ironic | 13:53 | |
*** jdandrea has joined #openstack-ironic | 13:54 | |
iurygregory | rpittau, I was looking at zuul and your change https://review.opendev.org/#/c/705395/ it's not on the check Queue | 13:56 |
patchbot | patch 705395 - ironic - Fix jsonpatch related tests - 2 patch sets | 13:56 |
rpittau | iurygregory: thanks, I have to make one adjustment anyway | 13:57 |
iurygregory | zuul is watching you | 13:57 |
iurygregory | that's why it didn't put on the queue XD | 13:57 |
rpittau | zuul knows | 13:58 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Workaround for Python virtualenv https://review.opendev.org/705060 | 14:01 |
TheJulia | good morning | 14:04 |
*** tzumainn has joined #openstack-ironic | 14:04 | |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Workaround for Python virtualenv https://review.opendev.org/705060 | 14:04 |
rpittau | good morning TheJulia :) | 14:04 |
*** Lucas_Gray has quit IRC | 14:04 | |
*** Lucas_Gray has joined #openstack-ironic | 14:05 | |
* TheJulia sips a delicious soy latte and looks at all the emails | 14:06 | |
openstackgerrit | Jerry Wang proposed openstack/ironic-inspector master: Update ironic inspector document for dropping py2.x https://review.opendev.org/705448 | 14:08 |
iurygregory | morning TheJulia | 14:10 |
iurygregory | rpittau, it seems that run 1 one command didn't help to avoid the error =( | 14:10 |
openstackgerrit | Julia Kreger proposed openstack/bifrost master: Sync pip_install files https://review.opendev.org/703502 | 14:10 |
iurygregory | https://zuul.opendev.org/t/openstack/build/f390e0a60bf1400aa3e8883f41b0f6f1/log/job-output.txt#93847 | 14:11 |
TheJulia | Just a rebase to master branch ^^^ | 14:11 |
* iurygregory is thinking in give more time to find the 14 nodes on the multinode job .-. | 14:11 | |
TheJulia | that is a trap | 14:14 |
TheJulia | you should need no more than ~4 minutes | 14:14 |
TheJulia | really 3 | 14:14 |
* rpittau wonders if we actually create the bridge at some point | 14:15 | |
TheJulia | That | 14:15 |
TheJulia | We had to add code to build the bridge on our own because infra's bridge standup was no longer processing at some point for the nodes | 14:15 |
rpittau | because the problem are not the interfaces anymore, but the bridge | 14:15 |
TheJulia | in the template.... | 14:15 |
TheJulia | There is an explicit role zuul has for doing that, fwiw | 14:15 |
TheJulia | I'm not sure what I ended up having to do the last time the bridge failed to startup... | 14:16 |
TheJulia | I guess the thing to look for is in the subnode logs, does anythign actually network boot? | 14:16 |
*** tkajinam has quit IRC | 14:16 | |
*** derekh has joined #openstack-ironic | 14:17 | |
TheJulia | rajinir: out of curiosity, have you looked at dell's 3rd party CI recently. About a half dozen changes I've looked at over the past week seem to have completely failed :( | 14:20 |
TheJulia | iurygregory: oh yes, your bridge is failing to stand-up | 14:20 |
iurygregory | TheJulia, yeah .-. I was trying with neutron people, they said we can use one command to do everything but didn't work | 14:30 |
TheJulia | yeah, the base bridge just isen't there... | 14:30 |
*** Lucas_Gray has quit IRC | 14:31 | |
*** Lucas_Gray has joined #openstack-ironic | 14:32 | |
*** dougsz has quit IRC | 14:33 | |
TheJulia | iurygregory: Last time I saw something like this, I found https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/multi-node-bridge/tasks/peer.yaml was not being invoked. in other the multi-node-bridge role. I had to add it explicitly as part of our job template een though I think the opendev infra role states it explicitly as well. | 14:34 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Fix jsonpatch related tests https://review.opendev.org/705395 | 14:36 |
iurygregory | TheJulia, ack I'll try to look at it | 14:36 |
TheJulia | rpittau: iurygregory: by chance have either of you had a chance to look back at https://review.opendev.org/#/c/703652 ? | 14:38 |
patchbot | patch 703652 - sushy - Fix 'None' field value processing - 2 patch sets | 14:38 |
iurygregory | it seems devstack-gate would take care of this, but in zuulv3 we are not using it | 14:38 |
TheJulia | iurygregory: yup | 14:38 |
TheJulia | devstack-gate also had code for it | 14:39 |
dtantsur | morning TheJulia | 14:39 |
* dtantsur gets back to a loooong presentation | 14:39 | |
TheJulia | dtantsur: ugh | 14:39 |
openstackgerrit | Merged openstack/bifrost stable/train: Fix variable checks in install-deps https://review.opendev.org/704956 | 14:40 |
rajinir | TheJulia: Good morning. dell CI is back and passing now. Last week we had several days of failures because of https://review.opendev.org/#/c/703735/ which is now patched | 14:41 |
patchbot | patch 703735 - devstack - Fix pip uncap fallout for nova and barbican (MERGED) - 6 patch sets | 14:41 |
*** cdearborn has joined #openstack-ironic | 14:42 | |
rpittau | TheJulia: I saw the discussion in the patch, I have a +2 there since a while, just waiting for the discussion to unfold | 14:43 |
TheJulia | I sense Dmitry has been heads down working on a presentation. I'm a bit worried about null and handling because for a while ansible was not converting inputs properly from yaml and humans would also not supply the right value, which ended in us needing kind of a similar bit of code to handle input variation that was not exactly valid but could happen. | 14:45 |
openstackgerrit | Jerry Wang proposed openstack/ironic-inspector master: Update ironic inspector document for dropping py2.x https://review.opendev.org/705448 | 14:45 |
rpittau | I'm sorry, I actually forgot to remove the +2 based on that discussion, I have the same concern, let me reply to that and adjust | 14:46 |
* dtantsur is working on slide #34 and envisions at least as many more to come | 14:47 | |
rpittau | oO | 14:47 |
TheJulia | rpittau: thanks! | 14:48 |
TheJulia | rpittau: I guess, I'm kind of also lacking a solid understanding of the actual problem. I'm sure I could look... just have been pulled in a few too many directions | 14:48 |
* dtantsur will probably take a short walk before continuing (thus missing the meeting) | 14:48 | |
openstackgerrit | Rajini Karthik proposed openstack/sushy master: Testing thirdparty sushy CI https://review.opendev.org/705289 | 14:48 |
* iurygregory hopes dtantsur wont have 100 slides and only 25 minutes | 14:48 | |
TheJulia | dtantsur: Okay :) | 14:49 |
rpittau | that would be an interesting presentation :) | 14:49 |
dtantsur | iurygregory: it's a full ironic workshop downstream, I'll have 3-4 hours, but that involves people doing stuff on their machines, soooo.. | 14:49 |
iurygregory | dtantsur, sounds cool *-* | 14:49 |
TheJulia | dtantsur: ugh, how long until you have to have that done? | 14:49 |
dtantsur | TheJulia: it's tomorrow afternoon.. | 14:49 |
TheJulia | ugh | 14:50 |
iurygregory | rpittau, my friend did this at fosdem and he also had a demo to fit in 25min XD | 14:50 |
dtantsur | on Sunday I finally got the ansible playbook to set up an environment working \o/ | 14:50 |
* TheJulia slides a elixer of productivity across to dtantsur | 14:50 | |
rpittau | iurygregory: was your friend Flash ? | 14:50 |
dtantsur | ++ | 14:50 |
* dtantsur actually goes for a walk | 14:50 | |
*** dtantsur is now known as dtantsur|brb | 14:50 | |
iurygregory | rpittau, he is a bit crazy I can say | 14:50 |
TheJulia | iurygregory: I have a few friends that did pre-timed slides completely and had to work out exact wording and body language because they wouldn't have a clicker or any ability to go back. It is a brutal presentation style :\ | 14:52 |
*** dougsz has joined #openstack-ironic | 14:52 | |
iurygregory | TheJulia, ouch =X | 14:52 |
iurygregory | brutal++ | 14:52 |
rpittau | "brutal presentation style" sounds like something to add to a resume | 14:54 |
jroll | morning ironic | 14:54 |
rpittau | hey jroll :) | 14:54 |
jroll | :) | 14:55 |
TheJulia | I think they told me it was a slide very 20 seconds... | 14:55 |
*** stendulker has joined #openstack-ironic | 14:57 | |
iurygregory | morning jroll | 14:57 |
jroll | \o | 14:57 |
rajinir | o/ | 15:00 |
TheJulia | #startmeeting ironic | 15:01 |
openstack | Meeting started Mon Feb 3 15:01:08 2020 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
TheJulia | o/ | 15:01 |
stendulker | o/ | 15:01 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
rpioso | o/ | 15:01 |
*** openstack changes topic to " (Meeting topic: ironic)" | 15:01 | |
iurygregory | o/ | 15:01 |
openstack | The meeting name has been set to 'ironic' | 15:01 |
TheJulia | Good morning everyone! | 15:01 |
rajinir | o/ | 15:01 |
cdearborn | \o | 15:01 |
bdodd | o/ | 15:01 |
kaifeng | o/ | 15:01 |
jerrywang1 | o/ | 15:01 |
arne_wiebalck | o/ | 15:02 |
TheJulia | Our agenda can be found on the wiki. Hopefully we should get through it fairly quickly. | 15:02 |
jroll | \o | 15:02 |
rloo | o/ | 15:02 |
TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:02 |
rpittau | o/ | 15:02 |
TheJulia | #topic Announcements/Reminders | 15:02 |
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)" | 15:02 | |
TheJulia | #info Ironic Mid-cycle February 25-26 @ CERN | 15:03 |
TheJulia | #link https://indico.cern.ch/event/863986/ | 15:03 |
TheJulia | #link https://etherpad.openstack.org/p/ironic-ussuri-midcycle | 15:03 |
arne_wiebalck | I'll try and send out a mail to the registered participants some time this week. | 15:03 |
TheJulia | arne_wiebalck: awesome | 15:03 |
arne_wiebalck | With some logistics details. | 15:03 |
* iurygregory probably won't go to the mid cycle =( | 15:03 | |
TheJulia | If anyone is looking at tickets from the states, it seems like flying in to CDG and taking a train is semi-reasonable. | 15:04 |
arne_wiebalck | iurygregory: :( | 15:04 |
TheJulia | #info Opendev + PTG Vancouver - June 8th through 11th | 15:04 |
rajinir | #info TheJulia: Announcing Dell 3rd Party Ironic CI now functional for Openstack/sushy library. Please take a look at https://review.opendev.org/#/c/705289/ | 15:05 |
patchbot | patch 705289 - sushy - Testing thirdparty sushy CI - 3 patch sets | 15:05 |
rpittau | rajinir: can you please a DNM to the commit message title? So it's clear we shouldn't merge that :) | 15:06 |
TheJulia | Additional info for the Opendev + PTG event | 15:06 |
rajinir | rpittau: sure | 15:06 |
rpittau | thanks! | 15:06 |
*** stendulker_ has joined #openstack-ironic | 15:06 | |
*** stendulker has quit IRC | 15:06 | |
openstackgerrit | Rajini Karthik proposed openstack/sushy master: Testing thirdparty sushy CI ( DON"T MERGE) https://review.opendev.org/705289 | 15:07 |
TheJulia | #info The foundation is looking for people interested in being part of the programming committee. If your interested in helping out, there is a form linked from the email Ashlee Ferguson sent out. | 15:07 |
TheJulia | #link http://lists.openstack.org/pipermail/openstack-discuss/2020-January/012239.html | 15:07 |
TheJulia | Does anyone else have anything to announce or remind us of? | 15:07 |
*** ociuhandu has joined #openstack-ironic | 15:07 | |
rpioso | TheJulia: Please see rajinir's message above. | 15:08 |
TheJulia | #info Announcing Dell 3rd Party Ironic CI now functional for Openstack/sushy library. Please take a look at https://review.opendev.org/#/c/705289/ | 15:08 |
patchbot | patch 705289 - sushy - Testing thirdparty sushy CI ( DON"T MERGE) - 4 patch sets | 15:08 |
TheJulia | Hopefully that will show up in the summary correctly :) | 15:08 |
rpioso | :) | 15:08 |
rajinir | rpioso, thank you | 15:09 |
TheJulia | It looks like we had no action items last week. | 15:09 |
rpioso | rajinir: yw | 15:09 |
TheJulia | I think everyone has been fairly heads-down so I'm wondering if it even makes sense to visit the subteam status reports? | 15:09 |
rloo | thank you rajinir! that is great that we have more testing :) | 15:09 |
rajinir | rloo: thank you | 15:10 |
TheJulia | I propose we jump to priorities for the week since it appears to not have made much progress the last week. | 15:11 |
*** khansa has quit IRC | 15:11 | |
rpittau | yep | 15:11 |
TheJulia | #topic Deciding on priorities for the coming week | 15:11 |
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:12 | |
TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:12 |
TheJulia | Line 169 | 15:12 |
TheJulia | It would be awesome if a couple people could review Ilya's L3 deployment spec revision today... It is actually also open on my other screen at the moment so I think he would appreciate feedback and maybe even an approved spec when he gets back from FOSDEM | 15:13 |
*** Lucas_Gray has quit IRC | 15:14 | |
TheJulia | Does anyone have anything to add to this list? | 15:14 |
rpittau | TheJulia: yes | 15:14 |
TheJulia | I guess one question is a stable/rocky fix. I'm wondering if we just abandon it and move on? | 15:14 |
rpittau | can we add this? https://review.opendev.org/705395 | 15:14 |
patchbot | patch 705395 - ironic - Fix jsonpatch related tests - 3 patch sets | 15:14 |
TheJulia | rpittau: Sure, please add! | 15:15 |
TheJulia | Anyone else? | 15:15 |
rpittau | done, FYI CI is broken without that :P | 15:15 |
TheJulia | \o/ | 15:16 |
TheJulia | Okay | 15:16 |
TheJulia | I guess we're good then? | 15:16 |
rloo | rocky is going into extended maintenance soon? | 15:16 |
TheJulia | rloo: yes, like a week or two I think | 15:17 |
rpittau | rloo: yes, the 24th | 15:17 |
rloo | don't we need rocky CI working at least before then? | 15:17 |
rloo | or do we decide now that we aren't going to 'support' rocky? | 15:17 |
rloo | or whatever the term is... | 15:17 |
TheJulia | It is not completely broken, it is a single job that breaks... If memory serves for completely unrelated reasons | 15:17 |
rloo | i just did a recheck on a patch for rocky and i was wondering about the state of rocky CI: https://review.opendev.org/#/c/692516/ | 15:18 |
patchbot | patch 692516 - ironic (stable/rocky) - Kill misbehaving `ipmitool` process - 1 patch set | 15:18 |
TheJulia | In the extended maintenance path, we have the option of killing jobs that are too much overhead to maintain. We could do that and merge the fix. | 15:18 |
*** Lucas_Gray has joined #openstack-ironic | 15:20 | |
rloo | wondering if we want to table that question until after Feb 24. For now, the situation is that we can't merge anything cuz CI rocky is broken. So unless someone fixes the CI, we're in the current state we're in, right? | 15:20 |
TheJulia | after the 24th, I'd just remove the job anyway | 15:20 |
*** mbeierl has joined #openstack-ironic | 15:20 | |
rloo | ok, we're good then. we'll wait til after 24th, the job will be removed, then we can merge. yes? | 15:21 |
TheJulia | yes, we're stuck unless we fix the issue which I believe is elsewhere, get that merged in the next two weeks, and then.. yeah | 15:21 |
TheJulia | Yes, that is entirely reasonable | 15:21 |
rloo | umm. when do we do a (last?) release of rocky? | 15:21 |
rpittau | whoops rechecked wrong patch :/ | 15:21 |
TheJulia | rloo: the release team does a final tag I believe around the 24th | 15:22 |
rloo | (ie, do we want these changes in the last official release of rocky) | 15:22 |
TheJulia | Nobody is screaming for it, so I suspect it is fine if it is something people can pickup later if needed | 15:22 |
rloo | TheJulia: ok, so I think that's really the only diff. Changes to Rocky avail in the last tag, or not. | 15:23 |
TheJulia | yup | 15:23 |
rloo | And yeah, if no one is 'asking', then let's go with our plan, unless someone wants to fix the CI for Rocky. | 15:23 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: WIP - Native zuulv3 grenade multinode multitenant https://review.opendev.org/705030 | 15:23 |
TheJulia | sounds good | 15:24 |
TheJulia | Moving on! | 15:24 |
TheJulia | #topic Discussion | 15:24 |
*** openstack changes topic to "Discussion (Meeting topic: ironic)" | 15:24 | |
TheJulia | #info stevebaker has proposed copying a subset of WSME type validation code into ironic. Steve's timezone really prohibits him attending this meeting as he should be quite asleep at the moment. | 15:25 |
*** mbeierl1 has joined #openstack-ironic | 15:25 | |
TheJulia | #link https://review.opendev.org/704486 | 15:25 |
patchbot | patch 704486 - ironic - WIP do all serialization in the expose decorator - 1 patch set | 15:25 |
*** mbeierl has quit IRC | 15:25 | |
*** mbeierl1 is now known as mbeierl | 15:25 | |
TheJulia | WSME is MIT licensed, and he is proposing moving about a thousand lines into ironic. I'm not really objecting to it as long as it is properly attributed. My only concern is really maintenance of the code, but copying the validations would at least put them into an active code base, but honestly I'd also rather see a more widely used piece of code out there for type validations | 15:26 |
rloo | I don't know enough about it, but it seems odd that we'd need to move (copy/paste) that code to ironic. is ironic the only project that needs that code? | 15:27 |
rpittau | in general is a good idea and sure helps the transition to Flask, as that's the final goal | 15:27 |
jerrywang1 | WSME was not only for ironic/pecan | 15:28 |
jerrywang1 | It supports other web frameworks also | 15:28 |
TheJulia | but I think we're one of the last projects using wsme? | 15:28 |
rloo | rpittau: so you're saying this wsme code is only temporary for ironic? | 15:28 |
rpittau | rloo: exactly | 15:28 |
rloo | rpittau: how 'temporary' are we thinking? | 15:28 |
kaifeng | did keystone use wsme before migration? | 15:28 |
TheJulia | I didn't percieve it as temporary | 15:29 |
TheJulia | kaifeng: I don't think so | 15:29 |
rpittau | rloo: hard for me to answer that :) | 15:29 |
rpittau | I think keystone was using wsme for some time.... | 15:30 |
rloo | i guess i'd like to know if any other openstack project is doing what ironic is doing, and whether this wsme copy is the way they've done it, and whether IF we copy, it should be in a library. | 15:30 |
kaifeng | but they moved to flask, wondering how they do validation now? | 15:30 |
rloo | I'm speaking/thinking from ignorance though | 15:30 |
jroll | "temporary" nearly always means forever, fwiw | 15:30 |
rloo | jroll: that's why i asked :) | 15:31 |
jroll | in my opinion, of course :) | 15:31 |
jroll | keystone uses jsonschema for validation now, afaik | 15:31 |
jroll | note that is strictly for validation, not for serialization to/from json/python objects | 15:32 |
jerrywang1 | validation was only one reason for WSME using at ironic, actually ironic used WSME's types and base classes also | 15:33 |
jroll | right, serialization is the other thing we used it for | 15:33 |
jerrywang1 | And expose function also | 15:33 |
TheJulia | kaifeng: I believe they wrote their own validation code... | 15:34 |
jerrywang1 | Did we would use flask in future? | 15:34 |
jroll | it feels like we could/should rip out the intermediate objects (not sure if now or later) | 15:34 |
jerrywang1 | If yes, current wsme code would change also | 15:34 |
jerrywang1 | for differnt web frame work WSME code was diffent | 15:34 |
TheJulia | wrote their own originally | 15:34 |
rpioso | TheJulia: How will driver vendor passthru APIs be converted? | 15:36 |
TheJulia | jroll: I feel like that might be reasonable, but I am struggling to visualize what that would look like | 15:36 |
jroll | TheJulia: I was just skimming code and wondering to myself if we could just use our existing o.vo objects or something. or just dicts in the api layer. | 15:37 |
TheJulia | rpioso: they would just be serialized into json and passed through, I believe that is what happens presently | 15:37 |
rpioso | TheJulia: Thank you! | 15:37 |
TheJulia | dict seem like the would be the most native and easiest to work with, but I suspect that also brings risk. Anyway, Steve has been focused on this, I suspect this is all feedback for him on thoughts and maybe he has ideas. There is also a thread on the mailing list that may be worth looking at/replying to | 15:38 |
TheJulia | I guess we should move on | 15:40 |
TheJulia | Any objections? | 15:40 |
TheJulia | Moving on! | 15:41 |
TheJulia | #topic Baremetal SIG | 15:41 |
*** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)" | 15:41 | |
TheJulia | Regarding the whitepaper! | 15:41 |
* arne_wiebalck has to get going on this one | 15:42 | |
TheJulia | I chatted with the foundation staff about getting the baremetal logo program submissions. The tl;dr is that they can't share them because of privacy issues and advised us that we should use content that was already published and can be attributed to that publication or that is directly submitted to the whitepaper. | 15:42 |
TheJulia | The tl;dr of that is that if there is content to be submitted, please do so soon. I'm working on getting Red Hat's submission released so we can put it in. I think that will help bolster the whitepaper, tbh. | 15:43 |
*** ociuhandu has quit IRC | 15:44 | |
arne_wiebalck | I won't be able to work much on this before the mid-cycle I'm afraid. | 15:44 |
TheJulia | Nor will I sadly, but I should at least have publication data. I also have an interesting reference that might work, but I need a few days to hunt through NSF proposals.. that is if I can find the public versions. | 15:45 |
*** ociuhandu has joined #openstack-ironic | 15:45 | |
TheJulia | Anyway, I guess we can proceed to Open Discussion if there is nothing else | 15:46 |
TheJulia | ? | 15:46 |
TheJulia | #topic Open Discussion with ?coffee? | 15:47 |
*** openstack changes topic to "Open Discussion with ?coffee? (Meeting topic: ironic)" | 15:47 | |
* TheJulia suspects coffee is required again | 15:48 | |
* rpittau sipping chocolate and pear tea | 15:48 | |
* arne_wiebalck needs a coffee as well | 15:48 | |
TheJulia | Well, I hope everyone has a wonderful week | 15:50 |
rpittau | thanks! | 15:51 |
rloo | crickets | 15:51 |
arne_wiebalck | thanks, TheJulia | 15:51 |
kaifeng | thx! | 15:51 |
TheJulia | Indeed! | 15:51 |
TheJulia | I'll end the meeting early if everyone promises to go review the L3 Deployments spec revision :)\ | 15:52 |
*** ociuhandu has quit IRC | 15:52 | |
rpittau | ok, promise :D | 15:52 |
TheJulia | https://review.opendev.org/#/c/672780/ | 15:52 |
patchbot | patch 672780 - ironic-specs - Update L3 based deployment spec - 31 patch sets | 15:52 |
TheJulia | :) | 15:52 |
TheJulia | Thanks everyone | 15:52 |
TheJulia | \o | 15:52 |
kaifeng | have to take a look tomorrow :) | 15:54 |
TheJulia | Thanks, I am just hoping we finally reach some level of consensus and are able to move forward since it is possibly the most impactful feature we could deliver this cycle. | 15:55 |
TheJulia | And with that, Thanks! | 15:55 |
TheJulia | #endmeeting | 15:55 |
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list." | 15:55 | |
openstack | Meeting ended Mon Feb 3 15:55:09 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:55 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-02-03-15.01.html | 15:55 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-02-03-15.01.txt | 15:55 |
openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-02-03-15.01.log.html | 15:55 |
kaifeng | but it seems no further discussion/update after my last check | 15:55 |
*** stendulker_ has quit IRC | 15:56 | |
TheJulia | It looks like Ilya responded two days later | 15:56 |
TheJulia | I've not had a chance to get that far in reading the update since I'm looking at the content, not the back and forth | 15:57 |
kaifeng | yeah, totally, back and forth is the outcome of no consensus | 15:58 |
TheJulia | :( | 15:58 |
TheJulia | we need to find a happy path that is viable and reasonable | 15:59 |
TheJulia | Anyway, I'll need to read about a month of that back and forth once I review the content | 15:59 |
TheJulia | well, finish reviewing | 15:59 |
kaifeng | it seems glean is desired choice, never heard of the glean tool before though, as long as it works I am ok, but I would avoid tying api input to the specific format | 16:02 |
TheJulia | Glean is a tool we've used with bifrost in the past and with-in openstack infra. | 16:03 |
kaifeng | yeah, this is only thing i know from the irc discussion before. | 16:04 |
TheJulia | Yeah, I'd rather avoid trying to create a bunch of validation code and make things hyper-specific | 16:04 |
TheJulia | but there is huge dislike of data pass-throughs | 16:04 |
*** gyee has joined #openstack-ironic | 16:04 | |
* TheJulia sighs | 16:05 | |
*** dtantsur|brb is now known as dtantsur | 16:05 | |
* TheJulia goes to next meeting | 16:06 | |
kaifeng | my general thinking is something like a network_info field for each port, filled with ip/mask, let conductor convert it to some backend. | 16:07 |
kaifeng | this is not pass through, we can validate input, we can assure compatible network data output. | 16:08 |
dtantsur | TheJulia, rloo, jroll, if the concern is wsme bits maintains, then I'm delighted to let you know that wsme itself is not maintained either :D | 16:08 |
dtantsur | * s/maintains/maintenance/ | 16:08 |
TheJulia | dtantsur: indeed :( | 16:08 |
jroll | heh | 16:09 |
rloo | (remind me why we're using wsme then) | 16:09 |
dtantsur | we barely made the CI pass again recently | 16:09 |
dtantsur | rloo: cargo-culted from nova? dunno, it was before me | 16:09 |
dtantsur | you're the last person here who has a chance to remember ;) | 16:09 |
rloo | never paid attention to that stuff, but that is a valid assumption. | 16:10 |
rloo | so the plan is to continue using (unsupported) wsme, even after we go from pecan->flask? | 16:10 |
rpittau | nope | 16:10 |
TheJulia | kaifeng: I fear the "what" becomes the problem. I thought we were making consensus on the single configuration drive data path, but... gah | 16:11 |
rloo | rpittau: is it documented somewhere, the plan? might be easier than for me to ask questions that someone already wrote up | 16:11 |
TheJulia | "make a configuration drive, use it" | 16:11 |
kaifeng | TheJulia: config-2 with cloud-init is not desired i remember. | 16:13 |
rpittau | rloo: you can check this story https://storyboard.openstack.org/#!/story/1651346, plus these patches https://review.opendev.org/#/q/topic:story/1651346+(status:open+OR+status:merged), and there is a discussion Steve started in the ML | 16:16 |
rloo | rpittau: i looked in our etherpad, it is titled 'Replacing WSME'. I then looked at the story ... ah, you beat me | 16:16 |
*** mkrai has quit IRC | 16:17 | |
rloo | rpittau: to be honest, that story itself, does not describe what the plan is. i don't think we can assume that people should be trying to follow the proposed PRs etc. I see PRs from Jerry, but nothing from Steve. So I am confused. | 16:17 |
rpittau | rloo: we probably need to expand the tasks there, and the description, although if you check the link to the patches I wrote before you'll see more patches from Steve | 16:19 |
rloo | rpittau: is the plan to use jsonschema, as described in one of the comments? I see mention of a spec, but the spec wasn't approved, and I see that it was rfe-approved. | 16:19 |
rloo | rpittau: yes, please describe. Hard for me to help make a decision about that wsme code, when I have no idea what is going on. Even if I knew what is going on, I might not be able to help but... :) | 16:20 |
*** ociuhandu has joined #openstack-ironic | 16:23 | |
*** tzumainn has quit IRC | 16:24 | |
*** Lucas_Gray has quit IRC | 16:26 | |
rpittau | rloo: I see a comment from kaifeng dated 2019-06-18 stating "We are moving to Flask, maybe we can mark rfe-approved since it's one of our tasks in Train priorities.". I don't remember when that was discussed or decided exactly, maybe in Denver? I was not there unfortunately, but I remember it was stated multiple times during meetings, as apparently the consensus was met. We definitely need to expand the story | 16:26 |
rpittau | and define it better, we can even bring it to the mid-cycle meetup for another review. | 16:26 |
kaifeng | wow, me neither | 16:29 |
rpittau | :) | 16:30 |
TheJulia | yeah, consensus was just get off pecan/wsme | 16:30 |
kaifeng | i remember the consensus at that time is to check how keystone do it and do the same change. copying part of wsme code seems a deviation. | 16:31 |
kaifeng | since we don't anticipate multiple solutions, so this is not strictly prohibited i guess. | 16:31 |
TheJulia | I think we're in a situation where the ends were wanted, and the intermediate was unknown. Ultimately it comes down to does the end justify the means of how we get there | 16:32 |
* rpittau needs to leave :/ | 16:33 | |
TheJulia | goodnight | 16:33 |
rpittau | goodnight! o/ | 16:33 |
*** rpittau is now known as rpittau|afk | 16:33 | |
*** ociuhandu has quit IRC | 16:34 | |
*** ociuhandu has joined #openstack-ironic | 16:55 | |
*** TxGirlGeek has joined #openstack-ironic | 16:59 | |
*** lucasagomes has quit IRC | 17:07 | |
*** ociuhandu has quit IRC | 17:09 | |
*** ociuhandu has joined #openstack-ironic | 17:10 | |
*** ociuhandu has quit IRC | 17:15 | |
*** ociuhandu has joined #openstack-ironic | 17:17 | |
*** kaifeng has quit IRC | 17:18 | |
*** ociuhandu has quit IRC | 17:20 | |
*** ociuhandu has joined #openstack-ironic | 17:21 | |
openstackgerrit | Zygimantas Matonis proposed openstack/ironic-inspector master: Added scope to introspection rules. https://review.opendev.org/702215 | 17:33 |
*** k_mouza has quit IRC | 17:39 | |
*** ociuhandu_ has joined #openstack-ironic | 17:54 | |
*** ociuhandu has quit IRC | 17:57 | |
*** ociuhandu_ has quit IRC | 17:59 | |
*** derekh has quit IRC | 18:00 | |
*** dougsz has quit IRC | 18:03 | |
*** k_mouza has joined #openstack-ironic | 18:11 | |
*** k_mouza has quit IRC | 18:15 | |
*** k_mouza has joined #openstack-ironic | 18:15 | |
rpittau|afk | Please stop recheck in ironic CI, at the moment is broken because of new jsonpatch version in upper-constraints, please review the fix here https://review.opendev.org/705395 | 18:32 |
patchbot | patch 705395 - ironic - Fix jsonpatch related tests - 3 patch sets | 18:32 |
TheJulia | rpittau|afk: will we need to backport that? | 18:36 |
TheJulia | nope | 18:38 |
rpittau|afk | TheJulia: if the version of jsonpatch has been updated for other branches other than master, yes | 18:38 |
rpittau|afk | I can't check at the moment :/ | 18:38 |
TheJulia | I just checked upper constraints | 18:39 |
TheJulia | jsonpatch 1.25 is current master only of requirements | 18:39 |
TheJulia | I approved the fix as a single core review | 18:39 |
rpittau|afk | Thanks TheJulia | 18:40 |
*** tzumainn has joined #openstack-ironic | 18:44 | |
*** stevebaker has joined #openstack-ironic | 19:15 | |
stevebaker | morning | 19:23 |
donnyd | I am hoping to get some clarification on how I can use the rule function on auto discovery | 19:49 |
*** efried has joined #openstack-ironic | 19:49 | |
donnyd | Lets say I have 100 nodes of varying size, and I want to apply resource classes to them on auto discovery based on say RAM or CPUs | 19:50 |
donnyd | Is there a way for me to craft a rule that says "if you have 2GB of ram, you are CUSTOM_BAREMETAL_SMALL , if you have more than 16G of RAM you are CUSTOM_BAREMETAL_MEDIUM" | 19:51 |
donnyd | so on and so forth | 19:52 |
donnyd | TheJulia: I know you were saying I needed to ask someone else about the discovery rules thing. who is the right person to ask that question to?> | 19:53 |
*** dtantsur is now known as dtantsur|afk | 19:53 | |
dtantsur|afk | donnyd: it's usually me, but I'm in Central European t ime | 19:53 |
dtantsur|afk | and busy as hell tomorrow, but will happily talk to you on Wednesday if nobody beats me to it | 19:54 |
*** tesseract has quit IRC | 20:01 | |
*** andy_ has quit IRC | 20:02 | |
*** irclogbot_0 has quit IRC | 20:05 | |
*** irclogbot_0 has joined #openstack-ironic | 20:06 | |
*** alexmcleod has quit IRC | 20:08 | |
donnyd | thank dtantsur|afk | 20:19 |
donnyd | If I don't get it solved by then I will be around and ask again | 20:19 |
TheJulia | donnyd: okay, sounds like you found the right person :) | 20:22 |
*** andy_ has joined #openstack-ironic | 20:23 | |
*** k_mouza has quit IRC | 20:34 | |
openstackgerrit | Merged openstack/ironic master: Fix jsonpatch related tests https://review.opendev.org/705395 | 20:43 |
*** TxGirlGeek has quit IRC | 20:51 | |
*** tzumainn has quit IRC | 20:53 | |
*** TxGirlGeek has joined #openstack-ironic | 21:00 | |
*** andy_ has quit IRC | 21:03 | |
*** TxGirlGeek has quit IRC | 21:07 | |
*** TxGirlGeek has joined #openstack-ironic | 21:07 | |
*** TxGirlGeek has quit IRC | 21:08 | |
*** Goneri has quit IRC | 21:16 | |
*** andy_ has joined #openstack-ironic | 21:21 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost master: Guard the status code checks https://review.opendev.org/705561 | 21:38 |
*** tzumainn has joined #openstack-ironic | 21:54 | |
*** bfournie has quit IRC | 22:33 | |
donnyd | dtantsur|afk: I figured it out with fancy rules - ironic is so awesome | 23:14 |
*** jdandrea has quit IRC | 23:41 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!