| iurygregory | Anyone have a Dell PowerEdge R450? Or any Server from 15G? seems like they use Numeric ID (1 vs System.Embedded.1 or 1 vs iDRAC.Embeedded.1) | 01:14 |
|---|---|---|
| TheJulia | What’s going on? | 02:20 |
| iurygregory | basically a machine where NumericDynamicSegmentsEnable is Disabled is freaking out... | 02:30 |
| iurygregory | they have numeric ID for Manager, which I personally find really weird for a Dell machine.. | 02:31 |
| rpittau | 'morning! looking for final approval for https://review.opendev.org/c/openstack/bifrost/+/932175 thanks! :) | 06:26 |
| opendevreview | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic-ui/+/998683 | 08:04 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Disable neutron-ovn-agent https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/998645 | 08:43 |
| opendevreview | Rowan Johns proposed openstack/tenks master: Add support for persistent networking https://review.opendev.org/c/openstack/tenks/+/998806 | 08:45 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Power off before ejecting redfish virtual media on ramdisk cleanup https://review.opendev.org/c/openstack/ironic/+/998640 | 10:59 |
| dtantsur | TheJulia: a wall of meetings today, but I'll try, thanks for heads-up | 11:54 |
| iurygregory | rpittau, +W | 12:03 |
| opendevreview | cid proposed openstack/ironic master: devstack: Support the container console provider https://review.opendev.org/c/openstack/ironic/+/995854 | 12:54 |
| opendevreview | cid proposed openstack/ironic master: Document the container console provider https://review.opendev.org/c/openstack/ironic/+/995855 | 12:54 |
| opendevreview | cid proposed openstack/ironic master: A non-voting job for the container console provider https://review.opendev.org/c/openstack/ironic/+/998934 | 12:54 |
| dtantsur | Team, could we get some eyes on https://review.opendev.org/c/openstack/ironic/+/998640 and https://review.opendev.org/c/openstack/ironic/+/998628 please? These affect Metal3 and cause regular failures in the BMO job. | 13:14 |
| TheJulia | dtantsur: yeah, I too have a wall of meetings today | 13:44 |
| TheJulia | :( | 13:44 |
| rpittau | Mahnoor: in https://review.opendev.org/c/openstack/ironic/+/998640 is it intended that we're doing a hard power off and not a soft one ? | 13:52 |
| TheJulia | So going back to Iury's question, it appears dell has introduced a "feature" to allow devices to be named based upon ID order in a list as opposed to the static expected resource names based upon the internal names. The issue seems to be, and Iury is working to confirm this, that some resources are still on their old device names even though we construct requests and URLs based upon the values discovered in place. In other words, | 13:52 |
| TheJulia | looking like a but in the BMC which needs to be fixed. | 13:52 |
| dtantsur | rpittau: very intended. Soft power off may not even work. | 13:52 |
| dtantsur | (it does fail in the BMO CI quite regularly since it relies on the operating system being up and accepting ACPI events) | 13:52 |
| rpittau | dtantsur: ack, thanks | 14:00 |
| opendevreview | Merged openstack/bifrost master: Use cirros 0.6.3 as default cirros version https://review.opendev.org/c/openstack/bifrost/+/932175 | 14:44 |
| rpittau | \o/ | 15:22 |
| JayF | cardoe: TheJulia: after going to Vegas, it' | 16:01 |
| JayF | cardoe: TheJulia: it's the food mecca which will bring us all together. It just needs a WH. | 16:01 |
| JayF | cardoe: TheJulia: Bojangles, In-n-out, Whataburger all in walking distance | 16:02 |
| TheJulia | where is this mystical, magical place with all three in close proximity | 16:02 |
| JayF | they have bojangles everywhere now. one in miracle mile shops, one being built in linq promenade | 16:03 |
| TheJulia | wait... wut | 16:18 |
| TheJulia | I'm deeply disappointed there is not a biscuitville | 16:20 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add kernel parameter parsing to _prepare_iso_image https://review.opendev.org/c/openstack/ironic/+/992307 | 16:21 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add kernel parameter parsing to build_extra_pxe_options https://review.opendev.org/c/openstack/ironic/+/992308 | 16:21 |
| opendevreview | Clif Houck proposed openstack/ironic master: Utilize KernelCommmandLine.parse() instead of ironic.common.utils.parse_kernel_params https://review.opendev.org/c/openstack/ironic/+/992309 | 16:21 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add a tool to parse kernel command lines https://review.opendev.org/c/openstack/ironic/+/992641 | 16:21 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add a deprecation notice for disable_kernel_parameter_parsing https://review.opendev.org/c/openstack/ironic/+/995750 | 16:21 |
| JayF | TheJulia: you know if you and Summer ever want to go on a Vegas trip, just let me know when and where 😂 | 16:32 |
| dtantsur | TheJulia: I'm not sure https://review.opendev.org/c/openstack/ironic-specs/+/998805 can even work, especially the power parts | 16:33 |
| dtantsur | unless we suddently become okay with embedding IPA into user's instances... | 16:33 |
| TheJulia | My reading is we pivot through the ramdisk again to restart the user instance | 16:37 |
| TheJulia | the point on the user instance is they are inaccessible and unreachable until the machine soft reboots | 16:37 |
| TheJulia | so its boot ramdisk, unlock devices, trigger soft reboot and pivot network to the instance network | 16:38 |
| JayF | talk about features that might be hard to test in CI | 16:38 |
| TheJulia | yeah, super difficult | 16:38 |
| dtantsur | TheJulia: soft reboot? why wouldn't that lock the device again? | 16:38 |
| TheJulia | no | 16:38 |
| TheJulia | so soft reboot is similar to the host just going "we're going back to the beginning" on the processor | 16:39 |
| TheJulia | where as hard is full on "RESET ALL THE THINGS" | 16:39 |
| TheJulia | and it actually holds down the interrupt to trigger it on the bus | 16:39 |
| TheJulia | so a hard reset, triggers all devices watching that interrupt as well which includes the disk controllers and ultimately disks because it permutates all the way across | 16:40 |
| TheJulia | often hard reset also includes cutting power | 16:40 |
| dtantsur | uh, interesting. except that.. to do network switch, we use power off, not reboot. | 16:40 |
| dtantsur | I think the spec overly focuses on API additions (redundant IMO) and less on semantics | 16:40 |
| TheJulia | that is a great callout, for this we would need switch similar to what we do with rescue | 16:40 |
| dtantsur | yeah, we got something for the disable_power_off feature | 16:41 |
| TheJulia | yeah, that was sort of my take, and I mentally viewed it as rescue with the callout. I think it could be a little more verbose as well in the interface they propose, but they have a working thing downstream so... kind of giving them a little grace that they have a lot of those details basically figured out and didn't want to get too deep in the details on the spec | 16:41 |
| JayF | What's our take on "we don't want this complexity for a limited use case"? :| | 16:42 |
| dtantsur | JayF: that's kind of where I'm getting with it. If it was just a bunch of service/deploy/clean steps, I'd be cool with it. | 16:44 |
| dtantsur | But we're talking about entire new state transitions and a serious rethink of power API. | 16:44 |
| TheJulia | Well, in the defense of the idea, we already do similar with rescue | 16:44 |
| JayF | and the more steps there are in a simple deployment, the more likely they are to break | 16:44 |
| TheJulia | but yeah, we would need to wire in on the power management calls to unlock disks | 16:44 |
| TheJulia | or just reset them | 16:44 |
| JayF | TheJulia: I think the fact rescue isn't frequently used (at least to my knowledge) is a nod to how that complexity drives folks way | 16:45 |
| JayF | **away | 16:45 |
| dtantsur | If rescue was invented today, it would be a service step combined with "hold" most likely. | 16:45 |
| TheJulia | Well, a lot of folks try to wedge that as a solution for things outside of our supported flows | 16:45 |
| TheJulia | dtantsur: yup | 16:45 |
| TheJulia | the key difference under the hood here is hardware disk locking | 16:45 |
| TheJulia | and the need to do the unlock which does, obviously have some impacts | 16:46 |
| dtantsur | I don't mind the feature but I do mind the scope of user-visible changes | 16:46 |
| dtantsur | Power actions becoming a state transition that uses IPA makes my head hurt. | 16:47 |
| TheJulia | yeah, that *feels* like it should be implied | 16:47 |
| TheJulia | but I can see why they do that today | 16:47 |
| TheJulia | its... weird and then how do you deal with that with the api, really, the api sort of needs to hide/mask that | 16:47 |
| JayF | I disagree a lot with that | 16:48 |
| TheJulia | well, depending on version | 16:48 |
| JayF | it should be easy for an operator to know when an IPA ramdisk is gonna boot on their node | 16:48 |
| JayF | hiding it as an implementation detail to hide the immense complexity of something that should be a basic action doesn't really improve things | 16:48 |
| TheJulia | I more so mean, if you send like api version 1.110, then you see the new states | 16:48 |
| TheJulia | but the key at the end of the day is it troubleshootable with an up to date client and use context | 16:49 |
| JayF | I'm going to open this up for a full review, but I am skeptical I'll ever find a +1 for this spec unless there is significant complexity that can be removed | 16:50 |
| dtantsur | Or they do it as a service step, which has a nice property of not modifying fundamental assumptions in Ironic | 16:50 |
| TheJulia | The real question even if it sosmehow a service step, is then how do you enable power to be translated across unless we have some flag/mechanism to re-map the step invocation to a series of steps | 16:52 |
| TheJulia | (which was sort of what I was thinking when reading it, as an ideal place to be, just not sure there is a magical get there in advance) | 16:52 |
| JayF | My answer to that would be similar to dtantsur's "this has no power control" flag | 16:53 |
| dtantsur | I don't quite know what you mean by "power to be translated across" and why it has to be several steps. | 16:54 |
| dtantsur | So yeah, in this deployment, normal power actions don't really work. You need to use "power_on_and_unlock" or "reboot_and_unlock" service/clean steps instead. | 16:54 |
| TheJulia | some way to almost replace task.driver.power.power_on to a list of steps | 16:54 |
| TheJulia | Maybe the right thing is a power driver wrapper | 16:55 |
| JayF | that feature comes in a bucket labelled "may also contain 100 security vulnerabilities" | 16:55 |
| dtantsur | You cannot touch the power interface itself - there must still be a way to make the low-level call. Chicken-and-egg problem otherwise. | 16:55 |
| JayF | this is a lot of complexity around one of the things we rely on the most to ensure security: powering off machines | 16:55 |
| TheJulia | well, you make a superset power_interface which understands there is additional workflow, but yeah, that gets weird | 16:55 |
| TheJulia | oh, we can always just power off the machine | 16:56 |
| TheJulia | the disks are locked as a result upon restart | 16:56 |
| dtantsur | I'm not talking about "where to put code" problem, I'm talking about "power action is now a state transition" problem. | 16:56 |
| TheJulia | Its the power_on which needs additional logic | 16:56 |
| TheJulia | fair | 16:56 |
| dtantsur | For low-level logic, whatever they need can be implemented in a derivative of the agent deploy interface most likely. Make sure every time we reboot for any reason, we tell IPA to unlock on return. | 16:57 |
| dtantsur | The user-facing power API will not work as expected, and I'd rather disable it for such nodes than rethink it entirely tbh | 16:58 |
| JayF | ^^ that is gonna be why I'm going to almost certainly -1 it | 16:58 |
| JayF | IPA required for power actions, and IPA boots themselves requiring a power action | 16:58 |
| JayF | means special casing and complexity being added to one of our most common paths | 16:58 |
| JayF | when the real answer is: don't use this style of drive locking with cloud provisioning | 16:58 |
| dtantsur | (On this positive note, I'm going to get some dinner, see you :) | 16:59 |
| TheJulia | The risk they are trying to manage is also, to use such devices to explicitly address risk of theft of the drive | 16:59 |
| TheJulia | or of the chassis | 16:59 |
| TheJulia | goes back to one of the older problems, ensuring data encrypted at rest | 16:59 |
| JayF | I mean, I appreciate the problem. Doesn't mean that Ironic's design choices enable this solution for it. | 17:00 |
| JayF | We have a very similar design in my downstream, but the vendor handles this part in the BMC, in hardware | 17:00 |
| JayF | because we understand that this model of disk locking doesn't fit with cloud server provisioning | 17:00 |
| TheJulia | ... interesting | 17:00 |
| TheJulia | so I was sort of thinking such a model where the bmc or even base firmware enables a case to some sort of control service would be ideal in our model, because we have to do it in advance and don't really have to think about all the workflow implications | 17:19 |
| JayF | yeah, I put a review on that with a strong -1 with a tl;dr: We gotta find a way to do this without booting IPA *to execute a power action* | 17:20 |
| JayF | and that I don't think that's possible with the hardware as laid out in the spec | 17:20 |
| JayF | you need something BMC/firmware side to help plug things in | 17:20 |
| TheJulia | the only real alternative I guess would be something like magical-sed-controller.efi which is signed and able to do it | 17:22 |
| JayF | yeah, most solutions to this problem in the wild are shaped like | 17:22 |
| JayF | remote server holds the key; BMC Queries remote server on boot to get the key and unlock the drive | 17:22 |
| JayF | remote server can say "no" if things are in wrong states, or whatnot | 17:22 |
| TheJulia | The other and likely challenge is an inbound connetion for the BMC to be driving that is likely a possible issue because the BMC is largely limited to the PLDM over the I2C bus | 17:32 |
| TheJulia | so it implies the bmc has a worrisome level of access or engagement with controllers which is, questionable | 17:32 |
| TheJulia | FWIW, I started a thread on the mailing list regarding threat modeling and perceptions. If anyone is interested, it would be good to chime in | 18:21 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/37.0: bugfix/stable: lock constraints for bugfix to 2026.1 https://review.opendev.org/c/openstack/ironic/+/998511 | 19:30 |
| TheJulia | JayF: I guess I'm curious why https://review.opendev.org/c/openstack/ironic-python-agent/+/997637/3/ironic_python_agent/inspector.py ? | 19:48 |
| clif | has anyone run into the fact that mypy doesn't seem to understand Ironic's objects' `fields`? As in, annotate a variable or method args with something like `objects.Node`, then try to access one of it's fields directly | 19:53 |
| clif | mypy will complain about the type not having the attribute and the tox mypy target will fail | 19:53 |
| clif | I'm not sure if there's some way to have mypy take 'fields' into account | 19:54 |
| JayF | TheJulia: backwards compat without having to add `or CONF.inspection_callback_url==mdns` everywhere | 19:54 |
| cardoe | clif: yeah I have the same issue. | 19:54 |
| JayF | TheJulia: think about the unmanaged inspection + mdns use case :( | 19:55 |
| TheJulia | yeah, but your overriding the value to it explicitly in that path by turning it on, but after the config loading | 19:55 |
| JayF | I'm turning it on /based on the inspection url/ | 19:55 |
| TheJulia | yeah, but if your putting a knob in, that should be set upfront | 19:55 |
| JayF | which is an explicit opt in | 19:55 |
| JayF | the goal of this PR: "only do mDNS if opted in" inspection_callback_url=mdns is a valid opt-in, just like [mdns]use_mdns=True is | 19:56 |
| clif | I haven't dug into it yet, I'm hoping there's something better than telling mypy not to type check the line cardoe | 19:56 |
| JayF | TheJulia: see: ironic_python_agent/agent.py:215 for the equivalent, non-api behavior | 19:56 |
| JayF | s/api/inspection/ | 19:56 |
| TheJulia | okay, looking at it through the slightly different lens I guess makes sense | 19:57 |
| TheJulia | it just does feel weird | 19:57 |
| JayF | I'll note: ipa-use-mdns is hooked up | 19:57 |
| JayF | to kernel command line | 19:57 |
| JayF | because if you can modify kernel cli, you can ALREADY set arbitrary commands including Ironic API | 19:58 |
| JayF | so my policy was "anything opted in explicitly, command line or in config file, wins" | 19:58 |
| JayF | this specific use case edge is something I worked a nontrivial amount on :) | 19:59 |
| cardoe | clif: yeah I hope there is as well. | 20:02 |
| TheJulia | I guess makes sense, doesn't completely explicitly close it, but at least the two patches makes it way more opt in, the second one functionally is the real lockout then :) | 20:02 |
| JayF | the [not yet written] third patch, to set a default value to the conf the 2nd patch adds, is a "better fix" potentially | 20:04 |
| JayF | I also think there's a door clearly open to get rid of url=mdns overrides and require config file opt-in on master | 20:05 |
| JayF | this was just written shaped in the ... least offensive, most backportable way possible | 20:05 |
| TheJulia | Yeah, I think it is then, the second patch really being the lockout on other values | 20:09 |
| JayF | TheJulia: to be explicit: 997637 resolves the issue where ipa-api-url being unset would automatically engage mDNS | 20:10 |
| JayF | TheJulia: now, someone must set api-url=mdns OR ipa-use-mdns=true / [mdns]/use_mdns=True for mDNS to engage | 20:11 |
| JayF | (ipa-api-url=null, ipa-use-mdns/use_mdns unset == doesn't automatically mDNS as it used to) | 20:12 |
| JayF | https://review.opendev.org/c/openstack/ironic-python-agent/+/997637/3#message-006d597301c69c8ed2c35b57b6b9d57c41f2a91e I put this in writing here | 20:12 |
| TheJulia | yeah, I guess in my brain I'm more coupling the two together | 20:12 |
| TheJulia | but inspection only *IS* different | 20:13 |
| JayF | ipa-api-url being unset can happen in *weird cases* | 20:13 |
| JayF | ipa-api-url (or inspection-callback-url) == mdns is never an accident | 20:13 |
| JayF | and while I don't love there are separate entrypoints for unmanaged inspection vs regular ironic use case ... meh, it's not that bad :) | 20:13 |
| TheJulia | I guess in the separation, we really don't need a doc update then | 20:13 |
| TheJulia | but we should just deprecate the feature out | 20:13 |
| JayF | yeah, but to me, the worst security hit of this bug -- a configuration being MISSING meaning we just go willy-nilly into mDNS -- is resolved by 997637 | 20:14 |
| TheJulia | yeah | 20:15 |
| Unknown | I've been playing with standalone ironic (metal3 container image to start with) and am amazed how well this has been put together. Thanks so much everyone! | 20:16 |
| *** Unknown is now known as Mike-- | 20:16 | |
| Mike-- | meh was not identified it seems | 20:16 |
| JayF | It's only taken us [check notes] about 12 years ;) | 20:17 |
| Mike-- | it's a very nice piece of software | 20:17 |
| TheJulia | JayF: do I have anything else to review for you that I forgot while you were on PTO? | 20:18 |
| JayF | I don't know the answer to that question yet lol | 20:19 |
| TheJulia | lol | 20:19 |
| JayF | I am still loading in my swapfile :) | 20:19 |
| TheJulia | oh my | 20:19 |
| TheJulia | i told you not to put your swapfiles on floppy disks! | 20:19 |
| TheJulia | ... I did that once | 20:19 |
| clif | cardoe: we may need to load oslo_versionedobjects.mypy plugin | 20:20 |
| clif | trying out a patch to enable this | 20:23 |
| cardoe | sweet | 20:23 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Add spec for sonic and nvue driver replacements. https://review.opendev.org/c/openstack/ironic-specs/+/998363 | 20:31 |
| TheJulia | would we have capacity... (as a group) to do some review jamming tomorrow ? | 20:36 |
| TheJulia | dtantsur: jfyi, I'm digging the diff on https://review.opendev.org/c/openstack/ironic-specs/+/995954/3..4/specs/approved/standalone-networking-v2.rst | 20:46 |
| clif | huzzah progress | 20:47 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add a custom configuration type KernelParameterString https://review.opendev.org/c/openstack/ironic/+/992306 | 20:48 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add UnsafeKernelCommandLine class https://review.opendev.org/c/openstack/ironic/+/995749 | 20:48 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add kernel parameter parsing to _prepare_iso_image https://review.opendev.org/c/openstack/ironic/+/992307 | 20:48 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add kernel parameter parsing to build_extra_pxe_options https://review.opendev.org/c/openstack/ironic/+/992308 | 20:48 |
| opendevreview | Clif Houck proposed openstack/ironic master: Utilize KernelCommmandLine.parse() instead of ironic.common.utils.parse_kernel_params https://review.opendev.org/c/openstack/ironic/+/992309 | 20:48 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add a tool to parse kernel command lines https://review.opendev.org/c/openstack/ironic/+/992641 | 20:48 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add a deprecation notice for disable_kernel_parameter_parsing https://review.opendev.org/c/openstack/ironic/+/995750 | 20:48 |
| opendevreview | Clif Houck proposed openstack/ironic master: Enable `oslo_versionedobjects.mypy` plugin for mypy https://review.opendev.org/c/openstack/ironic/+/999020 | 20:48 |
| JayF | dtantsur: TheJulia: So fun story: I have a need on a near-immediate timescale for portgroups in ironic standalone networking. | 20:48 |
| JayF | cc clif ^ | 20:48 |
| TheJulia | clif: I'm now super happy I didn't start into clif's patches yet | 20:49 |
| clif | oh boy | 20:49 |
| clif | I mean, the mypy thing is minor | 20:49 |
| clif | I just put it up front because one of the patches in that chain benefits from it | 20:49 |
| clif | im happy about the versionedobjects mypy plugin because I was afraid it was going to be a pita to get mypy working for that | 20:52 |
| clif | stephenfin: kudos to you for creating that plugin, thank you | 20:53 |
| TheJulia | cid: hey, https://review.opendev.org/c/openstack/ironic/+/995853 has me a bit worried since tests are not included in the commit. Could we get https://review.opendev.org/c/openstack/ironic/+/995856 folded into 995853 ? | 20:59 |
| TheJulia | easy backport: https://review.opendev.org/c/openstack/ironic/+/998018?usp=search | 21:28 |
| cid | TheJulia, alright. I will quickly do that now. | 21:28 |
| TheJulia | last call on https://review.opendev.org/c/openstack/sushy/+/997463 | 21:31 |
| opendevreview | Merged openstack/ironic-specs master: Firmware endpoint hardware identity https://review.opendev.org/c/openstack/ironic-specs/+/985800 | 21:32 |
| opendevreview | Verification of a change to openstack/ironic master failed: Remove disabling the scope enforcement https://review.opendev.org/c/openstack/ironic/+/996687 | 21:33 |
| opendevreview | Merged openstack/ironic-specs master: Asynchronous sensor data collection https://review.opendev.org/c/openstack/ironic-specs/+/972754 | 21:35 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder stable/2025.2: Fixed rescue ConditionPathExists flag https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/999025 | 21:38 |
| opendevreview | cid proposed openstack/ironic master: Add a container provider for console containers https://review.opendev.org/c/openstack/ironic/+/995853 | 21:38 |
| opendevreview | cid proposed openstack/ironic master: devstack: Support the container console provider https://review.opendev.org/c/openstack/ironic/+/995854 | 21:38 |
| opendevreview | cid proposed openstack/ironic master: Document the container console provider https://review.opendev.org/c/openstack/ironic/+/995855 | 21:38 |
| opendevreview | cid proposed openstack/ironic master: A non-voting job for the container console provider https://review.opendev.org/c/openstack/ironic/+/998934 | 21:38 |
| TheJulia | cardoe: I believe you've wanted this: https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/997659 | 21:41 |
| TheJulia | General reminder: ironic-week-prio open items can use some reviews | 21:59 |
| opendevreview | Merged openstack/networking-baremetal master: doc: include info about l2vni_subport_anchor_network and cross refs https://review.opendev.org/c/openstack/networking-baremetal/+/998530 | 22:04 |
| TheJulia | rpittau: when your up tomorrow, if you have any ideas on https://review.opendev.org/c/openstack/ironic/+/998511 lmk. Thanks in advance! | 22:12 |
| opendevreview | Merged openstack/ironic master: Limit LLDP TLV value length and interfaces https://review.opendev.org/c/openstack/ironic/+/990452 | 22:23 |
| opendevreview | Merged openstack/sushy master: Add openstack-python3-next-jobs https://review.opendev.org/c/openstack/sushy/+/997769 | 22:33 |
| opendevreview | Merged openstack/sushy master: Remove Python 3.10 support https://review.opendev.org/c/openstack/sushy/+/997770 | 22:33 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Add minimum pyproject.toml https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/998639 | 22:40 |
| opendevreview | Merged openstack/ironic master: Enable `oslo_versionedobjects.mypy` plugin for mypy https://review.opendev.org/c/openstack/ironic/+/999020 | 23:26 |
| opendevreview | Merged openstack/networking-generic-switch master: Disable neutron-ovn-agent https://review.opendev.org/c/openstack/networking-generic-switch/+/998644 | 23:29 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!