| NobodyCam | good evening ironic folks | 00:24 |
|---|---|---|
| TheJulia | good evening | 00:29 |
| opendevreview | Vanou Ishii proposed openstack/ironic master: Modify do_node_verify to avoid state machine stuck https://review.opendev.org/c/openstack/ironic/+/851950 | 04:46 |
| opendevreview | Vanou Ishii proposed openstack/ironic master: [WIP] Add verify_step to iRMC driver https://review.opendev.org/c/openstack/ironic/+/851952 | 05:04 |
| opendevreview | Vanou Ishii proposed openstack/ironic master: Modify do_node_verify to avoid state machine stuck https://review.opendev.org/c/openstack/ironic/+/851950 | 05:42 |
| rpittau | good morning ironic! o/ | 07:04 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/19.0: Use yoga upper-constraints for all tests https://review.opendev.org/c/openstack/ironic/+/851900 | 07:11 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/18.1: Fix prepare ramdisk for 'wait' states https://review.opendev.org/c/openstack/ironic/+/851526 | 07:14 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/19.0: Use yoga upper-constraints for all tests https://review.opendev.org/c/openstack/ironic/+/851900 | 07:50 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/18.1: Use xena upper-constraints for all tests https://review.opendev.org/c/openstack/ironic/+/851902 | 07:52 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/18.1: Fix prepare ramdisk for 'wait' states https://review.opendev.org/c/openstack/ironic/+/851526 | 07:53 |
| ajya | Good morning Ironic | 08:00 |
| ajya | thanks rpittau. Looks like Dell's 3rd party CI also needs some adjustments, now at least using master images. Pinging erbarr | 08:00 |
| rpittau | ajya: hi! no problem, bugfix branches need more attention as by default they will use master even though should be constrained to the closest stable | 08:02 |
| rpittau | I'm thinking to update the etherpad and make that explicit | 08:02 |
| ajya | +1 | 08:03 |
| rpittau | and probably need to find a way to have some sort of alias, I'm pretty sure I've seen something on zuul about thtat | 08:03 |
| kubajj | Good morning Ironic! | 08:09 |
| kubajj | We were thinking that after https://review.opendev.org/c/openstack/ironic-python-agent/+/850861 , we should introduce the skip list also into get_os_install_device function for deployment. Should I create a new change for it? Should I rename the function I created (its name says _for_cleaning)? | 08:11 |
| mgoddard | dtantsur: morning, have time for a quick bifrost review? https://review.opendev.org/c/openstack/bifrost/+/851907 | 08:22 |
| janders | good morning rpittau ajya kubajj mgoddard and Ironic o/ | 08:34 |
| ajya | hi, janders | 08:39 |
| *** tosky_ is now known as tosky | 08:47 | |
| opendevreview | Vanou Ishii proposed openstack/ironic master: Fix iRMC driver to use certification file in HTTPS https://review.opendev.org/c/openstack/ironic/+/851802 | 08:57 |
| mgoddard | morning janders | 08:59 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/19.0: Use yoga upper-constraints for all tests https://review.opendev.org/c/openstack/ironic/+/851900 | 09:22 |
| rpittau | hey janders :) | 09:23 |
| rpittau | mgoddard: have you tried bifrost on ubuntu 22.04 yet? I suppose you did since your latest patch, did you have any issue with pbr installing ironic from source? | 09:25 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/19.0: Use yoga upper-constraints for all tests https://review.opendev.org/c/openstack/ironic/+/851900 | 09:26 |
| mgoddard | rpittau: we are using it in kolla & kayobe CI. It's a bit different because it's in a container, but generally works AFAIK | 10:25 |
| frickler | not sure if someone has seen this already, ironic UT are failing with latest u-c updates https://zuul.opendev.org/t/openstack/build/450166a4dc9941be803e8b4013d496fd , confirmed locally that jsonschema is the culprit, would be great if someone could work on a fix | 10:52 |
| iurygregory | good morning Ironic | 10:53 |
| iurygregory | frickler, we merged a patch on Jul 18 to fix things with jsonschema https://review.opendev.org/c/openstack/ironic/+/849882 | 10:55 |
| iurygregory | maybe we missed something .-. | 10:55 |
| iurygregory | I will try to take a look at it today | 10:56 |
| frickler | iurygregory: iiuc that patch was to make >= 4.0.0 work. 4.7.2 is working fine now, but the most recent release is 4.9.1, which fails again | 11:04 |
| iurygregory | frickler, ack o/ | 11:05 |
| rpittau | mgoddard: thanks, I found what it seems a regression in pbr in Python 3.10 that shows only in our CI at the moment, I'll keep digging | 11:23 |
| rpittau | if anyone has a moment please check those 2 patches to fix CI in ironic bugfix branches, thanks! https://review.opendev.org/c/openstack/ironic/+/851902 https://review.opendev.org/c/openstack/ironic/+/851900 | 11:24 |
| rpittau | ajya: with the fix this is now green, feel free to rebase the rest of the chain https://review.opendev.org/c/openstack/ironic/+/851526 | 11:25 |
| rpittau | same should be for bugfix/19.0 | 11:25 |
| iurygregory | rpittau, done | 11:25 |
| rpittau | thanks iurygregory | 11:25 |
| iurygregory | everyone please don't forget to fill out the doodle for the Revise Ironic Vision https://doodle.com/meeting/organize/id/eZ4L5Xgb | 11:27 |
| ajya | rpittau: thanks, will do at some point | 11:29 |
| dtantsur | frickler: omg, thanks for the heads-up.. I wonder if it's just a question of changed error messages | 12:14 |
| dtantsur | anyone looking at jsonschema already? | 12:17 |
| opendevreview | Merged openstack/ironic bugfix/18.1: Use xena upper-constraints for all tests https://review.opendev.org/c/openstack/ironic/+/851902 | 12:28 |
| rpittau | dtantsur: for bugfix/19.0 we can recheck after https://review.opendev.org/c/openstack/ironic/+/851900 merges | 12:37 |
| rpittau | bugfix/20.0 is not really needed | 12:37 |
| dtantsur | k cool | 12:38 |
| rpittau | for xena we may need https://review.opendev.org/c/openstack/ironic/+/832306 first | 12:38 |
| opendevreview | Merged openstack/ironic bugfix/19.0: Use yoga upper-constraints for all tests https://review.opendev.org/c/openstack/ironic/+/851900 | 12:44 |
| opendevreview | Merged openstack/bifrost master: Repair MariaDB tables after upgrade https://review.opendev.org/c/openstack/bifrost/+/851907 | 12:49 |
| TheJulia | good morning | 13:03 |
| * TheJulia tries to wake up | 13:16 | |
| kubajj | dtantsur: I have added the documentation and the release note. Please let me know if everything is correct. | 13:32 |
| *** lifeless_ is now known as lifeless | 13:51 | |
| opendevreview | Aija Jauntēva proposed openstack/ironic bugfix/18.1: Fix prepare ramdisk for 'wait' states https://review.opendev.org/c/openstack/ironic/+/851526 | 14:33 |
| opendevreview | Aija Jauntēva proposed openstack/ironic bugfix/18.1: Fix redfish RAID failed tasks https://review.opendev.org/c/openstack/ironic/+/848591 | 14:33 |
| opendevreview | Aija Jauntēva proposed openstack/ironic bugfix/18.1: Fix Redfish RAID for non-immediate controllers https://review.opendev.org/c/openstack/ironic/+/851413 | 14:33 |
| opendevreview | Aija Jauntēva proposed openstack/ironic bugfix/18.1: Fix Redfish RAID to update raid_config https://review.opendev.org/c/openstack/ironic/+/851527 | 14:33 |
| opendevreview | Aija Jauntēva proposed openstack/ironic bugfix/19.0: Fix prepare ramdisk for 'wait' states https://review.opendev.org/c/openstack/ironic/+/851524 | 14:43 |
| opendevreview | Aija Jauntēva proposed openstack/ironic bugfix/19.0: Fix redfish RAID failed tasks https://review.opendev.org/c/openstack/ironic/+/848592 | 14:43 |
| opendevreview | Aija Jauntēva proposed openstack/ironic bugfix/19.0: Fix Redfish RAID for non-immediate controllers https://review.opendev.org/c/openstack/ironic/+/851411 | 14:43 |
| opendevreview | Aija Jauntēva proposed openstack/ironic bugfix/19.0: Fix Redfish RAID to update raid_config https://review.opendev.org/c/openstack/ironic/+/851525 | 14:43 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy stable/yoga: Handle mismatch between registry Identity and the way it's referred to https://review.opendev.org/c/openstack/sushy/+/851883 | 15:15 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy stable/xena: Handle mismatch between registry Identity and the way it's referred to https://review.opendev.org/c/openstack/sushy/+/851884 | 15:15 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy master: Better logging when registries are not available https://review.opendev.org/c/openstack/sushy/+/851023 | 15:18 |
| 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:22 |
| TheJulia | dtantsur: good suggestion w/r/t https://review.opendev.org/c/openstack/sushy/+/850547 | 15:46 |
| TheJulia | I've double checked the standard/schema stuffs just to be on the safe side | 15:47 |
| dtantsur | cool! | 15:54 |
| * dtantsur is horrified at "BMC shall respond with a 406 if it cannot handle a non-compressed response" | 15:54 | |
| opendevreview | Julia Kreger proposed openstack/sushy master: Do not send compression as acceptable encoding https://review.opendev.org/c/openstack/sushy/+/850547 | 15:54 |
| TheJulia | dtantsur: yeah..... | 15:56 |
| TheJulia | janders: ^^^ | 15:56 |
| TheJulia | janders: timeu_: ^^^ I remember somewhere there was a comment about the behavior changed on one of the endpoints and redirect to "/Pending" appeared. I have consulted the spec and I feel that is a firmware bug. | 15:57 |
| opendevreview | Riccardo Pittau proposed openstack/ironic stable/xena: Suppress Bandit B509 in snmp.py https://review.opendev.org/c/openstack/ironic/+/851885 | 16:28 |
| TheJulia | rpittau: did bandit strike? :) | 16:29 |
| rpittau | :D | 16:29 |
| rpittau | apparently | 16:29 |
| TheJulia | rpittau: speaking of quick reviews: https://review.opendev.org/c/openstack/ironic/+/851779 | 16:34 |
| rpittau | mmm deja vu | 16:34 |
| rpittau | +W | 16:34 |
| TheJulia | rpittau: deja vu? | 16:41 |
| rpittau | yeah, I thought I approved that already | 16:41 |
| rpittau | I probably need rest :/ | 16:43 |
| rpittau | good night! o/ | 16:43 |
| TheJulia | stevebaker[m]: so... idea for https://review.opendev.org/c/openstack/ironic/+/851681 follow-up could be "explicit ip via a VIF..." | 17:05 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Initial tempest test idea anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/835917 | 17:21 |
| opendevreview | Merged openstack/ironic bugfix/19.0: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847669 | 17:59 |
| opendevreview | Merged openstack/ironic bugfix/18.1: CI: Add iweb to the use tinyipa on list https://review.opendev.org/c/openstack/ironic/+/847810 | 17:59 |
| opendevreview | Merged openstack/ironic bugfix/19.0: Fix prepare ramdisk for 'wait' states https://review.opendev.org/c/openstack/ironic/+/851524 | 17:59 |
| opendevreview | Merged openstack/ironic bugfix/19.0: Fix redfish RAID failed tasks https://review.opendev.org/c/openstack/ironic/+/848592 | 18:01 |
| opendevreview | Merged openstack/ironic bugfix/18.1: Fix prepare ramdisk for 'wait' states https://review.opendev.org/c/openstack/ironic/+/851526 | 18:02 |
| opendevreview | Merged openstack/ironic bugfix/18.1: Fix redfish RAID failed tasks https://review.opendev.org/c/openstack/ironic/+/848591 | 18:02 |
| opendevreview | Merged openstack/ironic master: Suppress Chassis Not Found on API Operation https://review.opendev.org/c/openstack/ironic/+/851779 | 18:02 |
| opendevreview | Merged openstack/ironic bugfix/19.0: Fix compatibility with jsonschema>=4.0.0 https://review.opendev.org/c/openstack/ironic/+/851374 | 19:22 |
| opendevreview | Merged openstack/ironic stable/xena: Suppress Bandit B509 in snmp.py https://review.opendev.org/c/openstack/ironic/+/851885 | 19:51 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Initial tempest test idea anaconda deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/835917 | 20:56 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/811710/3/releasenotes/notes/version-foo-2eb39b768112547f.yaml#1 does release note name matter? | 21:30 |
| JayF | like the filename | 21:30 |
| TheJulia | JayF: not really | 21:43 |
| JayF | that is a weird filename in any event lol | 21:43 |
| JayF | I'll go back and land it | 21:43 |
| TheJulia | yeeaaah... | 21:44 |
| TheJulia | I spotted that too | 21:44 |
| * TheJulia tries to figure out a way to better test the session cache | 22:00 | |
| opendevreview | Merged openstack/ironic master: Modify do_node_verify to avoid state machine stuck https://review.opendev.org/c/openstack/ironic/+/851950 | 23:28 |
| opendevreview | Merged openstack/ironic master: Enable the ramdisk deploy by default https://review.opendev.org/c/openstack/ironic/+/851751 | 23:28 |
| opendevreview | Merged openstack/ironic master: Exit ipxe script if enable_netboot_fallback failed https://review.opendev.org/c/openstack/ironic/+/811710 | 23:28 |
| *** undefined is now known as Guest7047 | 23:34 | |
| opendevreview | Merged openstack/ironic-python-agent master: Use lsblk json output for safety_check_block_device https://review.opendev.org/c/openstack/ironic-python-agent/+/850497 | 23:47 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!