Friday, 2021-10-01

*** gmann is now known as gmann_afk00:07
tzumainnTheJulia, sure, that's fair! I actually think there's an explicit test that somehow passes under both set of conditionals, but I'll see if I can make it more exact01:10
*** gmann_afk is now known as gmann01:11
TheJuliatzumainn: ahh, yeah. My conccern is a comment on there as well. Fwiw :)01:13
tzumainnyep, the concern definitely makes sense!01:14
opendevreviewMerged openstack/python-ironicclient stable/wallaby: Fix distribution compatability for configdrive build  https://review.opendev.org/c/openstack/python-ironicclient/+/80938102:38
*** pmannidi is now known as pmannidi|brb04:12
*** pmannidi|brb is now known as pmannidi05:03
iurygregorygood morning Ironic, TGIF!05:52
arne_wiebalckGood morning iurygregory and Ironic!06:07
iurygregoryarne_wiebalck, o/06:16
rpittaugood morning ironic! Happy Friday! o/07:44
iurygregorymorning rpittau o/07:48
rpittauhey iurygregory :)07:49
*** pmannidi is now known as pmannidi|AFK08:06
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent master: Remove else/elif when not needed  https://review.opendev.org/c/openstack/ironic-python-agent/+/81210808:32
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent master: Remove else/elif when not needed  https://review.opendev.org/c/openstack/ironic-python-agent/+/81210808:36
dtantsurmorning folks, happy Friday08:44
dtantsurand slack is still down, happy Friday++!08:44
rpittauis slack down?08:45
dtantsuryeah08:45
dtantsurI think it's DNS, as always08:45
rpittauwork for me (TM)08:45
dtantsurit cannot, try updating something08:46
dtantsurstatus.slack.com doesn't even open08:46
rpittau"Less than 1% of users may be experiencing trouble connecting to Slack"08:46
dtantsurjudging by twitter, less than 1% is a huge understatement08:46
dtantsurbut maybe you're part of the lucky group :)08:46
rpittauit's DNS anyway according to their blog08:47
dtantsurof course it is :)08:47
rpittau"It will take some time for the DNS changes to resolve this issue for everyone. If organization policy permits, and if you are comfortable making this change, switching to Google DNS may improve things in the meantime."08:47
dtantsur"In order to resolve this faster, your ISP (Internet Service Provider) will need to flush their DNS record for http://slack.com."08:48
dtantsursweet08:48
iurygregorymornign dtantsur 08:57
TheJulia good morning!13:31
iurygregorygood morning TheJulia =)13:46
dtantsurTheJulia: morning!13:46
opendevreviewJulia Kreger proposed openstack/ironic-inspector master: DNM: Test UEFI default change impact for inspector  https://review.opendev.org/c/openstack/ironic-inspector/+/81213713:47
opendevreviewJulia Kreger proposed openstack/ironic-inspector master: CI: Make inspector configuration UEFI aware  https://review.opendev.org/c/openstack/ironic-inspector/+/81213714:04
opendevreviewJulia Kreger proposed openstack/bifrost master: DNM: Test ironic boot mode default change  https://review.opendev.org/c/openstack/bifrost/+/81213914:08
opendevreviewJulia Kreger proposed openstack/ironic master: Remove debian packages file for devstack  https://review.opendev.org/c/openstack/ironic/+/81031414:19
opendevreviewJulia Kreger proposed openstack/bifrost master: DNM: Test ironic boot mode default change  https://review.opendev.org/c/openstack/bifrost/+/81213914:37
rpittaubye everyone, have a great weekend! o/14:48
TheJuliao/14:48
opendevreviewJulia Kreger proposed openstack/ironic-inspector master: CI: Make inspector configuration UEFI aware  https://review.opendev.org/c/openstack/ironic-inspector/+/81213715:04
TheJuliabraaains15:52
dtantsurwhere??15:54
* TheJulia has an urge to zombie shuffle15:59
opendevreviewJulia Kreger proposed openstack/ironic master: CI: Change CI ipxe file to snponly  https://review.opendev.org/c/openstack/ironic/+/81216716:13
opendevreviewJulia Kreger proposed openstack/ironic-inspector master: CI: Make inspector configuration UEFI aware  https://review.opendev.org/c/openstack/ironic-inspector/+/81213716:13
opendevreviewMerged openstack/ironic-inspector master: Add Python3 yoga unit tests  https://review.opendev.org/c/openstack/ironic-inspector/+/81050816:23
* TheJulia ponders the devstack plugin16:32
dtantsurhave a great weekend folks!17:09
TheJuliayou too dtantsur 17:16
TheJuliaerbarr: I'm going to take a wild guess that your CI job config never expected us to change the boot mode of nodes by default: https://review.opendev.org/c/openstack/ironic/+/810249 You likely just need to set IRONIC_BOOT_MODE: bios on bios jobs.18:23
iurygregorymakes sense ^18:25
opendevreviewJulia Kreger proposed openstack/ironic master: CI: Change CI ipxe file to snponly  https://review.opendev.org/c/openstack/ironic/+/81216718:40
opendevreviewJulia Kreger proposed openstack/ironic master: Retool devstack plugin to use pxe loaders configuration  https://review.opendev.org/c/openstack/ironic/+/81218818:40
* TheJulia crosses fingers18:40
erbarrTheJulia, thanks! I'm stacking with IRONIC_BOOT_MODE=uefi for all jobs and had to change flavor to remove uefi capability from baremetal flavor and that was working. I'll change stacking accordingly now.18:41
TheJuliasooo much cruft in the devstack plugin :(18:41
TheJuliaerbarr: that default change hasn't merged yet, fwiw, but if not set it you can run into the wrong bootloader being set and such18:42
erbarrahh, thanks18:45
* TheJulia wonders if we could get away with boiling down to swapping the mode around.... but at the same time that is likely pointless given UEFI only chips are shipping18:57
-opendevstatus- NOTICE: The review.opendev.org Gerrit server has become unreachable as of approximately 19:10 UTC due to a networking issue in the provider, but should be reachable again shortly19:44
opendevreviewJulia Kreger proposed openstack/ironic-inspector master: CI: Make inspector configuration UEFI aware  https://review.opendev.org/c/openstack/ironic-inspector/+/81213721:59
opendevreviewJulia Kreger proposed openstack/ironic master: Retool devstack plugin to use pxe loaders configuration  https://review.opendev.org/c/openstack/ironic/+/81218822:22
opendevreviewJulia Kreger proposed openstack/ironic-inspector master: CI: Make inspector configuration UEFI aware  https://review.opendev.org/c/openstack/ironic-inspector/+/81213722:58
opendevreviewJulia Kreger proposed openstack/ironic master: Retool devstack plugin to use pxe loaders configuration  https://review.opendev.org/c/openstack/ironic/+/81218823:27

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