Tuesday, 2022-08-02

opendevreviewJulia Kreger proposed openstack/ironic master: Exit ipxe script if enable_netboot_fallback failed  https://review.opendev.org/c/openstack/ironic/+/81171000:27
opendevreviewJulia Kreger proposed openstack/ironic master: Suppress Chassis Not Found on API Operation  https://review.opendev.org/c/openstack/ironic/+/85177901:25
opendevreviewSteve Baker proposed openstack/ironic master: WIP Implement a DHCP driver backed by dnsmasq  https://review.opendev.org/c/openstack/ironic/+/85168102:31
opendevreviewSteve Baker proposed openstack/bifrost master: WIP Switch bifrost to use the dnsmasq DHCP driver  https://review.opendev.org/c/openstack/bifrost/+/85168202:48
opendevreviewSteve Baker proposed openstack/bifrost master: WIP Switch bifrost to use the dnsmasq DHCP driver  https://review.opendev.org/c/openstack/bifrost/+/85168202:52
opendevreviewJulia Kreger proposed openstack/ironic master: Suppress Chassis Not Found on API Operation  https://review.opendev.org/c/openstack/ironic/+/85177902:56
opendevreviewSteve Baker proposed openstack/ironic master: WIP Implement a DHCP driver backed by dnsmasq  https://review.opendev.org/c/openstack/ironic/+/85168104:22
opendevreviewVanou Ishii proposed openstack/ironic master: Fix iRMC driver to use certification file in HTTPS  https://review.opendev.org/c/openstack/ironic/+/85180207:37
opendevreviewwangjiaqi proposed openstack/tenks master: remove unicode from code  https://review.opendev.org/c/openstack/tenks/+/85181708:13
opendevreviewRadosław Piliszek proposed openstack/tenks master: Remove py2 unicode markers  https://review.opendev.org/c/openstack/tenks/+/85181708:16
opendevreviewwangjiaqi proposed openstack/sushy-tools master: remove unicode from code  https://review.opendev.org/c/openstack/sushy-tools/+/85181908:26
rpittaugood morning ironic! o/08:30
opendevreviewwangjiaqi proposed openstack/tenks master: Use TOX_CONSTRAINTS_FILE  https://review.opendev.org/c/openstack/tenks/+/85182108:32
opendevreviewMerged openstack/tenks master: Remove py2 unicode markers  https://review.opendev.org/c/openstack/tenks/+/85181709:02
dtantsurTheJulia: the last time I did https://review.rdoproject.org/r/c/rdoinfo/+/2819609:39
dtantsurTheJulia, iurygregory, let's issue a call for help wrt ironic-ui. Otherwise we may need to abandon it.09:40
*** tosky_ is now known as tosky10:24
opendevreviewHarald Jensås proposed openstack/networking-baremetal master: Add netconf-openconfig device driver  https://review.opendev.org/c/openstack/networking-baremetal/+/83532410:29
ajyaHi Ironic10:39
ajyarpittau those RAID bugfix backports are consistently failing now, can't see why, will take a closer look.10:40
rpittauajya: hi! Yep, I gave a quick look yesterday but was not able to replicate the issue, I suspect we're using wrong version of something 10:43
opendevreviewDmitry Tantsur proposed openstack/ironic master: Finally remove support for netboot and the boot_option capability  https://review.opendev.org/c/openstack/ironic/+/85024010:47
opendevreviewMerged openstack/sushy-tools master: remove unicode from code  https://review.opendev.org/c/openstack/sushy-tools/+/85181911:14
iurygregorygood morning Ironic11:40
iurygregorydtantsur, agree11:40
iurygregoryyou mean sending an email to openstack-discuss with [ironic][horizon] to see if they can help us?11:41
* iurygregory needs more coffee11:41
dtantsuriurygregory: one of the ways, yes. I also asked on the chat I'm in if there are any users who may be able to help.12:12
*** priteau_ is now known as priteau12:38
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic master: Add ironic-grenade-skip-level Job  https://review.opendev.org/c/openstack/ironic/+/83696612:56
TheJuliadtantsur: ++13:34
* TheJulia is a zombie13:34
TheJuliaI went through some old bugs last night and posted a "fix" for one https://review.opendev.org/c/openstack/ironic/+/85177913:37
TheJuliait is the realistic fix for complex environments short of over complicating and joining everything together every time.13:37
TheJuliaand by joining i mean queries13:37
TheJuliain a single transaction... and hoping a read replica is synced13:38
*** dansmith_ is now known as dansmith13:46
TheJuliayou know, it occurs to me another similar error exists on port lists...14:11
TheJuliadtantsur: thanks for the link14:22
dtantsursure thing (and good morning)14:22
* TheJulia makes vampire like sounds at the day star14:27
TheJuliarpittau: you around?14:39
rpittauTheJulia: yep14:52
opendevreviewAija Jauntēva proposed openstack/sushy-oem-idrac master: Add OEM Storage controller extension  https://review.opendev.org/c/openstack/sushy-oem-idrac/+/85189314:55
TheJuliahmm, I think I might know what is going on :\14:55
TheJuliarpittau: so I'm looking at the recent email from rdo, and I think backporting Ia79ad659098c3e68432214782aeb5b8a9df99eb0 would address the issue since the package actually seems to be okay...15:07
rpittauTheJulia: I don't think I saw the email15:10
TheJuliahmm I can't seem to locally reproduce it15:10
TheJuliaI guess backporting that patch wouldn't hurt anything really15:11
rpittauTheJulia: that should work up to py3.815:11
rpittaunot sure about lower versions15:11
TheJuliahmm... the job is running with py3.615:11
rpittauyeah15:12
rpittauso should be ok up to that point15:12
rpittauajya: I think I found the issue, we're running with the wrong upper-constraints15:22
TheJuliahmm, so the failure is ultimately the unit tests are running and and not finding python-scciclient15:22
opendevreviewRiccardo Pittau proposed openstack/ironic bugfix/19.0: Use yoga upper-constraints for all tests  https://review.opendev.org/c/openstack/ironic/+/85190015:24
opendevreviewRiccardo Pittau proposed openstack/ironic bugfix/19.0: Use yoga upper-constraints for all tests  https://review.opendev.org/c/openstack/ironic/+/85190015:29
opendevreviewJakub Jelinek proposed openstack/ironic-python-agent master: Enable skipping disks for cleaning  https://review.opendev.org/c/openstack/ironic-python-agent/+/85086115:29
TheJuliarpittau: any ideas w/r/t scciclient? I'm feeling stumped, but maybe I just need more coffee15:32
opendevreviewRiccardo Pittau proposed openstack/ironic bugfix/18.1: Use xena upper-constraints for all tests  https://review.opendev.org/c/openstack/ironic/+/85190215:32
rpittauTheJulia: sorry, I can't find the rdo email you mentioned, do you have a link with the failure ?15:33
TheJuliaoh, sure, one moment15:33
TheJuliahttps://review.rdoproject.org/zuul/build/e08a3c90f6a44a3aa23b19862c024a8415:34
rpittauthanks15:34
opendevreviewRiccardo Pittau proposed openstack/ironic bugfix/18.1: Use xena upper-constraints for all tests  https://review.opendev.org/c/openstack/ironic/+/85190215:34
TheJuliahttps://logserver.rdoproject.org/46/44246/1/check/DLRN-rpmbuild-centos8/e08a3c9/buildset/centos8/repos/f9/45/f945fb40d27148e81689c22b5e9d32cce4a14c68_dev/rpmbuild.log.gz15:34
TheJuliathe only thing I can think is the library is just not installed, but then "why is it just now failing"15:35
rpittaummm yeah scciclient is not there15:35
TheJuliawell, a change did just merge on the spec file 3 hours ago15:36
iurygregorydtantsur, verifying if the BMC for the node is V6 (for the external_http_url_v6),  I'm checking based on task.node.driver_info and retriving the redfish_address field , this would be ok right? 15:36
TheJuliamaybe dependency issue that I'm just not seeing?!15:36
rpittauTheJulia: hold on, the client is installed: DEBUG:  python3-scciclient                  noarch  0.10.2-0.20220729095350.b8e6e34.el8  delorean-component-baremetal  124 k15:36
TheJuliayeah, acts like it is not...15:37
TheJuliathat is what is bizzar15:37
rpittauok15:37
* TheJulia may just be loosing her mind15:37
TheJuliaoh I bet I know what is happening15:40
TheJuliaif it was missing diffused xml, then the import would have failed outright15:40
TheJuliaand just returned None15:40
rpittauthe patch you linked could actually fix that15:41
TheJuliayup15:41
TheJuliachecking to see when they added it as a dependency15:41
TheJulia2021-12-2815:42
TheJuliaso thatw as it15:42
TheJuliaokay, mystery soled15:42
TheJuliasolved15:42
* TheJulia makes more coffee15:42
dtantsuriurygregory: well, $some_address field, which is making everything a bit more complicated15:43
iurygregorydtantsur, oh right15:44
rpittaummm interesting zuul doesnt recognize the name of the new project-template I added....15:45
rpittauoh of course15:45
iurygregoryI also noticed I would need to update the override_api_url accordingly ...15:46
opendevreviewRiccardo Pittau proposed openstack/ironic bugfix/19.0: Use yoga upper-constraints for all tests  https://review.opendev.org/c/openstack/ironic/+/85190015:46
opendevreviewRiccardo Pittau proposed openstack/ironic bugfix/18.1: Use xena upper-constraints for all tests  https://review.opendev.org/c/openstack/ironic/+/85190215:47
rpittaubye everyone, see you tomorrow o/16:04
opendevreviewMark Goddard proposed openstack/bifrost master: Repair MariaDB tables after upgrade  https://review.opendev.org/c/openstack/bifrost/+/85190716:17
dtantsuro/17:18
TheJuliadtantsur: so I thought about https://review.opendev.org/c/openstack/ironic/+/851779 also sort of existing in ports, and turns out the row is skipped by the code. :) \o/17:19
TheJuliagoodnight dtantsur 17:20

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