| opendevreview | Merged openstack/ironic master: fix: remove unnecessary trailing slash https://review.opendev.org/c/openstack/ironic/+/968150 | 00:42 |
|---|---|---|
| opendevreview | Merged openstack/ironic master: ci: remove snmp ci job https://review.opendev.org/c/openstack/ironic/+/963215 | 01:22 |
| opendevreview | Merged openstack/ironic master: Configuration file for Trait Based Networking https://review.opendev.org/c/openstack/ironic/+/962598 | 01:44 |
| opendevreview | Merged openstack/ironic master: chore: disable pycodestyle and pyflakes double runs https://review.opendev.org/c/openstack/ironic/+/968151 | 01:54 |
| opendevreview | Steve Baker proposed openstack/ironic master: OciImageService detect bootc image https://review.opendev.org/c/openstack/ironic/+/966760 | 01:55 |
| opendevreview | Steve Baker proposed openstack/ironic master: Swap to bootc deploy interface for bootc images https://review.opendev.org/c/openstack/ironic/+/966761 | 01:55 |
| opendevreview | Verification of a change to openstack/ironic master failed: chore: fix spelling mistakes https://review.opendev.org/c/openstack/ironic/+/968152 | 03:41 |
| opendevreview | Merged openstack/ironic master: chore: fix spelling mistakes https://review.opendev.org/c/openstack/ironic/+/968152 | 05:51 |
| opendevreview | Merged openstack/ironic master: Remove inspector inspect interface https://review.opendev.org/c/openstack/ironic/+/967476 | 06:50 |
| rpittau | good morning ironic! o/ | 07:40 |
| opendevreview | Riccardo Pittau proposed openstack/tenks master: Modernize and fix CI https://review.opendev.org/c/openstack/tenks/+/968200 | 08:53 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Update release mappings for new ironic release 33.0 https://review.opendev.org/c/openstack/ironic/+/968306 | 09:53 |
| rpittau | preparation for next release ^ | 09:54 |
| rpittau | I put 33.0 to highlight the removal of inspector interface | 09:54 |
| opendevreview | Merged openstack/ironic master: Fail flat driver rebind when no VIFs are bound https://review.opendev.org/c/openstack/ironic/+/967778 | 10:38 |
| opendevreview | Merged openstack/ironic master: chore: bump to latest pre-commit tools https://review.opendev.org/c/openstack/ironic/+/968153 | 10:53 |
| dtantsur | rpittau: I assume you don't expect more API or RPC changes? | 11:58 |
| dtantsur | I know that alegacy's patches add a new entry to the mappings | 11:58 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: WIP add an OCI artifact registry https://review.opendev.org/c/openstack/bifrost/+/961388 | 12:38 |
| alegacy | cid: TheJulia: how would you feel about if I do a: s/portchannel/lag/ to rename that to a more vendor agnostic term. | 12:44 |
| alegacy | sorry, didn't mean cid: there... i mean cardoe: ^^ | 12:44 |
| * cid nodes in the background | 12:45 | |
| cardoe | Works for me | 12:55 |
| alegacy | k, thanks | 12:55 |
| rpittau | dtantsur: I was planning to cut bugfix this week, will alegacy patches land in time? | 13:36 |
| cardoe | rpittau: on stable branches? | 14:12 |
| rpittau | cardoe: on master | 14:13 |
| cardoe | ah okay | 14:13 |
| cardoe | The only show stopper on master for me is https://review.opendev.org/c/openstack/ironic/+/968258 | 14:14 |
| cardoe | I have to update the tests because the tests validated the broken behavior. | 14:14 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/968226 that'll fix the reported bug that cid triaged during the meeting yesterday | 14:16 |
| TheJulia | good morning | 14:16 |
| cardoe | we landed the inspector removal so the bug fix release will be the first without inspector | 14:17 |
| TheJulia | alegacy: I guess that works for me | 14:19 |
| rpittau | yeah I actually wanted to cut bugfix now because of that change :) | 14:19 |
| TheJulia | because of which change? | 14:19 |
| rpittau | TheJulia: the inspector interface removal | 14:19 |
| TheJulia | seems like cardoe has good reason to hold 33.0's release until he has tests written to confirm/verify the broken heavior he has with images, if he can make it appear quickly :) | 14:20 |
| cardoe | Let me make coffee and I'll do it right now. | 14:20 |
| TheJulia | k | 14:21 |
| cardoe | 100% the patch I did is correct and fixes the behavior. 100% the tests are wrong. | 14:21 |
| cardoe | We attempt to read the source format type from the dest file which doesn't yet exist because we create the dest file a couple of lines down. | 14:21 |
| cardoe | The problem is it's a spider web of "path" and "path_tmp" variables | 14:21 |
| rpittau | yeah, I... nvm :D | 14:21 |
| rpittau | I haven't proposed 33.0 yet, I was just updating the release mapping | 14:22 |
| rpittau | I don;t think cardoe's change influnces that | 14:22 |
| cardoe | The tests just do an assert.called_with() and its testing that we read the source format from the dest file. | 14:23 |
| TheJulia | dtantsur: any issue if I rev your bifrost OCI registry change? | 14:25 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: WIP add an OCI artifact registry https://review.opendev.org/c/openstack/bifrost/+/961388 | 14:28 |
| TheJulia | lets see! | 14:28 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2025.2: Fail flat driver rebind when no VIFs are bound https://review.opendev.org/c/openstack/ironic/+/968326 | 14:39 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2025.1: Fail flat driver rebind when no VIFs are bound https://review.opendev.org/c/openstack/ironic/+/968330 | 14:39 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/31.0: Fail flat driver rebind when no VIFs are bound https://review.opendev.org/c/openstack/ironic/+/968331 | 14:40 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/30.0: Fail flat driver rebind when no VIFs are bound https://review.opendev.org/c/openstack/ironic/+/968332 | 14:40 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Remove tinyipa support and switch to DIB-based IPA https://review.opendev.org/c/openstack/bifrost/+/964404 | 14:47 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Remove tinyipa support and switch to DIB-based IPA https://review.opendev.org/c/openstack/bifrost/+/964404 | 15:12 |
| TheJulia | dtantsur: fyi, next rev for your bifrost change already staged locally, which I suspect based upon the current failures will pass CI once proposed | 15:22 |
| dtantsur | TheJulia: sure, feel free to update the patch whichever way you want (just let me know when you're done) | 15:23 |
| TheJulia | dtantsur: Were you thinking a separate change to upload and use an artifact? | 15:23 |
| dtantsur | My plans for next patches are: download ORAS client to /opt/bifrost/bin and finally switch the default deployment logic to use the registry instead of a file in httpd | 15:23 |
| dtantsur | heh, good timing :) | 15:24 |
| dtantsur | yes, I did not want to keep piling things into this already large change | 15:24 |
| TheJulia | fair enough, so if you want to start on a separate change then, you definitely can, looks like the current one is close | 15:24 |
| dtantsur | Got it. Time to exercise a bit, then I can task my Claude with the ORAS one | 15:26 |
| TheJulia | heh, okay | 15:27 |
| TheJulia | Enjoy | 15:27 |
| * TheJulia makes a release note | 15:27 | |
| TheJulia | Changing your generated by to assisted by, btw. | 15:33 |
| dtantsur | fair, it has received enough human attention | 15:33 |
| TheJulia | Yeah, that is what I was thinking looking at it and the revs | 15:34 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix: use the correct path to the image when deep image inspection is off https://review.opendev.org/c/openstack/ironic/+/968258 | 15:50 |
| cardoe | That should pass the tests. Sorry. I had a 1:1 | 15:51 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/968226 if you guys wanna close out that bug | 15:52 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: add an OCI artifact registry https://review.opendev.org/c/openstack/bifrost/+/961388 | 16:09 |
| opendevreview | Julia Kreger proposed openstack/sushy master: Don't require Boot and Actions for Systems https://review.opendev.org/c/openstack/sushy/+/967715 | 16:51 |
| TheJulia | hmmmmm | 17:01 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: Set upper_bound column in bios_settings table to bigint https://review.opendev.org/c/openstack/ironic/+/968348 | 17:01 |
| TheJulia | dtantsur: so slight problem, keystone and the registry both want port 5000 | 17:03 |
| dtantsur | lol | 17:08 |
| TheJulia | yeaaah... | 17:08 |
| dtantsur | TheJulia: it should not be a big deal to redirect the registry to a different port | 17:08 |
| TheJulia | okay, I'll change it to $something | 17:08 |
| dtantsur | TheJulia: can we have a port in oci:// URLs? | 17:08 |
| TheJulia | afaik, yes | 17:08 |
| TheJulia | keep in mind, quay runs as just a normal https service | 17:09 |
| TheJulia | so you would have had to explicitly define 5000 to use it as is today | 17:09 |
| dtantsur | oh yeah, true, so even 5000 would need to be included explicitly | 17:09 |
| dtantsur | right | 17:09 |
| dtantsur | anyway, if it does not work, we fix it :) | 17:09 |
| TheJulia | ok | 17:11 |
| opendevreview | Merged openstack/ironic master: Update release mappings for new ironic release 33.0 https://review.opendev.org/c/openstack/ironic/+/968306 | 17:11 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: add an OCI artifact registry https://review.opendev.org/c/openstack/bifrost/+/961388 | 17:14 |
| cardoe | Does conductor need to host the files locally in its http server with OCI? | 17:16 |
| dtantsur | cardoe: not necessarily, depending on image_download_source | 17:16 |
| dtantsur | TheJulia: re docs: I was thinking about an update to https://docs.openstack.org/bifrost/latest/user/architecture.html (I can do it in a follow-up if you run out of cycles) | 17:16 |
| TheJulia | dtantsur: The more I was thinking about it, the more I was thinking it might just be best as a follow-up or with the actual OCI usage chagnes | 17:17 |
| TheJulia | just so we paint the entire picture at once | 17:18 |
| cardoe | So we generate a url which doesn’t need auth? | 17:18 |
| dtantsur | I think the conductor passes auth to IPA together with the URL. I don't remember for sure, TheJulia? | 17:18 |
| TheJulia | If auth is required, we're able to pass the session token out to the agent. Althoguh, standalone auth wise using with an htpasswod, I have no idea because its a digest auth, nor an actual session auth with a shared token | 17:19 |
| cardoe | I would love to get rid of conductor’s http server. | 17:22 |
| TheJulia | The conductor's local http server is still required for conductor side local caching and ipxe | 17:23 |
| dtantsur | yeah, we're not quite there yet. Virtual media also serves the ISO locally. | 17:23 |
| * dtantsur claude goes brrrrr | 17:24 | |
| TheJulia | Yeah, there is basically no getting rid of it short of kicking entire use cases to the curb | 17:24 |
| dtantsur | The biggest conundrum IMO is that both BMCs (for virtual media) and iPXE can be very picky about the source of their downloads. | 17:25 |
| dtantsur | Otherwise, we'd be able to just serve a virtual media ISO directly to the BMC | 17:25 |
| TheJulia | Its a challenge of the range of use cases, and what makes ironic powerful is a variety, not any singular pattern one must fit into | 17:31 |
| TheJulia | Granted, specific flows/patterns/tool/approaches may do some of the needful and eliminate the need in some cases, but that will always be a specific pattern and configuration which may not apply at all to what someone else needs to do | 17:32 |
| TheJulia | so technically, one could upload the vmedia iso to a swift registry as well, but we also know some vendors don't like things like & and ? in those URLs either. :\ | 17:34 |
| dtantsur | The ORAS patch is ready locally, are you by chance updating the parent patch now? | 17:34 |
| TheJulia | what is in CI should be good since I changed the port to 6000 on it | 17:34 |
| TheJulia | its running presently | 17:35 |
| dtantsur | ah, I can probably use --no-rebase to avoid changing it.. or not? | 17:35 |
| TheJulia | you can | 17:35 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: bifrost-registry-install: install the ORAS client https://review.opendev.org/c/openstack/bifrost/+/968355 | 17:35 |
| dtantsur | in any case, it worked. let's see what the CI says (I still don't have a working local bifrost, sigh) | 17:36 |
| TheJulia | LGTM, next would be upload I guess | 17:36 |
| dtantsur | yeah, it will require a bit more thinking than just telling claude what to do | 17:37 |
| TheJulia | yeah | 17:37 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Add configuration options for trait based networking https://review.opendev.org/c/openstack/ironic/+/968054 | 17:37 |
| dtantsur | TheJulia: TLS settings could be a potential headache | 17:39 |
| TheJulia | That is definitely a thing | 17:40 |
| dtantsur | oh, and I wonder if port 6000 is a good choice: https://www.speedguide.net/port.php?port=6000 | 17:40 |
| TheJulia | 5500 ? | 17:43 |
| dtantsur | TheJulia: we could even do 5050 to confuse people migrating from inspector :-P | 17:43 |
| dtantsur | (seriously though, maybe reusing a previously known code is not a terrible idea: it may already be enabled on firewalls) | 17:46 |
| dtantsur | s/code/port/ but you got it | 17:47 |
| TheJulia | That is just evil | 17:48 |
| TheJulia | That being said its looking good so far for 6000. I also fixed some of the value usage to use templated values | 17:49 |
| dtantsur | https://docs.openstack.org/ironic/latest/admin/oci-container-registry.html doesn't say much about the TLS situation, we need to add it | 17:54 |
| TheJulia | AIUI, non-https usage is actually not supposed to work for that stuff | 17:56 |
| TheJulia | its *all* expected to be httpsified at all times | 17:56 |
| dtantsur | yep, but the only configuration we have is webserver_verify_ca | 17:56 |
| dtantsur | and even there we only accept a boolean and crash if a path is provided :) | 17:57 |
| dtantsur | I want to do 2 things: first have an option specific to oci (with a fallback to webserver_verify_ca), second also support paths | 17:58 |
| * dtantsur hack hack hack | 17:58 | |
| TheJulia | reasonable I guess for self signed | 18:01 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Improve TLS support for OCI images https://review.opendev.org/c/openstack/ironic/+/968361 | 18:09 |
| dtantsur | something like this ^^ | 18:10 |
| opendevreview | Merged openstack/ironic stable/2025.2: fix: glance image member lookup resulted in an empty list always https://review.opendev.org/c/openstack/ironic/+/968226 | 18:11 |
| dtantsur | I wonder if we need per-node options too. I think we do, especially for container cases where modifying the global configuration is not trivial. | 18:12 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Enable TLS by default, deprecate enable_tls https://review.opendev.org/c/openstack/bifrost/+/968363 | 18:29 |
| dtantsur | Not sure what made me do ^^. Probably perfectionism. | 18:29 |
| TheJulia | seems reasonabkle, to me | 18:30 |
| TheJulia | reasonable | 18:30 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/968258 there's the passing test TheJulia | 18:37 |
| TheJulia | brraaaains | 18:39 |
| TheJulia | sorry, my uncle called me and gave me a flurry of details | 18:39 |
| cardoe | Just pinging cause that was the patch we wanted to land before rpittau made the bugfix release. | 18:40 |
| dtantsur | I wonder if there is a mirror of the registry container, so that we don't hit docker's limits https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_5cb/openstack/5cb8fe7cd9974fcfae0c2edd9eed5e2f/logs/bifrost-registry.log | 18:50 |
| TheJulia | uuugh | 18:53 |
| dtantsur | I wonder if quay is easy to install | 18:56 |
| TheJulia | ... I was thinking a similar question | 18:56 |
| * dtantsur checking https://docs.projectquay.io/welcome.html | 18:56 | |
| dtantsur | ugh, it has a lot of requirements | 18:57 |
| TheJulia | but there are containers... | 18:58 |
| TheJulia | except the flow is you have to go create an initial account on the webui | 18:59 |
| dtantsur | sigh | 18:59 |
| dtantsur | okay, I wonder if we can just install the registry binary from https://github.com/distribution/distribution/releases (similarly to my ORAS patch) | 18:59 |
| TheJulia | That I'm skeptical of | 19:02 |
| dtantsur | TheJulia: okay, so we can use the binary from ^^ which has the benefit of not messing with bloody podman. Or we use the mirror in the CI https://quay.io/repository/opendevmirror/registry?tab=tags | 19:02 |
| TheJulia | given the explicit redis requirement | 19:02 |
| dtantsur | sorry, I'm confusing things. the former docker's registry (now distribution) is a single binary | 19:02 |
| TheJulia | oh, wait, there is a mirror already | 19:02 |
| dtantsur | and yes, there is a mirror apparently | 19:02 |
| TheJulia | oh, okay | 19:02 |
| * TheJulia holds up the use the mirror sign | 19:02 | |
| TheJulia | Sorry, my uncle called me a little while ago and decided to provide. "here is everything going on" discussion, so I'm mentally derailed at the moment | 19:03 |
| dtantsur | It's too late for my brain, I see benefits in both options, so I'll leave it up to you I guess | 19:03 |
| dtantsur | ah, so you too. we can get back to it tomorrow | 19:03 |
| TheJulia | Get some rest, I'm going to go get some lunch and stop by the grocery store, and then get back to it | 19:03 |
| dtantsur | enjoy your lunch! | 19:04 |
| opendevreview | Doug Goldstein proposed openstack/networking-generic-switch master: WIP: l2vni plug case https://review.opendev.org/c/openstack/networking-generic-switch/+/968377 | 19:24 |
| cardoe | TheJulia: ^ yolo | 19:24 |
| cardoe | I did cisco since that's what ya got but I also think the command might need a name | 19:26 |
| opendevreview | Merged openstack/bifrost master: Add support for OVS as a virtual switch https://review.opendev.org/c/openstack/bifrost/+/962038 | 20:21 |
| TheJulia | cardoe: but what happens when that port goes away? :) | 21:38 |
| cardoe | you throw away all the hardware and start over? | 21:41 |
| cardoe | I'll wire it up but I was just trying to say it's not a big change. | 21:41 |
| cardoe | Once again... https://bugs.launchpad.net/networking-generic-switch/+bug/2114451 makes it hard | 21:41 |
| cardoe | Cause you have to walk all the ports to see what segments are in use | 21:42 |
| cardoe | Cause you need to know if that vlan is in use on that switch | 21:47 |
| cardoe | So you need to know if there's any ports on that segment. | 21:47 |
| TheJulia | Oh, I know its not a big change | 21:48 |
| TheJulia | I know its relatively small, the key is we need to also do the deletes/cleanup as needed which somehow needs to be aware in that model. Maybe we can figure out a way or not | 21:48 |
| TheJulia | and yeah, that is an awful bug | 21:48 |
| cardoe | There actually might be an escape hatch here. | 22:00 |
| TheJulia | Yeah, I concur, just can't look for it right now | 22:01 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: add an OCI artifact registry https://review.opendev.org/c/openstack/bifrost/+/961388 | 23:00 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: bifrost-registry-install: install the ORAS client https://review.opendev.org/c/openstack/bifrost/+/968355 | 23:00 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: WIP: Upload a disk image to OCI https://review.opendev.org/c/openstack/bifrost/+/968416 | 23:00 |
| opendevreview | Julia Kreger proposed openstack/bifrost master: WIP: upload and use the OCI image https://review.opendev.org/c/openstack/bifrost/+/968417 | 23:00 |
| TheJulia | dtantsur: claude attempts, change at will | 23:04 |
| opendevreview | Steve Baker proposed openstack/ironic master: Swap to bootc deploy interface for bootc images https://review.opendev.org/c/openstack/ironic/+/966761 | 23:06 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix: ensure that the bootloader path exists https://review.opendev.org/c/openstack/ironic/+/968418 | 23:12 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix: do not allow nested paths in loader_file_paths https://review.opendev.org/c/openstack/ironic/+/968419 | 23:12 |
| cardoe | The first one is a backport and the 2nd updates to the intended behavior | 23:14 |
| TheJulia | Thanks cardoe ! | 23:33 |
| opendevreview | Kaifeng Wang proposed openstack/ironic-specs master: Flow control on PXE related port requests https://review.opendev.org/c/openstack/ironic-specs/+/947961 | 23:53 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: DNM: Enable NGS to auto-prune unused vlans https://review.opendev.org/c/openstack/networking-generic-switch/+/968420 | 23:55 |
| TheJulia | cardoe: super-duper rough idea, but something ^^^^ could be a basis to unwind a vlan out of existence on detachments | 23:56 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!