Arnike/watcher
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Update zone_migration comment

Browse Source 
This patch updates zone_migration comment for document and
removes unnecessary TODO.

Change-Id: Ib1eadad6496fe202e406108f432349c82696ea88
This commit is contained in:
Hidekazu Nakamura
2018-01-29 14:12:29 +09:00
parent 0c8c32e69e
commit d3f23795f5
1 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
watcher/decision_engine/strategy/strategies/zone_migration.py
View File

@@ -10,6 +10,14 @@
# implied. # implied.
# See the License for the specific language governing permissions and # See the License for the specific language governing permissions and
# limitations under the License. # 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 from dateutil.parser import parse
import six import six
@@ -40,6 +48,7 @@ IN_USE = "in-use"
class ZoneMigration(base.ZoneMigrationBaseStrategy): class ZoneMigration(base.ZoneMigrationBaseStrategy):
"""Zone migration using instance and volume migration"""
def __init__(self, config, osc=None): def __init__(self, config, osc=None):
@@ -371,15 +380,6 @@ class ZoneMigration(base.ZoneMigrationBaseStrategy):
:param pool: pool name :param pool: pool name
:returns: host 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] return pool.split('@')[0]
def get_dst_node(self, src_node): def get_dst_node(self, src_node):