| opendevreview | Merged openstack/ironic stable/2026.1: Portgroup shard filter bypasses scope to project https://review.opendev.org/c/openstack/ironic/+/999762 | 00:25 |
|---|---|---|
| opendevreview | Merged openstack/ironic bugfix/37.0: Portgroup shard filter bypasses scope to project https://review.opendev.org/c/openstack/ironic/+/999764 | 00:25 |
| opendevreview | Merged openstack/ironic bugfix/38.0: Portgroup shard filter bypasses scope to project https://review.opendev.org/c/openstack/ironic/+/999763 | 00:25 |
| opendevreview | Verification of a change to openstack/ironic bugfix/34.0 failed: Portgroup shard filter bypasses scope to project https://review.opendev.org/c/openstack/ironic/+/999765 | 00:25 |
| rpittau | dtantsur: looks like the fix made BMO job happy :) | 06:42 |
| dtantsur | rpittau: nice! have you checked the logs? how much time do we lose in the end? | 07:31 |
| rpittau | I haven't checked the logs yet | 07:36 |
| dtantsur | (the revert of autodetect did not help, so your patch is all we have, I guess) | 07:37 |
| rpittau | fingers crossed | 07:38 |
| dtantsur | Retry after 7 seconds, not too bad | 07:40 |
| dtantsur | rpittau: can you address a small comment in the PR please? I don't want someone to remove the fix assuming it's accidental | 07:40 |
| rpittau | yes, I was going to do add that now actually | 07:41 |
| dtantsur | Thanks for looking into the issue! It was a wild ride | 07:42 |
| dtantsur | once you're done, anything blocking W+1 of https://review.opendev.org/c/openstack/ironic/+/999718? | 07:44 |
| rpittau | nope, I just didn't see Doug already +2ed that :D | 07:44 |
| rpittau | added +W | 07:44 |
| dtantsur | thx! | 07:46 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix fast-track deployment failure with autodetect deploy interface https://review.opendev.org/c/openstack/ironic/+/999699 | 07:50 |
| rpittau | comment added | 07:50 |
| dtantsur | thx! | 07:57 |
| dtantsur | https://review.opendev.org/c/openstack/ironic/+/997998 could you some input as well | 07:57 |
| opendevreview | Merged openstack/ironic master: Fix Redfish inspection without Storage API https://review.opendev.org/c/openstack/ironic/+/999718 | 08:37 |
| opendevreview | Yorick proposed openstack/sushy master: system: guard HttpBootUri writes against iDRAC 10 removal https://review.opendev.org/c/openstack/sushy/+/999743 | 09:24 |
| blanson[m] | Hello ironic ! I have a bit of an issue using the hardware raid feature with redfish on HPE DL365 Gen11 with megaraid controllers. These controllers do not accept a size parameter and will throw iLO.2.30.UnsupportedOperation. I did the mess of a fix locally that just retries without the parameter, but I'm wondering if you'd have a better idea that I could put upstream. | 10:31 |
| dtantsur | blanson[m]: hmm, do we send the size unconditionally or is it based on API input? | 10:59 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Refactor firmware update SimpleUpdate submission and error dispatch https://review.opendev.org/c/openstack/ironic/+/999230 | 11:21 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Unify non-BMC firmware updates under batched state machine https://review.opendev.org/c/openstack/ironic/+/999720 | 11:21 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Document batched firmware updates and add release note https://review.opendev.org/c/openstack/ironic/+/999721 | 11:21 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/2026.1: Fix Redfish inspection without Storage API https://review.opendev.org/c/openstack/ironic/+/999809 | 11:50 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/38.0: Fix Redfish inspection without Storage API https://review.opendev.org/c/openstack/ironic/+/999810 | 11:50 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/37.0: Fix Redfish inspection without Storage API https://review.opendev.org/c/openstack/ironic/+/999811 | 11:51 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Unify non-BMC firmware updates under batched state machine https://review.opendev.org/c/openstack/ironic/+/999720 | 11:54 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Document batched firmware updates and add release note https://review.opendev.org/c/openstack/ironic/+/999721 | 11:54 |
| blanson[m] | dtantsur: took me a little while to remember the whole thing but essentially: those megaraid controllers only support using the whole disk and don't take a size_gb value for the config. but using max we have to pass the exact disks, which we do not know because we use agent inspection for other reasons (getting switches lldp infos and such). I didn't find a way to surface the redfish disk infos in the inventory this way, so we | 11:59 |
| blanson[m] | have to pass the size_gb parameter and not MAX, which then breaks the megaraid controller that does not want it | 11:59 |
| blanson[m] | I believe that's our whole situation | 11:59 |
| dtantsur | blanson[m]: would you be fine with max using all available disks? | 12:04 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Unify non-BMC firmware updates under batched state machine https://review.opendev.org/c/openstack/ironic/+/999720 | 12:27 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Document batched firmware updates and add release note https://review.opendev.org/c/openstack/ironic/+/999721 | 12:27 |
| blanson[m] | yeah, usually we only plug in 2 drives on these controllers and we expect it to use everything to make a raid 1 | 12:33 |
| TheJulia | wow, I just had someone ask me about using conductor groups downstream... for 10 nodes. :( | 13:36 |
| TheJulia | sounds like the validation really just needs to maybe retry "popping" out the size_gb parameter? | 13:38 |
| TheJulia | (this also feels semi-known why MAX even came into being, but it sounds like something changed | 13:38 |
| TheJulia | last call on https://review.opendev.org/c/openstack/ironic-specs/+/997653 | 13:46 |
| dtantsur | blanson[m], TheJulia, I imagine we could relax the restriction and allow size=max without holder disks to mean "take all disks" | 13:55 |
| dtantsur | TheJulia: also, could you check https://review.opendev.org/c/openstack/ironic/+/999699 please? it's blocking BMO | 13:57 |
| dtantsur | I'm +1 on 997653 simply because I cannot find time for a full detailed read. But don't want for me in any case. | 13:58 |
| TheJulia | dtantsur: yeah, that is also reasonable I think | 14:01 |
| TheJulia | dtantsur: ack, okay | 14:01 |
| TheJulia | dtantsur: I'll make another cup of coffe and check that change out | 14:01 |
| blanson[m] | dtantsur: that was my initial though for it and then I was like "there is probbly a good reason that I don't know about for it not to be done this way", but if that sounds like a viable solution I'm all for it | 14:04 |
| dtantsur | blanson[m]: I don't remember the reason, but we might have been simple cautious | 14:05 |
| dtantsur | y'know, it's easier to relax a restriction than to introduce one.. | 14:05 |
| blanson[m] | gotchu, thanks for looking into it ! I will try to find some time to propose something upstream then you can choose if it meets with what you had in mind :) | 14:07 |
| dtantsur | nice, thanks! | 14:09 |
| TheJulia | dtantsur: I approved and workflowed that change. It confused me greatly until I re-read the commit message. | 14:23 |
| * dtantsur nods | 14:59 | |
| dtantsur | Fast track is a bit of a minefield | 14:59 |
| dtantsur | thank you! | 14:59 |
| opendevreview | Verification of a change to openstack/ironic-specs master failed: Advance connectivity/driver modeling in NGS https://review.opendev.org/c/openstack/ironic-specs/+/997653 | 15:24 |
| TheJulia | dtantsur: unfortunately, yeah. See movie: dodgeball | 15:31 |
| JayF | last call to get a review on [OSSA-2026-033] Ironic portgroup scoping https://review.opendev.org/c/openstack/ossa/+/999767 before it's advisoried | 15:32 |
| opendevreview | Merged openstack/ironic master: Fix fast-track deployment failure with autodetect deploy interface https://review.opendev.org/c/openstack/ironic/+/999699 | 15:34 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent bugfix/12.0: Update .gitreview for bugfix/12.0 https://review.opendev.org/c/openstack/ironic-python-agent/+/999860 | 15:35 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic bugfix/38.0: Fix fast-track deployment failure with autodetect deploy interface https://review.opendev.org/c/openstack/ironic/+/999870 | 15:55 |
| opendevreview | Merged openstack/ironic-python-agent bugfix/12.0: Update .gitreview for bugfix/12.0 https://review.opendev.org/c/openstack/ironic-python-agent/+/999860 | 16:12 |
| *** Unknown123 is now known as Mike-- | 16:28 | |
| opendevreview | cid proposed openstack/ironic-python-agent master: Ignore the transport prefix when matching allowed containers https://review.opendev.org/c/openstack/ironic-python-agent/+/998381 | 16:42 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Drop the unused container_conf_file option https://review.opendev.org/c/openstack/ironic-python-agent/+/998382 | 16:42 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Rename container_clean_step to generic_container_step https://review.opendev.org/c/openstack/ironic-python-agent/+/998383 | 16:42 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Verify registry TLS by default https://review.opendev.org/c/openstack/ironic-python-agent/+/999725 | 16:42 |
| opendevreview | Merged openstack/ironic-specs master: Advance connectivity/driver modeling in NGS https://review.opendev.org/c/openstack/ironic-specs/+/997653 | 17:48 |
| opendevreview | Serhii Ivanov proposed openstack/sushy-tools master: Add empty BootOptions collection stub and Boot navigation links https://review.opendev.org/c/openstack/sushy-tools/+/991910 | 18:54 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily at 20:00 UTC (an hour from now) while we restart for a patch upgrade, but should return within a few minutes. | 19:02 | |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will not be restarted at 20:00 UTC after all, due to unanticipated image build regressions, and will be rescheduled to a later time/date. | 19:13 | |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add [api] enable_ramdisk_endpoints config option https://review.opendev.org/c/openstack/ironic/+/999897 | 19:44 |
| TheJulia | JayF: ^^^ may be of interest to you | 19:44 |
| JayF | You gonna write up the OSSN to pair with it? | 19:51 |
| TheJulia | I guess I can if you want | 19:58 |
| JayF | I just don't wanna dupe work | 19:59 |
| JayF | 90% of the work for the OSSN will be copy+paste your doc edits into a different format lol | 19:59 |
| JayF | just let me know if you want me to knock it out | 19:59 |
| JayF | are we gonna backport 999897? | 20:00 |
| JayF | Technically it's a feature but I think it's easily justifible | 20:00 |
| TheJulia | yeah, that was why I was going to raise it | 20:01 |
| TheJulia | to your attention at least | 20:01 |
| TheJulia | We can, I don't see why not | 20:01 |
| opendevreview | Adam McArthur proposed openstack/ironic-specs master: Proposal for a Bulk Node Operations API https://review.opendev.org/c/openstack/ironic-specs/+/983183 | 20:01 |
| TheJulia | Yeah, its a justifable backport where we just change it to a "fix" ;) | 20:01 |
| cardoe | who was it that picked up openstack-exporter? | 20:16 |
| JayF | adamcarthur5 from my team | 20:19 |
| JayF | He's in the channel #openstack-exporter | 20:19 |
| opendevreview | Serhii Ivanov proposed openstack/sushy-tools master: Add one-time boot via libvirt reboot listener (SUSHY_EMULATOR_BOOT_ONCE) https://review.opendev.org/c/openstack/sushy-tools/+/991912 | 21:05 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add image_server_auth_hosts to restrict credential scope https://review.opendev.org/c/openstack/ironic/+/999744 | 21:10 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Change image_server_auth_permit_unknown_hosts default to False https://review.opendev.org/c/openstack/ironic/+/999907 | 21:10 |
| TheJulia | calling it a day folks | 21:11 |
| JayF | o/ | 21:11 |
| cardoe | TheJulia: but there's so much more yaks that need shaving! | 21:20 |
| TheJulia | cardoe: spending half my day trying to figure out why I had an email saying "you have $4017 in 17 outstanding expenses"... meaning migraine | 21:43 |
| TheJulia | so yeah, the yak's are limited today | 21:43 |
| cardoe | I mean that sounds like yaks of a different color | 21:46 |
| TheJulia | business process automation yaks which required very large electric shavers [of doom] | 21:50 |
| cardoe | Would you guys be against me removing some behavior in inspection rules to make it more clear/declarative? | 21:51 |
| cardoe | Right now you can pass the values as a list OR a dict. | 21:51 |
| cardoe | So if you have an operation "eq" which is defined as eq(arg1, arg2). You can pass.. {"args": {"values": [1, 2]}} or {"args": [1, 2]} | 21:53 |
| cardoe | But here's the FUN part | 21:53 |
| cardoe | {"args": {"values": ["blue", "blue"]}} or {"args": [["blue", "blue"]]} | 21:54 |
| cardoe | notice the double / nested list? | 21:54 |
| cardoe | And my examples suck cause the string one is the case where it returns true | 21:55 |
| cardoe | Cause {"args": ["blue", "blue"]} would return False | 21:55 |
| cardoe | Because it compares "b" against "l" | 21:56 |
| cardoe | Just feels like JayF's gonna be writing another OSSN in the future with so many different syntaxes and magical data marshalling. | 21:57 |
| * TheJulia twitches and falls over | 21:58 | |
| cardoe | I'd change it to require the "values" key always. | 21:58 |
| TheJulia | file my thoughts under "just don't break things" | 21:58 |
| cardoe | I mean I suspect I'm the one using these things the most. | 21:58 |
| TheJulia | and "keep things working in the support matrix across services" | 21:58 |
| TheJulia | eh, your doing so quite aggressively, that is for sure | 21:59 |
| TheJulia | (and there is nothing wrong with that) | 21:59 |
| JayF | > file my thoughts under "just don't break things" | 21:59 |
| JayF | literally exactly what I was thinking | 21:59 |
| JayF | there's so much customization and inconsistency here fixing it to be sensible while navigating stability is ... hard | 22:00 |
| cardoe | Basically if input isn't a dict, we make a dict. We look at the operation and grab the fields that the operation takes and make those dictionary keys. | 22:00 |
| cardoe | We walk the list and assign the list values to each input parameter | 22:01 |
| cardoe | But eq is actually eq(*args, optional_arg=None) syntax | 22:02 |
| cardoe | So we further check if the FIRST argument isn't enumerable then pass it to *args but if it is enumerable set the parameters differently. | 22:02 |
| cardoe | Basically it's a nightmare of un-safety | 22:03 |
| TheJulia | so, I'm good with making consistent and cleaning up, but again, version matrix of agents to ironic services is the thing to navigate | 22:04 |
| TheJulia | that is if stuff needs to chagne | 22:04 |
| TheJulia | and if rules need to change themselves, a path | 22:04 |
| cardoe | I'm 100% gonna locally patch it to get rid of all the hammering the data into the correct shape and just require people pass data correctly | 22:04 |
| TheJulia | and wwarning/error/etc | 22:04 |
| TheJulia | the upgrade check being the path there | 22:04 |
| cardoe | Well here's the even funner case since the variable gets interpolated as well... the behavior can change based on the inspected data of the hardware if you have an interpolated variable first. | 22:05 |
| cardoe | We only shipped this in 2026.1 | 22:06 |
| cardoe | So it just feels better to tighten it up now and make an upgrade note. | 22:06 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: Interpolate values for comparison operators in inspection rules https://review.opendev.org/c/openstack/ironic/+/999910 | 22:09 |
| TheJulia | ++++ | 22:16 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!