| TheJulia | cardoe: I'm going to try and pull the code state into devstack and start with a fresh look tomororw, it looks like my CI job attempts tickeled just a different enough config that I'll need to sit down and figure out what is really going on in not CI | 00:22 |
|---|---|---|
| opendevreview | Harald Jensås proposed openstack/sushy-tools master: novadriver: rescue-based net-boot support https://review.opendev.org/c/openstack/sushy-tools/+/977284 | 03:00 |
| opendevreview | Harald Jensås proposed openstack/sushy-tools master: novadriver: rescue-based net-boot support https://review.opendev.org/c/openstack/sushy-tools/+/977284 | 03:14 |
| opendevreview | Harald Jensås proposed openstack/sushy-tools master: novadriver: rescue-based net-boot support https://review.opendev.org/c/openstack/sushy-tools/+/977284 | 03:41 |
| kubajj | good morning Ironic! o/ | 07:25 |
| rpittau | hey kubajj :) | 08:44 |
| rpittau | dking: are you using a stable release of metal3/BMO/ironic-image? to you can try checking the driver_info to see if the values are available for the node, you shoould also see something from the metal3 pod logs | 08:49 |
| rpittau | dking: one more thing, for all metal3 related stuff better start asking in the metal3 slack channel, just because there's more people working principally with metal3 there :) | 10:02 |
| *** tkajinam_ is now known as tkajinam | 10:13 | |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/34.0: [bugfix only] remove metal3 integration job https://review.opendev.org/c/openstack/ironic/+/977224 | 10:32 |
| kubajj | rpittau: wait, the metal3-integration is already fixed? | 12:54 |
| kubajj | https://review.opendev.org/c/openstack/ironic/+/977006 | 12:54 |
| TheJulia | good morning | 12:54 |
| TheJulia | kubajj: that was a courtesy "have the change staged if we don't need to do anything locally" change. | 12:56 |
| opendevreview | cid proposed openstack/ironic stable/2025.1: ci: Pin `setuptools` to a range that still ships https://review.opendev.org/c/openstack/ironic/+/977317 | 13:04 |
| * cid my dev environment is messed up :D | 13:06 | |
| TheJulia | :( | 13:07 |
| opendevreview | cid proposed openstack/ironic master: ci: Pin `setuptools` to a range that still ships https://review.opendev.org/c/openstack/ironic/+/977318 | 13:10 |
| opendevreview | cid proposed openstack/ironic master: ci: Pin `setuptools` to a range that still ships https://review.opendev.org/c/openstack/ironic/+/977318 | 13:14 |
| opendevreview | cid proposed openstack/ironic master: ci: Pin `setuptools` to a range that still ships https://review.opendev.org/c/openstack/ironic/+/977318 | 13:19 |
| rpittau | kubajj: yes it';s fixed in metal3-dev-env :) | 13:22 |
| kubajj | that was quick | 13:23 |
| rpittau | well it took us a couple of days :D | 13:23 |
| rpittau | freaking docker.... | 13:23 |
| rpittau | cid: re setuptools pin: do we need that for master too? | 13:26 |
| dking | rpittau: I found my problem. The image sets up the configuration with a script (/bin/configure-ironic.sh). In there, it does a check to see whether the ramdisk and kernel actually exist as files before setting the variable. This is a race condition for me as I have an external container that downloads and manages those images, and that configuration only runs once. | 13:41 |
| rpittau | dking: ok, I see | 13:42 |
| dking | rpittau: I'm using IrSO, and I tried to set that with my ironics' .spec.extraConfig. It creates an environment variable OS_CONTROLLER__DEPLOY_KERNEL, but I'm not sure that's getting used anywhere. | 13:45 |
| rpittau | dking: IrSO maps the exrta config to env variables, but you will find yourself in the same condition if the files are not there | 13:48 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: WIP: Crazy idea: NDFC support in NGS? https://review.opendev.org/c/openstack/networking-generic-switch/+/968484 | 13:51 |
| cid | rpittau, I believe master has an indirect dependency as well (?) | 14:02 |
| rpittau | dking: yeah | 14:08 |
| dking | rpittau: It would seem to me that it would be useful to allow an override there that forces the variables, even if the files are not currently there. There would be an error eventually if they never do exist, but as is the case with any overrides, the end user would be aware of that risk. Now that I'm looking at it, it looks like the Ironic image itself would handle it, as there just needs to be a DEPLOY_KERNEL_URL and DEPLO | 14:09 |
| dking | RAMDISK_URL passed in, so maybe I need to to pass in those variables directly, rather than using extraConfig. | 14:09 |
| rpittau | dking: you should be able to do that in IrSO though, using extraConfig in the ironic custom resource | 14:12 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: be specific on exception checks https://review.opendev.org/c/openstack/networking-baremetal/+/976977 | 14:23 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: trunk-reconciler: Dial back logging https://review.opendev.org/c/openstack/networking-baremetal/+/977074 | 14:28 |
| *** dmellado47 is now known as dmellado | 14:40 | |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: fix orphanded localnet binding https://review.opendev.org/c/openstack/networking-baremetal/+/977329 | 15:32 |
| TheJulia | cardoe: stevebaker[m] ^^^ would be appreciated. | 15:32 |
| TheJulia | I'm hit it locally and was like "oh, yeah, that makes sense... doh!" | 15:33 |
| cardoe | boy I thought you read my mind on a launchpad bug I asked someone to file. | 15:33 |
| cardoe | We've got an issue with the console stuff stevebaker[m]. When the user's browser connection times out or errors in anyway... on the backend the console still stays logged in and the user cannot reconnect cause it says there's another session and it prompts the original session to allow sharing but nobody can click yes. | 15:34 |
| TheJulia | My port binds, I've got dhcp from OVN which means the bridge across is working, but routing connectivity is unhappy. Its further than I'm getting upstream but a starting point on my resurrected devstack | 15:43 |
| TheJulia | cardoe: session sharing on the BMC? | 15:44 |
| TheJulia | *likely* what needs to happen on a new connection request, we need to shutdown the prior | 15:44 |
| TheJulia | hmm, feels like that other ovn bug even though my ports are up. WUT. | 15:47 |
| cardoe | TheJulia: well not trying to session share the BMC. Just re-connect after getting dropped. | 17:30 |
| cardoe | So yes. Need to shutdown on a new connection request. | 17:31 |
| cardoe | At the doctor’s office with the kid. So many balls in the air. | 17:31 |
| opendevreview | cid proposed openstack/sushy master: Handle missing collection members gracefully https://review.opendev.org/c/openstack/sushy/+/977345 | 17:35 |
| TheJulia | oh, so logout needs to be an explicit thing then | 17:47 |
| TheJulia | your kid is obviously more important | 17:47 |
| TheJulia | :) | 17:47 |
| TheJulia | and another bug found | 18:29 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: fix orphanded localnet binding https://review.opendev.org/c/openstack/networking-baremetal/+/977329 | 18:56 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: Fix port stickiness https://review.opendev.org/c/openstack/networking-baremetal/+/977353 | 18:56 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: Fix mech looping and supported type declaration https://review.opendev.org/c/openstack/networking-baremetal/+/977218 | 19:53 |
| cardoe | cid: you're a dup of https://review.opendev.org/c/openstack/sushy/+/976329 would you prefer your implementation? | 22:15 |
| opendevreview | Doug Goldstein proposed openstack/sushy master: skip missing collection members instead of failing https://review.opendev.org/c/openstack/sushy/+/976329 | 22:16 |
| cardoe | He didn't put Closes-Bug in the commit message. I just added that. I hate that when you correct something in the commit message it drops previous reviews. | 22:17 |
| cid | Approved. I will abandon mine. | 22:20 |
| cardoe | TheJulia: for segment in context.segments_to_bind is the bug from my original push. | 22:22 |
| cardoe | That's my mistake | 22:22 |
| TheJulia | cardoe: doing it or the lack of doing it? | 22:22 |
| cardoe | No I was just reviewing your changes | 22:22 |
| TheJulia | ahh, okay | 22:23 |
| cardoe | And I was looking at our code. That's what we run and when I pushed the original to neutron, I wrote it wrong. | 22:23 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: vxlan: Fix mech looping and supported type declaration https://review.opendev.org/c/openstack/networking-baremetal/+/977218 | 22:23 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: CI: implement vxlan job https://review.opendev.org/c/openstack/networking-baremetal/+/977081 | 22:23 |
| cardoe | Cause I was splitting functionality apart in our code since we've got 1 monolithic mech that needs to get broken down | 22:23 |
| TheJulia | ahh, yeah | 22:23 |
| TheJulia | no worries | 22:24 |
| TheJulia | I just want to try to get it all to work in CI | 22:24 |
| TheJulia | so far though! I've got it working locally (attachment wise) | 22:24 |
| TheJulia | just... yeah | 22:24 |
| cardoe | alright I think you've convinced me to move sushy inside of ironic. | 22:32 |
| TheJulia | huh, what? how? | 22:32 |
| cardoe | *BUT* I wanna see the hardware workaround stuff move out of the scattered all over ironic to ironic/sushy | 22:32 |
| TheJulia | whaaaat?! | 22:32 |
| TheJulia | bartender, one whiskey please. | 22:33 |
| cardoe | Let's just say I spent way too long just now on a call about 2 machines which are apparently the same. | 22:33 |
| cardoe | They came from someone's dorm room in Texas. | 22:35 |
| TheJulia | but they have "slightly different" internals? | 22:35 |
| TheJulia | That is... so very standard | 22:35 |
| cardoe | Oh I wish. | 22:35 |
| TheJulia | *really* actually identical? | 22:35 |
| cardoe | Redfish thinks they're identical | 22:36 |
| * TheJulia raises an eyebrow | 22:36 | |
| cardoe | They're like identical twins that nobody can tell apart except their mother can. | 22:36 |
| TheJulia | even the serials? | 22:36 |
| cardoe | They're different down to the PCI revisions | 22:36 |
| cardoe | er the same down to the PCI revisions | 22:36 |
| cardoe | But they're different | 22:37 |
| * TheJulia pours cardoe a beverage | 22:37 | |
| cardoe | One needs to have NIC.Slot.1-1-1 enabled and one needs NIC.Slot.1-1 | 22:38 |
| * TheJulia adds more to the glass | 22:38 | |
| TheJulia | "Momma says we're the saaame!" | 22:39 |
| cardoe | One is running OEM firmware from the NIC manufacturer and one is running firmware from the Texas hardware vendor. | 22:39 |
| TheJulia | serious question, same *even* if you wipe out all the stored config on the bmc and force it to rebild it all? | 22:39 |
| TheJulia | OH! | 22:39 |
| TheJulia | yeah | 22:39 |
| TheJulia | thats totally a thing | 22:39 |
| cardoe | They usually don't reuse the same sub-vendor and sub-device IDs though | 22:40 |
| TheJulia | because the card speaks PLDM differently | 22:40 |
| cardoe | or the same PCI revision | 22:40 |
| TheJulia | and gets represented differently as a result | 22:40 |
| TheJulia | nah, its the special vendor builds | 22:41 |
| TheJulia | sorry, I haven't had braincells to look at any of that, heads down on vxlan stuffs | 23:09 |
| TheJulia | wrong window! | 23:10 |
| opendevreview | Julia Kreger proposed openstack/networking-baremetal master: WIP: CI: implement vxlan job https://review.opendev.org/c/openstack/networking-baremetal/+/977081 | 23:32 |
| TheJulia | https://paste.opendev.org/show/bqClqY8lBgffbsfv6q2E/ <-- neutron hates me | 23:38 |
| TheJulia | I think I know what is going on! | 23:51 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!