| opendevreview | Mohammed Naser proposed openstack/ovn-bgp-agent master: Run ovs-ofctl commands with OpenFlow1.3 https://review.opendev.org/c/openstack/ovn-bgp-agent/+/946317 | 01:33 |
|---|---|---|
| opendevreview | yatin proposed openstack/neutron master: [DNM] Check ovs job failures https://review.opendev.org/c/openstack/neutron/+/945433 | 01:53 |
| lajoskatona | ralonsoh: Hi, could you please check this one for fullstack if you have few free minutes: https://review.opendev.org/c/openstack/neutron/+/944013 ,thanks in adance | 07:54 |
| ralonsoh | lajoskatona, for sure | 07:56 |
| ralonsoh | ah yes, I remember this patch | 07:56 |
| ralonsoh | I had it in my pile, I was waiting for the CI... but then I forgot it | 07:57 |
| ralonsoh | I think it worth backporting it | 07:57 |
| hjensas | Good morning, I am looking at https://bugs.launchpad.net/nova/+bug/2106073 - the Ironic side of it. I am wondering, would there be a possiblity to set binding_host_id + some other binding attribute to ensure no port binding actually occurs? | 08:41 |
| hjensas | Currently we do not set binding host when Nova allocate ports, for security reason - see: https://opendev.org/openstack/nova/src/commit/9d910ec4bf2a12baf3b5f0ec3bc41686413538fb/nova/virt/ironic/driver.py#L1797 | 08:41 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Adapt BW limit tests to traffic shaping https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/946237 | 08:45 |
| lajoskatona | ralonsoh: thanks | 08:52 |
| ralonsoh | hjensas, but the port binding is something called from Nova | 08:54 |
| hjensas | ralonsoh: when ironic is involved, it is actually ironic doing a port update to set binding:host_id and binding:profile later. | 08:56 |
| ralonsoh | hjensas, ok but this is not a port binding | 08:56 |
| hjensas | ralonsoh: ok, so if I say "we need to tell neutron not to actually connect the port" - even if binding:host_id is set, deferring it until Ironic set/unset some attribute on the port. | 08:59 |
| hjensas | currently we ensure the port is not connected by not setting binding:host_id ref: https://opendev.org/openstack/nova/src/commit/9d910ec4bf2a12baf3b5f0ec3bc41686413538fb/nova/virt/ironic/driver.py#L1806-L1815 | 09:03 |
| ralonsoh | hjensas, right now, the 'create_port_binding' can be executed only by a service | 09:03 |
| ralonsoh | https://github.com/openstack/neutron/blob/master/neutron/conf/policies/port_bindings.py#L35-L46 | 09:04 |
| ralonsoh | this is the only way (right now) to prevent this call | 09:04 |
| hjensas | ralonsoh: ok, so what if nova created a port and set a vif_type to "deffer-binding", and then we check for that in https://opendev.org/openstack/neutron/src/branch/master/neutron/plugins/ml2/plugin.py#L660-L663 - or in check for that in neutron/plugins/ml2/managers.py to not call drivers? | 09:16 |
| ralonsoh | hjensas, sorry, we don't have this type of vif_type | 09:18 |
| ralonsoh | it can be unbound, failed or a specific one (depending on the mech driver) | 09:18 |
| ralonsoh | again, the port binding is a service call | 09:18 |
| hjensas | ralonsoh: I know, but I thinking if we add that vif_type (or something else) to allow setting the binding:host_id but still not call mech driver to actually connect anything. | 09:20 |
| ralonsoh | hjensas, to be honest, I don't know the implications of adding this new vif type | 09:28 |
| hjensas | ralonsoh: right, me neither. :) I think what I am asking for is: "As a user/service I want to be able to create a port and pre-seed all information, binding:<host_id, profile, *_type> to it, but deffer the actual "connecting" until a port update removing some flag." | 09:35 |
| opendevreview | Merged openstack/neutron master: fullstack: use shorter agent_down_time and report_interval https://review.opendev.org/c/openstack/neutron/+/944013 | 09:47 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Fix OVN DB sync when syncing an OVN LB from scratch https://review.opendev.org/c/openstack/ovn-octavia-provider/+/945140 | 10:29 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Adapt BW limit tests to traffic shaping https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/946237 | 11:34 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Adapt _validate_qos_rules_nbdb to flat and vlan networks https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/946253 | 13:39 |
| opendevreview | Brian Haley proposed openstack/neutron master: Update jobs based on testing runtime for 2025.2 https://review.opendev.org/c/openstack/neutron/+/944809 | 14:17 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Adapt _validate_qos_rules_nbdb to flat and vlan networks https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/946253 | 14:35 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily for a patch release update | 17:17 | |
| opendevreview | Candido Campos Rivas proposed x/whitebox-neutron-tempest-plugin master: Increase burst window size to avoid unstabilities in slow envs https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/944616 | 18:57 |
| opendevreview | Candido Campos Rivas proposed x/whitebox-neutron-tempest-plugin master: Increase burst window size to avoid unstabilities in slow envs https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/944616 | 19:15 |
| opendevreview | Merged openstack/neutron stable/2025.1: If OVS Manager creation failes retry to set values https://review.opendev.org/c/openstack/neutron/+/946142 | 19:21 |
| opendevreview | Merged openstack/neutron master: Avoid race condition by using persist_uuid https://review.opendev.org/c/openstack/neutron/+/923971 | 22:16 |
| opendevreview | Brian Haley proposed openstack/neutron-specs master: Spec folder for 2025.2 cycle https://review.opendev.org/c/openstack/neutron-specs/+/946454 | 22:31 |
| opendevreview | Brian Haley proposed openstack/neutron-specs master: Spec folder for 2025.2 cycle https://review.opendev.org/c/openstack/neutron-specs/+/946454 | 22:49 |
| opendevreview | Brian Haley proposed openstack/neutron-specs master: Spec folder for 2025.2 cycle https://review.opendev.org/c/openstack/neutron-specs/+/946454 | 22:52 |
| opendevreview | Brian Haley proposed openstack/neutron-specs master: Spec folder for 2025.2 cycle https://review.opendev.org/c/openstack/neutron-specs/+/946454 | 22:55 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!