opendevreview | Miguel Lavalle proposed openstack/neutron master: [PoC][DNM] Router flavors and service type for OVN https://review.opendev.org/c/openstack/neutron/+/883988 | 00:10 |
---|---|---|
mnaser | slaweq: vpnaas is the last hurdle for us to move to ovn .. https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/847007 has some comments/replies involving you -- i am going to try and drive https://review.opendev.org/c/openstack/neutron-vpnaas/+/765353 | 05:16 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Add FIP on LogicalSwitchPortUpdate event https://review.opendev.org/c/openstack/ovn-octavia-provider/+/888918 | 06:32 |
opendevreview | Slawek Kaplonski proposed openstack/neutron master: Remove RULE_NET_OWNER_MEMBER API policy rule https://review.opendev.org/c/openstack/neutron/+/889004 | 08:00 |
ralonsoh | slaweq, https://review.opendev.org/c/openstack/neutron/+/888932 a quick one. Thanks! | 08:30 |
slaweq | ralonsoh done | 08:33 |
slaweq | but please then check https://review.opendev.org/c/openstack/neutron/+/889004 - it's also quick cleanup only :) | 08:34 |
slaweq | thx in advance | 08:34 |
ralonsoh | for sure! | 08:34 |
slaweq | thx | 08:35 |
ralonsoh | slaweq, check the py38 tests | 08:38 |
ralonsoh | https://zuul.opendev.org/t/openstack/build/a10deb5a33934edfb76b59ebbac3e874 | 08:38 |
slaweq | thx, I will check it | 08:40 |
sahid | slaweq: o/ ++ on https://review.opendev.org/c/openstack/neutron/+/887776 looks like the proposal on LP was OK, thanks to have considered it | 09:10 |
opendevreview | Merged openstack/ovn-octavia-provider stable/zed: Fix port for Load Balancer Health Check for FIP https://review.opendev.org/c/openstack/ovn-octavia-provider/+/888545 | 09:15 |
opendevreview | yatin proposed openstack/neutron master: Add sanity check for dnsmasq 2.86 https://review.opendev.org/c/openstack/neutron/+/889015 | 09:21 |
slaweq | sahid yes, thx a lot for that suggestion :) | 09:49 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use the new network HA parameter https://review.opendev.org/c/openstack/neutron/+/881742 | 10:05 |
opendevreview | Merged openstack/neutron-vpnaas stable/victoria: [stable-only] Cap virtualenv/setuptools https://review.opendev.org/c/openstack/neutron-vpnaas/+/888720 | 10:14 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [UT] Reduce the binding retry loop in ``L3HATestFramework`` https://review.opendev.org/c/openstack/neutron/+/889071 | 10:17 |
sahid | slaweq: :-) | 10:18 |
opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Retry retrieving LSP hosting information https://review.opendev.org/c/openstack/neutron/+/889073 | 10:22 |
ralonsoh | slaweq, bcafarel I'm checking the stable branches (Neutron in particular, but that could affect other projects) and I think we should release new versions | 11:05 |
ralonsoh | I'll check tomorrow and I'll send the patches | 11:05 |
ralonsoh | no rush | 11:06 |
opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [OVN] Prevent Trunk creation/deletion with parent port bound https://review.opendev.org/c/openstack/neutron/+/887606 | 11:21 |
opendevreview | Merged openstack/ovn-octavia-provider stable/2023.1: Fix port for Load Balancer Health Check for FIP https://review.opendev.org/c/openstack/ovn-octavia-provider/+/888544 | 11:23 |
opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Retry retrieving LSP hosting information https://review.opendev.org/c/openstack/neutron/+/889073 | 11:27 |
*** damian___92183 is now known as damian___9218 | 11:28 | |
opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Disable the mcast_flood_reports option for LSPs https://review.opendev.org/c/openstack/neutron/+/888127 | 13:43 |
opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Hash Ring: Better handle Neutron worker failures https://review.opendev.org/c/openstack/neutron/+/888574 | 13:44 |
opendevreview | Merged openstack/ovn-octavia-provider master: Add FIP on LogicalSwitchPortUpdate event https://review.opendev.org/c/openstack/ovn-octavia-provider/+/888918 | 13:49 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2023.1: Add FIP on LogicalSwitchPortUpdate event https://review.opendev.org/c/openstack/ovn-octavia-provider/+/889097 | 13:54 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/zed: Add FIP on LogicalSwitchPortUpdate event https://review.opendev.org/c/openstack/ovn-octavia-provider/+/889098 | 13:56 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/yoga: Add FIP on LogicalSwitchPortUpdate event https://review.opendev.org/c/openstack/ovn-octavia-provider/+/889099 | 13:56 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/xena: Add FIP on LogicalSwitchPortUpdate event https://review.opendev.org/c/openstack/ovn-octavia-provider/+/889100 | 13:57 |
opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/wallaby: Add FIP on LogicalSwitchPortUpdate event https://review.opendev.org/c/openstack/ovn-octavia-provider/+/889101 | 13:57 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use the new network HA parameter https://review.opendev.org/c/openstack/neutron/+/881742 | 14:23 |
opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Add initial support for local OVN cluster instead of kernel-networking https://review.opendev.org/c/openstack/ovn-bgp-agent/+/881779 | 14:54 |
opendevreview | Merged openstack/neutron stable/victoria: [stable-only] Cap virtualenv/setuptools https://review.opendev.org/c/openstack/neutron/+/888691 | 15:15 |
opendevreview | Merged openstack/neutron master: [PostgreSQL] Subnet entity with ServiceType grouped by both tables https://review.opendev.org/c/openstack/neutron/+/888782 | 16:04 |
opendevreview | Merged openstack/ovn-octavia-provider stable/2023.1: Update pool upon HM deletion request https://review.opendev.org/c/openstack/ovn-octavia-provider/+/888538 | 16:16 |
opendevreview | Merged openstack/ovn-octavia-provider stable/zed: Update pool upon HM deletion request https://review.opendev.org/c/openstack/ovn-octavia-provider/+/888539 | 16:16 |
opendevreview | Merged openstack/ovn-octavia-provider stable/yoga: Update pool upon HM deletion request https://review.opendev.org/c/openstack/ovn-octavia-provider/+/888540 | 16:16 |
opendevreview | Merged openstack/ovn-octavia-provider stable/xena: Update pool upon HM deletion request https://review.opendev.org/c/openstack/ovn-octavia-provider/+/888542 | 16:16 |
opendevreview | Merged openstack/ovn-octavia-provider stable/wallaby: Update pool upon HM deletion request https://review.opendev.org/c/openstack/ovn-octavia-provider/+/888543 | 16:16 |
frickler | I'm a bit concerned by https://storyboard.openstack.org/#!/story/2010838 , did neutron really break backwards-compatibility or is that a user error? | 16:22 |
haleyb | frickler: there was a new API added for multiple gateways in https://review.opendev.org/c/openstack/neutron-lib/+/870887 but i don't think the sdk part was merged yet, assuming it's related? | 16:56 |
ralonsoh | frickler, haleyb this is not related to the multiple GW feature | 17:08 |
ralonsoh | Omer was yesterday using the SDK to add/remove GW to a router | 17:08 |
ralonsoh | using the add_gw and remove_gw methods | 17:09 |
ralonsoh | but the CLI (and the API) use router_update with the GW info | 17:09 |
ralonsoh | instead of using these methods | 17:09 |
ralonsoh | these 2 methods seems that are (until yesterday) not being used anywhere | 17:09 |
ralonsoh | more info: check neutronclient implementation, same as CLI: router update with "external_gateway_info" field | 17:14 |
opendevreview | Luis Tomas Bolivar proposed openstack/ovn-bgp-agent master: Add initial support for local OVN cluster instead of kernel-networking https://review.opendev.org/c/openstack/ovn-bgp-agent/+/881779 | 17:15 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Skip the port status UP update during a live migration https://review.opendev.org/c/openstack/neutron/+/888769 | 17:22 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][Trunk] Set the subports correct host during live migration https://review.opendev.org/c/openstack/neutron/+/888776 | 17:22 |
frickler | ralonsoh: the add_external_gateways was added with the api-ref change haleyb quoted, so it is new. the sdk add_gateway code is 7 years old, I have a hard time believing that it was broken all along | 17:31 |
ralonsoh | frickler, please check the SDK code, is 7 years old | 17:32 |
ralonsoh | https://review.opendev.org/c/openstack/openstacksdk/+/357954 | 17:33 |
ralonsoh | so no, that was incorrectly implemented before | 17:33 |
ralonsoh | as commented, to add/remove GW in a router, we are using router update | 17:33 |
ralonsoh | not these paths | 17:33 |
frickler | hmm, weird, but ok, weird things happen | 17:42 |
haleyb | ralonsoh: i only mentioned it since the change mentioned had a lot of the same code implementing those new calls | 17:57 |
*** Continuity__ is now known as Continuity | 18:54 | |
opendevreview | Ghanshyam proposed openstack/neutron master: DNM: testing Tempest run increased concurrency https://review.opendev.org/c/openstack/neutron/+/889133 | 20:54 |
opendevreview | Miguel Lavalle proposed openstack/neutron master: [PoC][DNM] Router flavors and service type for OVN https://review.opendev.org/c/openstack/neutron/+/883988 | 23:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!