Skip to content

Commit ec6595e

Browse files
Zuulopenstack-gerrit
authored andcommitted
Merge "[stable/2023.1 only] Fix incorrect cherry-pick for ext gw" into stable/2023.1
2 parents 872c6ad + 9bb3752 commit ec6595e

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -978,8 +978,10 @@ def add_gw_port_info_to_logical_router_port(self):
978978
cmds = []
979979
context = n_context.get_admin_context()
980980
for router in self._ovn_client._l3_plugin.get_routers(context):
981-
ext_gw_networks = [
982-
ext_gw['network_id'] for ext_gw in router['external_gateways']]
981+
ext_gw_net_id = (router.get('external_gateway_info') or
982+
{}).get('network_id')
983+
if not ext_gw_net_id:
984+
continue
983985
rtr_name = 'neutron-{}'.format(router['id'])
984986
ovn_lr = self._nb_idl.get_lrouter(rtr_name)
985987
for lrp in ovn_lr.ports:
@@ -992,7 +994,7 @@ def add_gw_port_info_to_logical_router_port(self):
992994
network_id = ovn_network_name.replace('neutron-', '')
993995
if not network_id:
994996
continue
995-
is_ext_gw = str(network_id in ext_gw_networks)
997+
is_ext_gw = str(network_id == ext_gw_net_id)
996998
external_ids = lrp.external_ids
997999
external_ids[ovn_const.OVN_ROUTER_IS_EXT_GW] = is_ext_gw
9981000
cmds.append(

neutron/tests/unit/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_maintenance.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -943,7 +943,7 @@ def test_add_gw_port_info_to_logical_router_port(self):
943943
internal_net_id = uuidutils.generate_uuid()
944944
routers_db = [{
945945
'id': uuidutils.generate_uuid(),
946-
'external_gateways': [{'network_id': ext_net_id}]}]
946+
'external_gateway_info': {'network_id': ext_net_id}}]
947947
ext_gw_lrp = fakes.FakeOvsdbRow.create_one_ovsdb_row(
948948
attrs={'external_ids': {
949949
constants.OVN_NETWORK_NAME_EXT_ID_KEY:

0 commit comments

Comments
 (0)