Cleanup get_plugin_name() from the tree

It does not seem to be used since Icehouse: the patch that removed its
last usage is: 1b36e2077141749495bb32b423f3df8cbbd0eb40

I don't think we should have it as part of abstract interface for
service plugins.

Change-Id: If03f67a617efd3a7cadb96d1a86283ac14088a83

diff --git a/neutron/extensions/metering.py b/neutron/extensions/metering.py
index 8485fc40d853bdec16ff785c606eff5fb196b28c..36db796dade348587eeab77cb2f5768eed0dd35d 100644 (file)
--- a/neutron/extensions/metering.py
+++ b/neutron/extensions/metering.py
@@ -137,9 +137,6 @@ class Metering(extensions.ExtensionDescriptor):
 @six.add_metaclass(abc.ABCMeta)
 class MeteringPluginBase(service_base.ServicePluginBase):
 
-    def get_plugin_name(self):
-        return constants.METERING
-
     def get_plugin_description(self):
         return constants.METERING
 

diff --git a/neutron/services/service_base.py b/neutron/services/service_base.py
index 0cc8b3bdae145218d5abef2f853019642f3f92b6..acae7a0f50b2b4fef16469a51edb4e4fff8cfd1f 100644 (file)
--- a/neutron/services/service_base.py
+++ b/neutron/services/service_base.py
@@ -41,15 +41,6 @@ class ServicePluginBase(extensions.PluginInterface):
         """
         pass
 
-    @abc.abstractmethod
-    def get_plugin_name(self):
-        """Return a symbolic name for the plugin.
-
-        Each service plugin should have a symbolic name. This name
-        will be used, for instance, by service definitions in service types
-        """
-        pass
-
     @abc.abstractmethod
     def get_plugin_description(self):
         """Return string description of the plugin."""

diff --git a/neutron/tests/unit/dummy_plugin.py b/neutron/tests/unit/dummy_plugin.py
index 0e7fcd98f4ef381810a2d8b0b03c2d6c25af661d..14251933e15ebe8b846217d7bd8c0b6a01c59959 100644 (file)
--- a/neutron/tests/unit/dummy_plugin.py
+++ b/neutron/tests/unit/dummy_plugin.py
@@ -24,7 +24,6 @@ from neutron.plugins.common import constants
 from neutron.services import service_base
 
 
-DUMMY_PLUGIN_NAME = "dummy_plugin"
 RESOURCE_NAME = "dummy"
 COLLECTION_NAME = "%ss" % RESOURCE_NAME
 
@@ -101,9 +100,6 @@ class DummyServicePlugin(service_base.ServicePluginBase):
     def get_plugin_type(self):
         return constants.DUMMY
 
-    def get_plugin_name(self):
-        return DUMMY_PLUGIN_NAME
-
     def get_plugin_description(self):
         return "Neutron Dummy Service Plugin"
 

diff --git a/neutron/tests/unit/extensions/test_servicetype.py b/neutron/tests/unit/extensions/test_servicetype.py
index cc834ffe5b4af11b46b0ece81255de1441f1d22b..13eab8fa2e612ed4fe8fa86ad9e42101f93079ca 100644 (file)
--- a/neutron/tests/unit/extensions/test_servicetype.py
+++ b/neutron/tests/unit/extensions/test_servicetype.py
@@ -32,9 +32,6 @@ from neutron.tests.unit import dummy_plugin as dp
 from neutron.tests.unit import testlib_api
 
 
-DEFAULT_SERVICE_DEFS = [{'service_class': constants.DUMMY,
-                         'plugin': dp.DUMMY_PLUGIN_NAME}]
-
 _uuid = test_base._uuid
 _get_path = test_base._get_path