Merge "Removes hash_distribution_replicas configuration option"

This commit is contained in:
Zuul 2019-05-01 13:27:53 +00:00 committed by Gerrit Code Review
commit 2fbe1d72fd
5 changed files with 8 additions and 15 deletions

View File

@ -285,7 +285,6 @@ default:
force_raw_images = True
graceful_shutdown_timeout = 60
grub_config_template = /opt/stack/ironic/ironic/common/grub_conf.template
hash_distribution_replicas = 1
hash_partition_exponent = 5
hash_ring_reset_interval = 180
host = ubuntu

View File

@ -427,9 +427,7 @@ class BaseConductorManager(object):
except exception.DriverNotFound:
return False
return self.host in ring.get_nodes(
node_uuid.encode('utf-8'),
replicas=CONF.hash_distribution_replicas)
return self.host in ring.get_nodes(node_uuid.encode('utf-8'))
def _fail_if_in_state(self, context, filters, provision_state,
sort_key, callback_method=None,

View File

@ -142,8 +142,7 @@ class ConductorAPI(object):
try:
ring = self.ring_manager.get_ring(node.driver,
node.conductor_group)
dest = ring.get_nodes(node.uuid.encode('utf-8'),
replicas=CONF.hash_distribution_replicas)
dest = ring.get_nodes(node.uuid.encode('utf-8'))
return dest.pop()
except exception.DriverNotFound:
reason = (_('No conductor service registered which supports '

View File

@ -187,15 +187,6 @@ hash_opts = [
'smooth in most cases. The default is suitable for up '
'to a few hundred conductors. Configuring for too many '
'partitions has a negative impact on CPU usage.')),
cfg.IntOpt('hash_distribution_replicas',
default=1,
help=_('[Experimental Feature] '
'Number of hosts to map onto each hash partition. '
'Setting this to more than one will cause additional '
'conductor services to prepare deployment environments '
'and potentially allow the Ironic cluster to recover '
'more quickly if a conductor instance is terminated.'),
deprecated_for_removal=True),
cfg.IntOpt('hash_ring_reset_interval',
default=15,
help=_('Time (in seconds) after which the hash ring is '

View File

@ -0,0 +1,6 @@
---
upgrade:
- |
The configuration option ``[DEFAULT]/hash_distribution_replicas`` was
deprecated in the Stein cycle. It has been removed
and is no longer supported.