trivial: Remove dead code
Picked up with vulture [1]. [1] https://pypi.org/project/vulture/ Change-Id: I0b318ea5309d201499f0a9ef6f205425c89e6946 Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
This commit is contained in:
@@ -30,13 +30,6 @@ from nova.policies import baremetal_nodes as bn_policies
|
|||||||
ironic_client = importutils.try_import('ironicclient.client')
|
ironic_client = importutils.try_import('ironicclient.client')
|
||||||
ironic_exc = importutils.try_import('ironicclient.exc')
|
ironic_exc = importutils.try_import('ironicclient.exc')
|
||||||
|
|
||||||
node_fields = ['id', 'cpus', 'local_gb', 'memory_mb', 'pm_address',
|
|
||||||
'pm_user', 'service_host', 'terminal_port', 'instance_uuid']
|
|
||||||
|
|
||||||
node_ext_fields = ['uuid', 'task_state', 'updated_at', 'pxe_config_path']
|
|
||||||
|
|
||||||
interface_fields = ['id', 'address', 'datapath_id', 'port_no']
|
|
||||||
|
|
||||||
CONF = nova.conf.CONF
|
CONF = nova.conf.CONF
|
||||||
|
|
||||||
|
|
||||||
|
@@ -207,9 +207,6 @@ valid_name_leading_trailing_spaces_regex = ValidationRegex(
|
|||||||
_("printable characters with at least one non space character"))
|
_("printable characters with at least one non space character"))
|
||||||
|
|
||||||
|
|
||||||
valid_name_regex_obj = re.compile(valid_name_regex.regex, re.UNICODE)
|
|
||||||
|
|
||||||
|
|
||||||
valid_description_regex_base = '^[%s]*$'
|
valid_description_regex_base = '^[%s]*$'
|
||||||
|
|
||||||
|
|
||||||
|
@@ -138,15 +138,6 @@ def get_flavor_by_flavor_id(flavorid, ctxt=None, read_deleted="yes"):
|
|||||||
return objects.Flavor.get_by_flavor_id(ctxt, flavorid, read_deleted)
|
return objects.Flavor.get_by_flavor_id(ctxt, flavorid, read_deleted)
|
||||||
|
|
||||||
|
|
||||||
def get_flavor_access_by_flavor_id(flavorid, ctxt=None):
|
|
||||||
"""Retrieve flavor access list by flavor id."""
|
|
||||||
if ctxt is None:
|
|
||||||
ctxt = context.get_admin_context()
|
|
||||||
|
|
||||||
flavor = objects.Flavor.get_by_flavor_id(ctxt, flavorid)
|
|
||||||
return flavor.projects
|
|
||||||
|
|
||||||
|
|
||||||
# NOTE(danms): This method is deprecated, do not use it!
|
# NOTE(danms): This method is deprecated, do not use it!
|
||||||
# Use instance.{old_,new_,}flavor instead, as instances no longer
|
# Use instance.{old_,new_,}flavor instead, as instances no longer
|
||||||
# have flavor information in system_metadata.
|
# have flavor information in system_metadata.
|
||||||
|
@@ -284,6 +284,7 @@ class KeyPair(API_BASE):
|
|||||||
nullable=False, server_default='ssh')
|
nullable=False, server_default='ssh')
|
||||||
|
|
||||||
|
|
||||||
|
# TODO(stephenfin): Remove this as it's now unused post-placement split
|
||||||
class ResourceClass(API_BASE):
|
class ResourceClass(API_BASE):
|
||||||
"""Represents the type of resource for an inventory or allocation."""
|
"""Represents the type of resource for an inventory or allocation."""
|
||||||
__tablename__ = 'resource_classes'
|
__tablename__ = 'resource_classes'
|
||||||
@@ -295,6 +296,7 @@ class ResourceClass(API_BASE):
|
|||||||
name = Column(String(255), nullable=False)
|
name = Column(String(255), nullable=False)
|
||||||
|
|
||||||
|
|
||||||
|
# TODO(stephenfin): Remove this as it's now unused post-placement split
|
||||||
class ResourceProvider(API_BASE):
|
class ResourceProvider(API_BASE):
|
||||||
"""Represents a mapping to a providers of resources."""
|
"""Represents a mapping to a providers of resources."""
|
||||||
|
|
||||||
@@ -325,6 +327,7 @@ class ResourceProvider(API_BASE):
|
|||||||
nullable=True)
|
nullable=True)
|
||||||
|
|
||||||
|
|
||||||
|
# TODO(stephenfin): Remove this as it's now unused post-placement split
|
||||||
class Inventory(API_BASE):
|
class Inventory(API_BASE):
|
||||||
"""Represents a quantity of available resource."""
|
"""Represents a quantity of available resource."""
|
||||||
|
|
||||||
@@ -356,6 +359,7 @@ class Inventory(API_BASE):
|
|||||||
foreign_keys=resource_provider_id)
|
foreign_keys=resource_provider_id)
|
||||||
|
|
||||||
|
|
||||||
|
# TODO(stephenfin): Remove this as it's now unused post-placement split
|
||||||
class Allocation(API_BASE):
|
class Allocation(API_BASE):
|
||||||
"""A use of inventory."""
|
"""A use of inventory."""
|
||||||
|
|
||||||
@@ -381,6 +385,7 @@ class Allocation(API_BASE):
|
|||||||
foreign_keys=resource_provider_id)
|
foreign_keys=resource_provider_id)
|
||||||
|
|
||||||
|
|
||||||
|
# TODO(stephenfin): Remove this as it's now unused post-placement split
|
||||||
class ResourceProviderAggregate(API_BASE):
|
class ResourceProviderAggregate(API_BASE):
|
||||||
"""Associate a resource provider with an aggregate."""
|
"""Associate a resource provider with an aggregate."""
|
||||||
|
|
||||||
@@ -394,6 +399,7 @@ class ResourceProviderAggregate(API_BASE):
|
|||||||
aggregate_id = Column(Integer, primary_key=True, nullable=False)
|
aggregate_id = Column(Integer, primary_key=True, nullable=False)
|
||||||
|
|
||||||
|
|
||||||
|
# TODO(stephenfin): Remove this as it's now unused post-placement split
|
||||||
class PlacementAggregate(API_BASE):
|
class PlacementAggregate(API_BASE):
|
||||||
"""A grouping of resource providers."""
|
"""A grouping of resource providers."""
|
||||||
__tablename__ = 'placement_aggregates'
|
__tablename__ = 'placement_aggregates'
|
||||||
@@ -597,6 +603,7 @@ class Trait(API_BASE):
|
|||||||
name = Column(Unicode(255), nullable=False)
|
name = Column(Unicode(255), nullable=False)
|
||||||
|
|
||||||
|
|
||||||
|
# TODO(stephenfin): Remove this as it's now unused post-placement split
|
||||||
class ResourceProviderTrait(API_BASE):
|
class ResourceProviderTrait(API_BASE):
|
||||||
"""Represents the relationship between traits and resource provider"""
|
"""Represents the relationship between traits and resource provider"""
|
||||||
|
|
||||||
@@ -614,6 +621,7 @@ class ResourceProviderTrait(API_BASE):
|
|||||||
nullable=False)
|
nullable=False)
|
||||||
|
|
||||||
|
|
||||||
|
# TODO(stephenfin): Remove this as it's unused
|
||||||
class Project(API_BASE):
|
class Project(API_BASE):
|
||||||
"""The project is the Keystone project."""
|
"""The project is the Keystone project."""
|
||||||
|
|
||||||
@@ -629,6 +637,7 @@ class Project(API_BASE):
|
|||||||
external_id = Column(String(255), nullable=False)
|
external_id = Column(String(255), nullable=False)
|
||||||
|
|
||||||
|
|
||||||
|
# TODO(stephenfin): Remove this as it's unused
|
||||||
class User(API_BASE):
|
class User(API_BASE):
|
||||||
"""The user is the Keystone user."""
|
"""The user is the Keystone user."""
|
||||||
|
|
||||||
@@ -644,6 +653,7 @@ class User(API_BASE):
|
|||||||
external_id = Column(String(255), nullable=False)
|
external_id = Column(String(255), nullable=False)
|
||||||
|
|
||||||
|
|
||||||
|
# TODO(stephenfin): Remove this as it's unused
|
||||||
class Consumer(API_BASE):
|
class Consumer(API_BASE):
|
||||||
"""Represents a resource consumer."""
|
"""Represents a resource consumer."""
|
||||||
|
|
||||||
|
@@ -80,7 +80,7 @@ class BaseLoader(object):
|
|||||||
in the directory that defines this class.
|
in the directory that defines this class.
|
||||||
"""
|
"""
|
||||||
classes = []
|
classes = []
|
||||||
for dirpath, dirnames, filenames in os.walk(self.path):
|
for dirpath, _, filenames in os.walk(self.path):
|
||||||
relpath = os.path.relpath(dirpath, self.path)
|
relpath = os.path.relpath(dirpath, self.path)
|
||||||
if relpath == '.':
|
if relpath == '.':
|
||||||
relpkg = ''
|
relpkg = ''
|
||||||
|
Reference in New Issue
Block a user