Friday, 2021-09-10

opendevreviewSteve Baker proposed openstack/bifrost master: WIP support grub network boot  https://review.opendev.org/c/openstack/bifrost/+/80722000:44
*** pmannidi is now known as pmannidi|brb03:27
*** pmannidi|brb is now known as pmannidi05:30
iurygregorygood morning Ironic o/05:59
iurygregoryhappy friday \o/05:59
opendevreviewMerged openstack/ironic master: update irmc document  https://review.opendev.org/c/openstack/ironic/+/80341207:28
opendevreviewAija Jauntēva proposed openstack/ironic master: Fix idrac-wsman set_power_state to wait on HW  https://review.opendev.org/c/openstack/ironic/+/80809808:16
iurygregorydtantsur, morning, I was looking at pushing the sushy-tools release, since we changed the default value for SUSHY_EMULATOR_VMEDIA_VERIFY_SSL (to match the real HW), should we release a major version or a minor is ok? (reno will probably recommend a major bump, I think)08:49
dtantsurgood morning ironic09:03
dtantsuriurygregory: in semantic versioning all 0.* versions are considered major09:03
dtantsurmore precisely, 0.* versions are considered unstable09:04
iurygregorylol to me it was always major.minor.patch 09:04
iurygregoryyeah that makes sense09:04
dtantsuryeah, major == 0 is an exception :)09:04
iurygregoryoh ok =)09:04
iurygregoryfor ipa-builder I'm thinking in pushing a 3.0.0 major because we have https://opendev.org/openstack/ironic-python-agent-builder/commit/0e243c4e51f918965021b34b6cdfdd829105ea7109:05
dtantsurmakes sense probably (although tinyIPA is not exactly a product)09:06
iurygregoryyeah, I was thinking about this 09:06
dtantsurIPA-builder is branched, right?09:06
iurygregoryright09:06
dtantsurdo you plan stable/xena from this release?09:06
iurygregorywe have wallaby with 2.7.009:06
iurygregorycorrect09:06
dtantsurI know there are efforts to support CentOS 9 asap, maybe we should include them?09:06
dtantsurI can check sshnaidm|off's patches today and update if necessary09:07
iurygregorysounds good to me09:07
iurygregoryI will talk with the release team that we will hold ipa-builder to include c9 support09:07
iurygregoryand push ironic-ui ipe 09:07
iurygregoryfor context http://lists.openstack.org/pipermail/openstack-discuss/2021-August/024534.html09:08
dtantsurI think we can do it today or on Monday09:10
* dtantsur checks out the patch09:10
dtantsurexcept that.. the debian job is broken in DIB09:11
iurygregorymove to non-voting?09:11
dtantsurprobably, and try to fix ASAP09:11
iurygregoryyeah09:11
dtantsurrpittau|afk is out, right?09:11
iurygregorycorrect09:12
iurygregorytill end of next week I think09:12
dtantsurokay, I'll take care of his patch too09:12
iurygregoryI will keep an eye and review the patches09:12
iurygregoryI can push the patch to make debian non-voting now09:12
dtantsuryes please09:13
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic-python-agent-builder master: Move Debian to non-voting to unblock gate  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80818709:17
opendevreviewMerged openstack/ironic master: Fix typo and add subscription docs  https://review.opendev.org/c/openstack/ironic/+/80721309:25
dtantsuriurygregory: a couple of comments there09:27
iurygregorydtantsur, ack09:27
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Support CentOS 9 Stream  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80680909:28
dtantsurneeds testing ^^09:28
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic-python-agent-builder master: Move Debian to non-voting to unblock gate  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80818709:33
opendevreviewMerged openstack/ironic master: Redfish RAID: Use identity instead of durable_name  https://review.opendev.org/c/openstack/ironic/+/80586609:38
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Support CentOS 9 Stream  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80680909:38
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Revert "Move Debian to non-voting to unblock gate"  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80819309:39
dtantsuriurygregory: I've created a revert with a dependency ^^09:40
iurygregorydtantsur, ack09:40
dtantsurwell, actually, hmm09:41
dtantsurI wonder if we just need to set the version explicitly..09:42
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent-builder master: CI: specify Debian version explicitly  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80819409:43
dtantsuriurygregory: this may be an alternative ^^09:43
iurygregoryinteresting!09:44
dtantsuryeah, it may happen that only the default doesn't work09:44
iurygregorylet's give some cookies to zuul09:44
dtantsuryeah, I'm trying it locally, and it seems to work09:44
dtantsurokay, now, what is broken in Fedora?09:50
iurygregorywoot?!09:52
dtantsurany opinions on no longer supporting Fedora in IPA-builder?10:01
dtantsurit's a moving target, and people should not care that much about what's in the ramdisk10:01
dtantsurokay, centos 9 doesn't work out-of-box10:02
iurygregoryI don't see any problem in not supporting Fedora10:03
dtantsurokay, I'll propose that10:03
dtantsuralso my Debian fix has passed the Debian jobs10:03
iurygregory\o/10:03
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Preliminary work to support CentOS 9 Stream  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80680910:06
* dtantsur is curious what version of ubuntu is in the ubuntu job10:11
dtantsurBionic. hmm.10:12
iurygregorywoot?!10:13
dtantsurand openSUSE 15.1. wtf.10:14
dtantsurthat's the price we pay for not being explicit10:14
iurygregoryyeah ++10:14
iurygregoryI have the feeling I will start drinking earlier today10:14
dtantsur++10:15
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Stop explicitly supported Fedora  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80819710:16
dtantsurokay, now I need to update these versions10:16
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Preliminary work to support CentOS 9 Stream  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80680910:17
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Use Ubuntu Focal and openSUSE 15.2  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80819810:19
dtantsurthis should be it, now lunch time10:21
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Use native Centos 8 Stream for dib images  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80595710:40
dtantsuriurygregory: I think these patches ^^^ are the scope of the 3.0.0 release10:41
iurygregorydtantsur, cool, I will keep an eye on them10:41
* iurygregory needs to order lunch...10:41
dtantsurmmm, openSUSE 15.2 is also broken in DIB10:42
iurygregory<not surprised gif>10:44
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Use Ubuntu Focal instead of the default Bionic  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80819810:45
dtantsurokay, I don't care enough, will just fix ubuntu10:46
opendevreviewDmitry Tantsur proposed openstack/ironic master: Document eject_vmedia for Redfish  https://review.opendev.org/c/openstack/ironic/+/80820111:08
Gamal8abd7el0moneamHello11:34
dtantsurwell, that's was quick11:41
dtantsuriurygregory: all patches passed the CI, except for one that failed with mirror issues. could you check them?12:42
iurygregorydtantsur, sure, looking now12:45
iurygregorysince the debian patch worked I will abandon the one to move to non-voting12:46
dtantsur++12:47
dtantsurI wonder who else is around for review. So quiet today!12:47
*** thiagop is now known as outbrito12:52
iurygregorytrue12:55
TheJuliagood morning13:12
iurygregorydtantsur, I'm looking at the C9 failure in the gate, locally you were able to build the image?13:12
dtantsuriurygregory: nope, same error13:12
dtantsurmorning TheJulia 13:12
iurygregorygood morning TheJulia =)13:12
TheJuliacentos stream 9?13:12
dtantsurTheJulia: once you caffeinate enough, could you check the open IPA-builder patches?13:12
iurygregoryTheJulia, scream :D13:12
dtantsurwe should release it really soon13:12
dtantsurTheJulia: yeah. sshnaidm|off started IPA-builder support, but DIB is still not there. Very much not there.13:13
TheJuliayeah :(13:13
dtantsurbut maybe when DIB is fixed, https://review.opendev.org/806809 will just work13:13
TheJuliaAre we worried that the Fedora breakages will appear in a future Centos version?13:14
dtantsurTheJulia: some of them will. But Fedora breaks more often than that, and is updated very frequently.13:14
dtantsurI.e. once CentOS 10 is a thing, we'll need to explicitly fix and add it. New versions of Fedora just fall on us from the skies.13:15
iurygregorywithout cookies13:15
dtantsuryep, no cookies at all13:15
TheJuliahttps://review.opendev.org/c/openstack/ironic-python-agent-builder/+/805957 makes me worry13:15
TheJuliadtantsur: true, yeah13:15
dtantsurwhy worry?13:16
dtantsurisn't it what we agreed to do?13:16
iurygregorydtantsur, TheJulia wasn't present on the second day of the midcycle I think13:16
TheJuliait is, and we've seen centos 8.4 get centos stream artifacts suddenly too13:16
dtantsuryeah. that's why I want to maintain good support for debian as a backup13:16
iurygregorywow!13:17
TheJuliaI was not... I was sick13:17
TheJuliadtantsur: ack, yeah13:17
dtantsurin any case, there won't be any other CentOS 8 in 4 months13:17
TheJuliavery true13:17
TheJuliaAre there any specific DIB patches which I could look at before I go to the eye doctor?13:23
iurygregoryso for c9s we need a fix in diskimage-builder .-., I'm wondering how we will handle..13:23
iurygregoryor should we just merge and fix things later on? 13:24
dtantsurI suggest ^^^13:24
dtantsurthere are DIB patches, but they seem incomplete13:25
* TheJulia approves steve's dib patches13:25
dtantsurokay, so dropping iSCSI bits13:27
TheJuliawe didn't from IPA itself13:27
dtantsurmmm?13:27
TheJuliaAnd I wouldn't really expect it until ?next? cycle13:27
dtantsurwe did remove the iscsi deploy support from IPA13:27
dtantsurif that's what you're trying to say13:28
TheJuliaoh13:28
TheJuliaI thought it was still there for some reason13:28
TheJuliashows you how recently I've dug into the IPA code outside of the bootloader/grub path13:28
dtantsurnope. we had a 2-cycle deprecation, then just pulled the trigger13:28
dtantsur:)13:28
dtantsurif you grep for 'iscsi', there is some code related to iBFT13:28
TheJuliaoh yeah13:28
TheJuliathat stuff is needed for the ibft table load13:29
dtantsurso I need to make sure we don't remove 'iscsistart' accidentally13:29
TheJulia++13:29
dtantsurso, it comes from iscsi-initiator-utils, which we don't install explicitly?13:30
TheJuliarutro13:31
iurygregorygood luck in the eye doctor TheJulia =)13:31
dtantsur++13:31
* TheJulia queues up a stack of dib patches13:33
dtantsurI wonder how iscsi-initiator-utils is called in ubuntu..13:33
dtantsuraha, open-iscsi13:34
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent-builder master: Remove support for iSCSI deploy from the DIB builder  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80821413:40
TheJuliaIf anyone wants to spend some quick api review time on https://review.opendev.org/c/openstack/ironic/+/803855/, I'm likely going to put in marker/limit (paging stuff) in later today, and fix the api ref stuffs13:45
* dtantsur tries not to review or produce large patches before disappearing for a week13:46
TheJulia:) ok13:46
TheJuliaAnything anyone else needs before I go? Asking because depending on how dialated my eyes will be, I might not be able to use the computer afterwards for 4-6 hours13:59
iurygregoryI think we are good14:15
opendevreviewOpenStack Release Bot proposed openstack/ironic-lib stable/xena: Update .gitreview for stable/xena  https://review.opendev.org/c/openstack/ironic-lib/+/80823114:31
opendevreviewOpenStack Release Bot proposed openstack/ironic-lib stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena  https://review.opendev.org/c/openstack/ironic-lib/+/80823514:31
opendevreviewOpenStack Release Bot proposed openstack/ironic-lib master: Add Python3 yoga unit tests  https://review.opendev.org/c/openstack/ironic-lib/+/80823814:31
opendevreviewOpenStack Release Bot proposed openstack/metalsmith stable/xena: Update .gitreview for stable/xena  https://review.opendev.org/c/openstack/metalsmith/+/80824514:32
opendevreviewOpenStack Release Bot proposed openstack/metalsmith stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena  https://review.opendev.org/c/openstack/metalsmith/+/80824814:32
opendevreviewOpenStack Release Bot proposed openstack/metalsmith master: Update master for stable/xena  https://review.opendev.org/c/openstack/metalsmith/+/80825014:32
opendevreviewOpenStack Release Bot proposed openstack/metalsmith master: Add Python3 yoga unit tests  https://review.opendev.org/c/openstack/metalsmith/+/80825314:32
opendevreviewOpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/xena: Update .gitreview for stable/xena  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/80825814:32
opendevreviewOpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/80826314:32
opendevreviewOpenStack Release Bot proposed openstack/python-ironic-inspector-client master: Update master for stable/xena  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/80826614:32
opendevreviewOpenStack Release Bot proposed openstack/python-ironic-inspector-client master: Add Python3 yoga unit tests  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/80827014:32
iurygregoryspam time14:32
opendevreviewOpenStack Release Bot proposed openstack/python-ironicclient stable/xena: Update .gitreview for stable/xena  https://review.opendev.org/c/openstack/python-ironicclient/+/80828414:32
opendevreviewOpenStack Release Bot proposed openstack/python-ironicclient stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena  https://review.opendev.org/c/openstack/python-ironicclient/+/80829014:32
opendevreviewOpenStack Release Bot proposed openstack/python-ironicclient master: Update master for stable/xena  https://review.opendev.org/c/openstack/python-ironicclient/+/80829414:32
opendevreviewOpenStack Release Bot proposed openstack/python-ironicclient master: Add Python3 yoga unit tests  https://review.opendev.org/c/openstack/python-ironicclient/+/80829914:32
opendevreviewOpenStack Release Bot proposed openstack/sushy stable/xena: Update .gitreview for stable/xena  https://review.opendev.org/c/openstack/sushy/+/80832714:33
opendevreviewOpenStack Release Bot proposed openstack/sushy stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena  https://review.opendev.org/c/openstack/sushy/+/80832914:33
opendevreviewOpenStack Release Bot proposed openstack/sushy master: Update master for stable/xena  https://review.opendev.org/c/openstack/sushy/+/80833014:34
opendevreviewOpenStack Release Bot proposed openstack/sushy master: Add Python3 yoga unit tests  https://review.opendev.org/c/openstack/sushy/+/80833214:34
cenne:p14:39
dtantsurouch14:39
TheJuliaSo, does this mean we should all be practicing yoga every day14:42
* iurygregory never did yoga in his whole life lol14:43
TheJuliasounds like we need to start?!14:43
iurygregoryprobably :D14:43
iurygregorybut if we s/g/d , we can just go and watch Star Wars....14:44
TheJuliawe tried that... I think the trademark lawyers would be unhappy14:45
iurygregoryyeah =(14:45
cenneomg iurygregory! really?14:45
iurygregorycenne, yup 14:47
cennewell, i think you might have done something, even if unknowingly ^^14:47
cenneever fall into to a weird position or even breathe deep for a few? hehe. that probably counts. 14:48
iurygregorycenne, the only thing I did was vote for Yoda instead of Yoga when the pool was open hehe14:49
cennehehe14:49
opendevreviewVerification of a change to openstack/ironic-python-agent-builder master failed: CI: specify Debian version explicitly  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80819414:53
opendevreviewVerification of a change to openstack/ironic-python-agent-builder master failed: Stop explicitly supported Fedora  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80819714:53
opendevreviewVerification of a change to openstack/ironic-python-agent-builder master failed: Use native Centos 8 Stream for dib images  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80595714:53
opendevreviewVerification of a change to openstack/ironic-python-agent-builder master failed: Preliminary work to support CentOS 9 Stream  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80680914:53
iurygregorytimeout \o/14:54
dtantsursigh14:56
cennedtantsur: something that'd probably be a quick review. https://review.opendev.org/c/openstack/openstacksdk/+/80452914:56
cennezuul seems to be happy with checking openstacksdk patches now. so just resurfacing it so anyone can take a look.14:56
cennebye iurygregory 14:57
iurygregorybye cenne 14:58
dtantsurbye cenne 15:07
cenneBye dtantsur 15:17
opendevreviewDmitry Tantsur proposed openstack/ironic-lib stable/xena: Avoid using blkid in favour of lsblk and parted  https://review.opendev.org/c/openstack/ironic-lib/+/80816615:32
dtantsurokay folks, time to drift into the weekend. see you in a week!15:36
NobodyCamGood morning 15:40
NobodyCamOpenstack Folks15:40
NobodyCamand OFC...15:41
NobodyCamTGIF!15:41
opendevreviewMerged openstack/metalsmith stable/xena: Update .gitreview for stable/xena  https://review.opendev.org/c/openstack/metalsmith/+/80824515:50
opendevreviewMerged openstack/metalsmith stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena  https://review.opendev.org/c/openstack/metalsmith/+/80824815:50
opendevreviewMerged openstack/ironic-lib stable/xena: Update .gitreview for stable/xena  https://review.opendev.org/c/openstack/ironic-lib/+/80823115:53
opendevreviewMerged openstack/ironic-lib stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena  https://review.opendev.org/c/openstack/ironic-lib/+/80823515:53
opendevreviewMerged openstack/python-ironicclient stable/xena: Update .gitreview for stable/xena  https://review.opendev.org/c/openstack/python-ironicclient/+/80828415:53
opendevreviewMerged openstack/python-ironic-inspector-client stable/xena: Update .gitreview for stable/xena  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/80825815:53
opendevreviewMerged openstack/python-ironic-inspector-client stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/80826315:53
opendevreviewMerged openstack/sushy stable/xena: Update .gitreview for stable/xena  https://review.opendev.org/c/openstack/sushy/+/80832715:53
opendevreviewMerged openstack/metalsmith master: Update master for stable/xena  https://review.opendev.org/c/openstack/metalsmith/+/80825016:01
opendevreviewMerged openstack/python-ironicclient stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena  https://review.opendev.org/c/openstack/python-ironicclient/+/80829016:01
opendevreviewMerged openstack/sushy stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena  https://review.opendev.org/c/openstack/sushy/+/80832916:01
iurygregorymorning NobodyCam o/16:54
TheJuliaanyone heard of anyone using openbmc?18:09
TheJuliagood morning NobodyCam 18:26
TheJuliaiurygregory: added a couple things to the ptg discussion list18:27
NobodyCamgood morning TheJulia 18:27
NobodyCamhappy Friday 18:28
NobodyCamGood Morning iurygregory !! totally missed you GM!18:28
TheJuliaeverything is too bright today18:35
NobodyCamheheheh18:54
NobodyCambut do you study nuclear science18:56
TheJuliahuh?19:05
NobodyCamhttps://www.youtube.com/watch?v=8qrriKcwvlY19:23
NobodyCam;p19:23
TheJuliaI was thinking more along the lines of https://www.youtube.com/watch?v=MPImhprnUsE19:35
NobodyCamOh ya!20:18
opendevreviewJulia Kreger proposed openstack/ironic master: Facilitate asset copy for bootloader ops  https://review.opendev.org/c/openstack/ironic/+/80681520:39
opendevreviewJulia Kreger proposed openstack/ironic master: Record node history and manage events in db  https://review.opendev.org/c/openstack/ironic/+/80329221:48
opendevreviewMerged openstack/ironic-python-agent-builder master: CI: specify Debian version explicitly  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80819421:56
opendevreviewMerged openstack/ironic-python-agent-builder master: Use native Centos 8 Stream for dib images  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80595722:14
opendevreviewMerged openstack/ironic-python-agent-builder master: Stop explicitly supported Fedora  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/80819722:14
*** pmannidi is now known as pmannidi|rebooting_laptop22:22
opendevreviewJulia Kreger proposed openstack/ironic master: API endpoints to get node history  https://review.opendev.org/c/openstack/ironic/+/80385523:45
TheJuliastevebaker: ^23:45
stevebakerTheJulia: thanks, I'll take a look on Monday23:45

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!