Tuesday, 2021-11-23

hgyHello, Ironic!03:17
hgyGood morning, janders o/03:18
arne_wiebalckGood morning, janders hgy_ and Ironic!07:13
hgy_Good morning, arne_wiebalck!07:14
iurygregorygood morning janders hgy_ arne_wiebalck and Ironic o/07:21
arne_wiebalckhey iurygregory o/07:22
rpittaugood morning ironic! o/07:35
arne_wiebalckhey rpittau o/08:00
jandershey hgy__ arne_wiebalck iurygregory rpittau and Ironic o/08:01
rpittauhey arne_wiebalck janders :)08:02
iurygregorymorning rpittau o/08:04
rpittauhey iurygregory :)08:04
opendevreviewMerged openstack/ironic stable/xena: Fix some of the SRBAC tests  https://review.opendev.org/c/openstack/ironic/+/81874108:05
opendevreviewMerged openstack/ironic stable/wallaby: Fix some of the SRBAC tests  https://review.opendev.org/c/openstack/ironic/+/81874208:16
dtantsurJayF: oh lol (re https://www.stackalytics.io/?release=all)08:56
dtantsurI guess I've been around for a while :)08:56
dtantsurgood morning ironic08:56
dtantsurJayF: do you think I get a prise for that or just more stuff to review? :)09:00
opendevreviewMerged openstack/sushy master: [Trivial] Fix docstring typo: s/SECORE/SECURE/g  https://review.opendev.org/c/openstack/sushy/+/81881609:04
iurygregorydtantsur, WOW09:23
iurygregorydtantsur, we had a spec/rfe for Introspection Rules ? I'm trying to find, but no luck yet .-.09:29
dtantsurI thought we did...09:29
iurygregoryat least I couldn't find in ironic-specs...09:31
dtantsuroh, wait, you mean some initial spec?09:31
iurygregoryyeah09:31
iurygregoryor an RFE with the idea09:31
dtantsurI think introspection rules happened back in the times when I was simply hacking on ironic-discoverd (as it was called) the way I wanted :D09:32
opendevreviewVerification of a change to openstack/ironic stable/train failed: CI: Work around tinyipa build issues  https://review.opendev.org/c/openstack/ironic/+/81873509:32
iurygregoryoh :D09:32
dtantsurmany problems come from that times :D09:32
iurygregorydark times...09:33
opendevreviewDmitry Tantsur proposed openstack/bifrost stable/xena: Fix the DHCP port in iptables (68 vs 67 and TCP vs UDP)  https://review.opendev.org/c/openstack/bifrost/+/81874609:33
opendevreviewDmitry Tantsur proposed openstack/bifrost stable/wallaby: Fix the DHCP port in iptables (68 vs 67 and TCP vs UDP)  https://review.opendev.org/c/openstack/bifrost/+/81874709:33
opendevreviewDmitry Tantsur proposed openstack/bifrost bugfix/11.1: Fix the DHCP port in iptables (68 vs 67 and TCP vs UDP)  https://review.opendev.org/c/openstack/bifrost/+/81874809:33
opendevreviewRiccardo Pittau proposed openstack/ironic master: Remove default option from create iso image  https://review.opendev.org/c/openstack/ironic/+/81887409:51
opendevreviewDmitry Tantsur proposed openstack/ironic stable/xena: Stop relying on explicit values of Redfish constants  https://review.opendev.org/c/openstack/ironic/+/81874910:01
opendevreviewDmitry Tantsur proposed openstack/bifrost master: Upgrade from Xena in the upgrade job  https://review.opendev.org/c/openstack/bifrost/+/81887810:41
opendevreviewMerged openstack/ironic-python-agent bugfix/8.1: Re-read the partition table with partx -a, part 2  https://review.opendev.org/c/openstack/ironic-python-agent/+/81742011:57
opendevreviewMerged openstack/bifrost stable/xena: Fix the DHCP port in iptables (68 vs 67 and TCP vs UDP)  https://review.opendev.org/c/openstack/bifrost/+/81874612:10
opendevreviewMerged openstack/ironic master: Add platform:rpm shim, grub packages to bindep  https://review.opendev.org/c/openstack/ironic/+/81538612:26
opendevreviewMerged openstack/ironic-python-agent master: Trivial: split away efibootmgr helpers  https://review.opendev.org/c/openstack/ironic-python-agent/+/81853712:26
rpittauls -l12:51
rpittauwhoops12:51
iurygregoryat least we don't have the output of ls -l =D13:13
opendevreviewRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Improve testing  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/81852313:23
*** sshnaidm|afk is now known as sshnaidm13:24
bkrahi folks, how can one set network settings for the deployed image? i cant seem to find ironic config that handles this. thanks13:24
bkramy ironic image deploys without any ipv4/6 network set in networkd/netplan13:25
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic-specs master: Yoga Themes  https://review.opendev.org/c/openstack/ironic-specs/+/81530813:36
dtantsurbkra: you can use configdrive feature to pass network_data https://docs.openstack.org/ironic/latest/install/configdrive.html13:40
bkrathx 13:42
opendevreviewMerged openstack/ironic stable/xena: Stop relying on explicit values of Redfish constants  https://review.opendev.org/c/openstack/ironic/+/81874913:45
opendevreviewJulia Kreger proposed openstack/ironic master: Add FIPS jobs  https://review.opendev.org/c/openstack/ironic/+/79773914:39
* dtantsur stares at wallaby bifrost failures14:39
dtantsuriurygregory, TheJulia, I wonder if it was a mistake to create bugfix branches on bifrost14:43
dtantsurI don't understand what mix of packages they're installing, how upgrades work there, etc14:44
iurygregoryI also don't remember why we did for bifrost...14:45
iurygregoryweird failures in the bugfix branches?14:45
TheJuliawow irccloud is borked today14:46
dtantsurbork-bork14:47
dtantsurso it bifrost wallaby on centos 814:47
TheJuliaI was about to ask14:47
TheJuliathat, it looks like at least to devstack, centosstream now says it is rhel814:47
dtantsurit fails to install libvirt-python, see in the bottom: https://00e428506f7eac65100a-ddf59c340d436068994b44c0bf057be9.ssl.cf1.rackcdn.com/818747/1/check/bifrost-integration-tinyipa-centos-8/55bd4f8/job-output.txt14:48
iurygregorywuut?!14:48
TheJuliaand /etc/redhat-release may not be present or contain centos, so some logic may be broken somewhere14:48
iurygregorywallaby with py3.8?14:49
iurygregorynvm14:49
TheJuliathat seems like a wholesale libvirt incompatability14:50
TheJuliawow14:50
opendevreviewMerged openstack/metalsmith master: CI: remove old centos7 job  https://review.opendev.org/c/openstack/metalsmith/+/81324914:50
opendevreviewDmitry Tantsur proposed openstack/bifrost stable/wallaby: Revert "Install libvirt-python from source instead of a wheel"  https://review.opendev.org/c/openstack/bifrost/+/81893414:50
dtantsurI'll see if ^^^ fixes it14:51
TheJulia++14:51
TheJulialikely a good idea14:51
TheJuliaso the interesting thing I'm also seeing on centos8 is issues authenticating to libvirt14:51
TheJuliabut I think it is just an issue rooted in our devstack plugin14:51
iurygregory=( I never tested devstack + centos so I can't provide any input 14:53
TheJuliadtantsur: so https://review.opendev.org/c/openstack/ironic/+/818553 looks like it is explicitly trying to invoke py38 as the env to test... but that also appears to toss an error :\15:12
dtantsurTheJulia: is there something you don't expect?15:13
dtantsurthe failure is real, sushy master does break ironic tests15:13
dtantsur(or used to - until my recently merged patch)15:13
TheJuliathe error I saw was an unexpected environment error15:17
TheJuliamaybe it is all better now?!15:17
opendevreviewDmitry Tantsur proposed openstack/sushy master: Migrate Chassis to enums  https://review.opendev.org/c/openstack/sushy/+/81894115:19
dtantsurah, well, I haven't looked at other jobs15:20
TheJuliaahh15:20
dtantsursorry, didn't quite get the question15:21
* TheJulia drops it15:21
opendevreviewRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Improve testing  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/81852315:45
opendevreviewRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Improve testing  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/81852316:09
opendevreviewRiccardo Pittau proposed openstack/python-ironic-inspector-client master: Improve testing  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/81852316:14
opendevreviewJulia Kreger proposed openstack/ironic master: Clarify driver load error message  https://review.opendev.org/c/openstack/ironic/+/81897316:16
dtantsurookay, https://review.opendev.org/c/openstack/bifrost/+/818934 seems to actually work16:22
TheJuliaI'm somehow not surprised16:29
rpittaunot sure if we care enough about fedora 34 support on wallaby, but that would not work unless we remove xinetd like we did in https://opendev.org/openstack/bifrost/commit/ce262837ab0d66922a6fd608fbf24dd808b9cd6816:33
dtantsuryeah, I was pondering that.. it's a large change :(16:33
rpittauyep16:33
opendevreviewMerged openstack/sushy master: Add support for NetworkAdapter resources  https://review.opendev.org/c/openstack/sushy/+/81383916:34
rpittaualso wondering if at this point we want to remove the bionic integration job, considering the yoga tested runtimes16:36
TheJuliaEarlier, I was actually kind of pondering dropping fedora as a focus at all16:38
rpittauheh didn't want to say that :)16:38
rpittauI guess we're moving into that direction16:38
TheJuliabut I think it is because centos stream is largely a thing and yeah16:38
dtantsurrpittau: we don't *need* to remove non-voting jobs16:39
iurygregorywell we should drop bionic jobs (is not a supported version for Yoga)16:39
dtantsurshould probably plan it for Z16:39
dtantsuralthough Bionic is pretty old, yeah16:39
* TheJulia needs to go drop off the car at the mechanic to get some parts replaced16:40
rpittaualright, I guess it can die as it is, just thought about preserving resources for other stuff as I think we'd like to start actively testing on arm6416:40
TheJuliaI'd much rather drop fedora for arm, personally16:43
TheJuliaand fedora has always been an odd-ball to support16:43
TheJuliain that it breaks really often16:43
TheJuliafor things we rarely see make it into distros16:43
dtantsurI have a patch deprecating Fedora support16:44
dtantsurhttps://review.opendev.org/c/openstack/bifrost/+/81878016:44
dtantsurnot removing it until it stops working though16:45
iurygregorysounds like a plan =)16:46
dtantsuranyone wants to work on Stream 9 support? :)16:47
rpittaudtantsur: I can help with that16:48
rpittauI'm starting to look at that for other reasons anyway :)16:50
opendevreviewMerged openstack/ironic stable/train: CI: Work around tinyipa build issues  https://review.opendev.org/c/openstack/ironic/+/81873516:55
opendevreviewMerged openstack/ironic-python-agent stable/train: Output verbose info from efibootmgr  https://review.opendev.org/c/openstack/ironic-python-agent/+/81702417:03
opendevreviewMerged openstack/ironic-python-agent stable/train: Delete EFI boot entry duplicate labels first  https://review.opendev.org/c/openstack/ironic-python-agent/+/81702517:03
opendevreviewMerged openstack/ironic-python-agent stable/train: Fix UEFI record regex  https://review.opendev.org/c/openstack/ironic-python-agent/+/81702617:05
rpittaugood night! o/17:11
NobodyCamgood Morning Ironic'ers17:28
TheJuliagood morning NobodyCam 17:34
NobodyCamGood Morning TheJulia 17:34
NobodyCam:) 0/17:34
dtantsurand good night :)17:41
dtantsuro/17:41
TheJuliagoodnight dtantsur 17:42
opendevreviewMerged openstack/bifrost stable/wallaby: Revert "Install libvirt-python from source instead of a wheel"  https://review.opendev.org/c/openstack/bifrost/+/81893417:45
opendevreviewMerged openstack/bifrost master: Refresh supported operating systems list  https://review.opendev.org/c/openstack/bifrost/+/81878017:45
opendevreviewMerged openstack/ironic master: Enable Redfish by default  https://review.opendev.org/c/openstack/ironic/+/81672117:48
TheJuliaso, NobodyCam was just asking me a question and I thought it is worth raising in chat. In his, and other snmp power strip users environments, with UEFI. We set an explicit default17:49
TheJuliawell, efibootmgr sets an explicit default, I'm wondering if we explicitly need a "don't set an explicit default" flag or... something... in IPA17:49
TheJuliadiscuss()17:49
NobodyCam:)17:51
NobodyCamwe are seeing a large number of the snmp controlled node reboot in to the deployed uefi os as opposed to boot into IPA for the delete / release cleaning 17:52
NobodyCam`I'm wondering if we explicitly need a "don't set an explicit default" flag` would be awesome. at lest for the power strip controlled nodes18:05
TheJuliaI suspect we kind of do18:38
TheJuliaor "always set network first" default in IPA18:38
TheJuliafiguring that out would be kind of a PITA on an enterprise grade system with 50 gazillion network ports18:40
* TheJulia twitches, having recently seen twelve entries for network ports on a machine18:40
NobodyCam:) 3 40gb ports in quad mode19:07
NobodyCamplus two on board nic's and a bmc for good measure 19:08
NobodyCamand its port 10 that's the one with the link19:09
* TheJulia twitches and falls over dead due to uefi19:19
stevebaker[m]morning19:31
TheJuliagood morning stevebaker[m] 19:41
NobodyCamMorining stevebaker[m] 19:48
stevebaker[m]hey NobodyCam 19:49
*** jamesdenton_alt is now known as jamesdenton20:12
opendevreviewSteve Baker proposed openstack/ironic master: Write master grub config on startup  https://review.opendev.org/c/openstack/ironic/+/81558020:17
opendevreviewSteve Baker proposed openstack/ironic master: Capture [pxe]loader_file_paths for distros  https://review.opendev.org/c/openstack/ironic/+/81539220:17
opendevreviewSteve Baker proposed openstack/bifrost master: Use distro conf loader_file_paths to copy ipxe/grub files  https://review.opendev.org/c/openstack/bifrost/+/81870920:27
opendevreviewSteve Baker proposed openstack/bifrost master: WIP support grub network boot  https://review.opendev.org/c/openstack/bifrost/+/80722020:27
opendevreviewSteve Baker proposed openstack/bifrost master: WIP support grub network boot  https://review.opendev.org/c/openstack/bifrost/+/80722020:29
opendevreviewJulia Kreger proposed openstack/ironic master: Add FIPS jobs  https://review.opendev.org/c/openstack/ironic/+/79773920:32
opendevreviewJulia Kreger proposed openstack/ironic master: Clarify driver load error message  https://review.opendev.org/c/openstack/ironic/+/81897320:40
* TheJulia goes and finds lunch21:20
stevebaker[m]TheJulia: OK, I think this change from 9 months ago will allow tripleo nova uefi boot https://review.opendev.org/c/openstack/tripleo-heat-templates/+/78502722:09
TheJulia\o/22:10
TheJuliastevebaker[m]: thanks for continuing to dig22:10
* TheJulia sits in car waiting for noms after first, and second lunch attempts failed22:10
TheJuliaThe tourists are like 1.5 months early. I need to buy land by the airport for a sign22:11
stevebaker[m]heck, its almost my lunch time22:13
stevebaker[m]TheJulia: also heat param NovaHWMachineType exists in stable/train so an overcloud could have q35 set for a deployment or an update22:15
NobodyCamwait 3 attempts at lunch 22:23
TheJuliaNobodyCam: bongos and blaze packed to the gills22:28
NobodyCamugh22:39
TheJuliastevebaker[m]: oh, good to know23:02
TheJuliaFATAL: kernel too old <-- ugh23:10
TheJuliastevebaker[m]: I'm starting to think upstream fips jobs are just not feasible23:26
TheJuliastevebaker[m]: start of major retooling23:26
TheJulias/start/short23:26
stevebaker[m]Bummer, why?23:28
TheJuliaWell, we can't rebuild tinycore23:28
TheJuliawhich means we're going to blow up on rackspace23:28
TheJuliaand..  newgrp raises crypt errors23:28
stevebaker[m]Right23:28
TheJuliaI dunno, I just feel like we're going to waste time/energy like crazy23:30
TheJuliashort of getting biger memory footprint VMs23:32
opendevreviewJulia Kreger proposed openstack/ironic master: Add FIPS jobs  https://review.opendev.org/c/openstack/ironic/+/79773923:45

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