| opendevreview | Kaifeng Wang proposed openstack/ironic-python-agent-builder master: Exclude unnecessary firmewares https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/959255 | 00:37 |
|---|---|---|
| opendevreview | Maksim Malchuk proposed openstack/bifrost stable/2025.1: Do not pass empty values to instance_info https://review.opendev.org/c/openstack/bifrost/+/959276 | 05:31 |
| opendevreview | Maksim Malchuk proposed openstack/bifrost stable/2024.2: Do not pass empty values to instance_info https://review.opendev.org/c/openstack/bifrost/+/959277 | 05:32 |
| opendevreview | Maksim Malchuk proposed openstack/bifrost stable/2024.1: Do not pass empty values to instance_info https://review.opendev.org/c/openstack/bifrost/+/959278 | 05:32 |
| abongale | good morning ironic o/ | 08:14 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-inspector master: sqlalchemy: Use built-in declarative https://review.opendev.org/c/openstack/ironic-inspector/+/953764 | 09:04 |
| TheJulia | good morning | 12:35 |
| opendevreview | Maksim Malchuk proposed openstack/bifrost stable/2024.1: [2024.1 only] Drop upgrade jobs since 2023.2 brach doesn't exists https://review.opendev.org/c/openstack/bifrost/+/959317 | 12:39 |
| opendevreview | Maksim Malchuk proposed openstack/bifrost stable/2024.1: Do not pass empty values to instance_info https://review.opendev.org/c/openstack/bifrost/+/959278 | 12:40 |
| clif | o/ | 13:30 |
| opendevreview | Pierre Riteau proposed openstack/bifrost master: [WIP] Test CentOS 10 support https://review.opendev.org/c/openstack/bifrost/+/958853 | 14:03 |
| opendevreview | Grzegorz Koper proposed openstack/networking-generic-switch master: Support 802.3ad port groups on Arista devices https://review.opendev.org/c/openstack/networking-generic-switch/+/959118 | 14:31 |
| dtantsur | TheJulia: hi! what was driving the limitation on deploy_kernel/deploy_ramdisk in https://docs.openstack.org/ironic/latest/admin/oci-container-registry.html#available-url-formats? | 14:32 |
| dtantsur | It was just raised as a potential adoption blocker in the Metal3 meeting | 14:32 |
| TheJulia | I don't remember offhand | 14:33 |
| TheJulia | did I note it in the code? | 14:33 |
| TheJulia | (On 2nd of ?4? hour long meetings this morning) | 14:33 |
| dtantsur | Not sure, I've just learnt about this limitation | 14:36 |
| JayF | you'd have to update the tag atomically across two things (x_kernel/x_ramdisk) to avoid issues | 14:38 |
| JayF | I could see it being logical to block off that route | 14:38 |
| dtantsur | I'm personally not terribly worried, compared to inability to use tags | 14:39 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/937896 I haven't found the limitation code | 14:42 |
| JayF | but that's the commit | 14:42 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add a new 'physical_network' field to the Portgroup object https://review.opendev.org/c/openstack/ironic/+/955625 | 15:00 |
| TheJulia | WOOHOO... an hour break | 15:01 |
| TheJulia | but I'm still on a meeting | 15:01 |
| TheJulia | dtantsur: I think someone just needs to try it out at this point. I think the base issue is more to what JayF was noting. It might be something early on that got removed through iteration | 15:11 |
| TheJulia | dtantsur: I *think* it might be tags require authentication, but I think it might just work based upon the current code state | 15:27 |
| TheJulia | Oh, looks like I need to revise those docs anyhow | 15:31 |
| dtantsur | ++ to just trying, yeah | 15:31 |
| TheJulia | Yay for super complicated features! | 15:31 |
| dtantsur | \o/ | 15:31 |
| TheJulia | set it up with quay, easiest path | 15:31 |
| dtantsur | If it makes you feel better, it opens a lot of potential for metal3 | 15:31 |
| dtantsur | Like, getting rid of this whole ipa-downloader stuff (if you know) | 15:31 |
| dtantsur | and from ad hoc image caches | 15:31 |
| TheJulia | the OCP image service doesn't support tags either, fwiw. | 15:32 |
| dtantsur | Yeah, you mentioned. But this all is in the upstream context, so I don't care. | 15:32 |
| TheJulia | yeah | 15:32 |
| dtantsur | (we also research OCI images for a feature downstream, but there we don't care about kernel/ramdisk) | 15:32 |
| TheJulia | I only very early did a bunch with quay, and then shifted to the more restricted OCI image service | 15:33 |
| dtantsur | OCP you mean? I found it very annoying, this image service. | 15:34 |
| TheJulia | yeah, OCP | 15:34 |
| TheJulia | sorry, BRRRAAAINS | 15:34 |
| dtantsur | Everything around authentication is just.. bizzare | 15:34 |
| * dtantsur -> quick walk | 15:34 | |
| TheJulia | It... makes sense. kind of. | 15:35 |
| TheJulia | We're getting pressure to ensure our releases for End of Cycle, we should dot our I's and cross our T's | 16:41 |
| TheJulia | cardoe: I met with some of our neutron folks today. I think we're going to need a spec, at least maybe in just the ironic context, which would talk about the rest of the picture in a standardized model. I do think they finally get the high level flow understood, but there are some questions and beginning to pin down another part of the known allows us to focus on the unknowns as well. | 17:08 |
| TheJulia | cardoe: I *suspect* I may be doing that unless I can somehow convince someone else to draft something | 17:08 |
| cardoe | sounds good. I really then need to sit down and hack on the spec some more. | 17:09 |
| cardoe | oh speaking of dotting Ts and crossing Is... | 17:10 |
| TheJulia | For those watching, this is in regards to attaching baremetal nodes to vxlan networks | 17:10 |
| JayF | I assume at some point we'll have to tackle the bigger problem, e.g. "model baremetal network concepts in neutron" starting with portgroups/bonding | 17:10 |
| cardoe | I've got a further revision to also say "vxlan attach or vlan attach" since the typical case is gonna be vlan attach. | 17:10 |
| JayF | I don't know how associated those issues are | 17:11 |
| TheJulia | I think portgroups are just super weird and maybe more noise for them, I dunno | 17:11 |
| TheJulia | its worth a discussion because they are aware it exists | 17:11 |
| cardoe | JayF: so it works and its werid | 17:11 |
| cardoe | Basically you still have one neutron port but it's binding info is a list instead | 17:11 |
| cardoe | Cause it's all just JSON free form being passed around willy nilly | 17:12 |
| TheJulia | I guess the open question for something else I'm needing to look at is... how do I know what the portchannel name is in the switch to appropriately tag vlans to? | 17:12 |
| TheJulia | Specifically because Cisco gear you have to do vlans on the port channel interface, not the port. | 17:12 |
| TheJulia | Where as some other vendors, its the inverse | 17:12 |
| cardoe | So technically the list is in a JSON object. We're slapping the portgroup info into that. | 17:13 |
| TheJulia | Yeah | 17:13 |
| cardoe | And something I pitched at neutron and got told to write a spec. | 17:14 |
| TheJulia | There is definitely some bridges to build there, in the form of shared context... and actual bridges | 17:15 |
| * TheJulia ducks | 17:15 | |
| cardoe | So one other thing I had on the redfish stuff... rather than all the hardware gathering in one big inspect function. I broke out each piece into _get_$THING_info(). And then drivers like idrac-redfish can override that if they want. | 17:15 |
| cardoe | But I don't love that so the other horrible idea I've had is to share the system sushy object to all the hooks in some fashion and allow hooks to be written to gather and populate more fields. | 17:16 |
| TheJulia | I think there a mixed bag of challenges since some of the BMCs take like a minimum of 20 seconds to return something usable just for one of those areas. | 17:18 |
| * TheJulia makes lunch before trying to do anything else | 17:27 | |
| TheJulia | Is there anything else we feel we must get merged before we release $allthethings ? | 17:27 |
| TheJulia | ^^^ ironic-cores | 17:27 |
| TheJulia | super easy logging improvement: https://review.opendev.org/c/openstack/ironic/+/955821 | 17:47 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Revert "Make ironic-tempest-uefi-redfish-vmedia-4k non-voting" https://review.opendev.org/c/openstack/ironic/+/958580 | 17:50 |
| TheJulia | Looking at the status dashboard, I think we need to go ahead and just cut the release | 17:52 |
| *** Guest22851 is now known as diablo_rojo_phone | 18:12 | |
| TheJulia | Anyone done portgroups with ironic and have the context relatively fresh in their minds? | 18:22 |
| TheJulia | nvmd, found the code | 18:26 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Support transport type as a root device hint https://review.opendev.org/c/openstack/ironic-python-agent/+/955742 | 19:21 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Support transport type as a root device hint https://review.opendev.org/c/openstack/ironic-python-agent/+/955742 | 19:52 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add a prelude for 2025.2 https://review.opendev.org/c/openstack/ironic/+/959389 | 20:51 |
| opendevreview | Merged openstack/ironic master: Revert "Make ironic-tempest-uefi-redfish-vmedia-4k non-voting" https://review.opendev.org/c/openstack/ironic/+/958580 | 21:18 |
| opendevreview | Merged openstack/ironic-python-agent master: Support transport type as a root device hint https://review.opendev.org/c/openstack/ironic-python-agent/+/955742 | 22:24 |
| TheJulia | woot | 22:51 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add a prelude for 2025.2 https://review.opendev.org/c/openstack/ironic/+/959389 | 23:43 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: WIP VXLAN networking https://review.opendev.org/c/openstack/ironic-specs/+/959401 | 23:45 |
| TheJulia | cardoe: ^^^ to try and begin to solidify the discussion a bit more | 23:46 |
| cardoe | Thank you. I literally was just thinking about this. | 23:47 |
| cardoe | Finding shelter from the rain in Nashville. | 23:48 |
| cardoe | +2’d https://review.opendev.org/c/openstack/ironic/+/955821 for the release. | 23:50 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!