| rpittau | good morning ironic! happy friday! o/ | 07:20 |
|---|---|---|
| rpittau | dtantsur: about bifrost on noble, apparently noble drops linuxefi in favor of the generic linux | 08:54 |
| rpittau | * New upstream version, 2.12~rc1 | 08:54 |
| rpittau | * build-efi-images: Drop linuxefi, using new loaders now | 08:54 |
| rpittau | I double-checked the presence of linuxefi.mod and it's been indeed removed since mantic | 08:54 |
| rpittau | I guess this impacts us deeply | 08:54 |
| *** jroll01 is now known as jroll0 | 09:00 | |
| opendevreview | Merged openstack/ironic-tempest-plugin master: The adoption test should not depend on the iscsi deploy https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/927029 | 10:28 |
| masghar | Hello ironic! For inspection rules, cid has laid the groundwork for CRUD here: https://review.opendev.org/c/openstack/ironic/+/918303 Reviews are very welcome! | 10:56 |
| masghar | Also, do you think it would be better to add the rules logic in a different patch (for easier reviews, for example)? (This one itself is pretty large) | 10:58 |
| dtantsur | rpittau: it is a huge problem, going well beyond bifrost, yes | 11:58 |
| dtantsur | we need to check if grub has any fallback mechanism where we could try one set of commands and fall back to the other | 11:59 |
| dtantsur | masghar: we generally try to avoid landing API without a backend (because any Ironic commit has to be fully functional) | 12:00 |
| dtantsur | in this case, maybe too few people actually care if we land them close to each other | 12:00 |
| dtantsur | ideally, we should land the rule engine first, then the API. but I can agree that it's less convenient for testing. | 12:01 |
| rpittau | dtantsur: this is probably what we're looking for https://www.gnu.org/software/grub/manual/legacy/Booting-fallback-systems.html | 12:46 |
| dtantsur | that could work, good catch! | 12:56 |
| rpittau | probably mention that during the meeting on monday, noble support is a community goal for next cycle so we'll have to act on it | 13:00 |
| dtantsur | indeed | 13:00 |
| dtantsur | wanna file a bug? | 13:01 |
| rpittau | yep, will do | 13:02 |
| rpittau | here https://bugs.launchpad.net/ironic/+bug/2081305 | 13:08 |
| rpittau | I need to skip now, we'll discuss on Monday, but I guess I'll see some talks before that :) | 13:09 |
| dtantsur | sure, thank you | 13:10 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost stable/2024.1: Fix missed occurrence of SETUPTOOLS_USE_DISTUTILS https://review.opendev.org/c/openstack/bifrost/+/930024 | 13:13 |
| opendevreview | Merged openstack/ironic-specs master: Support for hardware that cannot be powered off https://review.opendev.org/c/openstack/ironic-specs/+/926654 | 13:15 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Provide consistent spelling of the microversion header https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/930028 | 13:28 |
| dtantsur | Your Friday facepalm ^^^ | 13:28 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Check inspection data and abortion in the standalone tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/927928 | 13:29 |
| opendevreview | OpenStack Release Bot proposed openstack/bifrost stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/bifrost/+/930031 | 13:50 |
| opendevreview | OpenStack Release Bot proposed openstack/bifrost stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/bifrost/+/930032 | 13:50 |
| opendevreview | OpenStack Release Bot proposed openstack/bifrost master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/bifrost/+/930033 | 13:50 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/ironic-inspector/+/930034 | 13:50 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/ironic-inspector/+/930035 | 13:51 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/ironic-inspector/+/930036 | 13:51 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/930037 | 13:51 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/930038 | 13:51 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/930039 | 13:51 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/930040 | 13:51 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/930041 | 13:51 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/930042 | 13:51 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/ironic-python-agent/+/930043 | 13:52 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/ironic-python-agent/+/930044 | 13:52 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/ironic-python-agent/+/930045 | 13:52 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/ironic-ui/+/930046 | 13:52 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/ironic-ui/+/930047 | 13:52 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/ironic-ui/+/930048 | 13:52 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/ironic/+/930049 | 13:53 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/ironic/+/930050 | 13:53 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/ironic/+/930051 | 13:53 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/networking-baremetal/+/930052 | 13:53 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/networking-baremetal/+/930053 | 13:54 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-baremetal master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/networking-baremetal/+/930054 | 13:54 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-generic-switch stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/networking-generic-switch/+/930055 | 13:54 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-generic-switch stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/networking-generic-switch/+/930056 | 13:54 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-generic-switch master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/networking-generic-switch/+/930057 | 13:54 |
| JayF | I'm going to go grab some breakfast but if they're not landed by then, I'm going to go through and single core land all of the rote updates for Branch cutting. If someone wants to toss plus twos on them in the intervening 15-20 minutes that'd be cool | 14:12 |
| cardoe | dtantsur: so I saw your comment around the DHCP and metal3. Something I was thinking about was writing another dhcp driver or more so a wrapper of the existing dnsmasq one for now. But defining some CRDs around what we need to transmit to dnsmasq and storing those in k8s directly from the conductor. Then getting to the point of not having ironic write the dnsmaq file directly but a controller doing it for us. | 14:58 |
| cardoe | The CRDs would become like the RPC method to communicate that out. | 14:58 |
| cardoe | If that would be something interesting to metal3, I'd be happy to collaborate. | 15:01 |
| JayF | How would one set a firmware setting under bios/oem? e.g.: https://support.hpe.com/hpesc/public/docDisplay?docId=a00094361en_us&docLocale=en_US lists redfish/v1/systems/1/bios/oem/hpe/kmsconfig/ as a location with settings we can enable | 15:10 |
| JayF | but I can't make heads or tails of if it's possible to get our BiosInterface to set these; I'm thinking no right now but not certain | 15:11 |
| cardoe | JayF: you cannot | 15:15 |
| JayF | cardoe: can you help me understand why?\ | 15:15 |
| JayF | this is a bios setting to operators, even if redfish doesn't think so, so we need to hook it up for em | 15:16 |
| cardoe | I absolutely agree. | 15:16 |
| cardoe | Come to my PTG session... "sushy-oem-idrac future and a future for other oems" | 15:16 |
| JayF | can you help me understand the shape of the problem as someone who doesn't have access to hardware? | 15:16 |
| cardoe | Yep. | 15:16 |
| cardoe | So sushy essentially only talks to the non-OEM endpoints | 15:16 |
| cardoe | So Ironic only exposes those. | 15:17 |
| JayF | fun | 15:17 |
| cardoe | For a couple of things, the idrac driver overrides an interface. Which then uses the sushy-oem-idrac library, which talks to the OEM interface and provides that functionality. | 15:17 |
| JayF | yeah, that fits with my mental model and what I got from sushy, cool | 15:18 |
| cardoe | So what I'd like for us to do is just merge sushy-oem-idrac into sushy proper and have the oem in there. | 15:18 |
| JayF | yeah, but I need ilo oem stuff :D | 15:18 |
| JayF | so we can do both | 15:19 |
| JayF | cardoe: do you have any workarounds? any tricks for downstream patch to do it? | 15:19 |
| cardoe | But today it's a different interface still. You need to load and use the oem interfaces that sushy has | 15:19 |
| TheJulia | Why can’t folks just standardize or use the standard models… | 15:19 |
| JayF | e.g. sushy.do_specific_thing(blah) | 15:19 |
| cardoe | JayF: yeah | 15:19 |
| cardoe | TheJulia: so in a bunch of cases the vendors provided the feature before DMTF standardized it. They're then slow to update. | 15:20 |
| cardoe | JayF: So where we use sushy, we monkey patch the hell out of it. | 15:20 |
| JayF | TheJulia: I told my co-worker downstream that if stuff like this didn't exist, ironic probably wouldn't either lol | 15:20 |
| JayF | cardoe: that's a really bad answer :( | 15:20 |
| cardoe | I agree with you. | 15:21 |
| cardoe | That's why I wanna come up with a solution that the upstream community is happy with. | 15:21 |
| TheJulia | cardoe: I know, or intentionally implement different modeling | 15:21 |
| TheJulia | Below the standardization which it is incompatible | 15:21 |
| cardoe | So like we have sushy-rax. Which depends on sushy-oem-idrac and also has some ilo stuff in it. You use sushy-rax instead of sushy. | 15:22 |
| JayF | cardoe: is that a public fork? | 15:22 |
| cardoe | And when you try to hit the standard sushy interface, if the hardware needs it we just add that in | 15:22 |
| cardoe | It's not. | 15:22 |
| cardoe | I can probably make it public. | 15:23 |
| JayF | I doubt we'd use it directly, but it would be inspirational for both upstream and downstream ideas | 15:23 |
| cardoe | Yeah so my goal is for it to die. | 15:23 |
| cardoe | -ETIME :/ | 15:24 |
| cardoe | https://bugs.launchpad.net/sushy/+bug/2075979 so that bug for example. | 15:25 |
| JayF | I'm going to link that under sushy-oem-idrac | 15:25 |
| JayF | for ptg topic | 15:25 |
| JayF | that's a useful contextual bug | 15:25 |
| cardoe | So even when I got Dell to fix the firmware | 15:26 |
| cardoe | man I scattered brained today sorry. | 15:26 |
| JayF | Yeah, I think some of our headache here is that I suspect we'll be touching settings that may not make sense or even exist in most hardware in the world, even of similar brands | 15:26 |
| cardoe | So the firmware fix is there's not NIC.Slot.1-1 and NIC.Slot.1-1-1. It's just NIC.Slot.1-1 | 15:26 |
| JayF | so I need a path to ensure we can, preferably without patching the world, have a little more flexibility in OOB stuff like we do for IB stuff | 15:27 |
| cardoe | https://opendev.org/openstack/sushy/src/commit/cd94fc00b69827b488fee15367e6b69bbf09bea1/sushy/resources/system/ethernet_interface.py#L59 that function which is used by the in-band inspector only sees 2 NICs on my hardware | 15:27 |
| cardoe | Not the 6 that exist. | 15:27 |
| JayF | especially for people who have the experience of "I used to do X in a hardware manager in-band using OEM tools" to "hey look Ironic provides an interface ... but it can't do $5_percent_of_things" and it's not as trivially pluggable | 15:27 |
| cardoe | So in sushy-rax, that summary function is monkey patched | 15:28 |
| cardoe | It extends out and uses https://review.opendev.org/c/openstack/sushy/+/927838 | 15:28 |
| cardoe | To query not just the EthernetInterfaces but Port interfaces that are of type=Ethernet | 15:28 |
| cardoe | To provide me all the data. | 15:29 |
| cardoe | Probably a bad example cause of two endpoints. | 15:29 |
| cardoe | Your BIOS stuff might be a better example. | 15:29 |
| JayF | well, the thing that made me hrm about that | 15:30 |
| JayF | is it's not a /oem on the root | 15:30 |
| JayF | it's /bios/oem | 15:30 |
| cardoe | https://opendev.org/openstack/ironic/src/commit/5857ca0cd5bc4949f77b9f3b6c3e0d7ab97f6b88/ironic/drivers/modules/redfish/bios.py#L82 | 15:30 |
| JayF | I don't know redfish well enough to know if that's meaningful but it seems like at least partially in the right place? | 15:30 |
| cardoe | So that's another spot we've monkey patched | 15:31 |
| cardoe | So we always load an OEM Manager at the top level. | 15:31 |
| cardoe | It might be a no-op. | 15:31 |
| JayF | I'd love to know what actual url that translates to for most gear | 15:32 |
| JayF | so I could get my downstream to see if the setting is in there | 15:32 |
| JayF | hmmm | 15:32 |
| cardoe | But that line of code fetches not just system.bios.attributes but it fetches system.oem.bios.attributes | 15:32 |
| JayF | so it might just work? | 15:32 |
| cardoe | I can set those iLO settings locally. | 15:32 |
| JayF | that is system.oem.bios -- mine would be system.bios.oem? | 15:32 |
| cardoe | Well that's the attribute structure that sushy is making | 15:36 |
| cardoe | oh yes sorry it's system.bios.oem.attributes | 15:36 |
| JayF | so you're saying system.bios.oem.attributes had to be patched in for those settings to work | 15:37 |
| JayF | I'd /really really/ love to see that patched code | 15:37 |
| cardoe | BUT the attribute field is coming from base.BIOSInterface | 15:37 |
| cardoe | So there's ILOBios(base.BIOSInterface) | 15:37 |
| cardoe | So sushy creates an instance of RedfishBIOS at system.bios | 15:39 |
| cardoe | We create an instance of ILOBios at system.bios.oem | 15:40 |
| cardoe | If the ILO OEM extension was applied to that connection I think? | 15:40 |
| JayF | can you just see if you can share the code? It's OSS and I know, at least when I was there, it was pretty easy to get a 'yes' to that for patched oss | 15:41 |
| cardoe | openstack baremetal node bios setting list against a DL380 Gen10 machine shows "KeyManagementType" as an available option. | 15:41 |
| cardoe | Yeah | 15:41 |
| JayF | cardoe: bonus points if the patched version you're running happens to be xena :P | 15:42 |
| cardoe | uh it's older than that | 15:48 |
| cardoe | the problem is that this code is now more like sushy's cousin | 15:48 |
| cardoe | Cause the common ancestor according to git is 668c40dec4cb65d658aeb6b22e2009aac963bc8b | 15:49 |
| cardoe | and the ironic using it I think is train | 15:49 |
| JayF | I'd imagine so | 15:49 |
| JayF | I am not under the impression I'm ever going to like, use it directly | 15:49 |
| JayF | but use it as inspiration for understanding the problem and helping my downstream | 15:49 |
| dtantsur | cardoe: I've definitely seen a k8s controller for dnsmasq, although seemingly abandoned. | 15:56 |
| dtantsur | Let me see if my browsing history retains it.. | 15:57 |
| dtantsur | cardoe: yeah https://github.com/aenix-io/dnsmasq-controller | 15:57 |
| dtantsur | it won't solve the HA issues, I suspect, nor the stability problems | 15:57 |
| cardoe | yeah I have too. But I was thinking more for generically DHCP options for a device instead of something specifically managing dnsmasq | 15:57 |
| cardoe | So that another backend could eventually serve it up | 15:58 |
| cardoe | JayF: requested to change visibility... | 15:58 |
| JayF | thank you :D | 15:59 |
| dtantsur | cardoe: we could invent something for metal3 but I'd rather get it all the way to Ironic so that more community can benefit from it. | 16:00 |
| JayF | I mean, isn't kea dhcp backend pretty much ideal for kubernetes? | 16:04 |
| JayF | I thought it was a cluster-friendly server, since you can have a centralized backend store | 16:04 |
| dtantsur | Probably? All I say is that I'd rather not have anything metal3-specific. | 16:05 |
| dtantsur | Especially since the metal3 team is also small and overloaded.. | 16:05 |
| JayF | I always feel like the more we put in ironic the better it is for everyone | 16:05 |
| JayF | talking about both openstack and metal3-related features | 16:05 |
| JayF | it's like, you can help one community or the other, but if you put it in ironic directly, both sides benefit | 16:05 |
| * dtantsur nods | 16:06 | |
| JayF | not always an option but when it is, it's awesome | 16:06 |
| JayF | I try to convey that feel by helping folks in slack, although my hit rate on "stuff that's different in metal3" is pretty bad lol | 16:06 |
| dtantsur | welll.. there are VERY few people who understand both Ironic and Metal3 well. One hand is more than enough to count. | 16:07 |
| JayF | Yep, and by folks like you, riccardo, adam, and hopefully ma little bit of me, we can pull em a little closer | 16:08 |
| dtantsur | ++ | 16:08 |
| JayF | does metal3 intersect at all with project mercury, in terms of use case? | 16:10 |
| JayF | I know almost nothing about networking in the k8s world | 16:10 |
| dtantsur | it very well can in the future | 16:10 |
| JayF | one of the things I put on that PTG topic | 16:10 |
| dtantsur | we have our own IPAM now but it's pretty limited | 16:10 |
| JayF | is that we need like, a 5 year vision if we want people to invest more in that | 16:10 |
| JayF | if I can tell the story of where we're going, it's a lot easier to get the time to invest it | 16:11 |
| JayF | because -- at least for some of my use cases -- we'll need more than just project mercury to overhaul an architecture | 16:11 |
| TheJulia | We also need to show people a viable path which is really not the same old same old in terms of vendor ml2 plugin lockin pain | 16:11 |
| TheJulia | but yeah | 16:11 |
| * TheJulia returns to pto by hunting through electrical schematics to find a short circuit | 16:12 | |
| JayF | the theme for the PTG gonna be "make the present better by designing the future" :D | 16:12 |
| TheJulia | +++++ | 16:12 |
| JayF | we're going to prove you /can/ have your cake and eat it too | 16:12 |
| JayF | because we can't let the clouds that exist starve as we're getting there | 16:12 |
| TheJulia | ++++ | 16:15 |
| TheJulia | demos, many demos | 16:15 |
| * JayF now finally getting around to approving those branch cut passwords lol | 16:22 | |
| JayF | s/passwords/patchsets/ | 16:22 |
| JayF | I always love approving these, it's like a mini-release-celebration in my office :D | 16:36 |
| dtantsur | ironic-tempest-plugin is absolutely cursed | 16:37 |
| JayF | :( that inspector change still flailing (I left the typo because it fits lol) | 16:40 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Check inspection data and abortion in the standalone tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/927928 | 16:43 |
| opendevreview | Merged openstack/networking-generic-switch stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/networking-generic-switch/+/930055 | 16:44 |
| dtantsur | Some silly fixes for your review pleasure on Friday afternoon: https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/930028 https://review.opendev.org/c/openstack/ironic-python-agent/+/929766 https://review.opendev.org/c/openstack/bifrost/+/928896 | 16:45 |
| JayF | +2 all three, I already was looking at the first | 16:46 |
| opendevreview | Merged openstack/ironic-ui stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/ironic-ui/+/930046 | 16:47 |
| opendevreview | Merged openstack/ironic-ui stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/ironic-ui/+/930047 | 16:47 |
| opendevreview | Merged openstack/networking-generic-switch stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/networking-generic-switch/+/930056 | 16:48 |
| opendevreview | Merged openstack/ironic-python-agent stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/ironic-python-agent/+/930043 | 16:50 |
| opendevreview | Merged openstack/ironic-python-agent stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/ironic-python-agent/+/930044 | 16:54 |
| opendevreview | Merged openstack/networking-baremetal stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/networking-baremetal/+/930052 | 16:55 |
| opendevreview | Merged openstack/networking-baremetal stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/networking-baremetal/+/930053 | 16:55 |
| opendevreview | Merged openstack/ironic stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/ironic/+/930049 | 17:01 |
| opendevreview | Merged openstack/ironic stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/ironic/+/930050 | 17:01 |
| opendevreview | Merged openstack/ironic-inspector stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/ironic-inspector/+/930034 | 17:01 |
| opendevreview | Merged openstack/ironic-inspector stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/ironic-inspector/+/930035 | 17:01 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/930040 | 17:06 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/930041 | 17:06 |
| opendevreview | Merged openstack/ironic-prometheus-exporter stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/930037 | 17:08 |
| opendevreview | Merged openstack/ironic-prometheus-exporter stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/930038 | 17:08 |
| JayF | dtantsur: https://review.opendev.org/c/openstack/ironic-python-agent/+/928779 is conflicting in scope, if not in exact code, with changes I've got in flight | 18:19 |
| JayF | dtantsur: I'm going to operate under the assumption you're OK with me pulling that into my chain so it is happier | 18:20 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Remove use of ironic_lib i18n module https://review.opendev.org/c/openstack/ironic-python-agent/+/930080 | 18:20 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Migrate to oslo.utils-based format_inspector https://review.opendev.org/c/openstack/ironic-python-agent/+/928463 | 18:29 |
| opendevreview | Jay Faulkner proposed 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:29 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Remove use of ironic_lib i18n module https://review.opendev.org/c/openstack/ironic-python-agent/+/930080 | 18:29 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: Migrate more trivial code from ironic-lib https://review.opendev.org/c/openstack/ironic-python-agent/+/928779 | 18:29 |
| JayF | ^^ part of a 4 patch series, migrate to oslo.utils format_inspector, refactor/cleanup disk_utils usage, remove use of ironic_lib i18n module (we had ~10 strings with _() in IPA, and it's never translated -- we should do it proper or not at all), and the rebased version of dtantsur's patch | 18:30 |
| JayF | I suspect that first patch might need to be hrm'd at given our experience with raw vs gpt returns | 18:31 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!