| opendevreview | Jacob Anders proposed openstack/sushy master: Fix handling of BMCs that send duplicate ETag headers https://review.opendev.org/c/openstack/sushy/+/962554 | 00:09 |
|---|---|---|
| cardoe | TheJulia: Well that's the thing... most of the idrac-redfish driver is just quirks for the Dell. | 00:22 |
| TheJulia | Yeah, we had a time where stock redfish would work but not anymore sadly | 01:40 |
| TheJulia | So, someone has asked me about quota support. | 01:41 |
| TheJulia | Part of me wonders if we could just hammer it out quickly | 01:41 |
| TheJulia | Oh, cumulative action counters | 01:45 |
| TheJulia | That’s more complex | 01:45 |
| TheJulia | janders: seems reasonable to dedup | 02:58 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix BMC firmware update timeout issues https://review.opendev.org/c/openstack/ironic/+/962560 | 04:21 |
| janders | TheJulia ack, thank you! | 04:22 |
| janders | extended etag_handler() accordingly | 04:22 |
| janders | I've seen a few cases of it working around issues I had no awareness of, so while bit complex it's a good little thing | 04:23 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix power sync timeouts after BMC firmware update https://review.opendev.org/c/openstack/ironic/+/962560 | 04:26 |
| opendevreview | cid proposed openstack/ironic-python-agent-builder master: Remove support for Python 3.9 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/962490 | 06:12 |
| rpittau | good morning ironic! o/ | 06:34 |
| janders | hey rpittau o/ | 06:37 |
| rpittau | hey janders :) | 06:41 |
| rpittau | janders: about https://review.opendev.org/c/openstack/ironic/+/962560 I wonder if we should add a warning in the docs about the reboot wait timeout being at least 5 minutes, considering that 'wait' is a user facing parameter | 06:47 |
| opendevreview | Pierre Riteau proposed openstack/bifrost master: Add support for Rocky Linux 10 image download https://review.opendev.org/c/openstack/bifrost/+/962575 | 09:16 |
| opendevreview | Merged openstack/ironic-python-agent master: Fix skip block devices for RAID arrays https://review.opendev.org/c/openstack/ironic-python-agent/+/937342 | 09:17 |
| kubajj | rpittau: with backports, do I start with 2025.2 now? | 10:24 |
| *** ubuntu is now known as Guest28036 | 10:45 | |
| dtantsur | kubajj: 2025.2 is the newest branch, so presumably yes | 10:51 |
| * dtantsur does not know the context, to be clear | 10:52 | |
| kubajj | dtantsur: I was mostly asking because I am a bit confused since the release is tomorrow, but I assume that there has been a code freeze already | 10:52 |
| dtantsur | janders: omg. Yeah, I suspect we'll get these e-tags as a list from requests. | 10:53 |
| rpittau | kubajj: if you'rever in doubt check https://etherpad.opendev.org/p/IronicWhiteBoard#L58 | 11:08 |
| frickler | rpittau: stable/2023.2 should be gone for some time already? and "bugfix/31.0 (flamingo) supported until end of January 2025" should be 2026 rather? | 11:31 |
| rpittau | frickler: yep, thanks :) | 11:32 |
| priteau | Hello Ironic! I have a somewhat trivial patch for Bifrost ready for review: https://review.opendev.org/c/openstack/bifrost/+/962575 | 12:30 |
| opendevreview | Allain Legacy proposed openstack/ironic master: WIP: Add standalone networking service for ironic https://review.opendev.org/c/openstack/ironic/+/962493 | 13:22 |
| opendevreview | Allain Legacy proposed openstack/networking-generic-switch master: WIP: Add standalone driver interface https://review.opendev.org/c/openstack/networking-generic-switch/+/962037 | 13:33 |
| opendevreview | Allain Legacy proposed openstack/networking-generic-switch master: Add standalone driver interface https://review.opendev.org/c/openstack/networking-generic-switch/+/962037 | 13:33 |
| mostepha[m] | Hi Ironic o/ | 14:04 |
| mostepha[m] | I have been struggling with backports, e.g. https://review.opendev.org/c/openstack/ironic-python-agent/+/957909 | 14:04 |
| mostepha[m] | Does anybody have an idea about why the nodes timeout while trying to clean? | 14:04 |
| opendevreview | Morten Stephansen proposed openstack/ironic-python-agent stable/2025.2: Fix skip block devices for RAID arrays https://review.opendev.org/c/openstack/ironic-python-agent/+/962591 | 14:17 |
| opendevreview | Allain Legacy proposed openstack/bifrost master: Add support for OVS as a virtual switch https://review.opendev.org/c/openstack/bifrost/+/962038 | 14:22 |
| opendevreview | Allain Legacy proposed openstack/bifrost master: Add support for standalone ironic networking https://review.opendev.org/c/openstack/bifrost/+/962394 | 14:22 |
| TheJulia | iurygregory: I think it sort of makes sense to remove the version check, and explicitly log that a dell machine was detected, and the dell-isms should be selected. That sort of doesn't really break folks, we can likely backport that, and separately moving forward we can intentionally error out the boot interface call with an error. | 14:33 |
| iurygregory | TheJulia, ++ agree | 14:45 |
| TheJulia | Actually, we could put an upper bound in term of "known working" and log an error | 14:57 |
| mostepha[m] | I'm trying to backport this patch: https://review.opendev.org/c/openstack/ironic-python-agent/+/953122 | 14:58 |
| mostepha[m] | because another patch relies on it, but it gives me this error: "not Signed-off-by author/committer/uploader in message footer" | 14:58 |
| mostepha[m] | Should I sign it as the owner? (It feels a bit like committing fraud) | 14:58 |
| JayF | Signed-off-by / DCO is saying "I can attest this code is licensed right" | 14:58 |
| JayF | so basically if you took it from another one of our branches | 14:58 |
| JayF | you can be 100% sure it was apache 2.0 licensed and is safe | 14:58 |
| JayF | think of it more like a chain of custody | 14:58 |
| JayF | It's not fraud-y at all and is exactly the thing to do :D | 14:59 |
| TheJulia | JayF: ++ | 14:59 |
| opendevreview | Morten Stephansen proposed openstack/ironic-python-agent stable/2025.1: Fix software RAID creation on different physical devices https://review.opendev.org/c/openstack/ironic-python-agent/+/962594 | 15:01 |
| mostepha[m] | Thanks for the clarrification! :) | 15:02 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Trait Based Networking Filter Expression Parsing and Base Models https://review.opendev.org/c/openstack/ironic/+/961498 | 15:22 |
| opendevreview | Clif Houck proposed openstack/ironic master: Configuration file for Trait Based Networking https://review.opendev.org/c/openstack/ironic/+/962598 | 15:22 |
| TheJulia | alegacy: following up on https://review.opendev.org/c/openstack/ironic/+/960823 if you could check to see if I'm interpreting your impression properly, that would be helpful for me | 17:00 |
| TheJulia | alegacy: regarding https://review.opendev.org/c/openstack/ironic/+/962493 requirements issue, yes, you can propose a change to openstack/requirements, it just has to be present before we can merge it in a project. | 17:02 |
| TheJulia | alegacy: regarding your test tool, it actually seems useful, I'd just split it to another change at this point to make it easier to review the main change your trying to make | 17:03 |
| opendevreview | Allain Legacy proposed openstack/bifrost master: Add support for standalone ironic networking https://review.opendev.org/c/openstack/bifrost/+/962394 | 17:08 |
| opendevreview | Verification of a change to openstack/ironic stable/2025.1 failed: Fix cache image hard link between different file systems https://review.opendev.org/c/openstack/ironic/+/962328 | 17:25 |
| opendevreview | Merged openstack/ironic stable/2024.2: Fix cache image hard link between different file systems https://review.opendev.org/c/openstack/ironic/+/962332 | 17:26 |
| opendevreview | Merged openstack/ironic master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/ironic/+/961788 | 17:45 |
| TheJulia | so, I think we'll need to review jam on the networking stuff at some point | 18:11 |
| TheJulia | clif: question, your trait based networking stuff, shoul the config files be first? | 19:18 |
| clif | probably not, the configuration file depends on base models + filter expression parsing | 19:19 |
| TheJulia | okay, just seems to be weird wip before it | 19:20 |
| TheJulia | but that makes sense, its nice and self contained which is nice | 19:20 |
| clif | I should update the base commit, it's essentially complete | 19:21 |
| clif | at least from a functionality perspective, there may be other items I need to check off in these commits or in follow-ups | 19:22 |
| TheJulia | I did spot an issue, the new requirement needs to be added to global-requirements | 19:22 |
| TheJulia | I did not do a complete review of the wip change | 19:23 |
| clif | TheJulia: under section: openstack-general? | 19:27 |
| TheJulia | yes | 19:28 |
| JayF | yeah openstack/requirements repo | 19:28 |
| JayF | they should have docs somewhere (docs.openstack.org / something / requirements?) on how to propose a new global requirement | 19:28 |
| JayF | I haven't pushed on this primarily because it's release time which means it's either frozen or the folks working on it have 10x as much to do | 19:29 |
| clif | I understand | 19:30 |
| TheJulia | I only brought it up because we are approaching the time where that will be open for new changes | 19:33 |
| frickler | https://docs.openstack.org/project-team-guide/dependency-management.html#for-new-requirements | 19:34 |
| opendevreview | Clif Houck proposed openstack/ironic master: Trait Based Networking Filter Expression Parsing and Base Models https://review.opendev.org/c/openstack/ironic/+/961498 | 19:37 |
| opendevreview | Clif Houck proposed openstack/ironic master: Configuration file for Trait Based Networking https://review.opendev.org/c/openstack/ironic/+/962598 | 19:37 |
| clif | seems like lark is well maintained and also packaged in common distros (ubuntu, fedora) | 19:38 |
| JayF | yeah I put that comment on the change just now | 19:39 |
| JayF | they would've asked explicitly otherwise | 19:39 |
| clif | https://review.opendev.org/c/openstack/requirements/+/962626 | 19:39 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: [refactor] HardwareManager into it's own module https://review.opendev.org/c/openstack/ironic-python-agent/+/962629 | 20:48 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: [refactor] HardwareManager into it's own module https://review.opendev.org/c/openstack/ironic-python-agent/+/962629 | 21:17 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: WIP: move generichardwaremanager https://review.opendev.org/c/openstack/ironic-python-agent/+/962633 | 23:16 |
| iurygregory | Does anyone remember the page to get statistics for the project in each release? commits, LOC, number of contributors, bugs .. | 23:56 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!