| opendevreview | Steve Baker proposed openstack/ironic master: DNM Don't install graphical console deps with bindep https://review.opendev.org/c/openstack/ironic/+/943868 | 00:36 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/ironic master: DMN disable ir-novnc in ironic-standalone-redfish https://review.opendev.org/c/openstack/ironic/+/943871 | 02:29 |
| frickler | stevebaker[m]: error is still happening in both of ^^, note that https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/943529 made it non-fatal, you have to search for "ansible-playbook" in the log to see the error now | 06:25 |
| vsaienko | hello ironic community, is there any core reviewer around? can you please add workflow https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/942009 the patch had WF+1 but it was removed due to rebase on top of another change to unblock CI. Thank you. | 06:50 |
| rpittau | good morning ironic! o/ | 10:31 |
| dtantsur | Sigh, what do I need to manually hack in gertty this time to make it work after an upgrade? | 10:58 |
| dtantsur | Monday, am I right? | 10:58 |
| dtantsur | Ah, downgrade SQLAlchemy, of course. | 10:59 |
| JayF | TheJulia: I got the full update in the situation re: power status. Ironic did the right things, I got the full story. Hope you got home safe o/ | 11:59 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Add retries while waiting for SSH on server https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/942009 | 12:05 |
| vsaienko | ironic team, please review https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/941013 - adds tests to validate trunks, already has +2 and all dependent patches are merged. And vlan aware VMs patches https://review.opendev.org/c/openstack/networking-generic-switch/+/928490 also has +2 and https://review.opendev.org/c/openstack/networking-generic-switch/+/940966. Maybe we can land this feature? | 12:28 |
| opendevreview | Merged openstack/ironic master: centos devstack support: fix VM permissions https://review.opendev.org/c/openstack/ironic/+/942134 | 12:31 |
| TheJulia | dtantsur: sure sounds like a monday thing | 13:18 |
| rpittau | I've removed the block for the cycle highlights https://review.opendev.org/c/openstack/releases/+/943229 | 13:23 |
| rpittau | please review it ASAP in case we want to add/remove something :) | 13:23 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [CI] metal3 integration: reduce total time between node check https://review.opendev.org/c/openstack/ironic/+/943595 | 13:23 |
| opendevreview | Kaifeng Wang proposed openstack/ironic master: Add servicing to stuck states on recovery https://review.opendev.org/c/openstack/ironic/+/941017 | 13:45 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Use bigger partition as work dir for metal3 job https://review.opendev.org/c/openstack/ironic/+/943374 | 13:49 |
| TheJulia | Good Morning! | 14:00 |
| TheJulia | Are we meeting this morning? | 14:00 |
| TheJulia | err. | 14:00 |
| TheJulia | that is in an hour isent it | 14:00 |
| TheJulia | yup, in an hour | 14:01 |
| opendevreview | Merged openstack/bifrost stable/2024.1: pip: Use SETUPTOOLS_USE_STDLIB if python < 3.12 https://review.opendev.org/c/openstack/bifrost/+/943180 | 14:02 |
| TheJulia | rpittau: lgtm | 14:08 |
| rpittau | thanks! | 14:08 |
| *** dking is now known as Guest11037 | 14:31 | |
| opendevreview | Merged openstack/python-ironic-inspector-client stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/943539 | 14:38 |
| opendevreview | Merged openstack/python-ironicclient stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/python-ironicclient/+/943542 | 14:39 |
| opendevreview | Merged openstack/python-ironicclient stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/python-ironicclient/+/943543 | 14:50 |
| opendevreview | Verification of a change to openstack/python-ironic-inspector-client stable/2025.1 failed: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/943540 | 14:57 |
| TheJulia | Good morning! | 15:02 |
| TheJulia | #startmeeting ironic | 15:02 |
| opendevmeet | Meeting started Mon Mar 10 15:02:24 2025 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:02 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:02 |
| * TheJulia gets out the crickets | 15:02 | |
| rpittau | #st... | 15:02 |
| rpittau | ok | 15:02 |
| rpittau | :) | 15:02 |
| TheJulia | #chair rpittau | 15:02 |
| opendevmeet | Current chairs: TheJulia rpittau | 15:02 |
| rpittau | o/ | 15:02 |
| TheJulia | o/ | 15:02 |
| kubajj | o/ | 15:02 |
| frickler | \o | 15:02 |
| rpittau | Hello everyone! | 15:03 |
| rpittau | Welcome to our weekly meeting! | 15:03 |
| rpittau | The meeting agenda can be found here: | 15:03 |
| rpittau | https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_March_10.2C_2025 | 15:03 |
| rpittau | not a lot of things for today, I'l just start with the announcements | 15:03 |
| rpittau | #topic Announcements/Reminders | 15:03 |
| cid | o/ | 15:04 |
| rpittau | #info Standing reminder to review patches tagged ironic-week-prio and to hashtag any patches ready for review with ironic-week-prio: | 15:04 |
| rpittau | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:04 |
| rpittau | ooook there are some patches to review, I'll get to them after the meeting | 15:04 |
| rpittau | #info 2025.1 Epoxy Release Schedule | 15:05 |
| rpittau | #link https://releases.openstack.org/epoxy/schedule.html | 15:05 |
| rpittau | 3 weeks left!!! | 15:05 |
| TheJulia | Have we prepared our releases? | 15:05 |
| rpittau | we're going to start seeing releases very soon | 15:05 |
| TheJulia | ok | 15:05 |
| rpittau | we're missing just a couple actually, ironic, ipa and such | 15:06 |
| rpittau | waiting for the last patches to merge | 15:06 |
| rpittau | so please if you have patches that need to be included in the release upload them ASAP! | 15:06 |
| iurygregory | o/ | 15:07 |
| rpittau | #info Flamingo PTG will take place place April 7-11, 2025! | 15:07 |
| rpittau | PTG etherpad | 15:07 |
| rpittau | #link https://etherpad.opendev.org/p/ironic-ptg-april-2025 | 15:07 |
| rpittau | I'll start reserving days/times, then we can adjust based on topics and cross project needs | 15:08 |
| iurygregory | if you need help let me know | 15:08 |
| rpittau | thanks iurygregory :) | 15:08 |
| rpittau | anything else to announce/remind today? | 15:08 |
| rpittau | ok moving on! | 15:09 |
| rpittau | #topic Discussions | 15:09 |
| TheJulia | do we have a list of patches we're waiting on? | 15:09 |
| rpittau | yep the one in the ironic-week-prio list | 15:10 |
| rpittau | I have only one topic for today, which is the last chance to comment on the cycle highlights :) | 15:11 |
| rpittau | #link https://review.opendev.org/c/openstack/releases/+/943229 | 15:11 |
| rpittau | thanks TheJulia for the review! | 15:11 |
| cid | I have one | 15:11 |
| * cid rules | 15:11 | |
| rpittau | cid: shoot! | 15:11 |
| TheJulia | Introspection rules? ;) | 15:12 |
| cid | Yep. ^^ Re: TheJulia. I will really love to get inspection rules to the finish line | 15:12 |
| iurygregory | how we will rule the world? :D | 15:12 |
| cid | :) | 15:13 |
| TheJulia | Is everything tagged? What explicitly is outstanding? | 15:13 |
| cid | Everything is tagged, the API is waiting for the follow-up change | 15:13 |
| cid | ... to merge | 15:13 |
| cid | https://review.opendev.org/c/openstack/ironic/+/942112 | 15:13 |
| cid | https://review.opendev.org/c/openstack/ironic/+/939217 | 15:14 |
| * dtantsur planning to get to these early this week | 15:14 | |
| cid | Appreciated ++ | 15:15 |
| rpittau | cid: if they're tagged I'll get to that for sure tomorrow at the latest | 15:15 |
| priteau | May I ask, what about the NGS patches in ironic-week-prio? Are they going to be merged only for Flamingo? | 15:15 |
| cid | tks, rpittau | 15:15 |
| rpittau | priteau: we'll do our best to review them before the release | 15:15 |
| rpittau | I think I actually reviewed some of them already | 15:16 |
| frickler | what about the last two? patches shouldn't have prio while they're W-1 or in merge conflict for months? | 15:17 |
| rpittau | the last two are automatically excluued if they're not udpated | 15:17 |
| TheJulia | priteau: are there specific patches you have in mind which we can focus on? | 15:18 |
| frickler | well they won't get merged but I was thinking maybe also remove the hashtag to avoid the distraction | 15:19 |
| rpittau | frickler: of course, we do some cleaning usually, but some patches can slip from view :) | 15:20 |
| frickler | ok. I have another question then regarding https://bugs.launchpad.net/ironic/+bug/2101021 , I'm not sure whether that is only a CI issue or a real regression maybe? stevebaker[m] starting testing but no success so far | 15:21 |
| priteau | TheJulia: It's the VLAN support patches. They have a +2 from rpittau already. | 15:23 |
| priteau | i.e. https://review.opendev.org/c/openstack/networking-generic-switch/+/928490 and https://review.opendev.org/c/openstack/networking-generic-switch/+/940966 | 15:23 |
| priteau | But one of them depends on https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/941013 which is also active | 15:24 |
| *** Guest11037 is now known as dking | 15:27 | |
| TheJulia | You can't realistically depends-on a tempest chane | 15:27 |
| TheJulia | change | 15:27 |
| TheJulia | That signifies the change breaks something then | 15:27 |
| TheJulia | which is.. bad | 15:27 |
| rpittau | I think it's just adding tests | 15:27 |
| TheJulia | likely, if we can check and decouple it would be best | 15:28 |
| TheJulia | I suspect mnasiadka is likely on a plane today or was yesterday | 15:28 |
| rpittau | alright, anything else to discuss today? | 15:30 |
| rpittau | then that's all folks! | 15:32 |
| rpittau | thanks everyone! | 15:32 |
| rpittau | #endmeeting | 15:32 |
| opendevmeet | Meeting ended Mon Mar 10 15:32:34 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:32 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-03-10-15.02.html | 15:32 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-03-10-15.02.txt | 15:32 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2025/ironic.2025-03-10-15.02.log.html | 15:32 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Use bigger partition as work dir for metal3 job https://review.opendev.org/c/openstack/ironic/+/943374 | 15:32 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Use bigger partition as work dir for metal3 job https://review.opendev.org/c/openstack/ironic/+/943374 | 15:33 |
| priteau | TheJulia: the test being added to the tempest plugin is disabled by default. It gets activated only by changes to the devstack plugin: https://review.opendev.org/c/openstack/networking-generic-switch/+/928490/31/devstack/plugin.sh | 15:34 |
| TheJulia | doesn't matter, the aspects need to be disjointed | 15:34 |
| TheJulia | making anything dependent upon tempest is wrong and increases difficulty to land | 15:35 |
| TheJulia | in any event, I'm looking at the base change which adds the relation now, If I'm good with it, I'll remove depends-on | 15:35 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add vlan aware VMs support https://review.opendev.org/c/openstack/networking-generic-switch/+/928490 | 15:42 |
| opendevreview | Vasyl Saienko proposed openstack/networking-generic-switch master: Adds vlan aware VMs support for Cumulus NVUE and DellOS10 https://review.opendev.org/c/openstack/networking-generic-switch/+/940966 | 15:44 |
| priteau | Thank you TheJulia | 15:55 |
| TheJulia | JayF would appreciate reviews on: https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/927545 | 16:07 |
| JayF | TheJulia: that's my patch 😃 | 16:34 |
| opendevreview | cid proposed openstack/ironic master: Follow-up: Apply Inspection Rules https://review.opendev.org/c/openstack/ironic/+/942112 | 16:34 |
| opendevreview | cid proposed openstack/ironic master: API/Testing: Inspection rules migration https://review.opendev.org/c/openstack/ironic/+/939217 | 16:34 |
| dtantsur | | last_error | Failed to get power state for node 31a2bc92-e4e5-4ce8-ba0a-55350b0689b0. Error: maximum recursion depth exceeded while | | 16:43 |
| dtantsur | | | calling a Python object | 16:43 |
| dtantsur | Very-very not wow | 16:43 |
| dtantsur | and I have a serious concern it's actually eventlet-related | 16:44 |
| dtantsur | or, hmm, we have an infinite recursion in sushy | 16:45 |
| dtantsur | well, lovely, we enter an infinite recursion on invalid credentials | 16:47 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent-builder master: WIP Add Podman DIB for container-based cleaning https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/943348 | 16:54 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent-builder master: WIP Add Podman DIB for container-based cleaning https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/943348 | 16:56 |
| opendevreview | Verification of a change to openstack/ironic-tempest-plugin master failed: Add tests to validate trunks https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/941013 | 17:31 |
| opendevreview | Merged openstack/ironic master: Remove multinancy page https://review.opendev.org/c/openstack/ironic/+/942306 | 17:37 |
| cardoe | Looking for reviews on https://review.opendev.org/c/openstack/ironic/+/942496 it’ll need to be back ported to 2024.2 and 2025.1 if that’s been branched. | 17:42 |
| cardoe | I’m traveling this week so I’m on and off line a bunch. | 17:43 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent-builder master: WIP Add Podman DIB for container-based cleaning https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/943348 | 17:48 |
| opendevreview | Merged openstack/networking-generic-switch master: Configuration error handling for non-Layer-2 ports https://review.opendev.org/c/openstack/networking-generic-switch/+/943865 | 17:51 |
| TheJulia | dtantsur: I take it forever invalid credentials or some other fun? | 17:55 |
| cardoe | I’m good with cid’s rules stuff. I think as we play with it we’ll end up wanting to revise the spec a little bit and thus the implementation. I know dtantsur wanted to review things so I’ve held off waiting on him. | 18:43 |
| cardoe | I feel like cid implemented the spec. | 18:44 |
| opendevreview | Merged openstack/ironic master: Filter physnets when ports are pre-allocated to a segment https://review.opendev.org/c/openstack/ironic/+/941343 | 18:44 |
| opendevreview | cid proposed openstack/ironic master: Follow-up: Apply Inspection Rules https://review.opendev.org/c/openstack/ironic/+/942112 | 19:03 |
| opendevreview | cid proposed openstack/ironic master: API/Testing: Inspection rules migration https://review.opendev.org/c/openstack/ironic/+/939217 | 19:03 |
| opendevreview | cid proposed openstack/ironic master: Remove IPA build on cleanup https://review.opendev.org/c/openstack/ironic/+/943987 | 19:03 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Add vlan aware VMs support https://review.opendev.org/c/openstack/networking-generic-switch/+/928490 | 21:43 |
| opendevreview | Steve Baker proposed openstack/ironic master: DMN disable ir-novnc in ironic-standalone-redfish https://review.opendev.org/c/openstack/ironic/+/943871 | 21:45 |
| opendevreview | Verification of a change to openstack/ironic-tempest-plugin master failed: Add tests to validate trunks https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/941013 | 22:10 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!