| rpittau | good morning ironic! o/ | 07:38 |
|---|---|---|
| rpittau | JayF re https://review.opendev.org/c/openstack/virtualpdu/+/933882 that looks good to me, that's great :) | 08:31 |
| rpittau | JayF re Epoxy priorities: I was planning to do the work items as we usually do after every PTG like https://review.opendev.org/c/openstack/ironic-specs/+/916295 but I haven't started anything yet, if you have a draft already feel free to submit that and I will review that as highest priority | 08:31 |
| opendevreview | cid proposed openstack/ironic master: Allow special characters in patch field keys https://review.opendev.org/c/openstack/ironic/+/933743 | 09:03 |
| opendevreview | cid proposed openstack/ironic master: Save ``configdrive`` in an auxiliary table https://review.opendev.org/c/openstack/ironic/+/933622 | 09:41 |
| opendevreview | Merged openstack/ironic master: Correct duplicated names/entries in unit tests https://review.opendev.org/c/openstack/ironic/+/933679 | 10:04 |
| opendevreview | Merged openstack/ironic master: Remove trailing whitespace https://review.opendev.org/c/openstack/ironic/+/933684 | 11:11 |
| cid | Good afternoon Ironic, rpittau. | 11:24 |
| rpittau | hey cid :) | 11:24 |
| cid | Does networking-baremetal does fall under the Ironic umbrella. | 11:25 |
| rpittau | cid: it does indeed! | 11:25 |
| cid | Okay great. | 11:25 |
| cid | New bug I might need help triaging filled on it, I will be adding it to the meeting page, s | 11:27 |
| rpittau | thanks cid, sounds good | 11:27 |
| cid | ++ | 11:27 |
| opendevreview | yatin proposed openstack/ironic stable/2023.1: [Stable Only] pin virtualbmc to last released tag https://review.opendev.org/c/openstack/ironic/+/934036 | 12:42 |
| opendevreview | cid proposed openstack/ironic master: Save ``configdrive`` in an auxiliary table https://review.opendev.org/c/openstack/ironic/+/933622 | 12:45 |
| opendevreview | yatin proposed openstack/ironic stable/2023.1: [Stable Only] pin virtualbmc/ironic-tempest-plugin to last released tag https://review.opendev.org/c/openstack/ironic/+/934036 | 13:43 |
| opendevreview | yatin proposed openstack/ironic stable/2023.1: [Stable Only] pin virtualbmc/sushy-tools/ironic-tempest-plugin to last released tag https://review.opendev.org/c/openstack/ironic/+/934036 | 14:51 |
| rpittau | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Nov 4 15:00:26 2024 UTC and is due to finish in 60 minutes. The chair is rpittau. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
| dtantsur | o/ | 15:00 |
| rpittau | Hello everyone! | 15:00 |
| rpittau | Welcome to our weekly meeting! | 15:00 |
| rpittau | The meeting agenda can be found here: | 15:00 |
| rpittau | https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_Novembere_04.2C_2024 | 15:00 |
| rpittau | not a lot to discuss today, but let's see also if we have quorum | 15:01 |
| JayF | o/ | 15:01 |
| cid | o/ | 15:02 |
| rpittau | I will start with the announcements and see if more people joins | 15:03 |
| rpittau | #topic Announcements/Reminders | 15:03 |
| rpittau | #info Standing reminder to review patches tagged ironic-week-prio and to hashtag any patches ready for review with ironic-week-prio | 15:03 |
| rpittau | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:03 |
| rpittau | seems most of the patches there are updated and need a second review | 15:04 |
| rpittau | can probably have a look 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 | we're at R-21, nothing special, just related to that next bugfix branches will happen in 3 weeks and we should consider having releases for other projects too | 15:05 |
| rpittau | #info Epoxy OpenInfra PTG was October 21-25, 2024 | 15:07 |
| rpittau | #link https://etherpad.opendev.org/p/ironic-ptg-october-2024 | 15:07 |
| rpittau | I'm leaving this still here for review | 15:07 |
| rpittau | JayF have you seen my message for the work items? | 15:07 |
| JayF | Yes. I'll work on that today | 15:09 |
| rpittau | thanks | 15:09 |
| rpittau | moving on | 15:09 |
| rpittau | #topic Discussion topics | 15:09 |
| rpittau | do we have anything to discuss today? | 15:09 |
| rpittau | onward then! | 15:10 |
| rpittau | #topic Bug Deputy Updates | 15:11 |
| cid | Yea, there were two RFEs | 15:11 |
| rpittau | cid: I think https://bugs.launchpad.net/networking-baremetal/+bug/2086303 is on oslo and not on us | 15:11 |
| rpittau | or even kolla | 15:11 |
| cid | Oh, okay, should I take any specific action(s) | 15:12 |
| rpittau | there's a reference to another bug | 15:12 |
| cardoe | timezones... | 15:13 |
| rpittau | looks like the issue is caused by the issue is due to the fix for that | 15:13 |
| cid | So, invalid on networking-baremetal? | 15:14 |
| rpittau | cid: reading the other bug I'm not sure anymore, it may be valid on nbm | 15:15 |
| cid | I might just change the status from new, then we will wait to find out with time where exactly it's occuring | 15:15 |
| rpittau | cid: I would ask in the bug itselfif they want to propose a fix in nbm | 15:16 |
| JayF | While we're talking about bugs, there was a patch posted over the weekend to NGS completely introducing a new actions model using an agent. I put a comment in that rfe requesting a spec and marking it as needs-spec | 15:17 |
| JayF | I don't have the link at hand to the bug, but it's the one associated with the NGS patch that was posted on Sunday | 15:17 |
| rpittau | JayF: thanks :) | 15:17 |
| cid | rpittau, I will drop a message in there. | 15:17 |
| rpittau | thanks cid | 15:17 |
| rpittau | also for https://bugs.launchpad.net/ironic/+bug/2064655 that is not a bug | 15:17 |
| rpittau | it's just a permission issue | 15:18 |
| cid | rpittau, I suspected same. | 15:18 |
| rpittau | RFEs look both ok baseed on feedback from PTG | 15:19 |
| cid | ++ | 15:19 |
| cid | On 2064655 , can I invalidate the bug. | 15:20 |
| rpittau | yep | 15:20 |
| cid | Alright | 15:20 |
| rpittau | about that error in CI, any specific job that is failing for that? | 15:21 |
| cid | Seen it in one of mine. | 15:22 |
| cid | 1 sec, let me get the link | 15:22 |
| cid | https://review.opendev.org/c/openstack/ironic/+/933743 | 15:22 |
| cid | Same job and error failed here https://review.opendev.org/c/openstack/ironic/+/933622, though, there's a lot more other failing jobs that are related to the change. | 15:24 |
| rpittau | ok, so it's related to a patch? looks like in ironic-tempest-ovn-uefi-ipmi-pxe only | 15:25 |
| rpittau | it may be worth running a test-ci patch to see if it replicates | 15:25 |
| JayF | The unit test failure on the config drive change is related to the patch based on a cursory look | 15:26 |
| rpittau | yeah | 15:26 |
| rpittau | that's why we need a zero-change patch to verify | 15:26 |
| rpittau | anyway, moving on | 15:28 |
| rpittau | any volunteer for bug deputy for this week? | 15:29 |
| cid | I don't mind | 15:29 |
| rpittau | cid: thanks! | 15:30 |
| cid | My pleasure | 15:31 |
| rpittau | one thing I forgot to mention, next Monday I won't be able to chair the meeting, someone else will need to take my place | 15:31 |
| rpittau | any more topics for today? | 15:32 |
| rpittau | alright, thanks everyone! | 15:33 |
| rpittau | #endmeeting | 15:33 |
| opendevmeet | Meeting ended Mon Nov 4 15:33:20 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:33 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-11-04-15.00.html | 15:33 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-11-04-15.00.txt | 15:33 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-11-04-15.00.log.html | 15:33 |
| JayF | I should be able to run the meeting next week. I'll set myself a reminder | 15:37 |
| rpittau | thanks JayF :) | 15:43 |
| TheJulia | I totally forgot about time change | 16:14 |
| TheJulia | Also, conflicting meeting for reasons | 16:14 |
| rpittau | and that's why we should abolish time change | 16:18 |
| JayF | Lets abolish all change. | 16:20 |
| jamesdenton | Can an image created with block-device-efi-lvm be resized post-deployment? Most examples i see have the root partition set to something like 25G. Also curious if config-2 will be problematic? | 16:39 |
| cardoe | rpittau: I keep hoping that happens every year. | 16:43 |
| cardoe | So skrobul noticed that "baremetal node set 1ec32b3b-6e5b-46de-a723-426162a22e3d set --no-automated-clean" has a policy error by default if the nodes are in a project. But if you use system scope like it suggests it can't find the nodes. Do we need a tweak to the default policy? | 16:45 |
| cardoe | "baremetal:node:disable_cleaning": "role:admin and system_scope:all" requires a scope of ['system'], request was made with project scope. (HTTP 500) | 16:45 |
| cardoe | I'm happy to make a bug report or maybe poke him to do so. | 16:45 |
| JayF | cardoe: no, that's accurate | 16:49 |
| JayF | cardoe: security disablement is not a project decision; it's a system decision | 16:50 |
| JayF | If you're saying it doesn't work either way, I'd like to get some logs of it both direction | 16:50 |
| JayF | jamesdenton: don't know the answer for you, btw | 16:51 |
| JayF | jamesdenton: might be worth cross-posting in #openstack-dib | 16:51 |
| Pcmalih_ | Hi, I am facing OSError: [Errno 18] Invalid cross-device link: '/var/lib/ironic/master_iso_images/UUID' -> '/tmp/ahfgjsh/boot. Using Redfish virtual media HTTP URL and Glanc UUID both face this issue. Nay suggesstion? | 16:55 |
| JayF | Pcmalih_: is /tmp on a separate partition? | 16:56 |
| JayF | Pcmalih_: if yes; can you file a bug @ bugs.launchpad.net with this error, and the output of `mount` on your conductor | 16:57 |
| rpittau | good night! o/ | 16:57 |
| Pcmalih_ | Looks like . But i have hosted both in same machine beig used as controller | 16:57 |
| JayF | Pcmalih_: and putting master_path on the same partition as /tmp would resolve it | 16:57 |
| JayF | Pcmalih_: this looks a /lot/ like the error you'd get trying to hardlink across parititons, so I'm wondering if we have a bug/assumption that master_path and /tmp are the same partition | 16:57 |
| Pcmalih_ | Any way to set master_path ? | 16:58 |
| JayF | [pxe]/instance_master_path https://docs.openstack.org/ironic/latest/configuration/sample-config.html | 16:59 |
| JayF | or | 16:59 |
| JayF | [deploy]/iso_master_path | 16:59 |
| JayF | that path looks more like the iso_master_path default value to me | 16:59 |
| JayF | I'll note: this dir is basically the place images get cached in Ironic, so you probalby don't want *that dir* in /tmp | 17:00 |
| Pcmalih_ | But i am using Kolla-Ansible | 17:00 |
| JayF | I can't speak to how kolla-ansible sets stuff up, they can help you with that piece, but please do file a bug as this is our issue if it's what I think it is | 17:01 |
| JayF | but I need full logs to be sure | 17:01 |
| Pcmalih_ | How to customise tmp address or ironic/master_iso_images using Kolla-Ansible? | 17:01 |
| TheJulia | jamesdenton: re block-device-efi-lvm, the take a look at the growvols ?element? | 17:02 |
| TheJulia | cardoe: likely, the restriction to it only being system scope should be fixed | 17:03 |
| TheJulia | cardoe: but the default policy rule is realistically, as jayf pointed out, a system decision | 17:03 |
| JayF | Are you saying we should allow project admin to do it? | 17:04 |
| TheJulia | we should allow someone to set appropriate policy to meet their needs | 17:04 |
| TheJulia | to just error on forcing scope to be specific is wrong | 17:04 |
| JayF | okay, so we don't have the bits in place for someone to make a custom policy rule permitting project to do this | 17:04 |
| TheJulia | because then they don't get a real "you don't have access", we outright deny because your asking with the wrong scope | 17:04 |
| JayF | and we wanna add that while keeping the default | 17:05 |
| TheJulia | slightly nuanced difference | 17:05 |
| JayF | okay I am +1 to that change | 17:05 |
| JayF | as long as default policy still restricts to system admin | 17:05 |
| JayF | or system-wahtever | 17:05 |
| TheJulia | yeah | 17:05 |
| opendevreview | Julia Kreger proposed openstack/ironic master: trivial: Fix policy scope restriction for automated cleaning https://review.opendev.org/c/openstack/ironic/+/934065 | 17:15 |
| TheJulia | that ^^ | 17:15 |
| jamesdenton | Thanks JayF TheJulia - gives me something to look at | 17:15 |
| JayF | Do we want a release note on that ? | 17:16 |
| JayF | Seems like it's operator facing | 17:16 |
| TheJulia | mildly operator facing | 17:16 |
| opendevreview | Julia Kreger proposed openstack/ironic master: trivial: Fix policy scope restriction for automated cleaning https://review.opendev.org/c/openstack/ironic/+/934065 | 17:19 |
| opendevreview | Michal Nasiadka proposed openstack/networking-generic-switch master: Add vlan aware VMs support https://review.opendev.org/c/openstack/networking-generic-switch/+/928490 | 17:28 |
| cardoe | So am I doing it wrong by putting my hardware into a domain and project that's separate from the domain that my normal users/projects are in? | 17:58 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: prevent break on communications failure https://review.opendev.org/c/openstack/networking-baremetal/+/933149 | 18:24 |
| jamesdenton | mnasiadka - with your NGS work, are you observing network_data.json reflecting the VLAN configuration on the deployed baremetal instance? Like, is cloud-init implementing the subinterface configuration in the OS? | 18:25 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal stable/2024.2: avoid attribute error on bad password or config https://review.opendev.org/c/openstack/networking-baremetal/+/934070 | 18:26 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal stable/2024.1: avoid attribute error on bad password or config https://review.opendev.org/c/openstack/networking-baremetal/+/934071 | 18:26 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal stable/2023.2: avoid attribute error on bad password or config https://review.opendev.org/c/openstack/networking-baremetal/+/934072 | 18:26 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal stable/2023.1: avoid attribute error on bad password or config https://review.opendev.org/c/openstack/networking-baremetal/+/934073 | 18:27 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: prevent break on communications failure https://review.opendev.org/c/openstack/networking-baremetal/+/933149 | 18:36 |
| TheJulia | hjensas: so I've been thinking about maybe we should be raising https://docs.openstack.org/oslo.service/ocata/api/loopingcall.html#oslo_service.loopingcall.LoopingCallDone instead, if you can take a look/glance (hopefully you remember enough of the higher level interaction) | 18:38 |
| mnasiadka | jamesdenton: haven’t thought about it, I’ll check - but isn’t network_data.json a Nova metadata thing? | 18:39 |
| jamesdenton | indeed. just curious to see how tight the integrations were | 18:39 |
| TheJulia | cardoe: you may also have thoughts regarding my comment to hjensas above | 18:40 |
| jamesdenton | no worries and no rush | 18:40 |
| TheJulia | I've thought about the metadata, and it likely has a couple issues with bonds/mutliple ports | 18:40 |
| jamesdenton | i think you're right | 18:40 |
| JayF | mnasiadka: kinda, but also it's us... | 18:40 |
| JayF | let me show you | 18:40 |
| JayF | https://opendev.org/openstack/nova/src/commit/0a59078935b753ba30518915e15189e63c0dfc66/nova/virt/ironic/driver.py#L1045 | 18:41 |
| JayF | so essentailly, ironic's nova driver builds what goes in netowrk_data in the configdrive | 18:41 |
| JayF | but does *not* do the same for metadata service | 18:41 |
| JayF | so yes-ish it's coming from nova-the-service, but contents are code that is sorta jointly maintained (and is separate from other drivers) | 18:41 |
| TheJulia | cardoe: hjensas: at least calling LoopingCallDone would allow us to signal with -1... in theory | 18:43 |
| hjensas | TheJulia: will it fully stop if we raise LoopingCallDone? There are two "loops" running, one for reporting state, and the other "notify_agents" for the cluster of agents. | 18:57 |
| opendevreview | Merged openstack/ironic-python-agent master: Cleanup usage of imported-from-ironic-lib disk_utils https://review.opendev.org/c/openstack/ironic-python-agent/+/928466 | 18:57 |
| TheJulia | oh.... | 18:57 |
| TheJulia | yeah, that would prevent it from ever exiting then | 18:57 |
| TheJulia | eww | 18:57 |
| TheJulia | so... hmmmmm | 18:57 |
| TheJulia | well, calling stop then would at least exit it | 18:57 |
| TheJulia | in *theory* | 18:57 |
| hjensas | It would be good to exit with an error return code ... | 18:58 |
| TheJulia | I think the thing to do is to just have a "exit with a return code" to the definition of stop | 19:00 |
| TheJulia | .... and then force python to exit with it | 19:00 |
| mnasiadka | JayF: thanks for the pointers | 19:03 |
| TheJulia | hjensas: we could call os.abort() | 19:04 |
| TheJulia | it could result in a core dump, and will result in an exit code... 255 I think | 19:06 |
| TheJulia | modern linux distros don't save core dumps by default, so it should be fine... | 19:06 |
| hjensas | TheJulia: I wonder if we should have something similar in the _notify_peer_agents method, try to re-initialize the messaging transport? (But I think oslo messaging should provide quite good resiliency, with retries with back off etc already.) | 19:08 |
| TheJulia | so, it only retires... 9 times | 19:08 |
| TheJulia | 30 seconds apart | 19:09 |
| TheJulia | so... I'm not sure how simple it would be to do that or not | 19:09 |
| hjensas | oh, it stops after 9 tries? I figures it would keep on trying with 30 second interval. | 19:13 |
| TheJulia | yeah, the default is a limited number of tries from my reading of the oslo.messaging code | 19:14 |
| TheJulia | granted, I only skimmed it | 19:14 |
| TheJulia | I think the whole idea is "how often does the message bus break | 19:15 |
| TheJulia | hmm... I guess _notify_peer_agents could fail too | 19:15 |
| TheJulia | hmmm | 19:16 |
| hjensas | Then catching "Exception" there might be bad - MessageDeliveryFailure might be better? | 19:16 |
| TheJulia | so the case we found the failed host in, it wasn't logging anything | 19:18 |
| TheJulia | it could be the overall process was long "weged" | 19:18 |
| TheJulia | I bet self.notifier.info() never returns and gets hung | 19:21 |
| opendevreview | Merged openstack/ironic-python-agent master: Remove use of ironic_lib i18n module https://review.opendev.org/c/openstack/ironic-python-agent/+/930080 | 19:22 |
| TheJulia | so, there is no timeout | 19:23 |
| TheJulia | so the looping call could in theory just hang too | 19:23 |
| TheJulia | I guess that is a uniquely separate way the agent can just sort of enter a apparent frozen state | 19:30 |
| hjensas | self.notifier.info() is the listener, it will run if a info message is on the topic. | 19:32 |
| TheJulia | nope, there is a default timeout of 300 seconds, so *eventually* it should exit | 19:36 |
| TheJulia | yeah | 19:36 |
| TheJulia | my nope was for my own perception of it hanging | 19:37 |
| TheJulia | so the likely state was the self.notifier.info was still silently working | 19:37 |
| TheJulia | so I bet the *actual* failure was more than likely the client breaking | 19:40 |
| TheJulia | and I bet the message bus was in a slightly happier state | 19:40 |
| TheJulia | which I feel like it sort of aligns with the state, the only way to know otherwise was to look at network sockets I guess | 19:40 |
| cardoe | hmm getting caught up sorry. | 19:43 |
| cardoe | sounds like no good solution yet? | 19:43 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: prevent break on communications failure https://review.opendev.org/c/openstack/networking-baremetal/+/933149 | 19:47 |
| TheJulia | eh, I think ^ might cover it | 19:48 |
| TheJulia | it will cause it to self abort, looping call will cause things to eventually timeout | 19:49 |
| TheJulia | so as long as loopingcall doesn't entirely go on vacation | 19:49 |
| TheJulia | some failure under it should become detectable and the abort *should* get called | 19:50 |
| TheJulia | That is a few toooo many shoulds, but it feels sound enough | 19:50 |
| TheJulia | ... I think | 19:50 |
| JayF | OK something is upside-down in CI | 19:52 |
| JayF | https://review.opendev.org/c/openstack/ironic-python-agent/+/911598 and ironic-lib changes both appear to not have tempest installed in the Devstack runs | 19:53 |
| JayF | or the ironic-tempest-plugin | 19:53 |
| JayF | hmmm maybe this is a different shaped failure than ironic-lib | 19:54 |
| JayF | I'm going to recheck and see, but I suspect we got busted by something | 19:54 |
| TheJulia | hmmmmmm | 19:55 |
| JayF | We also have a stunning number of patches in ironic-week-prio | 19:57 |
| JayF | I'm going thru em now myself, but some nonzero % of them are mine | 19:57 |
| JayF | mnasiadka: https://review.opendev.org/c/openstack/networking-generic-switch/+/932541 just asking if you saw my comment here? Happy to help you get a release note on that if you want so we can land it | 19:58 |
| JayF | Question for the class: now that we have ironic-reviewers and ironic-approvers separated, how do we feel about self-approvals when the code review threshold is already met? | 20:00 |
| JayF | e.g. https://review.opendev.org/c/openstack/ironic/+/933685/1 has CID +2, Riccardo +2, but not landed (CID's review came second). Should I land it even though I was the author/ | 20:01 |
| JayF | I'm leaning towards "yes/best judgement" but don't wanna make such a call unilaterally :) | 20:01 |
| TheJulia | I generally refrain from doing so unless there is a needful aspect, i.e. "oh, trying to fix CI" | 20:02 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Vendor metrics library from Ironic-Lib & deprecate https://review.opendev.org/c/openstack/ironic-python-agent/+/933063 | 20:02 |
| JayF | honestly the only rush here is wanting the tip of that chain to merge before something else new being linted gets landed and breaks it | 20:03 |
| JayF | since I don't wanna have to redo that auditt | 20:03 |
| cardoe | Yeah that’s why I was trying to nudge that in. | 20:09 |
| opendevreview | Merged openstack/python-ironicclient master: bump minimum pbr version for pip 23.1 support https://review.opendev.org/c/openstack/python-ironicclient/+/933597 | 20:41 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Correct invalid docstrings; s/Found/Error/ https://review.opendev.org/c/openstack/ironic-python-agent/+/911598 | 20:50 |
| opendevreview | Merged openstack/ironic-python-agent stable/2024.1: Warn when the provided checksum algorithm does not match the detected https://review.opendev.org/c/openstack/ironic-python-agent/+/933105 | 21:33 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent stable/2023.2: Warn when the provided checksum algorithm does not match the detected https://review.opendev.org/c/openstack/ironic-python-agent/+/934091 | 21:35 |
| JayF | fyi looks like we're getting some clean ci runs now | 21:58 |
| JayF | idk what was broken on all those patches I checked but it's gone | 21:58 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Use patched dnsmasq from PPA https://review.opendev.org/c/openstack/ironic/+/933104 | 22:10 |
| JayF | ^^ that is ready for review, is idempotent, and should solve our issue (and we can back it out once upstream takes the backport) | 22:11 |
| JayF | well, *ubuntu* takes the backport, which is upstream of us but downstream of dnsmasq | 22:11 |
| opendevreview | Jay Faulkner proposed openstack/ironic-specs master: WIP: 2025.1 priorities https://review.opendev.org/c/openstack/ironic-specs/+/934092 | 23:03 |
| JayF | rpittau: ^ I commented where I stopped off, I have a meeting to go to | 23:04 |
| JayF | rpittau: that should be a good start though | 23:04 |
| opendevreview | Doug Goldstein proposed openstack/python-ironicclient master: fix port name in Port resource https://review.opendev.org/c/openstack/python-ironicclient/+/933746 | 23:34 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Correct invalid docstrings; s/Found/Error/ https://review.opendev.org/c/openstack/ironic-python-agent/+/911598 | 23:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!