| opendevreview | Steve Baker proposed openstack/ironic master: WIP autodetect deploy interface https://review.opendev.org/c/openstack/ironic/+/973187 | 01:01 |
|---|---|---|
| opendevreview | Steve Baker proposed openstack/ironic master: WIP autodetect deploy interface https://review.opendev.org/c/openstack/ironic/+/973187 | 03:14 |
| janders | good morning folks | 08:19 |
| janders | further to my yesterday's complaints about CI, from early feedback on non-voting ARM jobs (latest comment in https://review.opendev.org/c/openstack/ironic/+/966344), I am pretty sure things will keep failing | 08:20 |
| janders | what can I/we do to troubleshoot? | 08:21 |
| abongale | good morning o/ | 09:15 |
| opendevreview | Maksim Malchuk proposed openstack/bifrost master: Add the ability to override 'disable_boot_timeout' https://review.opendev.org/c/openstack/bifrost/+/973917 | 09:48 |
| rpittau | good morning ironic! o/ | 10:00 |
| kubajj | dtantsur: we are looking into auto-enrollment, do you want us to implement default conductor group config or is there more behind it that we do not see? | 10:13 |
| kubajj | https://opendev.org/openstack/ironic/src/branch/master/ironic/api/controllers/v1/ramdisk.py#L354 | 10:13 |
| dtantsur | kubajj: ugh, we already have default_conductor_group but it has never been wired in there.. | 10:20 |
| dtantsur | so yeah, go ahead | 10:20 |
| kubajj | ok, will do | 10:20 |
| kubajj | dtantsur: do you also know if there is something similar to inspector's enroll_node_fields? https://docs.openstack.org/ironic-inspector/latest/configuration/ironic-inspector.html#discovery.enroll_node_fields | 10:22 |
| dtantsur | kubajj: inspection rules :) | 10:23 |
| dtantsur | Which in the new implementation can also be provided in the configuration | 10:23 |
| kubajj | dtantsur: ok, will read on those :D | 10:23 |
| kubajj | dtantsur: these are cool, they look like they can replace all of our inspection downstream patches 👀 | 10:44 |
| priteau | Hello Ironic! Would it be possible to merge this bifrost bug fix? Interestingly the tls_certificate_path in playbooks/roles/bifrost-registry-install/defaults/main.yml is impacting other playbooks, even when the registry is disabled. | 12:02 |
| dtantsur | I assume you mean https://review.opendev.org/c/openstack/bifrost/+/973452? | 12:32 |
| dtantsur | priteau: yeah, ansible is terrible at namespacing... | 12:33 |
| dtantsur | approved | 12:34 |
| priteau | dtantsur: Oops, forgot the link. Thanks for the merge :) | 12:35 |
| dtantsur | Sure :) Seriously though, my list of "reasons not to use ansible" would start with this issue. | 12:36 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add hardware health monitoring via management interface https://review.opendev.org/c/openstack/ironic/+/966946 | 12:39 |
| opendevreview | Jacob Anders proposed openstack/python-ironicclient master: Add support for node health status field https://review.opendev.org/c/openstack/python-ironicclient/+/967055 | 12:40 |
| opendevreview | Jacob Anders proposed openstack/python-ironicclient master: Add support for node health status field https://review.opendev.org/c/openstack/python-ironicclient/+/967055 | 12:57 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add hardware health monitoring via management interface https://review.opendev.org/c/openstack/ironic/+/966946 | 12:57 |
| dtantsur | janders: we need https://review.opendev.org/c/openstack/ironic/+/973818 to merge to unblock the CI | 13:32 |
| opendevreview | Verification of a change to openstack/bifrost master failed: Fix path to bifrost.crt with non-default tls_root https://review.opendev.org/c/openstack/bifrost/+/973452 | 13:33 |
| janders | dtantsur thank you, this is great. Will keep an eye on it. | 13:35 |
| TheJulia | good morning | 14:04 |
| janders | Hi TheJulia o/ | 14:11 |
| opendevreview | Merged openstack/ironic master: Adapt bindep for debian-trixie https://review.opendev.org/c/openstack/ironic/+/973818 | 14:14 |
| dtantsur | janders: recheck all teh things! | 14:15 |
| janders | dtantsur recheck++ | 14:39 |
| cardoe | TheJulia: okay back at the desk today... just need to get caught up on a couple of things and then networking | 14:47 |
| opendevreview | Abhishek Bongale proposed openstack/python-ironicclient master: Fix field names for selection and json https://review.opendev.org/c/openstack/python-ironicclient/+/973948 | 14:48 |
| opendevreview | Doug Goldstein proposed openstack/python-ironicclient master: fix: test determining latest API version https://review.opendev.org/c/openstack/python-ironicclient/+/973440 | 16:23 |
| cardoe | So I've gotta start saying "assisted-by: claude code 2.0" cause they dynamically select models. So I'm using opus, sonnet, and haiku while I make my change. | 17:07 |
| cardoe | I'm also getting a massive error from autodoc when it tries to import modules cause its flip flopping between eventlet backend and thread backend | 17:08 |
| TheJulia | cardoe: I've noticed some of that locally with the doc builds. Ugh | 17:14 |
| cardoe | dtantsur: don't get kubajj's hopes up too high... there's a bunch of fixes that need to happen | 17:15 |
| dtantsur | Fixing something that exists is hopefully easier than something that does not :) | 17:16 |
| cardoe | Can I leave an existing page in place and just make it a link? | 17:26 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: docs: refactor the inspection docs https://review.opendev.org/c/openstack/ironic/+/973971 | 17:27 |
| cardoe | ^ I'd get rid of doc/admin/inspection.rst for doc/admin/inspection/index.rst entirely. | 17:27 |
| cardoe | but for the purposes of google I'd like to make that a redirect | 17:27 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: docs: refactor the inspection docs https://review.opendev.org/c/openstack/ironic/+/973971 | 17:34 |
| cardoe | Anyway... I hope that makes stuff clearer | 17:34 |
| opendevreview | Merged openstack/ironic master: Make post-firmware-update reboot conditional on component https://review.opendev.org/c/openstack/ironic/+/966344 | 17:38 |
| cardoe | kubajj: https://review.opendev.org/c/openstack/ironic/+/973381 something I'm working on which would likely help | 17:48 |
| dtantsur | cardoe: we tend to leave some placeholder in place, yeah | 17:52 |
| dtantsur | cardoe: you probably need to fix the reference to the old page in /admin/index | 17:53 |
| cardoe | There was none | 17:53 |
| dtantsur | strange.. | 17:54 |
| dtantsur | cardoe: okay, but I see a few references to /admin/inspection in other pages, could you grep for them? | 17:56 |
| cardoe | Yep | 17:56 |
| cardoe | Looks like I cannot have a ref in a nested path? | 18:01 |
| cardoe | doc/source/admin/drivers/ilo.rst:1050: WARNING: Failed to create a cross reference. A title or caption not found: 'capabilities-discovery' | 18:02 |
| cardoe | It got moved to a page in admin/inspection/blah.rst | 18:02 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix: refactor VIF network types to be an enum and ensure all are checked https://review.opendev.org/c/openstack/ironic/+/973557 | 18:04 |
| fungi | if i want to add a topic for next week's meeting, should i also update the title where it currently says yesterday's date? or is there some additional process? | 18:04 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add ConfigLoader class to TBN and use it in TaskManager https://review.opendev.org/c/openstack/ironic/+/972495 | 18:04 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Integrate TBN with vif_attach https://review.opendev.org/c/openstack/ironic/+/973690 | 18:04 |
| opendevreview | Clif Houck proposed openstack/ironic master: WIP: Update TBN simulator for vif_attach planning https://review.opendev.org/c/openstack/ironic/+/973691 | 18:04 |
| JayF | fungi: update the date | 18:05 |
| fungi | got it, thanks! | 18:06 |
| fungi | and done | 18:07 |
| opendevreview | Merged openstack/networking-baremetal master: Swap networking-baremetal to use pyproject.toml https://review.opendev.org/c/openstack/networking-baremetal/+/973539 | 18:07 |
| opendevreview | Doug Goldstein proposed openstack/networking-baremetal master: update pre-commit config to match ironic https://review.opendev.org/c/openstack/networking-baremetal/+/973979 | 18:08 |
| cardoe | TheJulia: ^ just an up date on yours | 18:08 |
| cardoe | looks like metal3-integration is failing... + wget --no-verbose -O clusterctl https://github.com/kubernetes-sigs/cluster-api/releases/download/v1.12.2/clusterctl-linux-amd64 https://github.com/kubernetes-sigs/cluster-api/releases/download/v1.12.2/clusterctl-linux-amd64: 2026-01-20 17:52:11 ERROR 404: Not Found. | 18:11 |
| cardoe | at least 12 patches are about to fail on that | 18:12 |
| TheJulia | cardoe: w/r/t pre-commit, ack, thanks | 18:26 |
| TheJulia | cardoe: super heads down banging my head gainst ovn | 18:26 |
| cardoe | Coming up along side ya as I unravel this code. | 18:27 |
| TheJulia | Yeah, I've got a few issues in the localnet stuff I posted and the logging, I need to push an update | 18:27 |
| TheJulia | but either my ovn is totally broken, or my localnet ports are just now entirely broken and sad | 18:27 |
| opendevreview | Verification of a change to openstack/ironic master failed: Add LLDP collect for DRAC Redfish inspection https://review.opendev.org/c/openstack/ironic/+/970630 | 18:35 |
| * JayF saw the sad state of our wiki and has been fixing it | 18:41 | |
| cardoe | ugh what horrible timing on that metal3-integration test... I feel like this has happened before. | 18:43 |
| cardoe | So basically cluster-api makes a tag for a release and once the tag is created it builds all the binary artifacts and the release | 18:44 |
| dtantsur | There are some issues with new clusterctl versions but in this case it looks like a github issue: the files can already be downloaded atm | 18:44 |
| cardoe | Yeah. The issue is that from tag to files being available it's about ~90 minutes | 18:45 |
| cardoe | Somewhere in the metal3-integration test it grabs the "latest" tag and uses that as the version | 18:45 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix BIOS firmware not flashing when task completes before first poll https://review.opendev.org/c/openstack/ironic/+/970999 | 18:51 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix firmware update failure when task remains in RUNNING state https://review.opendev.org/c/openstack/ironic/+/970998 | 18:55 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix BMC resource validation to catch all sushy exceptions https://review.opendev.org/c/openstack/ironic/+/971000 | 19:14 |
| opendevreview | Merged openstack/bifrost master: Fix path to bifrost.crt with non-default tls_root https://review.opendev.org/c/openstack/bifrost/+/973452 | 19:16 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: l2vni baremetal mech driver https://review.opendev.org/c/openstack/networking-baremetal/+/973889 | 19:24 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add ironic-networking network interface https://review.opendev.org/c/openstack/ironic/+/966470 | 19:30 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add standalone networking service installation guide https://review.opendev.org/c/openstack/ironic/+/966471 | 19:30 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Improve exception handling in switch driver factory https://review.opendev.org/c/openstack/ironic/+/969852 | 19:30 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Address remaining review comments for rpc methods https://review.opendev.org/c/openstack/ironic/+/971184 | 19:30 |
| janders | cardoe exactly as you said, a number of things failed metal3-integration. Is there anything we can do on our side to improve this situation? | 19:40 |
| janders | (question to all, not just cardoe really) | 19:41 |
| TheJulia | cardoe: I feel like I'm missing something with ovn... | 20:54 |
| cardoe | TheJulia: what part are you working on? | 20:55 |
| TheJulia | local port binding | 20:55 |
| TheJulia | for localnet | 20:55 |
| cardoe | Hmm yes. That’s why I make the fake port that mirrors the local net port | 20:57 |
| cardoe | Your local net is vnic_normal | 20:58 |
| TheJulia | I'm not seeing it even try to bridge the port over, but I guess it needs the fake port in your model to do so | 21:00 |
| TheJulia | but... the router should be good enough, imho, that it should just do it | 21:00 |
| cardoe | Can ya paste what you have in your router and your switch? | 21:02 |
| TheJulia | cardoe: https://paste.opendev.org/show/bUtmHeB58Z2tZwcaykT6/ | 21:03 |
| TheJulia | for a very very very short time, I somehow got a local port mapping in `ovs-vsctl show`, but I must channel my inner goth and say "nevermore" | 21:04 |
| TheJulia | okay, so your fake ports set to vnic_normal to force the ovn to do the patch port management | 21:09 |
| TheJulia | I feel un-lucky | 21:20 |
| cardoe | holy phone calls bat man | 21:35 |
| TheJulia | heh | 21:39 |
| cardoe | So I would think the packet would come into OVN via the switch on VLAN 123 | 21:40 |
| TheJulia | It is not even doing the needful side of on the ovs record for creating the patch record | 21:41 |
| TheJulia | it looks like none of the logic even triggers it outside of a VM, really. | 21:41 |
| TheJulia | https://paste.opendev.org/show/bSRVdPAI1yST38DyytQg/ | 21:42 |
| cardoe | What's brbm? | 21:48 |
| TheJulia | local bridge for VM interconnect | 21:48 |
| TheJulia | test vms at that, so its the ovs side which we plug our test VMs into | 21:49 |
| TheJulia | basically, the same config works when we do ovs or even ovn with vlans | 21:49 |
| cardoe | So my br-ex has patches plugged in for the switches | 21:49 |
| *** tosky_ is now known as tosky | 21:51 | |
| TheJulia | in my case, brbm is just configured as my external physical bridge | 21:53 |
| TheJulia | running to the post office and into town | 22:00 |
| TheJulia | cardoe: what version of OVN and ovs? | 22:16 |
| cardoe | It's a Ubuntu 24.04 box so OVN 22.03.3 and OVS 2.17.9 | 22:17 |
| cardoe | I'm first tossing all of our patched stuff and going to what ya uploaded | 22:20 |
| cardoe | To make sure I didn't miss something | 22:20 |
| cardoe | TheJulia: hrm did you write the lsp_add? | 22:28 |
| cardoe | Cause that's the difference between our code. | 22:29 |
| TheJulia | Oh, the local net code addition? | 22:32 |
| TheJulia | To force the binding, yeah | 22:32 |
| TheJulia | Otherwise I don’t see how it extend down | 22:32 |
| cardoe | https://github.com/rackerlabs/understack/blob/e7d61dda8f53a44b29bda271d0075a404c12298a/python/neutron-understack/neutron_understack/routers.py#L168-L188 | 22:34 |
| cardoe | Just trying to pop stack.. I think that's the place I just described what we were doing and didn't actually port it over to the patch. | 22:36 |
| cardoe | Cause I originally had Milan leave out the connection of the routers from the original patch | 22:36 |
| cardoe | oh yes now I remember | 22:42 |
| cardoe | We moved over to networking-baremetal but I had started to update my patch against neutron. | 22:46 |
| cardoe | And that's where we differ | 22:46 |
| cardoe | So I was calling the port "segment-{segment_id}" And you've got neutron-{network_id}-localnet-{physnet} there's no max column width is there? | 22:47 |
| cardoe | Milan's original called it "uplink-{segment_id}" | 22:48 |
| cardoe | nah. I bet its localnet_learn_fdb | 22:51 |
| cardoe | https://github.com/openstack/neutron/blob/5fb69ea54eaac96c0d9c531a6b0119c299c89daf/neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/test_mech_driver.py#L948 is the basis for what Milan did. | 23:05 |
| opendevreview | Merged openstack/ironic master: Rip out check for multiple physnets in vif_attach https://review.opendev.org/c/openstack/ironic/+/972494 | 23:08 |
| TheJulia | cardoe: the db is just json, so afaik no limit | 23:11 |
| TheJulia | hmm, interesting, the tests there do it as one shot, claude thought one couldn't | 23:48 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!