| rpittau | good morning ironic! o/ | 07:03 |
|---|---|---|
| opendevreview | Merged openstack/ironic stable/2025.1: Increase memory allocation for ironic-base https://review.opendev.org/c/openstack/ironic/+/962723 | 08:43 |
| opendevreview | Merged openstack/ironic stable/2024.2: Increase memory allocation for ironic-base https://review.opendev.org/c/openstack/ironic/+/962724 | 08:43 |
| opendevreview | Merged openstack/ironic master: fix idrac9 version handling in redfish vmedia boot interface https://review.opendev.org/c/openstack/ironic/+/962678 | 08:44 |
| opendevreview | Ettore Simone proposed openstack/ironic master: Allow multiple HTTP redirects for image source https://review.opendev.org/c/openstack/ironic/+/962676 | 08:47 |
| opendevreview | Ettore Simone proposed openstack/ironic master: Allow multiple HTTP redirects for image source https://review.opendev.org/c/openstack/ironic/+/962676 | 09:18 |
| dtantsur | JayF: re asyncio: the power loop is not easier: it doesn't just read information, it also writes it (== changes power state) | 10:42 |
| dtantsur | hey folks, is it expected that https://github.com/openstack/ironic/blob/stable/2025.2/ironic/drivers/modules/redfish/boot.py#L786-L788 does not work for standalone networking? | 10:48 |
| alegacy_ | dtantsur: ya, there is certainly no 'vif' handling in the standalone network driver. | 11:24 |
| alegacy_ | what use case does that code cover? it is preparing the node to pxe boot by initiating the boot via redfish? | 11:25 |
| dtantsur | alegacy_: that's what caused the discussion https://github.com/metal3-io/baremetal-operator/issues/2712 | 11:26 |
| alegacy_ | perhaps a new method on the network interface should be added to retrieve the mac of the pxe enabled port... rather than (or in addition to) the get_current_vif method. | 11:31 |
| dtantsur | yeah, I think we need to fall back to pxe_enabled | 11:51 |
| dtantsur | TheJulia: hey, does https://docs.openstack.org/ironic/latest/admin/dhcp-less.html need update with all the recent IPA-builder and IPA changes in this area? | 12:19 |
| dtantsur | do people still need to install simple-init aka glean? | 12:19 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix intermittent Redfish firmware update failures with BMC validation https://review.opendev.org/c/openstack/ironic/+/960230 | 12:23 |
| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] Make cache_firmware_components more resilient during upgrades https://review.opendev.org/c/openstack/ironic/+/960711 | 12:26 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix intermittent Redfish firmware update failures with BMC validation https://review.opendev.org/c/openstack/ironic/+/960230 | 12:43 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix intermittent Redfish firmware update failures with BMC validation https://review.opendev.org/c/openstack/ironic/+/960230 | 12:45 |
| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] Make cache_firmware_components more resilient during upgrades https://review.opendev.org/c/openstack/ironic/+/960711 | 12:46 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Make cache_firmware_components more resilient during upgrades https://review.opendev.org/c/openstack/ironic/+/960711 | 13:00 |
| TheJulia | dtantsur: I don't... think that is true anymore. It occurs to me that folks asking have disappeared as well which makes me think the world is a happier place?! | 13:03 |
| TheJulia | Once cafinated, I'll check my outstanding patches. I think there was one which just got ignored | 13:04 |
| dtantsur | Cool! I'm asking because I got asked :) | 13:10 |
| dtantsur | A related question: is it now safe to include simple-init in our published IPA images by default? | 13:15 |
| TheJulia | relatively, but I thought I fixed it for cloud-init or at least got the edges smoothed over mostly | 13:26 |
| TheJulia | well | 13:26 |
| TheJulia | the issue is simple-init and cloud-init fight a little | 13:26 |
| TheJulia | oh, and simple-init now uninstalls cloud-init I *believe* | 13:26 |
| TheJulia | So this is a semi-interesting RFE although I think creates some naming issues in IPA code: https://bugs.launchpad.net/ironic/+bug/2126644 Nothing insurmountable beyond a quick update, really. | 13:27 |
| TheJulia | cardoe: so, I guess the answer is "no" ? | 13:31 |
| cardoe | I'm going to this morning | 13:31 |
| cardoe | once this call is over | 13:31 |
| dtantsur | TheJulia: what I was told is that cloud-init does not work without meta_data.json | 13:31 |
| dtantsur | anyway | 13:32 |
| dtantsur | I'll rephrase: should we include simple-init in our official IPA builds so that dhcp-less works by default? | 13:32 |
| TheJulia | dtantsur: that is not exactly true, there are two launch modes one where it can do initial network configuration | 13:33 |
| TheJulia | and the other where it hunts for full metadata | 13:33 |
| dtantsur | ah, okay | 13:33 |
| TheJulia | and then operates based upon that | 13:33 |
| dtantsur | I guess it's not easy to discover | 13:33 |
| TheJulia | no, not in the slightest | 13:33 |
| TheJulia | Unfortunately | 13:33 |
| TheJulia | and of course, that might have changed on us slightly but I think I even made a CI job for this | 13:33 |
| TheJulia | cardoe: thank you | 13:34 |
| dtantsur | The person I'm talking to is not even using DIB/IPA-builder, so who knows what they ended up with.. | 13:34 |
| TheJulia | OH | 13:34 |
| TheJulia | that would do it | 13:34 |
| TheJulia | oh | 13:34 |
| TheJulia | uhhhh | 13:34 |
| dtantsur | So, my question about official images is not relevant to their case, more of a related thought | 13:34 |
| dtantsur | because it's been a pain in the metal3 community so far | 13:34 |
| TheJulia | hmm | 13:34 |
| TheJulia | okay, so sounds like the thing which needs to be done is a bit of a "if your not using community ramdisk building tools, you need to do x,y,z" | 13:35 |
| TheJulia | just having cloud-init *in* and start as part of systemd config on boot out of the box *wont* just work | 13:35 |
| TheJulia | because it has to be configured to go look/load thhe other network configuration | 13:36 |
| TheJulia | well, not other, but for any via configdrive | 13:36 |
| opendevreview | Takashi Kajinami proposed openstack/networking-generic-switch master: Drop ineffective ignore_base_python https://review.opendev.org/c/openstack/networking-generic-switch/+/962796 | 14:01 |
| iurygregory | open infra live happening now | 14:01 |
| TheJulia | looks like the test I have is slightly wrong, I think. | 14:22 |
| TheJulia | but the workload ends up booting (which is nice) | 14:23 |
| TheJulia | I suspect I'm going to need to revise the test as well, it happens unfortunately | 15:09 |
| opendevreview | cid proposed openstack/sushy master: WIP: async/await support prototype for PTG https://review.opendev.org/c/openstack/sushy/+/962832 | 15:23 |
| opendevreview | Merged openstack/bifrost master: Support IPA download with authenticated requests https://review.opendev.org/c/openstack/bifrost/+/945324 | 15:35 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: CI: Set the neutron network interface for dhcpless runs https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/962837 | 16:00 |
| priteau | dtantsur: thanks for approving the bifrost download patch, could you please check https://review.opendev.org/c/openstack/bifrost/+/962575 as well (trivial) | 16:03 |
| cardoe | TheJulia: https://bugs.launchpad.net/nova/+bug/2126675 | 16:25 |
| cardoe | https://github.com/rackerlabs/understack/blob/486e75098f4d7356cccbe2667518a8cedafd2115/python/cinder-understack/cinder_understack/dynamic_netapp_driver.py#L407 that's the one line change that can be done with the stock NetApp driver for NVMe | 16:28 |
| cardoe | That's what we're doing to Cinder. | 16:29 |
| cardoe | But then https://github.com/rackerlabs/understack/blob/main/containers/nova/patches/ironic-attach-debug.patch is just the hand wave to make volume attach/detach not error out. | 16:29 |
| cardoe | Currently we listen on the RabbitMQ event stream and run some commands to the BMC out of band but I'm gonna aim to do that with Ironic. | 16:30 |
| TheJulia | cardoe: and I guess confiugure the connector in ironic accordingly? | 16:30 |
| cardoe | Yes. We configure the Ironic connector with the network info needed. | 16:31 |
| cardoe | I gotta find that in the repo but I can link it all. | 16:31 |
| TheJulia | okay, What I kind of want to do is to be able to dynamically move, utilziing some of the existing work underway the network attachments to a storage network, if so requested, and then do the bmc config | 16:34 |
| cardoe | Yeah that's what I'd like as well. | 16:41 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: CI: Set the neutron network interface for dhcpless runs https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/962837 | 16:44 |
| TheJulia | friends don't let friends touch python when they are sleepy and their brain is in a meeting | 16:44 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM: check cloud-init advanced vmedia ops https://review.opendev.org/c/openstack/ironic/+/962860 | 17:43 |
| TheJulia | so the advanced job is using simple init, I don't think I have any outstanding cloud-init changes that i see, based upon results I'll rev the docs | 17:45 |
| cardoe | So Nidhi's got https://review.opendev.org/c/openstack/sushy/+/962680 as a follow on to her initial PCI device support. I'm having her add to the redfish inspector gathering the "pci_devices". Gonna go with a slightly different format from what IPA does today but I would probably update IPA. | 17:50 |
| cardoe | IPA today only gathers vendor and then saves the device as "product" | 17:51 |
| cardoe | I'm having her capture PCI class, vendor, device, sub_vendor, sub_device, revision because THAT is what's gonna uniquely identify the thing. | 17:52 |
| TheJulia | logical to get the entire thing including revision because some vendors have been doing some... fishy things versioning wise for some time | 17:53 |
| cardoe | You can say Dell. We're among friends here. | 17:53 |
| cardoe | I kid. I know its a problem in the industry. | 17:54 |
| TheJulia | Dell folks I know have tended to be vocal when they encounter stuff, but sometiems they don't control the thing which comes from another vendor | 18:01 |
| cardoe | They did get two different things from the vendor. | 18:01 |
| TheJulia | c'est la vie | 18:02 |
| opendevreview | Verification of a change to openstack/bifrost master failed: Add support for Rocky Linux 10 image download https://review.opendev.org/c/openstack/bifrost/+/962575 | 18:21 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/2025.2: Allow multiple HTTP redirects for image source https://review.opendev.org/c/openstack/ironic/+/962869 | 19:37 |
| opendevreview | Allain Legacy proposed openstack/ironic-specs master: Update standalone networking initial conditions https://review.opendev.org/c/openstack/ironic-specs/+/960550 | 20:37 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/2024.1: Increase memory allocation for ironic-base https://review.opendev.org/c/openstack/ironic/+/962725 | 20:44 |
| opendevreview | Merged openstack/ironic master: Allow multiple HTTP redirects for image source https://review.opendev.org/c/openstack/ironic/+/962676 | 20:49 |
| opendevreview | Allain Legacy proposed openstack/networking-generic-switch master: Add standalone driver interface https://review.opendev.org/c/openstack/networking-generic-switch/+/962037 | 20:55 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add support for multiple JSON-RPC configuration groups https://review.opendev.org/c/openstack/ironic/+/960593 | 20:55 |
| opendevreview | Allain Legacy proposed openstack/ironic master: Add force_dhcp option for managed inspection https://review.opendev.org/c/openstack/ironic/+/960824 | 20:55 |
| opendevreview | Allain Legacy proposed openstack/ironic master: WIP: Add standalone networking service for ironic https://review.opendev.org/c/openstack/ironic/+/962493 | 20:55 |
| opendevreview | Allain Legacy proposed openstack/bifrost master: Add support for standalone ironic networking https://review.opendev.org/c/openstack/bifrost/+/962394 | 20:59 |
| opendevreview | Verification of a change to openstack/bifrost master failed: Add support for Rocky Linux 10 image download https://review.opendev.org/c/openstack/bifrost/+/962575 | 21:39 |
| opendevreview | cid proposed openstack/ironic master: WIP: Asyncio prototype for PTG https://review.opendev.org/c/openstack/ironic/+/962708 | 22:13 |
| opendevreview | Steve Baker proposed openstack/ironic master: Replace Chrome/Selenium console with Firefox extension https://review.opendev.org/c/openstack/ironic/+/961434 | 22:39 |
| opendevreview | Steve Baker proposed openstack/ironic master: Add a kubernetes provider for console container https://review.opendev.org/c/openstack/ironic/+/962242 | 22:39 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!