From d3f23795f5aa05d3f178d686d2dc9379097b2d7b Mon Sep 17 00:00:00 2001 From: Hidekazu Nakamura Date: Mon, 29 Jan 2018 14:12:29 +0900 Subject: [PATCH] Update zone_migration comment This patch updates zone_migration comment for document and removes unnecessary TODO. Change-Id: Ib1eadad6496fe202e406108f432349c82696ea88 --- .../strategy/strategies/zone_migration.py | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/watcher/decision_engine/strategy/strategies/zone_migration.py b/watcher/decision_engine/strategy/strategies/zone_migration.py index 98dc8f52c..7e52e3970 100644 --- a/watcher/decision_engine/strategy/strategies/zone_migration.py +++ b/watcher/decision_engine/strategy/strategies/zone_migration.py @@ -10,6 +10,14 @@ # implied. # See the License for the specific language governing permissions and # limitations under the License. +# +""" +*Zone migration using instance and volume migration* + +This is zone migration strategy to migrate many instances and volumes +efficiently with minimum downtime for hardware maintenance. + +""" from dateutil.parser import parse import six @@ -40,6 +48,7 @@ IN_USE = "in-use" class ZoneMigration(base.ZoneMigrationBaseStrategy): + """Zone migration using instance and volume migration""" def __init__(self, config, osc=None): @@ -371,15 +380,6 @@ class ZoneMigration(base.ZoneMigrationBaseStrategy): :param pool: pool name :returns: host name """ - - # TODO(hidekazu) use this - # mapping = zonemgr.get_host_pool_mapping() - # for host, pools in six.iteritems(mapping): - # for _pool in pools: - # if pool == _pool: - # return host - # LOG.warning(self.msg_not_exist_corresponding_host % pool) - # return pool return pool.split('@')[0] def get_dst_node(self, src_node):