| opendevreview | Takashi Kajinami proposed openstack/sushy master: Remove Python 3.8 support https://review.opendev.org/c/openstack/sushy/+/931321 | 02:10 |
|---|---|---|
| rpittau | good morning ironic! happy friday! o/ | 06:46 |
| rpittau | so we're waiting for https://review.opendev.org/c/openstack/releases/+/931314 before proposing the epoxy release I guess | 09:32 |
| iurygregory | good morning ironic | 10:41 |
| iurygregory | rpittau, you mean because of the OSSA-2024-004? | 10:42 |
| iurygregory | I don't think we would need to release already (since the cycle just started...) | 10:43 |
| TheJulia | good morning | 14:01 |
| TheJulia | Any chance I can get eyes on https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/931069 ? | 14:01 |
| TheJulia | I've got a job I want to merge to master which uses centos as the target OS, which I guess doesn't ping by default..... :\ | 14:01 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM https://review.opendev.org/c/openstack/ironic/+/931061 | 14:02 |
| rpittau | TheJulia: you got a +2, you just need another one ;) | 14:11 |
| rpittau | I need to split, have a great weekend everyone! o/ | 14:12 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: Fallback to SSH if ping fails https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/931069 | 14:13 |
| TheJulia | rpittau: fixed slightly misleading message in ^ | 14:13 |
| TheJulia | rpittau: enjoy! | 14:13 |
| dansmith | JayF: on https://bugs.launchpad.net/ironic/+bug/2076289 -- are you not using glanceclient there? | 14:25 |
| dansmith | I would assume this https://github.com/openstack/ironic/blob/master/ironic/common/images.py#L382 is calling into glanceclient, is it not? | 14:26 |
| dansmith | or is the bug for the case where you're not using glance at all? | 14:26 |
| dansmith | TheJulia: ^ | 14:26 |
| TheJulia | it can use glance client, or in that specific bug it was using a requests client to directly download the image | 14:27 |
| dansmith | okay, glanceclient does the checksum by default as it streams, so it looks like maybe you'll double-checksum in that case? | 14:28 |
| TheJulia | image_service is, if it is a uuid, is a glance client | 14:28 |
| TheJulia | image_service is, if it is a URL, a wrapper around requests | 14:28 |
| TheJulia | The latter being the one at issue | 14:29 |
| TheJulia | It might mean we end up double checksuming, but at this point if your doing local caching by default, your likely not in an integrated openstack | 14:29 |
| dansmith | okay | 14:30 |
| TheJulia | the default path sort of leans into this direction of files coming out of swift directly | 14:31 |
| TheJulia | via tempurls | 14:31 |
| TheJulia | but there is just a number of different paths which are possible based upon operator preferences and the way they are running their deployment | 14:32 |
| TheJulia | iurygregory: i found one | 15:35 |
| iurygregory | TheJulia, \o/ | 15:41 |
| opendevreview | cid proposed openstack/ironic-specs master: [WIP] Kea DHCP backend https://review.opendev.org/c/openstack/ironic-specs/+/931025 | 15:53 |
| jamesdenton | does anyone know if ironic-neutron-agent is compatible with ml2/ovn deployments? Or is it ovs-centric? | 16:47 |
| TheJulia | jamesdenton: should be compatible, it is not ovs-centric. Largely it does the internal data updates for neutron to correctly do bridge mapping | 16:49 |
| TheJulia | aiui | 16:49 |
| jamesdenton | ok, good to know. thank you. | 16:50 |
| iurygregory | TheJulia, I'm happy to +W https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/931069 since previously you had rpittau +2 and the change in the new patch set was just some comment in the code | 18:09 |
| TheJulia | up to you, no rush | 18:10 |
| TheJulia | I kind of want to see https://review.opendev.org/c/openstack/ironic/+/931061 pass, but tempest plugin is tempest plugin :) | 18:11 |
| iurygregory | ack o/ | 18:12 |
| iurygregory | I will try to look at things after fighting the unit tests in golang | 18:13 |
| TheJulia | Enjoy, running into town for lunch with the wife before I fly out | 18:14 |
| jamesdenton | TheJulia no rush on a comment here, but i'm curious to know if ironic + ml2/ovn + neutron-ovn-metadata-agent is a known-working configuration. I do have the 'baremetal' plugin configured and seemingly functional, but port bindings fail for baremetal node deployments with the error "Refusing to bind port due to unsupported vnic_type: baremetal with no switchdev capability bind_port". Not sure if that's an anomaly or to be | 18:30 |
| jamesdenton | expected. I guess i thought maybe the baremetal ports would bind to the node running nova-compute for ironic? Instead the logs indicate its trying to bind the port to the actual baremetal node itself (Failed to bind port aac8799c-ea70-4fa2-ae89-4a4822d6c0c5 on host d0e4d74f-a239-4981-886a-60ca0072b659 for vnic_type baremetal using segments) | 18:30 |
| jamesdenton | the reason i mention the metadata agent is because i'm thinking its not spawning a ovnmeta namespace because the port is not bound | 18:38 |
| TheJulia | So complicated, but I thought ovn support for basement tagged ports was sorted, but you need networker nodes since there is not a traditional compute node is in use. It could be that is expected, did you check our known ovn issues doc? | 18:40 |
| jamesdenton | https://docs.openstack.org/ironic/latest/admin/ovn-networking.html ? | 18:40 |
| TheJulia | Yes | 18:41 |
| jamesdenton | I did, but i will check it out again and see if i missed something. My network node is a gateway (enable-chassis-as-gw) and the agent list reflects as such. I see mention in another document about using enable-chassis-as-extport-host as a potential destination for external ports instead. https://docs.openstack.org/neutron/latest/admin/ovn/external_ports.html | 18:49 |
| TheJulia | jamesdenton: hjensas might know, but I suspect he is asleep at the moment | 19:23 |
| jamesdenton | no worries, i'm about to call it. seeing some strange behavior with relation to the HA_Chassis_Group and all of the other stuff | 19:24 |
| jamesdenton | i get an ha_chassis_group for my port - neutron-extport-44c83bad-7b1c-4906-81d2-43ee4182a9f9 - but the ha_chassis list is empty. It's not until I *delete the baremetal instance* that i start to see that list get populated. but by then it's too late :D | 19:26 |
| opendevreview | Adam McArthur proposed openstack/ironic master: Added Localdev Docs on changing microversion https://review.opendev.org/c/openstack/ironic/+/931351 | 19:28 |
| jamesdenton | TheJulia thanks for the help. have a great weekend | 19:33 |
| opendevreview | Adam McArthur proposed openstack/ironic-python-agent master: Experimenting with IPA Eventlet removal https://review.opendev.org/c/openstack/ironic-python-agent/+/924634 | 20:49 |
| iurygregory | have a great weekend everyone o/ | 21:07 |
| opendevreview | Adam McArthur proposed openstack/ironic master: Added Localdev Docs on changing microversion https://review.opendev.org/c/openstack/ironic/+/931351 | 21:17 |
| opendevreview | Verification of a change to openstack/ironic-tempest-plugin master failed: Fallback to SSH if ping fails https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/931069 | 22:26 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!