Thursday, 2023-07-20

opendevreviewMiguel Lavalle proposed openstack/neutron master: [PoC][DNM] Router flavors and service type for OVN  https://review.opendev.org/c/openstack/neutron/+/88398800:10
mnaserslaweq: 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/+/76535305:16
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Add FIP on LogicalSwitchPortUpdate event  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88891806:32
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Remove RULE_NET_OWNER_MEMBER API policy rule  https://review.opendev.org/c/openstack/neutron/+/88900408:00
ralonsohslaweq, https://review.opendev.org/c/openstack/neutron/+/888932 a quick one. Thanks!08:30
slaweqralonsoh done08:33
slaweqbut please then check https://review.opendev.org/c/openstack/neutron/+/889004 - it's also quick cleanup only :)08:34
slaweqthx in advance08:34
ralonsohfor sure!08:34
slaweqthx08:35
ralonsohslaweq, check the py38 tests08:38
ralonsohhttps://zuul.opendev.org/t/openstack/build/a10deb5a33934edfb76b59ebbac3e87408:38
slaweqthx, I will check it08:40
sahidslaweq: o/ ++ on https://review.opendev.org/c/openstack/neutron/+/887776 looks like the proposal on LP was OK, thanks to have considered it09:10
opendevreviewMerged openstack/ovn-octavia-provider stable/zed: Fix port for Load Balancer Health Check for FIP  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88854509:15
opendevreviewyatin proposed openstack/neutron master: Add sanity check for dnsmasq 2.86  https://review.opendev.org/c/openstack/neutron/+/88901509:21
slaweqsahid yes, thx a lot for that suggestion :)09:49
opendevreviewRodolfo Alonso proposed openstack/neutron master: Use the new network HA parameter  https://review.opendev.org/c/openstack/neutron/+/88174210:05
opendevreviewMerged openstack/neutron-vpnaas stable/victoria: [stable-only] Cap virtualenv/setuptools  https://review.opendev.org/c/openstack/neutron-vpnaas/+/88872010:14
opendevreviewRodolfo Alonso proposed openstack/neutron master: [UT] Reduce the binding retry loop in ``L3HATestFramework``  https://review.opendev.org/c/openstack/neutron/+/88907110:17
sahidslaweq: :-)10:18
opendevreviewLucas Alvares Gomes proposed openstack/neutron master: [OVN] Retry retrieving LSP hosting information  https://review.opendev.org/c/openstack/neutron/+/88907310:22
ralonsohslaweq, bcafarel I'm checking the stable branches (Neutron in particular, but that could affect other projects) and I think we should release new versions11:05
ralonsohI'll check tomorrow and I'll send the patches11:05
ralonsohno rush 11:06
opendevreviewRodolfo Alonso proposed openstack/neutron stable/wallaby: [OVN] Prevent Trunk creation/deletion with parent port bound  https://review.opendev.org/c/openstack/neutron/+/88760611:21
opendevreviewMerged 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/+/88854411:23
opendevreviewLucas Alvares Gomes proposed openstack/neutron master: [OVN] Retry retrieving LSP hosting information  https://review.opendev.org/c/openstack/neutron/+/88907311:27
*** damian___92183 is now known as damian___921811:28
opendevreviewLucas Alvares Gomes proposed openstack/neutron master: [OVN] Disable the mcast_flood_reports option for LSPs  https://review.opendev.org/c/openstack/neutron/+/88812713:43
opendevreviewLucas Alvares Gomes proposed openstack/neutron master: [OVN] Hash Ring: Better handle Neutron worker failures  https://review.opendev.org/c/openstack/neutron/+/88857413:44
opendevreviewMerged openstack/ovn-octavia-provider master: Add FIP on LogicalSwitchPortUpdate event  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88891813:49
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider stable/2023.1: Add FIP on LogicalSwitchPortUpdate event  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88909713:54
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider stable/zed: Add FIP on LogicalSwitchPortUpdate event  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88909813:56
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider stable/yoga: Add FIP on LogicalSwitchPortUpdate event  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88909913:56
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider stable/xena: Add FIP on LogicalSwitchPortUpdate event  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88910013:57
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider stable/wallaby: Add FIP on LogicalSwitchPortUpdate event  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88910113:57
opendevreviewRodolfo Alonso proposed openstack/neutron master: Use the new network HA parameter  https://review.opendev.org/c/openstack/neutron/+/88174214:23
opendevreviewLuis 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/+/88177914:54
opendevreviewMerged openstack/neutron stable/victoria: [stable-only] Cap virtualenv/setuptools  https://review.opendev.org/c/openstack/neutron/+/88869115:15
opendevreviewMerged openstack/neutron master: [PostgreSQL] Subnet entity with ServiceType grouped by both tables  https://review.opendev.org/c/openstack/neutron/+/88878216:04
opendevreviewMerged openstack/ovn-octavia-provider stable/2023.1: Update pool upon HM deletion request  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88853816:16
opendevreviewMerged openstack/ovn-octavia-provider stable/zed: Update pool upon HM deletion request  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88853916:16
opendevreviewMerged openstack/ovn-octavia-provider stable/yoga: Update pool upon HM deletion request  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88854016:16
opendevreviewMerged openstack/ovn-octavia-provider stable/xena: Update pool upon HM deletion request  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88854216:16
opendevreviewMerged openstack/ovn-octavia-provider stable/wallaby: Update pool upon HM deletion request  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/88854316:16
fricklerI'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
haleybfrickler: 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
ralonsohfrickler, haleyb this is not related to the multiple GW feature17:08
ralonsohOmer was yesterday using the SDK to add/remove GW to a router17:08
ralonsohusing the add_gw and remove_gw methods17:09
ralonsohbut the CLI (and the API) use router_update with the GW info17:09
ralonsohinstead of using these methods17:09
ralonsohthese 2 methods seems that are (until yesterday) not being used anywhere17:09
ralonsohmore info: check neutronclient implementation, same as CLI: router update with "external_gateway_info" field17:14
opendevreviewLuis 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/+/88177917:15
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Skip the port status UP update during a live migration  https://review.opendev.org/c/openstack/neutron/+/88876917:22
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN][Trunk] Set the subports correct host during live migration  https://review.opendev.org/c/openstack/neutron/+/88877617:22
fricklerralonsoh: 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 along17:31
ralonsohfrickler, please check the SDK code, is 7 years old17:32
ralonsohhttps://review.opendev.org/c/openstack/openstacksdk/+/35795417:33
ralonsohso no, that was incorrectly implemented before17:33
ralonsohas commented, to add/remove GW in a router, we are using router update17:33
ralonsohnot these paths17:33
fricklerhmm, weird, but ok, weird things happen17:42
haleybralonsoh: i only mentioned it since the change mentioned had a lot of the same code implementing those new calls17:57
*** Continuity__ is now known as Continuity18:54
opendevreviewGhanshyam proposed openstack/neutron master: DNM: testing Tempest run increased concurrency  https://review.opendev.org/c/openstack/neutron/+/88913320:54
opendevreviewMiguel Lavalle proposed openstack/neutron master: [PoC][DNM] Router flavors and service type for OVN  https://review.opendev.org/c/openstack/neutron/+/88398823:42

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!