]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Removes unused nvp plugin config param
authorarmando-migliaccio <amigliaccio@nicira.com>
Thu, 14 Nov 2013 01:03:27 +0000 (17:03 -0800)
committerarmando-migliaccio <amigliaccio@nicira.com>
Thu, 14 Nov 2013 02:22:53 +0000 (18:22 -0800)
nvp_cluster_uuid is no longer used; it was missed
during the configuration file tweaks that was done
during Havana.

Change-Id: Id6dd684833c99453cd97d88584be04d99fd21550
Closes-bug: #1251086

etc/neutron/plugins/nicira/nvp.ini
neutron/plugins/nicira/README
neutron/plugins/nicira/common/config.py
neutron/tests/unit/nicira/etc/nvp.ini.agentless.test
neutron/tests/unit/nicira/etc/nvp.ini.full.test
neutron/tests/unit/nicira/test_nvpopts.py

index 921520387b9545d8ecdc57d946fce2d3acd6bcea..1e0aa3bee49b2d15033043327fe46a864b05e3cc 100644 (file)
 # tunneled isolated "Neutron" networks. This option MUST be specified, e.g.:
 # default_tz_uuid = 1e8e52cf-fa7f-46b0-a14a-f99835a9cb53
 
-# (Optional) UUID of the cluster in NVP.  It can be retrieved from NVP management
-# console "admin" section.
-# nvp_cluster_uuid = 615be8e4-82e9-4fd2-b4b3-fd141e51a5a7
-
 # (Optional) UUID for the default l3 gateway service to use with this cluster.
 # To be specified if planning to use logical routers with external gateways.
 # default_l3_gw_service_uuid =
index a5695451bf8d849907a8f5f273ff7f4a92711d66..5077024701cff045ebe7543a098ad898539b67cf 100644 (file)
@@ -31,8 +31,6 @@ NVP Plugin configuration
     - default_tz_uuid: This is uuid of the default NVP Transport zone that
       will be used for creating tunneled isolated "Neutron" networks. It
       needs to be created in NVP before starting Neutron with the nvp plugin.
-    - nvp_cluster_uuid: Optional paramter identifying the UUID of the cluster
-      in NVP.  This can be retrieved from NVP management console "admin" section.
     - nvp_controllers: describes the list of controllers
     More details can be found in etc/neutron/plugins/nicira/nvp.ini
 
index 09d7e232a274ecf0a5908b75e31913a845ab43f7..301181470e4843d9369162619e4a62bd20045d9d 100644 (file)
@@ -108,10 +108,6 @@ cluster_opts = [
                       "will be used for creating tunneled isolated "
                       "\"Neutron\" networks. It needs to be created in NVP "
                       "before starting Neutron with the nvp plugin.")),
-    cfg.StrOpt('nvp_cluster_uuid',
-               help=_("Optional paramter identifying the UUID of the cluster "
-                      "in NVP.  This can be retrieved from NVP management "
-                      "console \"admin\" section.")),
     cfg.StrOpt('default_l3_gw_service_uuid',
                help=_("Unique identifier of the NVP L3 Gateway service "
                       "which will be used for implementing routers and "
index 33550520ab4716cc9fceb31170681fd637f105dc..746dcba72ce51d4d8d063a747e1be0e9ab77f935 100644 (file)
@@ -1,7 +1,6 @@
 [DEFAULT]
 default_tz_uuid = fake_tz_uuid
 nova_zone_id = whatever
-nvp_cluster_uuid = fake_cluster_uuid
 nvp_controllers = fake_1, fake_2
 nvp_user = foo
 nvp_password = bar
index fdcdcb89806e01c0b651b52b7dfe787b3bf9bdad..b856b2af67284dec9d94469329dea0d1c7eb644e 100644 (file)
@@ -1,7 +1,6 @@
 [DEFAULT]
 default_tz_uuid = fake_tz_uuid
 nova_zone_id = whatever
-nvp_cluster_uuid = fake_cluster_uuid
 nvp_controllers = fake_1, fake_2
 nvp_user = foo
 nvp_password = bar
index a245ddca6b72e10830c086136e8978b5465e4fa9..ab667bbc223fdbcb886665b1407499d7e5997feb 100644 (file)
@@ -43,7 +43,6 @@ class NVPClusterTest(testtools.TestCase):
     cluster_opts = {'default_tz_uuid': uuidutils.generate_uuid(),
                     'default_l2_gw_service_uuid': uuidutils.generate_uuid(),
                     'default_l2_gw_service_uuid': uuidutils.generate_uuid(),
-                    'nvp_cluster_uuid': uuidutils.generate_uuid(),
                     'nvp_user': 'foo',
                     'nvp_password': 'bar',
                     'req_timeout': 45,
@@ -128,7 +127,6 @@ class ConfigurationTest(testtools.TestCase):
         self.assertEqual('stt', cfg.CONF.NVP.default_transport_type)
 
         self.assertIsNone(cfg.CONF.default_tz_uuid)
-        self.assertIsNone(cfg.CONF.nvp_cluster_uuid)
         self.assertEqual('admin', cfg.CONF.nvp_user)
         self.assertEqual('admin', cfg.CONF.nvp_password)
         self.assertEqual(30, cfg.CONF.req_timeout)