Wednesday, 2022-06-22

stevebaker[m]TheJulia: it might be this https://bugs.launchpad.net/python-swiftclient/+bug/1977867 https://review.opendev.org/c/openstack/python-swiftclient/+/84515702:37
opendevreviewJacob Anders proposed openstack/ironic master: [WIP] Prevent clear_job_queue failures on older iDRACs  https://review.opendev.org/c/openstack/ironic/+/84685905:04
stevebaker[m]TheJulia: rpittau has a fix for the ironic-grenade swift thing, https://review.opendev.org/c/openstack/ironic/+/84661005:09
opendevreviewVerification of a change to openstack/ironic stable/yoga failed: Add workaround for sha1 support removal from Swift  https://review.opendev.org/c/openstack/ironic/+/84661005:23
*** MahnoorAsghar_ is now known as MahnoorAsghar06:12
*** rpioso_ is now known as rpioso06:12
arne_wiebalck_Good morning, Ironic!06:18
*** arne_wiebalck_ is now known as arne_wiebalck06:19
rpittaugood morning ironic! o/06:51
kubajjdtantsur: Thank you for the tutorial how to test the IPA . I managed to compile it like that from source with ease. How do I tell devstack to use it now though? The IPA readme points to the devstack developer quick start which mentions an IPA flag to be set to true if we want to use our own, but how do we proceed from that?07:54
jandersgood morning arne_wiebalck rpittau kubajj and Ironic o/07:58
janderskubajj not sure if this is what you after, but could you just upload a new image to glance and point nodes to it?07:59
janderslet see if dtantsur has a better response07:59
rpittauhey janders :)08:07
opendevreviewMerged openstack/ironic stable/yoga: Add workaround for sha1 support removal from Swift  https://review.opendev.org/c/openstack/ironic/+/84661008:13
rpittaulet's the recheck party begin!08:15
rpittauand hopefully ends well :)08:15
dtantsurand I'm fixing docs for kubajj again :)08:28
dtantsurgood morning08:28
rpittauhey dtantsur :)08:28
opendevreviewVerification of a change to openstack/ironic master failed: Switch to q35 machine type for test nodes  https://review.opendev.org/c/openstack/ironic/+/84355808:45
opendevreviewDmitry Tantsur proposed openstack/ironic master: Docs: specify what to do with the created images  https://review.opendev.org/c/openstack/ironic/+/84713608:51
dtantsurjanders: now we have some docs ^^08:51
opendevreviewJacob Anders proposed openstack/ironic master: [WIP] Prevent clear_job_queue failures on older iDRACs  https://review.opendev.org/c/openstack/ironic/+/84685909:11
jandersdtantsur \o/ TY09:18
jandersajya I've been working on a patch enabling skipping iDRAC specific verify steps if the iDRAC generation doesn't support it. It would be great to have your feedback on the patch in general and in particular w/r/t the points that Dmitry raised: https://review.opendev.org/c/openstack/ironic/+/846859/ Thank you!09:19
jandersdinner time, bbiab09:19
ajyathanks, janders , will take a look later today09:21
jandersajya TY!09:23
*** sshnaidm_ is now known as sshnaidm11:05
iurygregorygood morning Ironic 11:18
jandershey iurygregory11:25
rpittauhey iurygregory :)11:26
rpittauso grenade job is worked-around until we get the new swiftclient11:28
rpittauI see if I can spend some time this afternoon to look deeper into ipa-tempest-bios-ipmi-direct-src, unless someone has some clue11:28
dtantsurI don't, any effort is truly appreciated11:28
iurygregoryrpittau, I'm looking at them since yesterday (trying to figure out what I can do) re ipa-tempest-bios11:49
opendevreviewVerification of a change to openstack/ironic master failed: [Minor] Fix misspellings of "insufficient"  https://review.opendev.org/c/openstack/ironic/+/84578912:03
jandersit seems https://review.opendev.org/c/openstack/ironic/+/846859/ passed CI about 1.5hr ago - does this mean the swift / grenade fix was released? that would be great12:41
janderssee you tomorrow Ironic o/12:41
rpittaujanders: yep, grenade should work now, see you tomorrow o/12:46
janders\o/12:49
ajyajanders: left some comments. jfyi, I'll be off from tomorrow till Monday.13:01
opendevreviewVerification of a change to openstack/ironic master failed: [Minor] Fix misspellings of "insufficient"  https://review.opendev.org/c/openstack/ironic/+/84578913:11
opendevreviewMerged openstack/ironic-python-agent master: Fix discovering WWN/serial for devicemapper devices  https://review.opendev.org/c/openstack/ironic-python-agent/+/84578813:45
opendevreviewMerged openstack/ironic master: Swap the metalsmith UEFI job for a legacy one  https://review.opendev.org/c/openstack/ironic/+/84208313:45
opendevreviewVerification of a change to openstack/ironic master failed: Remove netboot jobs from the gate  https://review.opendev.org/c/openstack/ironic/+/84393713:45
dtantsurmerging started \o/13:53
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.6: Fix discovering WWN/serial for devicemapper devices  https://review.opendev.org/c/openstack/ironic-python-agent/+/84715513:59
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent stable/yoga: Fix discovering WWN/serial for devicemapper devices  https://review.opendev.org/c/openstack/ironic-python-agent/+/84715613:59
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent stable/xena: Fix discovering WWN/serial for devicemapper devices  https://review.opendev.org/c/openstack/ironic-python-agent/+/84715713:59
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.3: Fix discovering WWN/serial for devicemapper devices  https://review.opendev.org/c/openstack/ironic-python-agent/+/84715814:00
dtantsurhuh, why on earth do I have a conflict only on 8.4, but not on 8.3 and 8.6 Oo14:01
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent stable/wallaby: Fix discovering WWN/serial for devicemapper devices  https://review.opendev.org/c/openstack/ironic-python-agent/+/84715914:01
opendevreviewDmitry Tantsur proposed openstack/ironic-python-agent bugfix/8.1: Fix discovering WWN/serial for devicemapper devices  https://review.opendev.org/c/openstack/ironic-python-agent/+/84716014:01
dtantsuriurygregory: have we backported the multipath change to 8.4? do we want to?14:05
iurygregorydtantsur, so I was planning in have a discussion about closing some bugfix branches14:07
dtantsurahhh right14:07
iurygregoryin the upstream meeting to see what people think about closing some of them (I think only *we* RH uses)14:08
iurygregorynot having things in 8.4 won't hurt in our case...14:09
rpittauiurygregory, dtantsur, from my perspective this is what *we* need: https://paste.openstack.org/show/bbNM99hKttJrovM4tbsl/14:10
iurygregoryrpittau, yup =)14:11
iurygregorymy idea would be to close the others if our community is ok with that14:12
iurygregoryand also some older stable branches pre-train basically14:12
iurygregorymaybe I should just send an email and see what people think about it, instead of waiting till we have quorum in the meeting14:12
rpittauiurygregory: you have my blessing, as far as we keep what's in that list :)14:13
iurygregorymuch appreciated rpittau =D14:13
opendevreviewMerged openstack/ironic master: Switch to q35 machine type for test nodes  https://review.opendev.org/c/openstack/ironic/+/84355814:22
opendevreviewVerification of a change to openstack/ironic master failed: Remove netboot jobs from the gate  https://review.opendev.org/c/openstack/ironic/+/84393714:25
rpittauI like how erratic the CI is at this moment14:28
dtantsurchaotic good14:30
rpittau:)14:31
JayF"like"?!14:43
rpittaubye everyone, see you tomorrow o/15:15
JayFo/15:20
opendevreviewRiccardo Pittau proposed openstack/ironic master: [DNM] test removing pecan.testing  https://review.opendev.org/c/openstack/ironic/+/84718115:22
TheJuliaiurygregory: +2 on closing up pre-train branches15:39
dtantsursee you tomorrow folks o/15:45
TheJuliagoodnight dtantsur 15:45
opendevreviewVerification of a change to openstack/ironic master failed: [Minor] Fix misspellings of "insufficient"  https://review.opendev.org/c/openstack/ironic/+/84578917:22
cboucharHi ironic folks: Ran into a Selinux issue trying to do pxe boot18:14
cboucharkernel & ramdisk need type tftpdir_t.       I have code for that; however, it seems these files beneath /var/lib/tftpboot are actually linked to files beneath '/var/lib/ironic/master_images/'.  When I change the type, it affects the link and master_image file. Not sure thats the right thing.18:17
cboucharwhat sent me down this path is I was getting a permission error being generated by ironic host when target host requests the file.18:18
TheJuliaI'm having deja vu of long ago...18:45
opendevreviewVerification of a change to openstack/ironic master failed: Remove netboot jobs from the gate  https://review.opendev.org/c/openstack/ironic/+/84393718:52
opendevreviewVerification of a change to openstack/ironic master failed: Remove netboot jobs from the gate  https://review.opendev.org/c/openstack/ironic/+/84393721:23
jandersgood morning Ironic o/21:26
jandersajya thank you for the review, I appreciate the pointers21:26
stevebaker[m]good morning21:36
TheJuliaiurygregory: https://github.com/OpenStackweb/ironic-website/pull/50/files21:39
TheJuliacbouchar: unfortunately I don't have any good answers for you w/r/t to selinux stuffs. It might resonate with dtantsur's brain in his morning21:40
stevebaker[m]cbouchar: I think I fixed that in bifrost as part of another change, I should probably break it out into its own change https://review.opendev.org/c/openstack/bifrost/+/807220/32/playbooks/roles/bifrost-ironic-install/tasks/bootstrap.yml#51721:42
iurygregoryTheJulia, looking now22:16
TheJuliaugh, my formatting is broken22:27
TheJuliaI can fix that, but not today22:27
TheJuliatomorrw22:27
iurygregorylgtm =)22:55
iurygregoryI liked the pictures \o/22:55

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