Monday, 2020-02-03

*** k_mouza has joined #openstack-ironic00:57
*** k_mouza has quit IRC01:05
*** root has joined #openstack-ironic01:15
*** k_mouza has joined #openstack-ironic01:15
*** root is now known as Guest2113701:15
*** Guest21137 is now known as jerrywang101:15
*** k_mouza has quit IRC01:19
*** k_mouza has joined #openstack-ironic01:56
*** k_mouza has quit IRC02:00
*** rcernin has joined #openstack-ironic02:27
*** jdandrea has quit IRC02:40
*** jerrywang1 has quit IRC03:10
*** root has joined #openstack-ironic03:15
*** root is now known as jerrywang103:16
*** rcernin has quit IRC03:31
openstackgerritKaifeng Wang proposed openstack/ironic master: Fix ipxe interface to perform ipxe boot without ipxe_enabled enabled  https://review.opendev.org/69763403:50
*** mkrai has joined #openstack-ironic03:55
openstackgerritMerged openstack/ironic-python-agent-builder master: Use correct command for Python virtualenv  https://review.opendev.org/70485404:57
*** yedongcan has joined #openstack-ironic05:23
*** yedongcan has quit IRC05:40
*** yedongcan has joined #openstack-ironic05:41
*** ricolin has quit IRC06:24
*** HagunKim has quit IRC06:32
*** Lucas_Gray has joined #openstack-ironic06:44
*** kaifeng has joined #openstack-ironic07:04
*** dtantsur|afk is now known as dtantsur07:20
dtantsurmorning ironic07:20
*** kaifeng has quit IRC07:33
*** kaifeng has joined #openstack-ironic07:35
kaifengmorning dtantsur07:36
*** hwoarang has quit IRC07:36
*** hwoarang has joined #openstack-ironic07:37
*** pcaruana has joined #openstack-ironic07:43
*** iurygregory has joined #openstack-ironic07:51
iurygregorygood morning Ironic o/07:51
kaifenghi iurygregory o/07:54
*** yedongcan has quit IRC07:57
*** Lucas_Gray has quit IRC07:58
*** yedongcan has joined #openstack-ironic07:59
iurygregorykaifeng, o/07:59
*** Lucas_Gray has joined #openstack-ironic08:03
arne_wiebalckGood morning, ironic!08:05
*** rpittau|afk is now known as rpittau08:05
rpittaugood morning ironic! o/08:05
kaifenghi arne_wiebalck, rpittau o/08:10
rpittauhey kaifeng :)08:10
arne_wiebalckgm kaifeng o/08:10
*** tesseract has joined #openstack-ironic08:14
kaifengis there some wrong with ironic ci? I am seeing unit test failing after rebasing which is ok before.08:15
rpittaukaifeng: which tests ? or do you have a log?08:16
kaifengthis patch: https://review.opendev.org/#/c/697634/08:16
patchbotpatch 697634 - ironic - Fix ipxe interface to perform ipxe boot without ip... - 5 patch sets08:16
rpittaucould it be a change in a package? I see lower-constraints is passing08:19
kaifengmaybe, the failed tests seemed unrelated with the change, while the change is rebased on the latest master08:21
*** yedongcan has quit IRC08:21
rpittauyeah, I don't see any patch that could have caused that08:22
*** yedongcan has joined #openstack-ironic08:23
*** khansa has joined #openstack-ironic08:27
kaifengjsonpatch is suspecious08:30
rpittauyeah, I'm testing with a precedent version08:36
rpittauI saw it was updated recently in upper-constraints08:36
*** sziviani has quit IRC08:37
kaifengi have to reestablish a whole working environment at home :(08:38
*** sziviani has joined #openstack-ironic08:38
rpittaukaifeng: I confirm the suspect, downgrading ONLY jsonpatch makes everything OK08:38
kaifengrpittau: thanks, maybe we need to cap upper version08:41
rpittaukaifeng: 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 IRC08:42
kaifengif it is a breaking change, it would not make sense to bump in the global requirement i think.08:43
*** sziviani has joined #openstack-ironic08:45
*** ociuhandu has joined #openstack-ironic08:46
iurygregorymorning rpittau arne_wiebalck o/08:47
rpittaukaifeng: orrr maybe now it's working as intended https://github.com/stefankoegl/python-json-patch/blob/master/jsonpatch.py#L7708:47
rpittauhey iurygregory :)08:47
arne_wiebalckhey iurygregory o/08:47
iurygregorywe can fix in our side or should we talk with requirements?08:48
rpittauok I see what changed08:50
rpittaulet me verify that08:50
rpittauwe might need to fix on our side08:50
*** ociuhandu has quit IRC08:50
rpittauthis is the breaking change, which seems to actually make sense https://github.com/stefankoegl/python-json-patch/commit/b3726f3a8bdcdf0f0841e078228014de8477b0ec08:51
*** dougsz has joined #openstack-ironic08:53
rpittauwe'll have to adjust the tests and probably raise the jsonpatch version08:53
*** lucasagomes has joined #openstack-ironic08:55
rpittaumy laptop needs coffee08:57
*** ociuhandu has joined #openstack-ironic09:02
kaifengfor a list the change makes sense, it use another jsonpointer stuff so i didn't follow though09:03
*** yolanda has joined #openstack-ironic09:03
rpittaukaifeng: the point is that now instead of raising a python error, it's intercepted by JsonPatchConflict09:04
kaifengit sounds we capture that then everything will be ok?09:05
*** ociuhandu has quit IRC09:07
*** k_mouza has joined #openstack-ironic09:15
*** dougsz has quit IRC09:18
*** dougsz has joined #openstack-ironic09:21
openstackgerritRiccardo Pittau proposed openstack/ironic master: Fix jsonpatch related tests  https://review.opendev.org/70539509:22
rpittau^ this should help, we might need to raise jsonpatch version in lowerconstraints though09:23
*** Lucas_Gray has quit IRC09:23
kaifengrpittau: ack, thanks! bump lower version to 1.25 seems aggressive,  maybe we can avoid that by adjusting our tests.09:28
rpittaukaifeng: I agree, although it doesn't change a lot between 1.24 and 1.25, except for the breaking thing :D09:29
rpittaulet's see if that fixes the upper-contraints test and then we can think about adjusting the tests09:30
kaifenghmm, 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 exclude09:33
openstackgerritRiccardo Pittau proposed openstack/ironic master: Fix jsonpatch related tests  https://review.opendev.org/70539509:34
*** derekh has joined #openstack-ironic09:39
*** Lucas_Gray has joined #openstack-ironic09:39
*** hjensas has joined #openstack-ironic09:47
*** yedongcan has quit IRC09:51
*** alexmcleod has joined #openstack-ironic09:52
*** yedongcan has joined #openstack-ironic09:52
*** lucasagomes_ has joined #openstack-ironic09:55
*** lucasagomes has quit IRC09:55
*** lucasagomes_ is now known as lucasagomes09:56
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Workaround for Python 2 installation  https://review.opendev.org/70506010:07
*** Lucas_Gray has quit IRC10:09
iurygregoryrpittau, it seems that the "errors" from the ovs-vsctl are normal .-.10:11
iurygregoryaccording to neutron ppl10:11
rpittauiurygregory: well, they're not normal, the operations need to be executed in a different way to avoid the error anyway10:15
rpittauor betterm to avoit to generate the error and as a consequence fail the script10:16
rpittauso 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 command10:18
rpittauas I said before, we need to modify the code in setup_vxlan_network10:18
*** ociuhandu has joined #openstack-ironic10:18
rpittauhttps://opendev.org/openstack/ironic/src/branch/master/devstack/lib/ironic#L289210:18
*** dougsz has quit IRC10:19
openstackgerritMerged openstack/bifrost master: Drop python 2.7 support and testing  https://review.opendev.org/69580010:20
* iurygregory asked on neutron channel XD10:20
rpittauiurygregory: check what slaweq said10:20
*** yedongcan has quit IRC10:21
*** ociuhandu has quit IRC10:23
*** dougsz has joined #openstack-ironic10:27
*** lucasagomes has quit IRC10:27
*** khansa has quit IRC10:27
*** khansa has joined #openstack-ironic10:28
*** lucasagomes has joined #openstack-ironic10:40
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: WIP - Native zuulv3 grenade multinode multitenant  https://review.opendev.org/70503010:43
* iurygregory prays for the CI jobs10:46
*** ociuhandu has joined #openstack-ironic10:47
*** Lucas_Gray has joined #openstack-ironic10:59
*** Lucas_Gray has quit IRC11:09
*** Lucas_Gray has joined #openstack-ironic11:10
*** tkajinam has joined #openstack-ironic11:28
*** Lucas_Gray has quit IRC11:31
*** Lucas_Gray has joined #openstack-ironic11:41
*** rpittau is now known as rpittau|bbl11:42
*** ociuhandu has quit IRC11:44
*** ociuhandu has joined #openstack-ironic11:45
*** ociuhandu has quit IRC11:48
*** ociuhandu has joined #openstack-ironic11:48
*** ociuhandu has quit IRC11:51
*** Lucas_Gray has quit IRC12:05
*** ociuhandu has joined #openstack-ironic12:09
*** Lucas_Gray has joined #openstack-ironic12:11
*** ociuhandu has quit IRC12:15
*** ociuhandu has joined #openstack-ironic12:16
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Run only one ovs-vsctl in setup_vxlan_network  https://review.opendev.org/70543012:23
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic stable/train: Run only one ovs-vsctl in setup_vxlan_network  https://review.opendev.org/70543112:24
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: WIP - Native zuulv3 grenade multinode multitenant  https://review.opendev.org/70503012:29
*** ociuhandu has quit IRC12:54
*** ociuhandu has joined #openstack-ironic12:54
*** bfournie has quit IRC12:57
*** rh-jelabarre has joined #openstack-ironic12:58
*** ociuhandu has quit IRC13:00
*** hjensas has quit IRC13:16
*** derekh has quit IRC13:24
*** hjensas has joined #openstack-ironic13:27
*** rpittau|bbl is now known as rpittau13:34
*** rloo has joined #openstack-ironic13:38
*** Goneri has joined #openstack-ironic13:40
* iurygregory is sad, move the function to one command didn't help =(13:49
*** bfournie has joined #openstack-ironic13:53
*** jdandrea has joined #openstack-ironic13:54
iurygregoryrpittau, I was looking at zuul and your change https://review.opendev.org/#/c/705395/ it's not on the check Queue13:56
patchbotpatch 705395 - ironic - Fix jsonpatch related tests - 2 patch sets13:56
rpittauiurygregory: thanks, I have to make one adjustment anyway13:57
iurygregoryzuul is watching you13:57
iurygregorythat's why it didn't put on the queue XD13:57
rpittauzuul knows13:58
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Workaround for Python virtualenv  https://review.opendev.org/70506014:01
TheJuliagood morning14:04
*** tzumainn has joined #openstack-ironic14:04
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Workaround for Python virtualenv  https://review.opendev.org/70506014:04
rpittaugood morning TheJulia :)14:04
*** Lucas_Gray has quit IRC14:04
*** Lucas_Gray has joined #openstack-ironic14:05
* TheJulia sips a delicious soy latte and looks at all the emails14:06
openstackgerritJerry Wang proposed openstack/ironic-inspector master: Update ironic inspector document for dropping py2.x  https://review.opendev.org/70544814:08
iurygregorymorning TheJulia14:10
iurygregoryrpittau, it seems that run 1 one command didn't help to avoid the error =(14:10
openstackgerritJulia Kreger proposed openstack/bifrost master: Sync pip_install files  https://review.opendev.org/70350214:10
iurygregoryhttps://zuul.opendev.org/t/openstack/build/f390e0a60bf1400aa3e8883f41b0f6f1/log/job-output.txt#9384714:11
TheJuliaJust 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
TheJuliathat is a trap14:14
TheJuliayou should need no more than ~4 minutes14:14
TheJuliareally 314:14
* rpittau wonders if we actually create the bridge at some point14:15
TheJuliaThat14:15
TheJuliaWe 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 nodes14:15
rpittaubecause the problem are not the interfaces anymore, but the bridge14:15
TheJuliain the template....14:15
TheJuliaThere is an explicit role zuul has for doing that, fwiw14:15
TheJuliaI'm not sure what I ended up having to do the last time the bridge failed to startup...14:16
TheJuliaI guess the thing to look for is in the subnode logs, does anythign actually network boot?14:16
*** tkajinam has quit IRC14:16
*** derekh has joined #openstack-ironic14:17
TheJuliarajinir: 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
TheJuliaiurygregory: oh yes, your bridge is failing to stand-up14:20
iurygregoryTheJulia, yeah .-. I was trying with neutron people, they said we can use one command to do everything but didn't work14:30
TheJuliayeah, the base bridge just isen't there...14:30
*** Lucas_Gray has quit IRC14:31
*** Lucas_Gray has joined #openstack-ironic14:32
*** dougsz has quit IRC14:33
TheJuliaiurygregory: 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
openstackgerritRiccardo Pittau proposed openstack/ironic master: Fix jsonpatch related tests  https://review.opendev.org/70539514:36
iurygregoryTheJulia, ack I'll try to look at it14:36
TheJuliarpittau: iurygregory: by chance have either of you had a chance to look back at https://review.opendev.org/#/c/703652 ?14:38
patchbotpatch 703652 - sushy - Fix 'None' field value processing - 2 patch sets14:38
iurygregoryit seems devstack-gate would take care of this, but in zuulv3 we are not using it14:38
TheJuliaiurygregory: yup14:38
TheJuliadevstack-gate also had code for it14:39
dtantsurmorning TheJulia14:39
* dtantsur gets back to a loooong presentation14:39
TheJuliadtantsur: ugh14:39
openstackgerritMerged openstack/bifrost stable/train: Fix variable checks in install-deps  https://review.opendev.org/70495614:40
rajinirTheJulia: 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 patched14:41
patchbotpatch 703735 - devstack - Fix pip uncap fallout for nova and barbican (MERGED) - 6 patch sets14:41
*** cdearborn has joined #openstack-ironic14:42
rpittauTheJulia: I saw the discussion in the patch, I have a +2 there since a while, just waiting for the discussion to unfold14:43
TheJuliaI 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
openstackgerritJerry Wang proposed openstack/ironic-inspector master: Update ironic inspector document for dropping py2.x  https://review.opendev.org/70544814:45
rpittauI'm sorry, I actually forgot to remove the +2 based on that discussion, I have the same concern, let me reply to that and adjust14:46
* dtantsur is working on slide #34 and envisions at least as many more to come14:47
rpittauoO14:47
TheJuliarpittau: thanks!14:48
TheJuliarpittau: 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 directions14:48
* dtantsur will probably take a short walk before continuing (thus missing the meeting)14:48
openstackgerritRajini Karthik proposed openstack/sushy master: Testing thirdparty sushy CI  https://review.opendev.org/70528914:48
* iurygregory hopes dtantsur wont have 100 slides and only 25 minutes 14:48
TheJuliadtantsur: Okay :)14:49
rpittauthat would be an interesting presentation :)14:49
dtantsuriurygregory: 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
iurygregorydtantsur, sounds cool *-*14:49
TheJuliadtantsur: ugh, how long until you have to have that done?14:49
dtantsurTheJulia: it's tomorrow afternoon..14:49
TheJuliaugh14:50
iurygregoryrpittau, my friend did this at fosdem and he also had a demo to fit in 25min XD14:50
dtantsuron 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
rpittauiurygregory: was your friend Flash ?14:50
dtantsur++14:50
* dtantsur actually goes for a walk14:50
*** dtantsur is now known as dtantsur|brb14:50
iurygregoryrpittau, he is a bit crazy I can say14:50
TheJuliaiurygregory: 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-ironic14:52
iurygregoryTheJulia, ouch =X14:52
iurygregorybrutal++14:52
rpittau"brutal presentation style" sounds like something to add to a resume14:54
jrollmorning ironic14:54
rpittauhey jroll :)14:54
jroll:)14:55
TheJuliaI think they told me it was a slide very 20 seconds...14:55
*** stendulker has joined #openstack-ironic14:57
iurygregorymorning jroll14:57
jroll\o14:57
rajiniro/15:00
TheJulia#startmeeting ironic15:01
openstackMeeting 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
TheJuliao/15:01
stendulkero/15:01
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:01
rpiosoo/15:01
*** openstack changes topic to " (Meeting topic: ironic)"15:01
iurygregoryo/15:01
openstackThe meeting name has been set to 'ironic'15:01
TheJuliaGood morning everyone!15:01
rajiniro/15:01
cdearborn\o15:01
bdoddo/15:01
kaifengo/15:01
jerrywang1o/15:01
arne_wiebalcko/15:02
TheJuliaOur agenda can be found on the wiki. Hopefully we should get through it fairly quickly.15:02
jroll\o15:02
rlooo/15:02
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:02
rpittauo/15:02
TheJulia#topic Announcements/Reminders15:02
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)"15:02
TheJulia#info Ironic Mid-cycle February 25-26 @ CERN15:03
TheJulia#link https://indico.cern.ch/event/863986/15:03
TheJulia#link https://etherpad.openstack.org/p/ironic-ussuri-midcycle15:03
arne_wiebalckI'll try and send out a mail to the registered participants some time this week.15:03
TheJuliaarne_wiebalck: awesome15:03
arne_wiebalckWith some logistics details.15:03
* iurygregory probably won't go to the mid cycle =( 15:03
TheJuliaIf 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_wiebalckiurygregory: :(15:04
TheJulia#info Opendev + PTG Vancouver - June 8th through 11th15: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
patchbotpatch 705289 - sushy - Testing thirdparty sushy CI - 3 patch sets15:05
rpittaurajinir: can you please a DNM to the commit message title? So it's clear we shouldn't merge that :)15:06
TheJuliaAdditional info for the Opendev + PTG event15:06
rajinirrpittau: sure15:06
rpittauthanks!15:06
*** stendulker_ has joined #openstack-ironic15:06
*** stendulker has quit IRC15:06
openstackgerritRajini Karthik proposed openstack/sushy master: Testing thirdparty sushy CI ( DON"T MERGE)  https://review.opendev.org/70528915: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.html15:07
TheJuliaDoes anyone else have anything to announce or remind us of?15:07
*** ociuhandu has joined #openstack-ironic15:07
rpiosoTheJulia: 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
patchbotpatch 705289 - sushy - Testing thirdparty sushy CI ( DON"T MERGE) - 4 patch sets15:08
TheJuliaHopefully that will show up in the summary correctly :)15:08
rpioso:)15:08
rajinirrpioso, thank you15:09
TheJuliaIt looks like we had no action items last week.15:09
rpiosorajinir: yw15:09
TheJuliaI think everyone has been fairly heads-down so I'm wondering if it even makes sense to visit the subteam status reports?15:09
rloothank you rajinir! that is great that we have more testing :)15:09
rajinirrloo: thank you15:10
TheJuliaI 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 IRC15:11
rpittauyep15:11
TheJulia#topic Deciding on priorities for the coming week15:11
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:12
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:12
TheJuliaLine 16915:12
TheJuliaIt 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 FOSDEM15:13
*** Lucas_Gray has quit IRC15:14
TheJuliaDoes anyone have anything to add to this list?15:14
rpittauTheJulia: yes15:14
TheJuliaI guess one question is a stable/rocky fix. I'm wondering if we just abandon it and move on?15:14
rpittaucan we add this? https://review.opendev.org/70539515:14
patchbotpatch 705395 - ironic - Fix jsonpatch related tests - 3 patch sets15:14
TheJuliarpittau: Sure, please add!15:15
TheJuliaAnyone else?15:15
rpittaudone, FYI CI is broken without that :P15:15
TheJulia\o/15:16
TheJuliaOkay15:16
TheJuliaI guess we're good then?15:16
rloorocky is going into extended maintenance soon?15:16
TheJuliarloo: yes, like a week or two I think15:17
rpittaurloo: yes, the 24th15:17
rloodon't we need rocky CI working at least before then?15:17
rlooor do we decide now that we aren't going to 'support' rocky?15:17
rlooor whatever the term is...15:17
TheJuliaIt is not completely broken, it is a single job that breaks... If memory serves for completely unrelated reasons15:17
rlooi 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
patchbotpatch 692516 - ironic (stable/rocky) - Kill misbehaving `ipmitool` process - 1 patch set15:18
TheJuliaIn 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-ironic15:20
rloowondering 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
TheJuliaafter the 24th, I'd just remove the job anyway15:20
*** mbeierl has joined #openstack-ironic15:20
rloook, we're good then. we'll wait til after 24th, the job will be removed, then we can merge. yes?15:21
TheJuliayes, we're stuck unless we fix the issue which I believe is elsewhere, get that merged in the next two weeks, and then.. yeah15:21
TheJuliaYes, that is entirely reasonable15:21
rlooumm. when do we do a (last?) release of rocky?15:21
rpittauwhoops rechecked wrong patch :/15:21
TheJuliarloo: the release team does a final tag I believe around the 24th15:22
rloo(ie, do we want these changes in the last official release of rocky)15:22
TheJuliaNobody is screaming for it, so I suspect it is fine if it is something people can pickup later if needed15:22
rlooTheJulia: ok, so I think that's really the only diff. Changes to Rocky avail in the last tag, or not.15:23
TheJuliayup15:23
rlooAnd yeah, if no one is 'asking', then let's go with our plan, unless someone wants to fix the CI for Rocky.15:23
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: WIP - Native zuulv3 grenade multinode multitenant  https://review.opendev.org/70503015:23
TheJuliasounds good15:24
TheJuliaMoving on!15:24
TheJulia#topic Discussion15: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-ironic15:25
TheJulia#link https://review.opendev.org/70448615:25
patchbotpatch 704486 - ironic - WIP do all serialization in the expose decorator - 1 patch set15:25
*** mbeierl has quit IRC15:25
*** mbeierl1 is now known as mbeierl15:25
TheJuliaWSME 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 validations15:26
rlooI 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
rpittauin general is a good idea and sure helps the transition to Flask, as that's the final goal15:27
jerrywang1WSME was not only for ironic/pecan15:28
jerrywang1It supports other web frameworks also15:28
TheJuliabut I think we're one of the last projects using wsme?15:28
rloorpittau: so you're saying this wsme code is only temporary for ironic?15:28
rpittaurloo: exactly15:28
rloorpittau: how 'temporary' are we thinking?15:28
kaifengdid keystone use wsme before migration?15:28
TheJuliaI didn't percieve it as temporary15:29
TheJuliakaifeng: I don't think so15:29
rpittaurloo: hard for me to answer that :)15:29
rpittauI think keystone was using wsme for some time....15:30
rlooi 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
kaifengbut they moved to flask, wondering how they do validation now?15:30
rlooI'm speaking/thinking from ignorance though15:30
jroll"temporary" nearly always means forever, fwiw15:30
rloojroll: that's why i asked :)15:31
jrollin my opinion, of course :)15:31
jrollkeystone uses jsonschema for validation now, afaik15:31
jrollnote that is strictly for validation, not for serialization to/from json/python objects15:32
jerrywang1validation was only one reason for WSME using at ironic, actually ironic used WSME's types and base classes also15:33
jrollright, serialization is the other thing we used it for15:33
jerrywang1And expose function also15:33
TheJuliakaifeng: I believe they wrote their own validation code...15:34
jerrywang1Did we would use flask in future?15:34
jrollit feels like we could/should rip out the intermediate objects (not sure if now or later)15:34
jerrywang1If yes, current wsme code would change also15:34
jerrywang1for differnt web frame work WSME code was diffent15:34
TheJuliawrote their own originally15:34
rpiosoTheJulia: How will driver vendor passthru APIs be converted?15:36
TheJuliajroll: I feel like that might be reasonable, but I am struggling to visualize what that would look like15:36
jrollTheJulia: 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
TheJuliarpioso: they would just be serialized into json and passed through, I believe that is what happens presently15:37
rpiosoTheJulia: Thank you!15:37
TheJuliadict 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 to15:38
TheJuliaI guess we should move on15:40
TheJuliaAny objections?15:40
TheJuliaMoving on!15:41
TheJulia#topic Baremetal SIG15:41
*** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)"15:41
TheJuliaRegarding the whitepaper!15:41
* arne_wiebalck has to get going on this one15:42
TheJuliaI 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
TheJuliaThe 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 IRC15:44
arne_wiebalckI won't be able to work much on this before the mid-cycle I'm afraid.15:44
TheJuliaNor 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-ironic15:45
TheJuliaAnyway, I guess we can proceed to Open Discussion if there is nothing else15: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 again15:48
* rpittau sipping chocolate and pear tea15:48
* arne_wiebalck needs a coffee as well15:48
TheJuliaWell, I hope everyone has a wonderful week15:50
rpittauthanks!15:51
rloocrickets15:51
arne_wiebalckthanks, TheJulia15:51
kaifengthx!15:51
TheJuliaIndeed!15:51
TheJuliaI'll end the meeting early if everyone promises to go review the L3 Deployments spec revision :)\15:52
*** ociuhandu has quit IRC15:52
rpittauok, promise :D15:52
TheJuliahttps://review.opendev.org/#/c/672780/15:52
patchbotpatch 672780 - ironic-specs - Update L3 based deployment spec - 31 patch sets15:52
TheJulia:)15:52
TheJuliaThanks everyone15:52
TheJulia\o15:52
kaifenghave to take a look tomorrow :)15:54
TheJuliaThanks, 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
TheJuliaAnd with that, Thanks!15:55
TheJulia#endmeeting15: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
openstackMeeting ended Mon Feb  3 15:55:09 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:55
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-02-03-15.01.html15:55
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-02-03-15.01.txt15:55
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-02-03-15.01.log.html15:55
kaifengbut it seems no further discussion/update after my last check15:55
*** stendulker_ has quit IRC15:56
TheJuliaIt looks like Ilya responded two days later15:56
TheJuliaI've not had a chance to get that far in reading the update since I'm looking at the content, not the back and forth15:57
kaifengyeah, totally, back and forth is the outcome of no consensus15:58
TheJulia:(15:58
TheJuliawe need to find a happy path that is viable and reasonable15:59
TheJuliaAnyway, I'll need to read about a month of that back and forth once I review the content15:59
TheJuliawell, finish reviewing15:59
kaifengit 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 format16:02
TheJuliaGlean is a tool we've used with bifrost in the past and with-in openstack infra.16:03
kaifengyeah, this is only thing i know from the irc discussion before.16:04
TheJuliaYeah, I'd rather avoid trying to create a bunch of validation code and make things hyper-specific16:04
TheJuliabut there is huge dislike of data pass-throughs16:04
*** gyee has joined #openstack-ironic16:04
* TheJulia sighs16:05
*** dtantsur|brb is now known as dtantsur16:05
* TheJulia goes to next meeting16:06
kaifengmy 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
kaifengthis is not pass through, we can validate input, we can assure compatible network data output.16:08
dtantsurTheJulia, rloo, jroll, if the concern is wsme bits maintains, then I'm delighted to let you know that wsme itself is not maintained either :D16:08
dtantsur* s/maintains/maintenance/16:08
TheJuliadtantsur: indeed :(16:08
jrollheh16:09
rloo(remind me why we're using wsme then)16:09
dtantsurwe barely made the CI pass again recently16:09
dtantsurrloo: cargo-culted from nova? dunno, it was before me16:09
dtantsuryou're the last person here who has a chance to remember ;)16:09
rloonever paid attention to that stuff, but that is a valid assumption.16:10
rlooso the plan is to continue using (unsupported) wsme, even after we go from pecan->flask?16:10
rpittaunope16:10
TheJuliakaifeng: I fear the "what" becomes the problem. I thought we were making consensus on the single configuration drive data path, but... gah16:11
rloorpittau: is it documented somewhere, the plan? might be easier than for me to ask questions that someone already wrote up16:11
TheJulia"make a configuration drive, use it"16:11
kaifengTheJulia: config-2 with cloud-init is not desired i remember.16:13
rpittaurloo: 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 ML16:16
rloorpittau: i looked in our etherpad, it is titled 'Replacing WSME'. I then looked at the story ... ah, you beat me16:16
*** mkrai has quit IRC16:17
rloorpittau: 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
rpittaurloo: 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 Steve16:19
rloorpittau: 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
rloorpittau: 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-ironic16:23
*** tzumainn has quit IRC16:24
*** Lucas_Gray has quit IRC16:26
rpittaurloo: 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 story16:26
rpittauand define it better, we can even bring it to the mid-cycle meetup for another review.16:26
kaifengwow, me neither16:29
rpittau:)16:30
TheJuliayeah, consensus was just get off pecan/wsme16:30
kaifengi 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
kaifengsince we don't anticipate multiple solutions, so this is not strictly prohibited i guess.16:31
TheJuliaI 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 there16:32
* rpittau needs to leave :/16:33
TheJuliagoodnight16:33
rpittaugoodnight! o/16:33
*** rpittau is now known as rpittau|afk16:33
*** ociuhandu has quit IRC16:34
*** ociuhandu has joined #openstack-ironic16:55
*** TxGirlGeek has joined #openstack-ironic16:59
*** lucasagomes has quit IRC17:07
*** ociuhandu has quit IRC17:09
*** ociuhandu has joined #openstack-ironic17:10
*** ociuhandu has quit IRC17:15
*** ociuhandu has joined #openstack-ironic17:17
*** kaifeng has quit IRC17:18
*** ociuhandu has quit IRC17:20
*** ociuhandu has joined #openstack-ironic17:21
openstackgerritZygimantas Matonis proposed openstack/ironic-inspector master: Added scope to introspection rules.  https://review.opendev.org/70221517:33
*** k_mouza has quit IRC17:39
*** ociuhandu_ has joined #openstack-ironic17:54
*** ociuhandu has quit IRC17:57
*** ociuhandu_ has quit IRC17:59
*** derekh has quit IRC18:00
*** dougsz has quit IRC18:03
*** k_mouza has joined #openstack-ironic18:11
*** k_mouza has quit IRC18:15
*** k_mouza has joined #openstack-ironic18:15
rpittau|afkPlease 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/70539518:32
patchbotpatch 705395 - ironic - Fix jsonpatch related tests - 3 patch sets18:32
TheJuliarpittau|afk: will we need to backport that?18:36
TheJulianope18:38
rpittau|afkTheJulia: if the version of jsonpatch has been updated for other branches other than master, yes18:38
rpittau|afkI can't check at the moment :/18:38
TheJuliaI just checked upper constraints18:39
TheJuliajsonpatch 1.25 is current master only of requirements18:39
TheJuliaI approved the fix as a single core review18:39
rpittau|afkThanks TheJulia18:40
*** tzumainn has joined #openstack-ironic18:44
*** stevebaker has joined #openstack-ironic19:15
stevebakermorning19:23
donnydI am hoping to get some clarification on how I can use the rule function on auto discovery19:49
*** efried has joined #openstack-ironic19:49
donnydLets 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 CPUs19:50
donnydIs 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
donnydso on and so forth19:52
donnydTheJulia: 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|afk19:53
dtantsur|afkdonnyd: it's usually me, but I'm in Central European t ime19:53
dtantsur|afkand busy as hell tomorrow, but will happily talk to you on Wednesday if nobody beats me to it19:54
*** tesseract has quit IRC20:01
*** andy_ has quit IRC20:02
*** irclogbot_0 has quit IRC20:05
*** irclogbot_0 has joined #openstack-ironic20:06
*** alexmcleod has quit IRC20:08
donnydthank dtantsur|afk20:19
donnydIf I don't get it solved by then I will be around and ask again20:19
TheJuliadonnyd: okay, sounds like you found the right person :)20:22
*** andy_ has joined #openstack-ironic20:23
*** k_mouza has quit IRC20:34
openstackgerritMerged openstack/ironic master: Fix jsonpatch related tests  https://review.opendev.org/70539520:43
*** TxGirlGeek has quit IRC20:51
*** tzumainn has quit IRC20:53
*** TxGirlGeek has joined #openstack-ironic21:00
*** andy_ has quit IRC21:03
*** TxGirlGeek has quit IRC21:07
*** TxGirlGeek has joined #openstack-ironic21:07
*** TxGirlGeek has quit IRC21:08
*** Goneri has quit IRC21:16
*** andy_ has joined #openstack-ironic21:21
openstackgerritJulia Kreger proposed openstack/bifrost master: Guard the status code checks  https://review.opendev.org/70556121:38
*** tzumainn has joined #openstack-ironic21:54
*** bfournie has quit IRC22:33
donnyddtantsur|afk: I figured it out with fancy rules - ironic is so awesome23:14
*** jdandrea has quit IRC23:41

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!