| opendevreview | Brian Haley proposed openstack/neutron master: Add more debugging to common agent code https://review.opendev.org/c/openstack/neutron/+/881645 | 03:00 |
|---|---|---|
| opendevreview | ZhouHeng proposed openstack/neutron master: Set result when lswitch port exist https://review.opendev.org/c/openstack/neutron/+/881771 | 03:05 |
| opendevreview | ZhouHeng proposed openstack/neutron master: Set result when lswitch port exist https://review.opendev.org/c/openstack/neutron/+/881771 | 06:16 |
| ricolin | hey forks, FYI I just propose update release for zed as we just merged subnet policy bug | 06:33 |
| ricolin | https://review.opendev.org/c/openstack/releases/+/887138 | 06:33 |
| opendevreview | ZhouHeng proposed openstack/neutron master: [ovn]Synchronize large-scale ACLs by committing multiple transactions https://review.opendev.org/c/openstack/neutron/+/885224 | 06:35 |
| lajoskatona | ricolin: Hi, good idea, as the same fix for srbac was merged also to antelope (stable/2023.1) could you propose for that also the new version? | 06:51 |
| ricolin | lajoskatona: right! sure thing | 06:53 |
| ricolin | lajoskatona: updated | 07:00 |
| lajoskatona | ricolin: thanks | 07:02 |
| ralonsoh | lajoskatona, sorry, what fix? | 07:31 |
| gkoper | ralonsoh, Digging up yesterdays thread(known limitations in dvr docs). I am using ML2 OVS. | 07:47 |
| ralonsoh | gkoper, and are you using HA + DVR? | 07:48 |
| gkoper | ralonsoh, yep | 07:49 |
| ralonsoh | I really don't know what this limitation refers to. This is the first time I see it | 07:50 |
| ralonsoh | is that a new environment? or an existitng one? | 07:50 |
| gkoper | existing one | 07:51 |
| opendevreview | yatin proposed openstack/neutron master: [ovn][ipv6] Add some more tests to skiplist https://review.opendev.org/c/openstack/neutron/+/887142 | 07:51 |
| ralonsoh | sorry, but in my company we have an internal limitation: we provide HA or DVR | 07:51 |
| gkoper | I was just curious about the limitation myself, i am troubleshooting a random l3 agent failover and was just reading through the docs wondering... :D | 07:51 |
| ralonsoh | but not both with ML2/OVS, because it was not tested | 07:51 |
| ralonsoh | but l2population is for the OVS agent | 07:52 |
| ralonsoh | not the L3 agent | 07:52 |
| opendevreview | Merged openstack/neutron master: Delete network namespace on last port deletion https://review.opendev.org/c/openstack/neutron/+/880006 | 07:53 |
| gkoper | Yes, it not rly related to my case, just something that caught my eye . | 07:53 |
| lajoskatona | ralonsoh: https://review.opendev.org/q/Ia494872b58f368581fb29fa40b7da17e1071db22 this was the fix for "create_subnet policy allows users to create subnet in the shared networks" (https://bugs.launchpad.net/neutron/+bug/2023679 ) | 08:10 |
| ralonsoh | lajoskatona, ah yes, thanks! | 08:10 |
| ralonsoh | slaweq, please, add a follow-up patch to include a reno for https://review.opendev.org/c/openstack/neutron/+/886983 | 08:16 |
| ralonsoh | (don't push a new PS, the patch is in the gate) | 08:17 |
| slaweq | ralonsoh sure | 08:17 |
| opendevreview | ZhouHeng proposed openstack/neutron master: [ovn]Synchronize large-scale ACLs by committing multiple transactions https://review.opendev.org/c/openstack/neutron/+/885224 | 08:25 |
| slaweq | ralonsoh ykarel actually I'm not sure we really need release note there - this patch actually fixed error 500 returned when You try to directly remove router port without IP address to the correct 40x (Conflict) error | 08:30 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Ensure agent is protected again wrong/missing bridge mappings https://review.opendev.org/c/openstack/ovn-bgp-agent/+/886617 | 08:31 |
| slaweq | it's IMO simple bug fix, nothing else as removal of such port wasn't really possible before this patch neighter | 08:31 |
| ykarel | slaweq, ack got it, /me agrees | 08:32 |
| ykarel | i asked for it as user behavior is changing, but with this just Error type is changing | 08:33 |
| slaweq | ykarel effectively yes, just error is changed from 500 to 40x :) | 08:38 |
| opendevreview | Merged openstack/neutron-lib master: Add port hardware offload extension https://review.opendev.org/c/openstack/neutron-lib/+/882726 | 09:01 |
| ralonsoh | folks: https://review.opendev.org/c/openstack/releases/+/887147 | 09:12 |
| opendevreview | Rodolfo Alonso proposed openstack/ovn-bgp-agent master: Explicitly define the MAC address when creating the interface https://review.opendev.org/c/openstack/ovn-bgp-agent/+/887149 | 09:25 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: [functional testing] enhance _get_device aux function https://review.opendev.org/c/openstack/ovn-bgp-agent/+/886625 | 09:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVS] Check the datapath ID set by the creation method https://review.opendev.org/c/openstack/neutron/+/887150 | 09:40 |
| opendevreview | Merged openstack/networking-odl stable/zed: [stable-only] Remove the periodic weekly queue https://review.opendev.org/c/openstack/networking-odl/+/887080 | 10:27 |
| opendevreview | Merged openstack/networking-odl stable/2023.1: [stable-only] Remove the periodic weekly queue https://review.opendev.org/c/openstack/networking-odl/+/887077 | 10:27 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron master: Ensure traffic is not centralized if DVR is enabled https://review.opendev.org/c/openstack/neutron/+/886626 | 10:28 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Improve ovn_l3/plugin.py exception logging https://review.opendev.org/c/openstack/neutron/+/887153 | 10:30 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Improve ovn_l3/plugin.py exception logging https://review.opendev.org/c/openstack/neutron/+/887153 | 10:34 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add neutron-tempest-plugin-linuxbridge job to the periodic queue https://review.opendev.org/c/openstack/neutron/+/887158 | 11:01 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Add neutron-tempest-plugin-linuxbridge job to the experimental queue https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/887159 | 11:03 |
| maximkorezkij[m] | Hey, is there something missing from my site regarding: https://review.opendev.org/c/openstack/neutron/+/885240 | 11:03 |
| maximkorezkij[m] | or do I just need to wait for review | 11:03 |
| slaweq | maximkorezkij I think You just need to wait for review. Maybe ping otherwiseguy lucasagomes and ralonsoh to look at it :) | 11:06 |
| maximkorezkij[m] | slaweq okey thanks, will do | 11:06 |
| ralonsoh | let me check | 11:07 |
| lucasagomes | maximkorezkij[m], I will take a look today | 11:07 |
| lucasagomes | ralonsoh, ^ check the patch above please, I think it conflicts with one of ur patches which makes the maint task writes to the SB DB (to remove the dup chassis) | 11:15 |
| ralonsoh | I'm on it, one sec | 11:16 |
| ralonsoh | lucasagomes, I'll reply on the patch. If we can issue a db write command to the backup node, ok. If that is not possible but the performance improvement is noticeable, we can always do https://review.opendev.org/c/openstack/neutron/+/812805/12/neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py | 11:29 |
| ralonsoh | (I'm going for lunch) | 11:31 |
| lucasagomes | ralonsoh, yeah, I think otherwiseguy can comment better about it. I have the same doubt | 11:43 |
| opendevreview | Adam proposed openstack/neutron master: Create net ns in init rather than initialize https://review.opendev.org/c/openstack/neutron/+/887036 | 12:12 |
| *** amoralej is now known as amoralej|lunch | 12:16 | |
| opendevreview | Merged openstack/ovn-bgp-agent master: Explicitly define the MAC address when creating the interface https://review.opendev.org/c/openstack/ovn-bgp-agent/+/887149 | 12:41 |
| opendevreview | Merged openstack/ovn-bgp-agent master: [functional testing] enhance _get_device aux function https://review.opendev.org/c/openstack/ovn-bgp-agent/+/886625 | 12:43 |
| fnordahl | Dear esteemed Neutron cores, we would love to hear review feedback on the remainder of the Active-Active L3 GW series: https://review.opendev.org/q/topic:2023-aa-l3-gw-multihoming+status:open cc ralonsoh slaweq lajoskatona and others | 12:45 |
| lajoskatona | fnordahl: ack, I try to allocate time to the review of it | 12:59 |
| fnordahl | lajoskatona: thanks, much appreciated! | 13:00 |
| opendevreview | Luis Tomas Bolivar proposed openstack/ovn-octavia-provider master: Add support for SOURCE_IP session persistence https://review.opendev.org/c/openstack/ovn-octavia-provider/+/885433 | 13:01 |
| *** amoralej|lunch is now known as amoralej | 13:01 | |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron master: Ensure traffic is not centralized if DVR is enabled https://review.opendev.org/c/openstack/neutron/+/886626 | 14:08 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add missing port_binding policy https://review.opendev.org/c/openstack/neutron/+/884983 | 14:14 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [WIP] [S-RBAC] Add service role in neutron policy https://review.opendev.org/c/openstack/neutron/+/886724 | 14:14 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Add missing port_binding policies https://review.opendev.org/c/openstack/neutron/+/884983 | 14:15 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [WIP] [S-RBAC] Add service role in neutron policy https://review.opendev.org/c/openstack/neutron/+/886724 | 14:15 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib master: Add is_service_role property to the context class https://review.opendev.org/c/openstack/neutron-lib/+/887191 | 14:54 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron master: Ensure traffic is not centralized if DVR is enabled https://review.opendev.org/c/openstack/neutron/+/886626 | 15:05 |
| opendevreview | Lajos Katona proposed openstack/neutron-lib master: api-ref: Move sfc api-ref to neutron-lib https://review.opendev.org/c/openstack/neutron-lib/+/887193 | 15:19 |
| *** amoralej is now known as amoralej|off | 16:54 | |
| opendevreview | Adam Oswick proposed openstack/neutron master: Load FIP information during initialize not init https://review.opendev.org/c/openstack/neutron/+/887036 | 20:30 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [PoC][DNM] Router flavors and service type for OVN https://review.opendev.org/c/openstack/neutron/+/883988 | 21:23 |
| opendevreview | Adam Oswick proposed openstack/neutron master: Load FIP information during initialize not init https://review.opendev.org/c/openstack/neutron/+/887036 | 23:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!