opendevreview | Julia Kreger proposed openstack/ironic master: Exit ipxe script if enable_netboot_fallback failed https://review.opendev.org/c/openstack/ironic/+/811710 | 00:27 |
---|---|---|
opendevreview | Julia Kreger proposed openstack/ironic master: Suppress Chassis Not Found on API Operation https://review.opendev.org/c/openstack/ironic/+/851779 | 01:25 |
opendevreview | Steve Baker proposed openstack/ironic master: WIP Implement a DHCP driver backed by dnsmasq https://review.opendev.org/c/openstack/ironic/+/851681 | 02:31 |
opendevreview | Steve Baker proposed openstack/bifrost master: WIP Switch bifrost to use the dnsmasq DHCP driver https://review.opendev.org/c/openstack/bifrost/+/851682 | 02:48 |
opendevreview | Steve Baker proposed openstack/bifrost master: WIP Switch bifrost to use the dnsmasq DHCP driver https://review.opendev.org/c/openstack/bifrost/+/851682 | 02:52 |
opendevreview | Julia Kreger proposed openstack/ironic master: Suppress Chassis Not Found on API Operation https://review.opendev.org/c/openstack/ironic/+/851779 | 02:56 |
opendevreview | Steve Baker proposed openstack/ironic master: WIP Implement a DHCP driver backed by dnsmasq https://review.opendev.org/c/openstack/ironic/+/851681 | 04:22 |
opendevreview | Vanou Ishii proposed openstack/ironic master: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/851802 | 07:37 |
opendevreview | wangjiaqi proposed openstack/tenks master: remove unicode from code https://review.opendev.org/c/openstack/tenks/+/851817 | 08:13 |
opendevreview | Radosław Piliszek proposed openstack/tenks master: Remove py2 unicode markers https://review.opendev.org/c/openstack/tenks/+/851817 | 08:16 |
opendevreview | wangjiaqi proposed openstack/sushy-tools master: remove unicode from code https://review.opendev.org/c/openstack/sushy-tools/+/851819 | 08:26 |
rpittau | good morning ironic! o/ | 08:30 |
opendevreview | wangjiaqi proposed openstack/tenks master: Use TOX_CONSTRAINTS_FILE https://review.opendev.org/c/openstack/tenks/+/851821 | 08:32 |
opendevreview | Merged openstack/tenks master: Remove py2 unicode markers https://review.opendev.org/c/openstack/tenks/+/851817 | 09:02 |
dtantsur | TheJulia: the last time I did https://review.rdoproject.org/r/c/rdoinfo/+/28196 | 09:39 |
dtantsur | TheJulia, 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 tosky | 10:24 | |
opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Add netconf-openconfig device driver https://review.opendev.org/c/openstack/networking-baremetal/+/835324 | 10:29 |
ajya | Hi Ironic | 10:39 |
ajya | rpittau those RAID bugfix backports are consistently failing now, can't see why, will take a closer look. | 10:40 |
rpittau | ajya: 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 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Finally remove support for netboot and the boot_option capability https://review.opendev.org/c/openstack/ironic/+/850240 | 10:47 |
opendevreview | Merged openstack/sushy-tools master: remove unicode from code https://review.opendev.org/c/openstack/sushy-tools/+/851819 | 11:14 |
iurygregory | good morning Ironic | 11:40 |
iurygregory | dtantsur, agree | 11:40 |
iurygregory | you mean sending an email to openstack-discuss with [ironic][horizon] to see if they can help us? | 11:41 |
* iurygregory needs more coffee | 11:41 | |
dtantsur | iurygregory: 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 priteau | 12:38 | |
opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add ironic-grenade-skip-level Job https://review.opendev.org/c/openstack/ironic/+/836966 | 12:56 |
TheJulia | dtantsur: ++ | 13:34 |
* TheJulia is a zombie | 13:34 | |
TheJulia | I went through some old bugs last night and posted a "fix" for one https://review.opendev.org/c/openstack/ironic/+/851779 | 13:37 |
TheJulia | it is the realistic fix for complex environments short of over complicating and joining everything together every time. | 13:37 |
TheJulia | and by joining i mean queries | 13:37 |
TheJulia | in a single transaction... and hoping a read replica is synced | 13:38 |
*** dansmith_ is now known as dansmith | 13:46 | |
TheJulia | you know, it occurs to me another similar error exists on port lists... | 14:11 |
TheJulia | dtantsur: thanks for the link | 14:22 |
dtantsur | sure thing (and good morning) | 14:22 |
* TheJulia makes vampire like sounds at the day star | 14:27 | |
TheJulia | rpittau: you around? | 14:39 |
rpittau | TheJulia: yep | 14:52 |
opendevreview | Aija Jauntēva proposed openstack/sushy-oem-idrac master: Add OEM Storage controller extension https://review.opendev.org/c/openstack/sushy-oem-idrac/+/851893 | 14:55 |
TheJulia | hmm, I think I might know what is going on :\ | 14:55 |
TheJulia | rpittau: 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 |
rpittau | TheJulia: I don't think I saw the email | 15:10 |
TheJulia | hmm I can't seem to locally reproduce it | 15:10 |
TheJulia | I guess backporting that patch wouldn't hurt anything really | 15:11 |
rpittau | TheJulia: that should work up to py3.8 | 15:11 |
rpittau | not sure about lower versions | 15:11 |
TheJulia | hmm... the job is running with py3.6 | 15:11 |
rpittau | yeah | 15:12 |
rpittau | so should be ok up to that point | 15:12 |
rpittau | ajya: I think I found the issue, we're running with the wrong upper-constraints | 15:22 |
TheJulia | hmm, so the failure is ultimately the unit tests are running and and not finding python-scciclient | 15:22 |
opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/19.0: Use yoga upper-constraints for all tests https://review.opendev.org/c/openstack/ironic/+/851900 | 15:24 |
opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/19.0: Use yoga upper-constraints for all tests https://review.opendev.org/c/openstack/ironic/+/851900 | 15:29 |
opendevreview | Jakub Jelinek proposed openstack/ironic-python-agent master: Enable skipping disks for cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/850861 | 15:29 |
TheJulia | rpittau: any ideas w/r/t scciclient? I'm feeling stumped, but maybe I just need more coffee | 15:32 |
opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/18.1: Use xena upper-constraints for all tests https://review.opendev.org/c/openstack/ironic/+/851902 | 15:32 |
rpittau | TheJulia: sorry, I can't find the rdo email you mentioned, do you have a link with the failure ? | 15:33 |
TheJulia | oh, sure, one moment | 15:33 |
TheJulia | https://review.rdoproject.org/zuul/build/e08a3c90f6a44a3aa23b19862c024a84 | 15:34 |
rpittau | thanks | 15:34 |
opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/18.1: Use xena upper-constraints for all tests https://review.opendev.org/c/openstack/ironic/+/851902 | 15:34 |
TheJulia | https://logserver.rdoproject.org/46/44246/1/check/DLRN-rpmbuild-centos8/e08a3c9/buildset/centos8/repos/f9/45/f945fb40d27148e81689c22b5e9d32cce4a14c68_dev/rpmbuild.log.gz | 15:34 |
TheJulia | the only thing I can think is the library is just not installed, but then "why is it just now failing" | 15:35 |
rpittau | mmm yeah scciclient is not there | 15:35 |
TheJulia | well, a change did just merge on the spec file 3 hours ago | 15:36 |
iurygregory | dtantsur, 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 |
TheJulia | maybe dependency issue that I'm just not seeing?! | 15:36 |
rpittau | TheJulia: hold on, the client is installed: DEBUG: python3-scciclient noarch 0.10.2-0.20220729095350.b8e6e34.el8 delorean-component-baremetal 124 k | 15:36 |
TheJulia | yeah, acts like it is not... | 15:37 |
TheJulia | that is what is bizzar | 15:37 |
rpittau | ok | 15:37 |
* TheJulia may just be loosing her mind | 15:37 | |
TheJulia | oh I bet I know what is happening | 15:40 |
TheJulia | if it was missing diffused xml, then the import would have failed outright | 15:40 |
TheJulia | and just returned None | 15:40 |
rpittau | the patch you linked could actually fix that | 15:41 |
TheJulia | yup | 15:41 |
TheJulia | checking to see when they added it as a dependency | 15:41 |
TheJulia | 2021-12-28 | 15:42 |
TheJulia | so thatw as it | 15:42 |
TheJulia | okay, mystery soled | 15:42 |
TheJulia | solved | 15:42 |
* TheJulia makes more coffee | 15:42 | |
dtantsur | iurygregory: well, $some_address field, which is making everything a bit more complicated | 15:43 |
iurygregory | dtantsur, oh right | 15:44 |
rpittau | mmm interesting zuul doesnt recognize the name of the new project-template I added.... | 15:45 |
rpittau | oh of course | 15:45 |
iurygregory | I also noticed I would need to update the override_api_url accordingly ... | 15:46 |
opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/19.0: Use yoga upper-constraints for all tests https://review.opendev.org/c/openstack/ironic/+/851900 | 15:46 |
opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/18.1: Use xena upper-constraints for all tests https://review.opendev.org/c/openstack/ironic/+/851902 | 15:47 |
rpittau | bye everyone, see you tomorrow o/ | 16:04 |
opendevreview | Mark Goddard proposed openstack/bifrost master: Repair MariaDB tables after upgrade https://review.opendev.org/c/openstack/bifrost/+/851907 | 16:17 |
dtantsur | o/ | 17:18 |
TheJulia | dtantsur: 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 |
TheJulia | goodnight dtantsur | 17:20 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!