Wednesday, 2025-06-25

opendevreviewDoug Goldstein proposed openstack/ironic master: inspection: define inventory shape with TypedDict  https://review.opendev.org/c/openstack/ironic/+/94486800:18
opendevreviewIury Gregory Melo Ferreira proposed openstack/sushy master: Don't fail to eject vmedia in case there is no vmedia  https://review.opendev.org/c/openstack/sushy/+/95273002:14
opendevreviewDoug Goldstein proposed openstack/ironic master: inspection: define inventory shape with dataclasses  https://review.opendev.org/c/openstack/ironic/+/94486804:26
opendevreviewDoug Goldstein proposed openstack/ironic master: fix port vif attach with networks with dynamic segments  https://review.opendev.org/c/openstack/ironic/+/95216805:01
rpittaugood morning ironic! o/06:28
rpittaucardoe: re py3.9 and up: we're still testing with python 3.9, so we should be good for now06:30
opendevreviewAbhishek Bongale proposed openstack/ironic-tempest-plugin master: WIP Add Tempest tests for inspection rules in Ironic  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/95176108:21
abongaleGood morning Ironic!08:23
opendevreviewcid proposed openstack/sushy master: Add retry logic to connection failures  https://review.opendev.org/c/openstack/sushy/+/95324808:26
opendevreviewAbhishek Bongale proposed openstack/ironic-tempest-plugin master: WIP Add Tempest tests for inspection rules in Ironic  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/95176109:50
dtantsurguilhermesp: it does not look like my patch is applied09:55
dtantsur> raid_utils.create_raid_device(index, logical_disk)09:55
dtantsurmy patch adds the 3rd argument there, it's not in your traceback09:55
dtantsurcardoe: I'm totally fine with 3.9+ at this point09:57
stephenfincid: fyi https://bugs.launchpad.net/ironic/+bug/211533210:47
stephenfincid: spotted while working on https://review.opendev.org/c/openstack/openstacksdk/+/942705/ The ironic job wasn't voting so we missed the fact that quite a few of the tests in https://review.opendev.org/c/openstack/openstacksdk/+/945700 weren't working. We live and learn10:49
cid👀10:49
cidWe do live and learn. I will check that out, hopefully before the end of the week.10:51
cidThanks, stephenfin.10:51
stephenfinnw10:51
iurygregorygood morning ironic11:03
opendevreviewAbhishek Bongale proposed openstack/ironic-tempest-plugin master: WIP Add Tempest tests for inspection rules in Ironic  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/95176111:07
ContinuityHey Ironic11:48
ContinuityAny idea what ironic-neutron-agent-member-manager-pool queue is used for11:48
ContinuityIt seems to have a lot of messages which are not being consumed11:48
ContinuityWe have multiple of them, all with lots of messages in them11:51
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent-builder master: [WIP] Build ironic-python-agent DIB image with CentOS Stream 10  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/95096511:52
iurygregoryContinuity, I've found https://storyboard.openstack.org/#!/story/2004938 (seems similar to what you are describing), but it was fixed already..12:00
opendevreviewcid proposed openstack/ironic master: WIP: Migrate conductor threads to a thread pool  https://review.opendev.org/c/openstack/ironic/+/95293912:18
guilhermespdtantsur: yep my bad, i did a fetch instead of a pull in master branch. I just ran a clean here and looks like it completed. I will inspect logs soon and deploy a system and provide you some feedback :) 13:15
dtantsur++13:15
TheJuliaContinuity: Interesting.... Once I've caffinated my brain I'll take a look13:22
* TheJulia caffinates13:22
ContinuityIurygregory: thanks, yeah I was reading that as well13:23
TheJuliaoh wow, yeah13:23
ContinuityTheJulia: sounds good. We came across it when investigating high disk usage. We are on 2023.213:23
TheJuliaContinuity: Are they continuing to grow?13:48
guilhermesphttps://www.irccloud.com/pastebin/8k8TQf9x/13:52
guilhermespdtantsur: bingo? :P 13:53
dtantsur\o/13:55
guilhermespgood stuff! thank you for that 13:56
opendevreviewVerification of a change to openstack/networking-generic-switch master failed: Replace LOG.warn  https://review.opendev.org/c/openstack/networking-generic-switch/+/95305614:02
dtantsurnp, I'll finish the patch when I have a minute14:03
abongaleI'm writing a Tempest test for the *Inspection Rule API* to verify updates to `actions`, `conditions`, and `priority`.14:19
abongaleThe test is currently failing due to a mismatch in the `actions` field during assertion. Specifically, the returned patched inspection rule `actions` include an unexpected extra variable: `loop`.14:19
abongaleI guess it's wrong? . AFAIK, `loop` should be a valid variable since we're following the Ansible pattern for both `actions` and `conditions`14:19
abongaleRef: Test https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/951761/30/ironic_tempest_plugin/tests/api/admin/test_inspection_rules.py#12714:19
abongalehttps://paste.openstack.org/show/b55InrObK3MgxI2iJmm2/14:20
abongaletraceback call ^^14:20
abongaleI believe we might have to update the `convert_actions` method and make it similar to `convert_conditions` https://paste.openstack.org/show/bXGGzI9wRSl9J2GdjB5S/14:21
TheJuliaContinuity: I suspect that they may be from a prior upgrade or state, but I'm also looking at the code wondering if it is just still happening based upon the state of the code, so some orientation would be helpful.14:23
TheJuliaabongale: so to reframe it, your not expecting loop to be added to the field by the API, but it ?is? getting added in it seems14:25
TheJuliaabongale: offhand, it seems like you might have found the issue in the ironic code, cid any thoughts?14:26
abongaleTheJulia : I am not expecting to update the actions with loop field but returned updated response have a loop variable in the actions.14:30
TheJuliaAre there any internal tests which place the expectation? I guess I'm wondering where the expectation came from which is why I'm hoping cid can chime in14:31
opendevreviewMerged openstack/ironic-specs master: Trait based port selection and dynamic portgroups  https://review.opendev.org/c/openstack/ironic-specs/+/94564214:31
cidabongale, TheJulia, I remember there was some special handling of a field. It's likely the 'loop' field that's added during rule creation.14:33
cidI will need to look at the code to refresh my memory14:33
* cid Goes looking...14:35
cidabongale, I think it makes sense to update convert_actions, clearly the spec does not indicate it as a required field.14:45
cidI could push that update now, unless you want to take care of it.14:45
abongale+++, I will create PR14:46
abongaleI can make changes. Thank you @cid14:47
cidAlright. More thanks to you, abongale.14:49
cidBy the way, stephenfin found something that I'm picking up right about now. Maybe I will just push fix your finding alongside.14:49
ContinuityThejulia: I shall investigate and come back to you...14:52
TheJuliaContinuity: Thanks!14:52
ContinuityTheJulia: yes they do seem to be growing14:55
TheJuliaokay, yeah, that kind of lines up with what I'd expect based upon the code14:55
Continuityironic-neutron-agent-member-manager-pool-c9d73d2d-3445-454d-a633-106e114c1a92   479251314:55
Continuity1 min14:55
Continuityironic-neutron-agent-member-manager-pool-c9d73d2d-3445-454d-a633-106e114c1a92   479251914:55
ContinuityDo you want me to throw a bug into launchpad14:56
TheJuliaoh, so c9d73d2d-3445-454d-a633-106e114c1a92  is the agent uuid?14:56
ContinuityYeah14:56
ContinuityThats the output from rabbitmqctl 14:56
TheJuliaYeah, go ahead and create a bug in launchpad then14:56
ContinuityWilco14:57
ContinuityDo you need any other detail 14:57
TheJuliaJust sort of the facts we've identified here14:58
TheJuliathe fact it is the agent uuid based entry which is getting recorded but not read/removed. I might need to get hjensas to chime in but it is clearly a bug we then have downstream as well which means we're also going to need to track it14:58
opendevreviewAbhishek Bongale proposed openstack/ironic master: fix: handle unexpected 'loop' in actions field  https://review.opendev.org/c/openstack/ironic/+/95332315:04
opendevreviewAbhishek Bongale proposed openstack/ironic master: fix: handle unexpected 'loop' in actions field  https://review.opendev.org/c/openstack/ironic/+/95332315:34
mnaserdtantsur: thanks for helping guilhermesp with the patch, can i help in finishing up that patch if you've got a bunch of stuff on your table, maybe if you put some guidance on whats needed for https://review.opendev.org/c/openstack/ironic-python-agent/+/953122 to land? :)15:47
opendevreviewMerged openstack/networking-generic-switch stable/2025.1: Cast numeric Netmiko kwargs to native types.  https://review.opendev.org/c/openstack/networking-generic-switch/+/95260015:56
opendevreviewMerged openstack/networking-generic-switch stable/2024.1: Cast numeric Netmiko kwargs to native types.  https://review.opendev.org/c/openstack/networking-generic-switch/+/95260215:56
opendevreviewMerged openstack/networking-generic-switch stable/2024.2: Cast numeric Netmiko kwargs to native types.  https://review.opendev.org/c/openstack/networking-generic-switch/+/95260115:56
dtantsurmnaser: oh, that would be nice! I think I've handled the unit tests, so it only needs a release note and check if the docs need any update (either in IPA itself or a follow-up in Ironic).16:08
mnaserdtantsur: ok cool, let me try and check that out16:09
dtantsurthanks!16:09
opendevreviewMohammed Naser proposed openstack/ironic-python-agent master: Fix software RAID creation on different physical devices  https://review.opendev.org/c/openstack/ironic-python-agent/+/95312216:23
mnaserdtantsur: ^ done =)16:23
dtantsur\o/16:24
guilhermesp++16:27
dtantsurrpittau: 2025 OpenInfra Summit Europe - Session Accepted! \o/16:36
opendevreviewHarald JensÃ¥s proposed openstack/ironic master: Stop using oslo.service graceful_shutdown_timeout  https://review.opendev.org/c/openstack/ironic/+/95333516:44
opendevreviewMaksim Malchuk proposed openstack/bifrost master: Do not pass empty values to instance_info  https://review.opendev.org/c/openstack/bifrost/+/95333616:48
ContinuityTheJulia: https://bugs.launchpad.net/networking-baremetal/+bug/2086303 looks like a copy of what we were seeing17:19
iurygregorydtantsur, rpittau congrats!17:24
TheJuliaContinuity: are you using kolla?17:25
iurygregorythe one I have with janders was also accepted :D17:25
TheJuliaCongrats guys17:25
iurygregorynow into the complicated part, budget... :D 17:25
dtantsurcongrats too!17:38
iurygregoryty17:42
opendevreviewDoug Goldstein proposed openstack/ironic master: fix port vif attach with networks with dynamic segments  https://review.opendev.org/c/openstack/ironic/+/95216817:53
FreemanBoss[m]Congratulations rpittau18:15
opendevreviewDoug Goldstein proposed openstack/ironic master: inspection: define inventory shape with dataclasses  https://review.opendev.org/c/openstack/ironic/+/94486818:32
abongalecongrats rpittau!!18:33
opendevreviewMerged openstack/networking-generic-switch master: Clean up add_subports_on_trunk, del_subports_on_trunk  https://review.opendev.org/c/openstack/networking-generic-switch/+/94983318:36
cardoeSo TheJulia you might not like my inventory shape patch. I know dtantsur won't like it.18:39
cardoeHoping someone might have some suggestions.18:39
cardoeI've found a bunch of code paths that the fields in the inventory matter cause we read them with code like inventory["some_key_here"]18:40
cardoeNow I'm enforcing a shape... whether its the right shape I dunno18:41
cardoeBut if it doesn't match it treats it like the box has no inventory.18:41
cardoeWhich is certainly better than conductor crashing I think.18:41
TheJuliacardoe: cool cool, I'll try to take a look tomorrow at this point, I'm sort of heads down on some eventlet stuff at the moment18:42
cardoeNo rush cause it's not 100% ready.18:42
cardoeThere's at least 5 shapes I've found. ironic-inspector migrated data. IPA web documented fields. IPA code itself. IPA's example for implementing your own hardware manager. And lastly redfish inspection.18:44
cardoeHaseeb on my team is fixing up redfish fields because we want to have inspector hooks and rules run on redfish still.18:45
cardoeI figure once we can agree on the shape, I can also add JSON schema validation into the API endpoint that IPA hits to submit the data.18:47
opendevreviewHarald JensÃ¥s proposed openstack/ironic master: Stop using oslo.service graceful_shutdown_timeout  https://review.opendev.org/c/openstack/ironic/+/95333518:50
opendevreviewDoug Goldstein proposed openstack/ironic master: fix port vif attach with networks with dynamic segments  https://review.opendev.org/c/openstack/ironic/+/95216818:58
TheJuliacid: regarding eventlet removal, any thoughts on starting to peel it out from services in general. I ask because the ironic-novncproxy is a bit of an all or nothing removal, seems to work fine if we remove the patching, but there is no clean intermediate runtime state short of adding logic to handle the state where it is still being patched to use eventlet explicitly.19:04
TheJuliastevebaker[m]: ^^^19:04
TheJuliaI guess I'm comfortable doing so with ironic-novncproxy in general since it has minimal (none, yet?) usage given its such a new feature19:06
TheJuliabut that means changing the other pattern and does sort of also line us up for a per-service removal, once we're happy with removing it from the API (and it does seem to work, FWIW as well... my dev machine only monkey_patches the conductor process.)19:06
stevebaker[m]Is there a way of detecting if eventlet is being used without importing eventlet? 19:07
TheJuliathe pattern in the libraries is I think imports, but not monkey patches19:08
TheJuliaand then there is a method to determine if it is patched, or not19:08
TheJuliaI'm strongly thinking just removing __init__.py and moving the contents into each service for now19:08
opendevreviewDoug Goldstein proposed openstack/ironic master: fix port vif attach with networks with dynamic segments  https://review.opendev.org/c/openstack/ironic/+/95216819:11
TheJuliasteve has reminded me downstream of the single process launcher19:14
* TheJulia hunts for it19:18
* cid sees that single process launcher might be benefitting from the module-level eventlet patching, but feels it might not require any major updates.19:29
cidTheJulia, I think we literally just rip out the module-level eventlet patching, like we're doing with the threadpool change. Looking through the codebase, novncproxy might need some independent work, but I honestly don't know how to test it. Aside from that, are there even any other services?19:29
TheJuliaI'm working on it right now, singleprocess is the only hurdle so I think it will have some conditional code until we can remove the generalized monkey patch invokation19:30
TheJuliainvocation19:30
cidCool, tks19:34
TheJuliamy goal was to rip it out, but... looks like we can't just go directly to remove so... one step at a time :)19:34
TheJuliaokay, that seems to work, now to just re-test by turning eventlet back on :)19:44
opendevreviewDoug Goldstein proposed openstack/ironic master: fix port vif attach with networks with dynamic segments  https://review.opendev.org/c/openstack/ironic/+/95216819:51
opendevreviewJulia Kreger proposed openstack/ironic master: Change the novncproxy service code to handle eventlet-lessness  https://review.opendev.org/c/openstack/ironic/+/95335120:21
TheJuliastevebaker[m]: cid: ^^^20:36
cid\o/20:38
opendevreviewAdam McArthur proposed openstack/ironic master: Update firmware and shard schemas to require 'created_at' and 'updated_at' fields  https://review.opendev.org/c/openstack/ironic/+/95335220:49
adamcarthur5TheJulia appreciate your early review. Jay was pretty keen for me to get this out in any form before he left. Its very pre-baked so give me some time before your next review? I.e. I want to make a tonne of changes21:06
TheJuliaCool cool21:07
adamcarthur5I'll get in touch via IRC when its ready. Your comments will help me greatly in shaping it so appreciate it a lot21:09
opendevreviewAdam McArthur proposed openstack/ironic-specs master: Add emergency bulk operations  https://review.opendev.org/c/openstack/ironic-specs/+/95253321:11
iurygregoryif anyone has some time https://review.opendev.org/c/openstack/sushy/+/952730 o/21:33

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