| TheJulia | oh wow, its a fun edge case. | 00:05 |
|---|---|---|
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: OCI Basic Auth/Tag access https://review.opendev.org/c/openstack/ironic/+/969314 | 00:46 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: add an OCI artifact registry https://review.opendev.org/c/openstack/bifrost/+/961388 | 00:46 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: bifrost-registry-install: install the ORAS client https://review.opendev.org/c/openstack/bifrost/+/968355 | 00:46 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: Upload a disk image to OCI https://review.opendev.org/c/openstack/bifrost/+/968416 | 00:46 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: Upload to and use artifact from OCI registry https://review.opendev.org/c/openstack/bifrost/+/968417 | 00:46 |
| TheJulia | dtantsur: so, before that, it looks like we're at the agent not being able to talk to the registry, hopefully the re-stack in bifrost addresses that. | 00:46 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Add type annotations https://review.opendev.org/c/openstack/ironic-python-agent/+/958333 | 02:01 |
| *** hroy_ is now known as hroy | 02:37 | |
| opendevreview | OpenStack Release Bot proposed openstack/ironic bugfix/33.0: Update .gitreview for bugfix/33.0 https://review.opendev.org/c/openstack/ironic/+/969559 | 05:42 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Disable metal3 integration job https://review.opendev.org/c/openstack/ironic/+/969565 | 08:06 |
| rpittau | good morning ironic! o/ | 08:10 |
| opendevreview | Merged openstack/ironic bugfix/33.0: Update .gitreview for bugfix/33.0 https://review.opendev.org/c/openstack/ironic/+/969559 | 08:33 |
| FreemanBoss[m] | Good morning ironic | 08:43 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/33.0: Disable metal3 integration job https://review.opendev.org/c/openstack/ironic/+/969566 | 08:56 |
| opendevreview | Jakub Jelinek proposed openstack/ironic master: Add documentation regarding the is_root_volume RAID config https://review.opendev.org/c/openstack/ironic/+/969419 | 13:05 |
| rpittau | if anyone has a moment can I please get reviews on https://review.opendev.org/c/openstack/bifrost/+/964404 ? I think it's ready to be merged | 13:57 |
| TheJulia | good morning | 14:17 |
| dtantsur | TheJulia: morning! I see that CS10 jobs are unhappy on your OCI patch because of something around firewalld? | 14:23 |
| dtantsur | but the ubuntu ones seems to work, so yay? | 14:23 |
| TheJulia | yeah, dunno | 14:36 |
| TheJulia | I'm about to post what claude thinks will fix it | 14:36 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Redfish: guard retrieve_nic_components against MissingAttributeError https://review.opendev.org/c/openstack/ironic/+/969788 | 14:37 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: add an OCI artifact registry https://review.opendev.org/c/openstack/bifrost/+/961388 | 14:37 |
| TheJulia | dtantsur: thanks! | 14:38 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: bifrost-registry-install: install the ORAS client https://review.opendev.org/c/openstack/bifrost/+/968355 | 14:38 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: Upload a disk image to OCI https://review.opendev.org/c/openstack/bifrost/+/968416 | 14:39 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: Upload to and use artifact from OCI registry https://review.opendev.org/c/openstack/bifrost/+/968417 | 14:39 |
| * TheJulia gets out the rebase party | 14:39 | |
| TheJulia | So, the OCI stuff only really gets used when we build artifacts, which is fine, really | 14:46 |
| TheJulia | but I *think* that is only on the centos10 primary job | 14:46 |
| TheJulia | dtantsur: two items are needed, 1) a http fallback option. More so an artifact of OCI registries are really expected to support https and in CI we're just not doing that today, but it makes sense to permit it if opted into so I propoosed 969277. 2) Explicitly supporting basic auth. I did run into a case in the entire code path where a tag can cause an exception to be rasied in an check which is likely not really needed (a | 14:49 |
| TheJulia | bit of a hold over code wise for conductor guard railing). I need to wrap it up, but if you could take a glance at 969314 that would help | 14:49 |
| dtantsur | Nice! (yak shaving, as always..) | 14:50 |
| TheJulia | (it was stupidly hard to find that check. I might add extra debug logging as well to help save other human's sanity. | 14:50 |
| TheJulia | woot woot, looks like it deployed via oci | 15:20 |
| dtantsur | \o/ | 15:20 |
| TheJulia | That was with agent credential passing too | 15:21 |
| TheJulia | I'll verify once the job logs are collected | 15:21 |
| TheJulia | awww yeah: https://d001d2ec41cae0b9083d-58126eca793a2b87b83ccf22d3c107a6.ssl.cf2.rackcdn.com/openstack/583be0fdbcd94eb186ea9ccc0ca73d50/logs/deploy-ramdisk/4e41df61-84b1-5856-bfb6-6b5f2cd3dd11_testvm1_2025-12-04-15-16-41.tar.gz | 15:33 |
| dtantsur | 'image_source': 'oci://192.168.122.1:5500/bifrost/images@sha256:41bf093ec8ee1ac99df282076100baa24efeee03db1ee972738ee80d82bdb2f7' \o/ | 15:54 |
| rpittau | cool! :D | 16:01 |
| opendevreview | Merged openstack/bifrost master: Remove tinyipa support and switch to DIB-based IPA https://review.opendev.org/c/openstack/bifrost/+/964404 | 16:12 |
| rpittau | \o/ | 16:20 |
| opendevreview | Julia Kreger proposed openstack/ironic master: OCI Basic Auth/Tag access https://review.opendev.org/c/openstack/ironic/+/969314 | 17:57 |
| TheJulia | That cleans up the second patch in that series to get it to work. The I'll go tag them but overall should be pretty good. The last bifrost patch which is dependent on the two isr echecking now | 18:08 |
| opendevreview | Clif Houck proposed openstack/ironic master: Generate network plan based on trait based networking config https://review.opendev.org/c/openstack/ironic/+/964895 | 18:27 |
| opendevreview | Clif Houck proposed openstack/ironic master: Trait Based Networking Simulator https://review.opendev.org/c/openstack/ironic/+/966202 | 18:27 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Add configuration options for trait based networking https://review.opendev.org/c/openstack/ironic/+/968054 | 18:27 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Integrate TBN https://review.opendev.org/c/openstack/ironic/+/969835 | 18:27 |
| TheJulia | alegacy: you around? | 18:36 |
| TheJulia | alegacy: specifically: https://review.opendev.org/c/openstack/ironic/+/966469/4/ironic/networking/switch_drivers/driver_translators.py#167 looks like its not extended later and has me scratching my head. | 18:37 |
| alegacy | TheJulia: ... you mean extended by a subclass later; therefore, why the heck does it exist if it is only doing a .get()? | 18:54 |
| TheJulia | I mean, it sets a singular value of like "netmiko_ovs_linux" which I'm not sure pans out so its just confusing and I didn't see any additional changes there so its just feeling confusing | 18:55 |
| TheJulia | dtantsur: looks like the change passed on a recheck of the bifrost change, so everything is ready to review | 18:56 |
| alegacy | ya, originally was doing some massaging of that value but then I settled on just passing thru the ngs values directly so now does nothing and I guess I got lazy and forgot to remove it in favor of just doing the .get() directly when necessary. | 18:56 |
| alegacy | I can certainly remove it. | 18:57 |
| opendevreview | Julia Kreger proposed openstack/sushy master: Don't require Boot and Actions for Systems https://review.opendev.org/c/openstack/sushy/+/967715 | 18:59 |
| TheJulia | If its just not going to be used that way, then yeah lets remove it. If there is a future patch or change to that where it makes sense to keep and use it then perhaps its fine. | 19:00 |
| alegacy | Ok, will re-spin that patch and remove it. I wish I could blame this on my friend Claude, but I believe this was my own dumbass doing. | 19:01 |
| TheJulia | eh, it happens | 19:01 |
| TheJulia | We get ideas, sometimes we draw lines or extend things to do a thing | 19:02 |
| TheJulia | sometimes we change and pivot, its all okay | 19:02 |
| * TheJulia is certainly guilty of that. | 19:02 | |
| TheJulia | I'm going to sit down first thing tomorrow with https://review.opendev.org/c/openstack/ironic-python-agent/+/958333. It is massive since its type work cid has been doing | 19:09 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Use per-node external_http_url for configdrive ISO https://review.opendev.org/c/openstack/ironic/+/901777 | 19:20 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add a kubernetes provider for console container https://review.opendev.org/c/openstack/ironic/+/962242 | 19:24 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Switch graphical console image to ubuntu by default https://review.opendev.org/c/openstack/ironic/+/964201 | 19:25 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Add a kubernetes provider for console container https://review.opendev.org/c/openstack/ironic/+/962242 | 19:26 |
| opendevreview | Merged openstack/ironic stable/2025.1: Add generic redfish compatible vendor name https://review.opendev.org/c/openstack/ironic/+/966781 | 19:42 |
| opendevreview | Merged openstack/ironic master: split error messages in image cache for more clarity https://review.opendev.org/c/openstack/ironic/+/968246 | 20:22 |
| opendevreview | Merged openstack/ironic bugfix/33.0: Disable metal3 integration job https://review.opendev.org/c/openstack/ironic/+/969566 | 20:22 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Separate states code from states constants https://review.opendev.org/c/openstack/ironic/+/969318 | 21:00 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Implement networking service RPC API methods https://review.opendev.org/c/openstack/ironic/+/966468 | 21:02 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add generic switch driver support https://review.opendev.org/c/openstack/ironic/+/966469 | 21:02 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add ironic-networking network interface https://review.opendev.org/c/openstack/ironic/+/966470 | 21:02 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add standalone networking service installation guide https://review.opendev.org/c/openstack/ironic/+/966471 | 21:02 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Improve exception handling in switch driver factory https://review.opendev.org/c/openstack/ironic/+/969852 | 21:02 |
| opendevreview | Clif Houck proposed openstack/ironic master: Trait Based Networking Simulator https://review.opendev.org/c/openstack/ironic/+/966202 | 21:34 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Add configuration options for trait based networking https://review.opendev.org/c/openstack/ironic/+/968054 | 21:34 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Integrate TBN https://review.opendev.org/c/openstack/ironic/+/969835 | 21:34 |
| opendevreview | Clif Houck proposed openstack/ironic master: Trait Based Networking Simulator https://review.opendev.org/c/openstack/ironic/+/966202 | 21:37 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Add configuration options for trait based networking https://review.opendev.org/c/openstack/ironic/+/968054 | 21:37 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Integrate TBN https://review.opendev.org/c/openstack/ironic/+/969835 | 21:37 |
| JayF | TheJulia: I'm rebasing your OVN VTEP removal change. Hopefully we can land it before any other API version additions invalidate it. | 21:50 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Remove ovn vtep support https://review.opendev.org/c/openstack/ironic/+/966140 | 21:55 |
| TheJulia | much appreciated! | 22:01 |
| opendevreview | Merged openstack/ironic master: Generate network plan based on trait based networking config https://review.opendev.org/c/openstack/ironic/+/964895 | 22:55 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!