openstack/neutron
Code Issues Proposed changes

Remove note about migration from lib/neutron-legacy to lib/neutron

Browse Source 
In [1] we finally got rid of the unfinished lib/neutron module and kept
only lib/neutron-legacy. It's renamed to lib/neutron now and it's the
only neutron related module in Devstack.
So this patch removes left over todo comments about things to do when
migration to the new lib/neutron module will be finished.

[1] https://review.opendev.org/c/openstack/devstack/+/865014

Change-Id: Icd7030d36a2462263c9379e02c756d23f6ff6440
This commit is contained in:
Sławek Kapłoński 2022-12-20 11:05:05 +01:00
parent a96c8cafe1
commit 4edf94041f
4 changed files with 1 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
devstack/lib/ml2
View File

@ -1,6 +1,6 @@
function configure_ml2_extension_drivers { function configure_ml2_extension_drivers {
if is_neutron_legacy_enabled; then if is_neutron_legacy_enabled; then
# NOTE(yamamoto): This overwrites what neutron-legacy set, # NOTE(yamamoto): This overwrites what Devstack's neutron module set,
# with the latest set of drivers. # with the latest set of drivers.
# While we modifies Q_ML2_PLUGIN_EXT_DRIVERS (via # While we modifies Q_ML2_PLUGIN_EXT_DRIVERS (via
# neutron_ml2_extension_driver_add calls) in the post-config phase, # neutron_ml2_extension_driver_add calls) in the post-config phase,

6
zuul.d/grenade.yaml
View File

@ -148,9 +148,6 @@
$NEUTRON_CONF: $NEUTRON_CONF:
DEFAULT: DEFAULT:
router_distributed: True router_distributed: True
# NOTE(slaweq): We can get rid of this hardcoded absolute path when
# devstack-tempest job will be switched to use lib/neutron instead of
# lib/neutron-legacy
"/$NEUTRON_CORE_PLUGIN_CONF": "/$NEUTRON_CORE_PLUGIN_CONF":
ml2: ml2:
mechanism_drivers: openvswitch,l2population mechanism_drivers: openvswitch,l2population
@ -197,9 +194,6 @@
$NEUTRON_CONF: $NEUTRON_CONF:
DEFAULT: DEFAULT:
router_distributed: True router_distributed: True
# NOTE(slaweq): We can get rid of this hardcoded absolute path when
# devstack-tempest job will be switched to use lib/neutron instead of
# lib/neutron-legacy
"/$NEUTRON_CORE_PLUGIN_CONF": "/$NEUTRON_CORE_PLUGIN_CONF":
agent: agent:
enable_distributed_routing: True enable_distributed_routing: True

6
zuul.d/tempest-multinode.yaml
View File

@ -83,9 +83,6 @@
DEFAULT: DEFAULT:
router_distributed: True router_distributed: True
l3_ha: True l3_ha: True
# NOTE(slaweq): We can get rid of this hardcoded absolute path when
# devstack-tempest job will be switched to use lib/neutron instead of
# lib/neutron-legacy
"/$NEUTRON_CORE_PLUGIN_CONF": "/$NEUTRON_CORE_PLUGIN_CONF":
ml2: ml2:
mechanism_drivers: openvswitch,l2population mechanism_drivers: openvswitch,l2population
@ -141,9 +138,6 @@
$NEUTRON_CONF: $NEUTRON_CONF:
DEFAULT: DEFAULT:
router_distributed: True router_distributed: True
# NOTE(slaweq): We can get rid of this hardcoded absolute path when
# devstack-tempest job will be switched to use lib/neutron instead of
# lib/neutron-legacy
"/$NEUTRON_CORE_PLUGIN_CONF": "/$NEUTRON_CORE_PLUGIN_CONF":
agent: agent:
enable_distributed_routing: True enable_distributed_routing: True

3
zuul.d/tempest-singlenode.yaml
View File

@ -171,9 +171,6 @@
quota_floatingip: 500 quota_floatingip: 500
quota_security_group: 100 quota_security_group: 100
quota_security_group_rule: 1000 quota_security_group_rule: 1000
# NOTE(slaweq): We can get rid of this hardcoded absolute path when
# devstack-tempest job will be switched to use lib/neutron instead of
# lib/neutron-legacy
"/$NEUTRON_CORE_PLUGIN_CONF": "/$NEUTRON_CORE_PLUGIN_CONF":
ml2_type_vlan: ml2_type_vlan:
network_vlan_ranges: foo:1:10 network_vlan_ranges: foo:1:10