Monday, 2020-03-09

« Sunday, 2020-03-08 Index Tuesday, 2020-03-10 »
*** threestrands has joined #openstack-ironic00:12
*** diurnalist has quit IRC00:45
*** diurnalist has joined #openstack-ironic00:51
*** diurnalist has quit IRC00:56
openstackgerritlkk proposed openstack/ironic-tempest-plugin master: test show detail of chassis  https://review.opendev.org/71186501:13
*** jerry2 has joined #openstack-ironic01:19
*** jerrywang has quit IRC01:19
*** jdandrea has joined #openstack-ironic01:23
*** bfournie has joined #openstack-ironic01:24
openstackgerritlkk proposed openstack/ironic-tempest-plugin master: List detailed Ports by Node  https://review.opendev.org/71186601:37
*** jdandrea has quit IRC02:40
*** diurnalist has joined #openstack-ironic02:42
*** diurnalist has quit IRC02:48
*** diurnalist has joined #openstack-ironic02:48
openstackgerritVerification of a change to openstack/bifrost failed: Explicitely set ansible version to use  https://review.opendev.org/71150103:10
openstackgerritVerification of a change to openstack/bifrost failed: Remove unused check_get_module function  https://review.opendev.org/71162103:10
*** diurnalist has quit IRC03:12
*** diurnalist has joined #openstack-ironic03:19
*** igordc has joined #openstack-ironic03:19
*** diurnalist has quit IRC03:24
*** mkrai has joined #openstack-ironic03:37
*** ricolin has quit IRC03:43
*** ricolin has joined #openstack-ironic04:27
*** diurnalist has joined #openstack-ironic04:31
*** diurnalist has quit IRC04:36
*** diurnalist has joined #openstack-ironic05:36
*** diurnalist has quit IRC05:41
*** sziviani has quit IRC06:10
*** sziviani has joined #openstack-ironic06:11
*** igordc has quit IRC06:31
*** threestrands has quit IRC06:42
*** ricolin has quit IRC06:52
arne_wiebalckGood morning, ironic!06:55
*** rcernin has quit IRC07:06
*** sziviani has quit IRC07:15
*** sziviani has joined #openstack-ironic07:16
*** rpittau|afk is now known as rpttau07:42
*** rpttau is now known as rpittau07:42
rpittaugood morning ironic! o/07:42
openstackgerritRiccardo Pittau proposed openstack/virtualbmc master: [trivial] change vbmc error message  https://review.opendev.org/71187107:54
*** tkajinam has quit IRC08:07
*** tesseract has joined #openstack-ironic08:12
*** iurygregory has joined #openstack-ironic08:20
iurygregorygood morning Ironic!08:20
openstackgerritVerification of a change to openstack/bifrost failed: Do not use tests as filters  https://review.opendev.org/71181508:21
*** amoralej|off is now known as amoralej08:26
*** ricolin_ has joined #openstack-ironic08:33
*** sziviani_ has joined #openstack-ironic08:34
*** dtantsur|afk is now known as dtantsur08:35
dtantsurmorning ironic08:35
openstackgerritKaifeng Wang proposed openstack/ironic master: Expand user-image doc  https://review.opendev.org/71187608:42
iurygregorymorning dtantsur08:53
*** rpittau is now known as rpittau|bbl08:56
*** lucasagomes has joined #openstack-ironic08:56
*** ricolin_ has quit IRC09:00
*** dougsz has joined #openstack-ironic09:00
*** ociuhandu has joined #openstack-ironic09:01
*** ricolin_ has joined #openstack-ironic09:02
*** yolanda has quit IRC09:11
*** yolanda has joined #openstack-ironic09:11
*** ricolin_ has quit IRC09:23
openstackgerritMark Goddard proposed openstack/bifrost master: DNM: Test virtualbmc<2  https://review.opendev.org/71188009:26
*** k_mouza has joined #openstack-ironic09:27
*** khansa has joined #openstack-ironic09:33
*** derekh has joined #openstack-ironic09:37
*** diurnalist has joined #openstack-ironic09:38
*** ijw has joined #openstack-ironic09:42
*** diurnalist has quit IRC09:43
*** ijw has quit IRC09:46
*** ociuhandu has quit IRC09:48
*** alexmcleod has joined #openstack-ironic09:48
*** fdegir has joined #openstack-ironic09:53
*** dsneddon has joined #openstack-ironic09:53
*** mkrai has quit IRC09:54
openstackgerritMark Goddard proposed openstack/bifrost master: DNM: Test virtualbmc<2  https://review.opendev.org/71188010:04
* dtantsur sighs and tries to rebase the deploy steps patch10:19
*** mkrai has joined #openstack-ironic10:22
mgoddarddtantsur: looks like virtualbmc 2.0.0 broke bifrost CI :(10:27
openstackgerritMark Goddard proposed openstack/bifrost master: DNM: Test virtualbmc<2  https://review.opendev.org/71188010:28
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: In-band deploy steps  https://review.opendev.org/69876810:33
openstackgerritDmitry Tantsur proposed openstack/ironic-lib master: Add a function to find all devices by hints  https://review.opendev.org/71189710:49
*** mkrai has quit IRC10:50
dtantsurarne_wiebalck: ^^ started looking into target devices for RAID.10:51
arne_wiebalckdtantsur: oh, nice! will have a look!10:52
dtantsurarne_wiebalck: that's the same code you'll need to modify to add a "type" hint, so may be of interest.10:52
openstackgerritMerged openstack/virtualbmc master: [trivial] change vbmc error message  https://review.opendev.org/71187110:58
dtantsurTheJulia: I think we broke vmedia with the agent token patches :( https://zuul.opendev.org/t/openstack/build/2278390504e5425bae6f07f717a3bbf1/log/controller/logs/screen-ir-cond.txt?severity=311:03
dtantsuroh, sorry, the patch hasn't merged yet11:03
mgoddardok, figured out the bifrost/vbmc issue. 2.0.0 dropped daemon autostart11:09
arne_wiebalcketingof: I am not able to reproduce the missing backtrace issue for in compatible hardware_types/default_management_interfaces in devstack: oslo_service prints the backtrace there ...11:11
openstackgerritMark Goddard proposed openstack/bifrost master: Run vbmcd as a Systemd service  https://review.opendev.org/71190411:12
arne_wiebalcketingof: I looked at our oslo config, but cannot see how we would disable logging the backtrace.11:12
arne_wiebalcketingof: There is still a difference, since we run Stein, while devstack is master.11:13
*** ricolin_ has joined #openstack-ironic11:13
etingofarne_wiebalck, o/ yeah, could something be fixed (in oslo?) since the version that exhibits the missing tb behavior?11:13
arne_wiebalcketingof: yeah, could be11:14
arne_wiebalcketingof: but even then the logging could be improved as from the error it is not clear that it is the *default* value to look at11:15
arne_wiebalcketingof: one has to look the function names in the backtrace to understand11:15
arne_wiebalcketingof: http://paste.openstack.org/show/790478/11:16
etingofarne_wiebalck, error message seems intelligible to me, no?11:17
arne_wiebalcketingof: it may be worth to catch the exc in default_interface(), log a message and re-raise it to help operators to understand which parameter to look at11:17
*** ociuhandu has joined #openstack-ironic11:17
etingofarne_wiebalck, so how the ideal error message would look like?11:18
arne_wiebalcketingof: it would contain the parameter name, ideally11:18
*** ociuhandu has quit IRC11:19
arne_wiebalcketingof: since it is not enabled_management_interfaces but default_management_interface11:20
*** ociuhandu has joined #openstack-ironic11:20
etingofarne_wiebalck, aha, so default vs explicitly enabled mgmt interface is important to have in the error message?11:20
arne_wiebalcketingof: which I, for instance, did not touch when I enabled redfish, so it took me a while to understand I need to change sth I hadn't touched11:20
arne_wiebalcketingof: it would have helped me, I think :)11:21
etingofarne_wiebalck, well, I can push a patch unless you do then ;)11:21
arne_wiebalcketingof: well, I had no message at all :-D11:21
etingofthat develops your ironic intuition!11:21
arne_wiebalcketingof: I can take care11:21
arne_wiebalcketingof: LOL11:22
etingofgreat, thank you!11:22
arne_wiebalcketingof: we should add this to the docs: "Make sure you have some Ironic intuition."11:22
etingof... or hire arne_wiebalck otherwise11:23
arne_wiebalckthat's also an optio:)11:23
arne_wiebalckoption11:23
dtantsurmgoddard: correct. do we still rely on it?11:26
mgoddarddtantsur: indeed we do. Just pushed a patch to address it11:26
dtantsurthx!11:28
*** ociuhandu_ has joined #openstack-ironic11:41
*** ociuhandu has quit IRC11:44
arne_wiebalcketingof: I think it would be sufficient for clarity if https://github.com/openstack/ironic/blob/master/ironic/common/driver_factory.py#L140 raised NoValidDefaultForInterface, rather than IncompatibleInterface. So a try/catch around get_interface() and raising with the correct exception is probably enough ... wdyt?11:47
*** bfournie has quit IRC11:49
etingofarne_wiebalck, looks reasonable to me. btw, NoValidDefaultForInterface is documented there, but is it ever raised?11:54
arne_wiebalcketingof: it is there further down, but I am not fully clear on the logic11:58
arne_wiebalcketingof: it is certainly not raised in the circumstances I am dealing with11:58
etingofarne_wiebalck, it seems to raise when there is no default interface set in the configuration, but it won't raise when the default is present, but invalid12:02
etingofso it acts as it was named as NoDefaultForInterface12:03
arne_wiebalcketingof: that is my understanding at fits what I see12:03
arne_wiebalcks/at/and/12:03
arne_wiebalcketingof: that's why I think we should catch the exc from get_interface and reraise with NoValidDefaultForInterface12:04
etingofyeah, sounds reasonable to me12:05
arne_wiebalckthanks!12:06
*** uzumaki has joined #openstack-ironic12:14
*** bfournie has joined #openstack-ironic12:36
*** uzumaki has quit IRC12:37
*** Goneri has joined #openstack-ironic12:40
*** rh-jelabarre has joined #openstack-ironic12:41
*** rpittau|bbl is now known as rpittau12:44
derekhTrying to use ironic iwth susy-emulator with nodes set to UEFI boot mode, and ironic is getting "Unknown boot mode requested: UEFI" when trying to set the boot mode,12:48
derekhit works when I use curl (at least I get a new unrealted error), with Uefi instead of UEFI12:48
derekhshould it be case sensitive or not ?12:48
iurygregoryderekh, i think it's case sensitive the mappings we have is 'UEFI' : 'uefi'12:51
iurygregoryhttps://github.com/openstack/sushy-tools/blob/4279ee0622db5654cbbb7479cbcf21747b29780c/sushy_tools/emulator/resources/systems/novadriver.py#L5012:51
iurygregoryhttps://opendev.org/openstack/sushy-tools/src/branch/master/sushy_tools/emulator/resources/systems/libvirtdriver.py#L483-L484 according to the libvirt driver you need to pass UEFI12:53
derekhhmm, I'm confused as its "Uefi" that is working, maybe I've an old version, checking12:55
iurygregoryyeah it's strange .-. maybe etingof have more ideas =)12:55
etingofderekh, iurygregory these literals are different in bare Redfish REST API vs ironic configuration settings12:56
derekhmust have been fixed at some stage, mine says "Uefi" in the code12:56
etingofcan you share the code you have in mind please?12:56
etingofI think in ironic node config it should be 'capabilities': 'boot_mode:uefi12:58
derekhetingof: looks like you fixed it in Oct 4279ee0622db5654cbbb7479cbcf21747b29780c12:58
derekhhttps://review.opendev.org/#/c/688458/12:58
patchbotpatch 688458 - sushy-tools - Fix `BootSourceOverrideMode` enumeration (MERGED) - 6 patch sets12:58
derekhhttps://review.opendev.org/#/c/688458/6/sushy_tools/emulator/resources/systems/libvirtdriver.py12:58
patchbotpatch 688458 - sushy-tools - Fix `BootSourceOverrideMode` enumeration (MERGED) - 6 patch sets12:58
*** rh-jelabarre has quit IRC12:58
etingofderekh, right! sorry, I thought of something else13:00
etingofderekh, so just get fresh sushy-toole? ;)13:00
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: In-band deploy steps  https://review.opendev.org/69876813:00
*** ricolin_ has quit IRC13:00
derekhetingof: yup, version was froozen here https://github.com/metal3-io/metal3-dev-env/blob/master/resources/sushy-tools/Dockerfile#L5  , will update it after lunch13:01
*** rloo has joined #openstack-ironic13:04
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent-builder master: Use tinycore 11  https://review.opendev.org/71164713:08
*** ricolin_ has joined #openstack-ironic13:10
*** ociuhandu_ has quit IRC13:19
*** ociuhandu has joined #openstack-ironic13:19
*** bdodd has joined #openstack-ironic13:21
openstackgerritMark Goddard proposed openstack/bifrost master: Run vbmcd as a Systemd service  https://review.opendev.org/71190413:25
openstackgerritMark Goddard proposed openstack/bifrost master: Run vbmcd as a Systemd service  https://review.opendev.org/71190413:26
openstackgerritDmitry Tantsur proposed openstack/ironic master: Remove compatibility with pre-deploy steps drivers  https://review.opendev.org/71191813:29
*** jdandrea has joined #openstack-ironic13:29
*** cdearborn has joined #openstack-ironic13:29
*** ociuhandu has quit IRC13:31
*** ociuhandu has joined #openstack-ironic13:31
*** ricolin_ has quit IRC13:33
*** amoralej is now known as amoralej|lunch13:34
*** tzumainn has joined #openstack-ironic13:36
*** rh-jelabarre has joined #openstack-ironic13:37
*** diurnalist has joined #openstack-ironic13:40
TheJuliagood morning13:42
iurygregorygood morning TheJulia13:42
openstackgerritMark Goddard proposed openstack/bifrost master: Run vbmcd as a Systemd service  https://review.opendev.org/71190413:43
*** diurnalist has quit IRC13:44
dtantsurmorning TheJulia13:44
dtantsurfinally home?13:45
TheJuliayup, Summer got the flu on her last flight home. I promptly got it13:48
dtantsurOUCH13:48
dtantsurI hope you both recover soon13:48
dtantsurTheJulia: get some rest maybe? I can drive the meeting today.13:49
*** erbarr has joined #openstack-ironic13:49
*** rloo has quit IRC13:49
TheJuliaI called our doctor as soon as I got home and picked up an antiviral called Tamiflu. It makes a big difference13:49
TheJuliaI actually feel most ly human13:49
*** rloo has joined #openstack-ironic13:49
dtantsurOh nice! And how does it feel to be a human? :)13:50
etingofprobably tiresome?13:50
TheJuliaa little tiresome13:51
dtantsurI would expect so (although catching mice by ear is not always trivial either)13:51
TheJuliaso, thinking about it, yes it does seem we broke older ipa images with virtual media13:51
TheJuliasilly pregenerated flag13:51
TheJuliaat least, in that patch13:52
TheJuliadtantsur: have you seen such a failure anywhere else?13:52
dtantsurTheJulia: no, I mistaken it for another failure. A patch of mine has passed today, so we're fine (for now).13:53
dtantsurTheJulia: but your vmedia token patch doesn't pass with older IPA.13:53
*** ociuhandu has quit IRC13:53
TheJuliaI think it is still an issue in that patch13:53
TheJuliayeah13:53
TheJuliaI realize that now :(13:53
TheJuliano good way to make it pass I suspect13:53
TheJuliacoffee time13:54
dtantsurcoffee++13:54
*** ociuhandu has joined #openstack-ironic13:54
dtantsurTheJulia: we can make pregenerated tokens also not mandatory (until V)13:57
TheJuliaThat really is the only option13:57
TheJuliastill generate, still embed13:58
dtantsurwe are going to provide an option to opt in mandatory tokens, right?13:58
TheJuliajust don't go "oh, nope!" if it doesn't work out of the box.13:58
TheJuliaI think we already did13:58
*** ociuhandu has quit IRC13:58
TheJuliawhich would allow operators to lock down the behavior sooner rather than later13:58
dtantsurthen it's cool, I guess14:00
TheJuliayeah, just me being overly aggressive on the changes14:00
TheJuliaI can look at fixing it shortly after the meeting if nothing else distracts me14:01
openstackgerritIlya Etingof proposed openstack/sushy master: Make MessageRegistryFile.Registry attribute non-required  https://review.opendev.org/71192514:03
openstackgerritIlya Etingof proposed openstack/sushy master: Make MessageRegistryFile.Registry attribute non-required  https://review.opendev.org/71192514:07
*** amoralej|lunch is now known as amoralej14:15
TheJuliamjturek: you may want to take a glance at https://review.opendev.org/#/c/69615614:15
patchbotpatch 696156 - ironic-python-agent - Software RAID: Add UEFI support - 5 patch sets14:15
*** Lucas_Gray has joined #openstack-ironic14:22
*** uzumaki has joined #openstack-ironic14:30
rpiosoGood morning, ironicers14:30
iurygregorymorning rpioso14:31
rpittauhey rpioso :)14:33
openstackgerritFatih Degirmenci proposed openstack/bifrost master: Do not use tests as filters  https://review.opendev.org/71181514:33
*** rh-jelabarre has quit IRC14:34
*** rh-jelabarre has joined #openstack-ironic14:34
uzumakihey richard!14:35
rpiosoiurygregory, rpittau: o/ :-)14:35
uzumakio/14:35
rpiosouzumaki: Hi14:35
uzumakiit's noor, I just joined this channel for the first time today. Sounds like a friendly place!14:36
dtantsurhi uzumaki, it is :)14:38
uzumakithat sounds great! I'm going to be working on the bios-interface for idrac-wsman14:38
uzumakido you guys hang out here daily?14:38
dtantsuron working days largely. and depending on each one's timezone14:39
dtantsur(and good news re bios-interface)14:39
uzumakiI see... well, excited to start working here! It's my first time working on an opendev project14:40
TheJuliawelcome uzumaki!14:41
uzumakithanks TheJulia !14:41
TheJuliawait, what good news regarding bios interfaces?14:41
* TheJulia is under the monday morning pre-meeting rock of going through review priorities14:41
dtantsurTheJulia: uzumaki is going to do the implementation for idrac14:42
openstackgerritDmitry Tantsur proposed openstack/ironic master: Remove compatibility with pre-deploy steps drivers  https://review.opendev.org/71191814:42
TheJuliaawesome14:43
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: In-band deploy steps  https://review.opendev.org/69876814:43
TheJuliauzumaki: if you want, you can pickup https://review.opendev.org/#/c/707254/ and revise it to your heart's content14:43
patchbotpatch 707254 - ironic - POC: idrac redfish bios? - 1 patch set14:43
dtantsurthis ^^ is so close, but for some reason deploy never finishes and just loops on the same step...14:43
* uzumaki is excited to start working on it14:43
TheJuliadtantsur: that sounds super familiar to a clean-step issue that took lots of coffee and then some whiskey to identify14:44
dtantsurTheJulia: do you recall anything (if only which whiskey :)14:44
dtantsur?14:44
TheJuliait required some additional logic in the end of the next step logic to determine that there was actually no more work to do and that we needed to exit14:45
dtantsurhmm, it should be already in place for deploy..14:45
TheJuliashould, but maybe it got changed. I think it ended up in _do_next_clean_step or something like that14:45
TheJuliawhich means it should be in the deploy step code14:45
TheJuliauzumaki: re that patch I linked, I was focusing the concept of just reading through redfish with an understanding writing is... more complex.14:46
uzumakiTheJulia, yeah, doing just that14:47
* TheJulia wonders if she could make an omlete in 10 minutes14:49
dtantsurnot impossible, but challenging14:49
TheJuliayeaaah....14:49
iurygregory10min, yes you can14:49
openstackgerritMerged openstack/bifrost master: Run vbmcd as a Systemd service  https://review.opendev.org/71190414:55
dtantsurmore than 1100 cases of the new virus in Germany. meh. it actually tends to double every few days..14:55
* iurygregory =O14:57
* rpittau looks at Italy and despair14:57
dtantsuryeah, could be worse :(14:58
dtantsur500 cases in NRW, but relatively far from me.. for now.14:58
rpittauwaiting for France to raise the alert level any moment14:59
* TheJulia returns with cheesy eggs14:59
rpittauthey're canceling all the events with more than 1000 people14:59
* etingof read that this virus does not like hot weather14:59
TheJuliabecause omlete failed14:59
dtantsuretingof: it's not unfortunately 100% known..14:59
rpittauetingof: I hope so...14:59
etingofhmmm, I read it's 99% certain14:59
dtantsurthere is a similar virus that feels pretty well in Afrika15:00
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Mar  9 15:00:11 2020 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
TheJuliaGreetings everyone!15:00
iurygregoryo/15:00
dtantsuro/15:00
JayFo/15:00
etingof\o15:00
rpittauo/15:00
arne_wiebalcko/15:00
TheJuliaWelcome to our weekly agenda!15:00
cdearborno/15:00
TheJuliaerr, meeting!15:00
rlooo/15:00
TheJuliaWell this meeting is going off wonderfully already15:00
erbarro/15:00
khansao/15:00
rpioso\o15:00
TheJuliaOur agenda can be found on the wiki, as always15:00
mgoddard\o15:00
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:00
TheJulia#topic Announcements / Reminders15:01
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)"15:01
TheJulia#info Rocky has entered extended maintenance and no more releases will be performed.15:01
uzumakio/15:01
TheJulia#info This week is R-9, so we're beginning to run out of time for this cycle.15:02
dtantsuroh, again?15:02
TheJulia#info Reviewers, please spend some time reviewing, we have a number of items people are pushing on and I'm sure everyone will appreciate some additional reviews15:02
TheJuliadtantsur: 3 weeks until non-client library freeze15:03
dtantsurso, metalsmith, sushy and ironic-lib15:03
TheJuliadtantsur: Realisticaly we'll need to cut our release in 6 weeks.15:03
dtantsurI guess sushy is particularly active15:03
dtantsuryeah15:03
openstackgerritMerged openstack/bifrost master: Prepare to switch CentOS 8 job voting again  https://review.opendev.org/70395415:04
dtantsurand feature freeze in around 4 weeks, I guess?15:04
*** beekneemech is now known as bnemec15:04
TheJuliaI was hoping to have a write-up of the mid-cycle posted by now, but haven't quite gotten to it15:04
TheJuliadtantsur: realistically yeah15:04
dtantsurTheJulia: if you share an etherpad/gdoc with me, we can work on the contents together15:05
iurygregoryI can help if necessary also o/15:05
TheJuliadtantsur: that being said, if there is limited scope or sufficient confidence in a feature I'm less concerned about a hard/formal feature freeze15:05
TheJuliaMuch appreciated guys, expect a link later today :)15:05
dtantsuryeah, I'm not advocating for a hard freeze, rather a soft freeze as previously15:05
iurygregoryack15:05
TheJuliadtantsur: ++15:05
TheJuliaDoes anyone else have anything to announce this week?15:06
TheJuliaSeems we had no action items so we can skip that section of the meeting15:06
dtantsur++15:06
TheJuliaAre we good to proceed onward!?15:07
rpittaulet's15:07
dtantsurchoo-choo!15:07
TheJulia#topic Review sub-team status reports15:07
*** openstack changes topic to "Review sub-team status reports (Meeting topic: ironic)"15:07
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:07
TheJuliaStarting at line 298.15:08
TheJuliaI've not had spoons to work on the whitepaper, but the doc is there. Hopefully the next month will be nice and quiet!15:09
dtantsurI don't think I've got anything for the whitepaper in the end :(15:09
* arne_wiebalck hopes that, too15:09
TheJuliawe need to copy and attribute some content sadly. We expected people proposing content to do so... that was hopeful I think.15:10
*** kaifeng_ has joined #openstack-ironic15:10
TheJuliaI guess nobody wants to take the role of the editor15:10
TheJuliaAnyway, moving on!15:10
TheJuliadtantsur: thanks for the update on deploy steps!15:10
arne_wiebalckthe node retirement item can be closed, no?15:11
dtantsurnp, it's fun to work on :D15:11
TheJuliatzumainn awesome progress on multitenancy!15:11
dtantsurarne_wiebalck: the sdk patch hasn't merged yet15:11
tzumainnTheJulia, thanks!15:11
dtantsuror has it?15:11
dtantsurtzumainn++15:11
rpittauarne_wiebalck: the openstacksdk patch is not merged yet, but it should be today15:11
arne_wiebalckI think it has.15:11
arne_wiebalckalmost, ok15:11
TheJuliain the gate \o/15:11
TheJuliaReviews of https://review.opendev.org/#/c/672780/ would be much appreciated!15:12
patchbotpatch 672780 - ironic-specs - Update L3 based deployment spec - 36 patch sets15:12
rpittauI need to fix one small thing in the functional test, but will do in a follow-up15:12
arne_wiebalcknice, thanks!15:13
TheJuliastevebaker is not aorund, but I'm wondering if we've reached a forward path on the wsme work?15:13
TheJulias/aor/aro/15:13
dtantsurI haven't seen new patches on that topic15:13
TheJuliaLooks like the software raid patches are ready for review15:14
arne_wiebalckI rebased and tested the UEFI RAID patches on real h/w.15:15
arne_wiebalckYes, are good to go from my POV.15:15
TheJuliaarne_wiebalck: the standalone raid tests are failing on the ironic patch, so something might be breaking it on that patch15:15
TheJuliaAre we basically good python3 wise for this cycle?15:16
arne_wiebalckTheJulia: ah, the ironic one?15:16
TheJuliaarne_wiebalck: yeah15:16
*** diurnalist has joined #openstack-ironic15:16
iurygregoryTheJulia, mostly15:16
dtantsurpython 3.. has someone checked what is happening in the metalsmith CI?15:16
rpittauI think we're missing still some stuff15:17
iurygregorytenks15:17
mgoddardtenks has patches ready to go15:17
iurygregorymgoddard, yeah =)15:17
TheJuliamgoddard: could you add it to the whiteboard? :)15:17
mgoddardhttps://review.opendev.org/#/c/695881 and https://review.opendev.org/#/c/71010015:18
patchbotpatch 695881 - tenks - CentOS 8 support - 20 patch sets15:18
patchbotpatch 710100 - tenks - Bump ansible version constraints to >=2.7,<2.10 - 5 patch sets15:18
mgoddardTheJulia: sure15:18
rpittaudtantsur: I was going to check metalsmith, didn't have time today15:18
TheJuliaThanks!15:18
dtantsurnice, thanks15:18
rpittauand we're still waiting for the fix to centos8 libvirt pkg for bifrost, somewhat related15:18
TheJuliaSo cross-project wise, one thing to raise awareness. The nova team found an issue with the hash ring logic. Turns out someone renamed one of their machines after the deployment was already online to something with mixed case.15:19
TheJuliaI noticed we kind of had a similar bug around conductor start-ups, so I put up a patch for that. Since nobody has reported it, I'm not too worried about getting the fix merged quickly.15:19
rloooh. interesting (or not) :-(15:20
arne_wiebalckTheJulia:do you have a link to a bug?15:20
TheJuliaAnyone have anything else to ask questions about on the list or note status ?15:20
TheJuliaarne_wiebalck: https://review.opendev.org/#/c/711680/15:20
patchbotpatch 711680 - nova - Lowercase ironic driver hash ring and ignore case ... - 3 patch sets15:20
arne_wiebalckTheJulia: thanks!15:20
TheJuliahttps://review.opendev.org/#/c/711765/ <-- what I posted after looking at our code15:21
patchbotpatch 711765 - ironic - Make reservation checks caseless - 2 patch sets15:21
TheJuliaAre we good to proceed ?15:21
dtantsur++15:21
openstackgerritHarald Jensås proposed openstack/ironic master: Additional IP addresses to IPv6 stateful ports  https://review.opendev.org/70000215:22
TheJulia#topic Deciding on priorities for the coming week15:22
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:22
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:22
TheJuliaLine 16515:22
*** jdandrea has quit IRC15:22
*** jdandrea has joined #openstack-ironic15:23
TheJuliaiurygregory: looks like the IPE patches merged, is there anything else needed for that right now?15:23
iurygregoryTheJulia, I have some backports to stable/train, and I noticed that we don't have the documentation for the ipe (I'm working on it)15:24
TheJuliaiurygregory: Thanks, and thanks for noting the backports15:25
TheJuliaI really hope folks have some review bandwidth this week. :(15:25
iurygregoryTheJulia, I just added the patches in "Potential candidates for backport"15:25
TheJuliaAre we good with the list of patches today?15:26
rloowhat does the 'for the next meeting' mean?15:26
dtantsurI guess we can remove the "For the next meeting" line?15:26
rlooL 22715:26
dtantsurrloo: stuff that I proposed earlier today15:26
rlooheh, thx dtantsur15:26
TheJuliaRemoved15:26
rlooquick question wrt backports. now that rocky is extended maintenance.15:27
rlooi see at L244, 'agreed... to take fixes to stein'.15:27
TheJuliarloo: sure15:27
rlooi'm not sure when 'last week' was. But is that the agreement?15:27
dtantsurI'm not sure either, but my take would be that backports to any non-EM releases are recommended, Rocky and Queens are optional15:28
TheJulia++15:28
dtantsurnon-EM releases are Train and Stein15:28
dtantsurPike and Ocata are likely dead already, and nobody has bothered with them for a while15:28
rlooi'm good with that. if no disagreements, i'll update that wording and link to this discussion.15:28
TheJuliarloo: thanks for spotting that!15:29
*** rh-jelabarre has quit IRC15:29
dtantsurrloo++15:29
TheJuliaAre we good to proceed to discussion items?15:29
openstackgerritRiccardo Pittau proposed openstack/metalsmith master: [DNM] Testing CI  https://review.opendev.org/71193515:29
* dtantsur is good15:29
iurygregory++15:29
TheJulia#topic Discussion15:30
*** openstack changes topic to "Discussion (Meeting topic: ironic)"15:30
TheJuliaWe have a single topic, which relates to what we were just actually talking about!15:30
iurygregoryyay15:30
TheJuliadtantsur inquires if we should keep the ocata and pike branches in Extended Maintenance or stop supporting them completely as in closing the stable branches and no longer accepting patches for those branches.15:31
rloowhat do other projects do?15:31
dtantsurgood question15:31
rloo(ok, nova!)15:31
rloomy heros. ha ha.15:32
* dtantsur looks at nova15:32
dtantsurI see that nova still has stable/ocata15:32
rpittauif we're not going to to backports, why should we keep them open?15:32
rloohow does it hurt to keep them open?15:32
TheJuliarpittau: Well right now, it is up to us if we want to accept patches on those branches15:32
dtantsuron the other hand, they have pretty recent patches to ocata: https://review.opendev.org/#/q/project:openstack/nova+branch:stable/ocata15:32
TheJuliawe just won't cut the EM brancehs15:33
TheJuliabranches15:33
dtantsurrloo: it doesn't, maybe just wrong messaging15:33
TheJuliaThe thing we kind of need to keep in mind is that it is OKAY to drop test jobs if they are a burden for EM branches15:33
dtantsurlike, I'm not going to revive the CI on Pike and Ocata. If anybody wants to - be my guest :)15:33
*** rh-jelabarre has joined #openstack-ironic15:33
rlooi suppose if some bug comes up that someone really wants fixed, and if it eg coincides with a nova patch in ocata?15:33
rpittauextended maintenance means that there are community members maintaining it15:33
rlooi thought we already turned off CI for pike and ocata.15:33
TheJuliaLike EM grenade tests, we didn't need anymore.15:33
iurygregorythis is also related if we care about holding the configurations in ironic-tempest-plugin e.g. https://github.com/openstack/ironic-tempest-plugin/blob/0cbe05eff6b1618e0cba700e246680f91e029832/ironic_tempest_plugin/config.py#L5515:34
dtantsurah, right, iurygregory raises a good point15:34
iurygregoryhttps://github.com/openstack/ironic-tempest-plugin/blob/0cbe05eff6b1618e0cba700e246680f91e029832/ironic_tempest_plugin/config.py#L21615:34
*** TxGirlGeek has joined #openstack-ironic15:34
dtantsurironic-tempest-plugin is branchless, so in theory it has to support all open branches15:34
dtantsurI've heard something about neutron (?) capping their tempest plugin15:35
rpittauI guess we need to keep queens compatibility15:35
TheJuliarpittau: the TC's intent is that the branches in EM are open for largely non-core community maintainers to submit patches to.15:35
dtantsurRH has business interest in queens, so I'm up for keeping it alive for now15:35
rloocan we change the rules so that the plugin only supports... (what?) branches?15:35
TheJuliaI'm all for saying "if it is in EM, we no longer need it in tempest"15:35
rlooyeah, what TheJulia sez ^^!15:35
dtantsurthat will mean that the tempest jobs will no longer pass there15:35
dtantsurOR we'll have to cap the plugin (which is not entirely impossible)15:36
rpittauTheJulia: I understand that, just thinking even if we have patches submitted but no core reviewing or taking care of the CI, well.....15:36
dtantsurand this ^^^15:36
rpittaumaintaining CI for 4 releases is already an humungous job15:36
dtantsuroh yes15:36
TheJuliadtantsur: well, is the burden of keeping CI working worth it. At some point only unit tests are going to work for some of these branches no matter what we do and how many hours we pour into them15:36
openstackgerritMerged openstack/bifrost master: Do not use tests as filters  https://review.opendev.org/71181515:36
rpittauhumongous*15:36
dtantsurTheJulia: right, but then have do we merge something there?15:37
dtantsurI'm not going to +2 a change that doesn't pass at least some testing15:37
TheJuliaIf we want to hold the requirement of some integration tests working, then where do we cut that off at?15:38
openstackgerritDmitry Tantsur proposed openstack/bifrost stable/train: Do not use tests as filters  https://review.opendev.org/71193815:38
TheJuliadtantsur: I think the TC expectation was that at some point really old branches that would never be released again may only end up with unit testing15:38
TheJuliabut that it is also up to each team15:38
rloolet's just cut them off then. there aren't enough people here to maintain all that, and if we won't +2 if there are no tests, then no use. Is that how everyone feels, no one will +2 w/o (functional) tests?15:39
TheJuliaI'm kind of neutral and all for reducing pain15:39
dtantsurrealistically, we don't see a lot of patches there15:39
dtantsur(to put it mildly)15:39
dtantsurif there was a strong interest in e.g. ocata, I would probably force myself into maintaining it15:39
rlooso.. maybe turn off. leave proposed patches around. people can merge those to their local copies?15:40
dtantsurbut the last patch was proposed to ocata in August15:40
TheJuliarloo: that was also the expected intent at some point15:40
rlooi mean if we can' guarantee that a patch works, we don't want to +2. so it is going to be people-beware...15:40
TheJuliaI don't think there is strong interest in maintaining ocata... nor really pike15:40
dtantsurexactly15:41
TheJuliaso whatever reduces the pain, I'm all for15:41
dtantsurwe can make it a documentation issue: document which branches we maintain to which extent15:41
dtantsursimilar to releases.o.o15:41
rloois it worth talking to nova to get their viewpoint on why they added changes to ocata recently? wow, just looking now, and it was quite recent.15:41
dtantsurmaybe just somebody cares to? :)15:42
rpittaumaybe there was an interest downstream15:42
* rloo continues to be impressed with nova15:43
dtantsurnote that ironic had been a fast-moving project until very recently15:43
TheJuliadtantsur: I say we update the docs, we drop tests to minimal on those branches, we document we're not intending to merge anything to those branches, and move on15:43
rloofwiw, we are using ocata downstream but I don't expect changes to be backported to that...15:43
dtantsurmaybe even skip dropping tests. if somebody wants to fix them - cool!15:43
TheJuliadtantsur: cool! Love it!15:43
TheJuliarloo: at that point, I guess you've got downstream branches already and most fixes are environment/local need specific15:44
rlooTheJulia: i make us feel the pain of being on an old branch by manually backporting things that we need...15:45
* TheJulia wonders if there is a movie reference that is appropriate...15:45
TheJuliaAnyway, I'm all for minimal pain, lets just document stance and move on.15:46
TheJuliaSince we have no RFEs listed nor the Baremetal SIG has anything listed, I suggest we go directly to Open Discussion15:46
dtantsur#action dtantsur to document supported branches15:47
TheJuliadtantsur: thanks!15:47
dtantsurunless somebody beats me to it15:47
iurygregoryso we support till queens right? =)15:47
rpittauiurygregory: yeah15:47
iurygregoryawesome =)15:48
rlooto be clear, we were talking about ocata & pike, right?15:48
* iurygregory updates ironic-tempest-plugin since ocata will be no longer supported15:48
rpittaurloo: yep15:49
rloook, wanted to make sure 'til queens' included queens :)15:49
rpittau:)15:49
iurygregoryrloo, yeah =) sorry15:49
TheJulia\o/15:49
TheJuliaIf there are no objections, Open Discussion time?15:49
rpittaulet's open the open discussion15:50
dtantsuropenly15:50
TheJulia#topic Open Discussion15:50
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:50
TheJuliaSo how is everyone?!?15:50
rpittauoh gosh.... reserve question ?15:51
TheJulialol15:51
dtantsurWorried, tired. Nothing new :)15:51
TheJulia*sigh* I know that feeling15:51
rpittauone thing I have, I'm switching the jbos in ironic to dib but there is an issue with ironic-tempest-ipa-wholedisk-bios-agent_ipmitool that really bugs me, I was thinking to leave it to tinycore until we can figure that out ?15:52
dtantsurdo you have a clue what is happening there?15:52
TheJuliaOn a plus side, hopefully I can stop taking tamiflu in a few days.15:53
rpittaudtantsur: https://4ae2de5d02c97f2930b7-ff2d888c3307ad9718a14b305143c473.ssl.cf2.rackcdn.com/710278/13/check/ironic-tempest-ipa-wholedisk-bios-agent_ipmitool/419eb12/controller/logs/ironic-bm-logs/node-0_console_2020-03-05-16%3A01%3A18_log.txt15:53
dtantsurthat's an improvement!15:53
dtantsuroh15:53
rpittauyeah :/15:53
dtantsurrpittau: I seem to recall that DIB-built images hate having >1 NICs15:53
openstackgerritMark Goddard proposed openstack/tenks master: WIP: Support linux bridge in addition to OVS  https://review.opendev.org/71168215:54
dtantsurcan it be the cause here?15:54
dtantsurI recall setting NIC number to 1 for these jobs15:54
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Change default driver  https://review.opendev.org/71194115:54
rpittaumaybe, I see it's failing checking for 2 macs15:54
TheJuliaThat is... alarming15:54
TheJuliacould lookup be broken?15:54
rpittaummm15:54
openstackgerritMerged openstack/ironic master: Expose allocation owner to additional policy checks  https://review.opendev.org/70730715:55
dtantsurmm, yeah, the NIC problem would manifest itself as "no route to host" or something like that15:55
dtantsur404 from lookup smells fishy15:55
TheJuliavery fishy15:55
rpittauI think I'll split the patch anyway, we can move forward with the others, they look fine15:55
iurygregory++15:55
*** Lucas_Gray has quit IRC15:56
dtantsuryeaahh15:56
TheJuliait is supposed to split on a command and look them up individually... at least that is what I remember of it15:56
dtantsurbroken lookup doesn't explain why only this job fails15:56
iurygregorymoving most of the jobs is awesome already...15:56
TheJuliarpittau: splitting soudns like a good idea because we could also be increasing hypervisor load with some of the job changes15:56
rpittauTheJulia: exactly15:57
*** ociuhandu has joined #openstack-ironic15:57
dtantsurwrapping up?15:58
TheJuliaYup, Does anyone have anything else?15:59
*** Lucas_Gray has joined #openstack-ironic15:59
TheJuliaThanks everyone! Have a wonderful week!15:59
dtantsurthank you!15:59
rpiosoTheJulia: ty15:59
rpittauthanks!15:59
iurygregoryty15:59
TheJulia#endmeeting16:00
*** 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."16:00
openstackMeeting ended Mon Mar  9 16:00:24 2020 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:00
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-03-09-15.00.html16:00
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-03-09-15.00.txt16:00
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2020/ironic.2020-03-09-15.00.log.html16:00
openstackgerritMerged openstack/bifrost master: Explicitely set ansible version to use  https://review.opendev.org/71150116:02
*** uzumaki has quit IRC16:03
*** kaifeng_ has quit IRC16:05
*** Lucas_Gray has quit IRC16:06
*** Lucas_Gray has joined #openstack-ironic16:08
openstackgerritMerged openstack/bifrost master: Remove unused check_get_module function  https://review.opendev.org/71162116:16
mgoddardTheJulia: hi, should have brought this up during the meeting: https://review.opendev.org/#/c/709849/116:23
patchbotpatch 709849 - releases - Switch networking-baremetal to cycle-with-rc - 1 patch set16:23
mgoddarddo we need to do intermediate releases for that lib?16:24
mgoddardhappy to push back if so16:24
openstackgerritTzu-Mainn Chen proposed openstack/ironic master: Add separate policies for updating node instance_info and extra  https://review.opendev.org/70771916:27
TheJuliaI really hate how the release team has decided to be so pushy with mdoels16:33
TheJuliamodels16:33
TheJuliaThen again, it is purely in maintenance, so cycle-with-rc is likely okay, but I think we need to reserve the right to break that out if we need to push additional releases down the road16:34
arne_wiebalckbye everyone o/16:42
TheJuliagoodnight!16:49
*** ociuhandu has quit IRC16:54
*** ociuhandu has joined #openstack-ironic16:55
dtantsurTheJulia: this was my thought: cycle-with-intermediary gives us certain freedom..16:55
*** ijw has joined #openstack-ironic16:57
*** ijw has quit IRC16:59
*** ijw has joined #openstack-ironic17:00
*** ociuhandu has quit IRC17:01
*** ociuhandu has joined #openstack-ironic17:03
openstackgerritMerged openstack/bifrost stable/train: Do not use tests as filters  https://review.opendev.org/71193817:03
*** ijw_ has joined #openstack-ironic17:05
*** lucasagomes has quit IRC17:06
*** ijw has quit IRC17:08
TheJuliadtantsur: And I think we should be able to re-assert that freedom should need arise. The TC is not going to object to projects trying to do the right thing for their users.17:13
dtantsur++17:14
rpittaugood night! o/17:27
*** rpittau is now known as rpittau|afk17:27
*** dtantsur is now known as dtantsur|afk17:30
dtantsur|afkg'night!17:30
openstackgerritTzu-Mainn Chen proposed openstack/ironic master: Add node lessee field  https://review.opendev.org/70686417:31
*** k_mouza has quit IRC17:34
*** ociuhandu_ has joined #openstack-ironic17:35
mgoddardthanks TheJulia, I've updated the review with a comment to that effect17:35
TheJuliamgoddard: thanks17:36
TheJuliaI'm going to lay down for a little bit, body is demanding it17:37
*** ociuhandu has quit IRC17:38
mgoddardTheJulia: close your 3 lids (eyes and laptop) for a while17:38
*** ociuhandu_ has quit IRC17:39
*** derekh has quit IRC18:00
etingofthe novel way of breaking sushy -- https://bugzilla.redhat.com/show_bug.cgi?id=1804127#c1118:01
openstackbugzilla.redhat.com bug 1804127 in python-sushy "Importing results in "Error: Cannot mix str and non-str arguments"" [Medium,New] - Assigned to ietingof18:01
*** Lucas_Gray has quit IRC18:08
*** dougsz has quit IRC18:15
*** alexmcleod has quit IRC18:17
*** amoralej is now known as amoralej|off18:31
*** ociuhandu has joined #openstack-ironic18:44
TheJuliaetingof: fun!18:45
*** ociuhandu has quit IRC19:10
*** ociuhandu has joined #openstack-ironic19:11
*** ociuhandu has quit IRC19:16
*** gyee has joined #openstack-ironic19:24
*** tesseract has quit IRC19:33
*** cdearborn has quit IRC20:19
*** k_mouza has joined #openstack-ironic20:43
*** k_mouza has quit IRC20:47
*** trident has quit IRC20:57
*** bfournie has quit IRC20:57
*** trident has joined #openstack-ironic20:58
*** trident has quit IRC21:04
*** trident has joined #openstack-ironic21:05
*** rh-jelabarre has quit IRC21:21
*** rcernin has joined #openstack-ironic21:36
*** bfournie has joined #openstack-ironic21:51
*** jdandrea has quit IRC21:54
*** khansa has quit IRC21:54
JayFdtantsur|afk: fun fact: you're two commits away from dethroning jroll and I from top contributor to IPA all time https://github.com/openstack/ironic-python-agent/graphs/contributors21:55
*** ociuhandu has joined #openstack-ironic22:03
*** ociuhandu has quit IRC22:07
*** zigo has quit IRC22:13
*** jdandrea has joined #openstack-ironic22:14
*** jdandrea has quit IRC22:15
*** bdodd has quit IRC22:18
*** zigo has joined #openstack-ironic22:19
*** bdodd has joined #openstack-ironic22:21
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Log the agent version  https://review.opendev.org/71199522:44
*** tkajinam has joined #openstack-ironic22:55
openstackgerritJulia Kreger proposed openstack/ironic master: Enable agent_token for virtual media boot  https://review.opendev.org/69758423:10
*** chenhaw has quit IRC23:15
*** chenhaw has joined #openstack-ironic23:16
*** gyee has quit IRC23:24
*** diurnalist has quit IRC23:26
openstackgerritJulia Kreger proposed openstack/ironic master: Make image_checksum optional if other checksum is present  https://review.opendev.org/71181623:32
*** erbarr has quit IRC23:32
openstackgerritJulia Kreger proposed openstack/ironic stable/stein: Disable grenade jobs from Stein  https://review.opendev.org/71200223:38
TheJuliaJayF: that is quite a interesting fun fact. I'm surprised by the amount of lines that it says I've added....23:48
*** diurnalist has joined #openstack-ironic23:49
*** diurnalist has quit IRC23:57
*** diurnalist has joined #openstack-ironic23:58
openstackgerritJulia Kreger proposed openstack/ironic master: Send our token back to the agent  https://review.opendev.org/69758523:59
openstackgerritJulia Kreger proposed openstack/ironic master: Signal agent token is required  https://review.opendev.org/69930423:59
« Sunday, 2020-03-08 Index Tuesday, 2020-03-10 »

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