| opendevreview | Merged openstack/ironic master: Eventlet: Migrate API & JSON-RPC to cheroot https://review.opendev.org/c/openstack/ironic/+/951054 | 01:09 |
|---|---|---|
| opendevreview | Iury 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/+/952730 | 02:32 |
| rpittau | good morning ironic! o/ | 07:09 |
| queensly[m] | Good morning | 07:44 |
| dtantsur | JayF: admit it, you just wanted to reserve the API version 1.100 for yourself ;) | 08:00 |
| abongale | good morning ironic! | 08:01 |
| opendevreview | Abhishek 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/+/951761 | 08:44 |
| opendevreview | Merged openstack/sushy master: Implement set_datetime method in Manager and refresh local state after PATCH https://review.opendev.org/c/openstack/sushy/+/950539 | 09:45 |
| iurygregory | good morning Ironic | 11:23 |
| opendevreview | Iury 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/+/952730 | 11:25 |
| opendevreview | Abhishek 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/+/951761 | 11:46 |
| opendevreview | Abhishek 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/+/951761 | 12:23 |
| JayF | dtantsur: realistically was just looking for a straight line something to distract me | 13:07 |
| JayF | We are likely going to have to take our surviving cat to the vet hospital today because he hasn't been eating due to a combination of grief and illness | 13:07 |
| TheJulia | :( | 13:09 |
| dtantsur | ouch, I'm sorry to hear :( | 13:13 |
| TheJulia | JayF: bonded pair I take it? | 13:21 |
| opendevreview | Abhishek 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/+/951761 | 13:36 |
| JayF | TheJulia: honestly, somewhere between bickering Brothers and sworn enemies until Luna died a few years back. Then they were much more friendly. Although I will say, he just ate more breakfast than he's eaten in the last 24 hours combined, so maybe all the medicines were stuffing in him are finally taking effect and he's turning a corner | 13:45 |
| JayF | Hope is dangerous | 13:46 |
| JayF | But he also apparently has mild pancreatitis as well according to the blood work, so I think it's some grief and some sick all combined | 13:46 |
| TheJulia | Ugh, rough. :( | 14:40 |
| opendevreview | cid proposed openstack/ironic master: Add retry during node tear down https://review.opendev.org/c/openstack/ironic/+/952835 | 15:04 |
| stephenfin | Does anyone have a DevStack w/ Ironic handy? If so, could you dump the output of the root ironic document (/baremetal) somewhere for me? | 15:09 |
| stephenfin | (saves me deploying one just for that) | 15:09 |
| dtantsur | stephenfin: does it have to be devstack or any ironic will work? | 15:10 |
| stephenfin | any ironic | 15:10 |
| dtantsur | hold on then | 15:10 |
| opendevreview | Abhishek 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/+/951761 | 15:11 |
| dtantsur | stephenfin: https://paste.opendev.org/show/bl4r70GZYUje7rSYh4hs/ is this what you're looking for? | 15:12 |
| dtantsur | (btw, it's not very hard to start Ironic locally without an OpenStack environment) | 15:12 |
| stephenfin | It looks like it. Could you also share the response for the `/v1` path if you don't mind? (This is for fleshing out some Gophercloud tests) | 15:13 |
| stephenfin | (and trying both ironic and cinder in standalone mode is on my rather long todo list :)) | 15:14 |
| dtantsur | stephenfin: https://paste.opendev.org/show/bnkdnRWLfBQ97Wc5zZaq/ | 15:15 |
| stephenfin | dtantsur++ Thank you 🙏 | 15:15 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add specific "use 24.04" note to devstack-guide https://review.opendev.org/c/openstack/ironic/+/952838 | 15:16 |
| JayF | can we land ^ that? | 15:17 |
| JayF | I know someone lost a good amount of time using the upstream-devstack reccomendation of 22.04 | 15:17 |
| JayF | which seems outta date too, and will be patched there | 15:17 |
| dtantsur | JayF: will "VM image" be clear enough in this context? | 15:17 |
| JayF | in the previous sentence, we say "use a VM for this" | 15:18 |
| JayF | more or less | 15:18 |
| dtantsur | we do mention a virtual machine slightly above | 15:18 |
| JayF | that's exactly why I chose that verbiage. | 15:18 |
| dtantsur | I"m just worried about a mixup between the OS on which Ironic is running and any images they're trying to deploy | 15:18 |
| dtantsur | maybe I'm reading too much into it | 15:18 |
| JayF | oh, I see | 15:18 |
| dtantsur | will +2 | 15:18 |
| JayF | I can make that more clear | 15:18 |
| JayF | that's a good point | 15:18 |
| JayF | 1 sec | 15:18 |
| dtantsur | waiting | 15:18 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add specific "use 24.04" note to devstack-guide https://review.opendev.org/c/openstack/ironic/+/952838 | 15:19 |
| dtantsur | perfect | 15:20 |
| JayF | fwiw also https://review.opendev.org/c/openstack/devstack/+/952839 because I think 22.04 is wrong generally | 15:29 |
| opendevreview | Merged openstack/ironic master: Add specific "use 24.04" note to devstack-guide https://review.opendev.org/c/openstack/ironic/+/952838 | 15:56 |
| slyon_ | Hey folks! Can somebody support us with verifying this Ubuntu stable-release update using the Ironic CI? https://bugs.launchpad.net/ubuntu/+source/dnsmasq/+bug/2026757 | 16:02 |
| slyon_ | Official .debs are now in noble-proposed for Ubuntu 24.04 LTS and can be found here: https://launchpad.net/ubuntu/+source/dnsmasq/2.90-2ubuntu0.1 (cc JayF) | 16:02 |
| JayF | can you point me at a curl-able .deb link or a ppa I can setup/ | 16:03 |
| JayF | I essentially have to rewrite https://opendev.org/openstack/ironic/src/branch/master/devstack/lib/ironic#L4344 in order to test this in our CI | 16:03 |
| JayF | so if you want bonus points, just rewrite that for something ubuntu specific, I'll review and pop it into CI | 16:04 |
| JayF | slyon_: ^ | 16:07 |
| * JayF has never seen a ddeb file before | 16:10 | |
| opendevreview | Jay Faulkner proposed openstack/ironic master: DO NOT MERGE: UBUNTU SRU SCIENCE https://review.opendev.org/c/openstack/ironic/+/952848 | 16:17 |
| JayF | slyon_: ^ I found the links hopefully that works for ya | 16:17 |
| JayF | I'll look at CI output and confirm it all worked | 16:17 |
| JayF | in a couple of hours when it's done + when I get time | 16:17 |
| slyon_ | Thanks a lot! I’m looking forward to those test results! | 16:23 |
| JayF | lol > 2025-06-18 16:32:11.212621 | controller | dpkg: error: requested operation requires superuser privilege | 16:33 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: DO NOT MERGE: UBUNTU SRU SCIENCE https://review.opendev.org/c/openstack/ironic/+/952848 | 16:33 |
| TheJulia | doh ;) | 16:35 |
| JayF | TheJulia: original bug filed by you: mid-2023. dnsmasq upstream fix: late-2024. ubuntu SRU summer-2025 | 16:36 |
| JayF | nice to know we aren't the only project running on openstack time :D | 16:36 |
| JayF | and persistence pays off lol | 16:36 |
| TheJulia | heh | 16:36 |
| TheJulia | yeah | 16:36 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: DO NOT MERGE: UBUNTU SRU SCIENCE https://review.opendev.org/c/openstack/ironic/+/952848 | 16:57 |
| slyon_ | Hmm the pipeline says something about arm64, is it correct to use the amd64 dnsmasq binaries? | 17:03 |
| JayF | we have two pipelines: a non-voting unit test pipeline on arm64 hosts, and our amd64 pipeline with unit tests + a bunch of functional tests | 17:04 |
| slyon_ | Ack. | 17:05 |
| JayF | we do also have a job that provisions arm64 VMs on an amd64 host | 17:05 |
| JayF | https://zuul.opendev.org/t/openstack/status?change=952848 if you wanna see it go | 17:05 |
| JayF | I've been preempting the failures early because I'll se them fail in setup phase | 17:05 |
| JayF | if they get started, runtime is nearly 2 hours if not longer | 17:05 |
| slyon_ | Cool, I’ll check back on the results later. TheJulia or JayF feel free to post the results to that launchpad bug, when ready. | 17:08 |
| JayF | yeah, I'm about to hit my meeting slate for the day so it'll be a while before I can followup | 17:08 |
| JayF | there won't be a smoking gun, but I can post a yep :D | 17:08 |
| slyon_ | That should be good enough (I hope). I’ll extend it with Ubuntu‘s CI test results. Together that should provide enough confidence for the Ubuntu stable release team. | 17:13 |
| JayF | The fun thing about this bug is that we're pretty sure it would've been unlikely to impact ironic deployments in the real world | 17:13 |
| JayF | because the race in CI is that dnsmasq crashes on reconfigure, but the VM boots before it can be respawned | 17:14 |
| JayF | in the real world servers take 10+ minutes to boot, giving it time to respawn | 17:14 |
| JayF | but I still suspect this is removing a source of flakiness from the system | 17:14 |
| slyon_ | Ack | 17:15 |
| adamcarthur5 | Appreciate the comments TheJulia. I'll get to them ASAP | 17:37 |
| TheJulia | cool cool | 20:46 |
| TheJulia | JayF: I feel like we need to double check some jobs, but looking pretty good | 20:58 |
| TheJulia | s/jobs/job logs/ | 20:58 |
| JayF | nice | 20:58 |
| JayF | I'll dig into those once clif and I are done and update the ticket | 20:58 |
| TheJulia | ++ | 21:02 |
| opendevreview | Clif Houck proposed openstack/ironic-specs master: Trait based port selection and dynamic portgroups https://review.opendev.org/c/openstack/ironic-specs/+/945642 | 21:23 |
| JayF | slyon_: it's good \o/ commenting in the bug now. Thank you. | 21:48 |
| JayF | slyon_: https://bugs.launchpad.net/ironic/+bug/2026757/comments/66 | 21:50 |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Add Tempest tests for inspection rules in Ironic https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/951761 | 21:55 |
| opendevreview | Merged openstack/ironic stable/2025.1: fix: 'built_in' KeyError on invalid priority. https://review.opendev.org/c/openstack/ironic/+/952771 | 23:24 |
| iurygregory | Quick question, I'm expanding the firmware updates to include NIC, I've noticed I would need to change things in the Schema https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/modules/redfish/firmware_utils.py#L78 (currently we only accept "bios", "bmc") and the nics doesn't have any standard name... should we have some sort of prefix ? | 23:40 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!