Change default spawn defer placement to True
By default we use the host and available zone that ICM chooses, now I will change it to the design that "Ignore the host that ICM selected, defer it to powervc scheduler". Closes-Bug: #1538452 Change-Id: I621154fab32325099cb3c57ed272c1e5935e06f9
This commit is contained in:
parent
6fc9a48cae
commit
63cd893180
|
@ -867,7 +867,7 @@ class PowerVCDriver(driver.ComputeDriver):
|
|||
raise exception.MigrationPreCheckError(reason=reason)
|
||||
|
||||
# check if the host selection will be defer to PowerVC
|
||||
isDefer = self._check_defer_placement(instance_ref)
|
||||
isDefer = self._check_defer_placement(instance_ref, is_spawn=True)
|
||||
if not isDefer:
|
||||
valid_hosts = self._service.get_valid_destinations(instance_ref)
|
||||
LOG.debug('compute manager host is %s, valid destination hosts '
|
||||
|
@ -1425,7 +1425,7 @@ class PowerVCDriver(driver.ComputeDriver):
|
|||
# TODO: Need to revisit this method after confirmation with powervc
|
||||
return ':' + instance['host']
|
||||
|
||||
def _check_defer_placement(self, instance):
|
||||
def _check_defer_placement(self, instance, is_spawn=False):
|
||||
"""
|
||||
Get instance meta data from instance
|
||||
such as "powervm:defer_placement" : "true"
|
||||
|
@ -1447,9 +1447,19 @@ class PowerVCDriver(driver.ComputeDriver):
|
|||
defer_val = entry.get('value', None)
|
||||
if defer_key == u'powervm:defer_placement':
|
||||
return str2bool(defer_val)
|
||||
return False
|
||||
# During spawn a vm, by default is defer=true
|
||||
if is_spawn:
|
||||
LOG.debug("defer_plancement not set, defer pvc to spanw")
|
||||
return True
|
||||
else:
|
||||
return False
|
||||
|
||||
isDefer = False
|
||||
# During spawn a vm, by default is defer=true
|
||||
if is_spawn:
|
||||
LOG.debug("defer_plancement not set, defer pvc to spanw")
|
||||
isDefer = True
|
||||
else:
|
||||
isDefer = False
|
||||
meta = instance.get('metadata', None)
|
||||
if meta:
|
||||
isDefer = get_defer_key_value(meta)
|
||||
|
|
Loading…
Reference in New Issue