Merge "Modify variable's using method in Log Messages"
This commit is contained in:
commit
49606d9099
|
@ -123,7 +123,7 @@ class BayClient(client.MagnumClient):
|
|||
lambda: self.does_bay_exist(bay_id), 10, 1800)
|
||||
except Exception:
|
||||
# In error state. Clean up the bay id if desired
|
||||
self.LOG.error(_LE('Bay %s entered an exception state.') % bay_id)
|
||||
self.LOG.error(_LE('Bay %s entered an exception state.'), bay_id)
|
||||
if delete_on_error:
|
||||
self.LOG.error(_LE('We will attempt to delete bays now.'))
|
||||
self.delete_bay(bay_id)
|
||||
|
@ -139,35 +139,35 @@ class BayClient(client.MagnumClient):
|
|||
resp, model = self.get_bay(bay_id)
|
||||
if model.status in ['CREATED', 'CREATE_COMPLETE',
|
||||
'ERROR', 'CREATE_FAILED']:
|
||||
self.LOG.info(_LI('Bay %s succeeded.') % bay_id)
|
||||
self.LOG.info(_LI('Bay %s succeeded.'), bay_id)
|
||||
return True
|
||||
else:
|
||||
return False
|
||||
except exceptions.NotFound:
|
||||
self.LOG.warning(_LW('Bay %s is not found.') % bay_id)
|
||||
self.LOG.warning(_LW('Bay %s is not found.'), bay_id)
|
||||
return False
|
||||
|
||||
def does_bay_exist(self, bay_id):
|
||||
try:
|
||||
resp, model = self.get_bay(bay_id)
|
||||
if model.status in ['CREATED', 'CREATE_COMPLETE']:
|
||||
self.LOG.info(_LI('Bay %s is created.') % bay_id)
|
||||
self.LOG.info(_LI('Bay %s is created.'), bay_id)
|
||||
return True
|
||||
elif model.status in ['ERROR', 'CREATE_FAILED']:
|
||||
self.LOG.error(_LE('Bay %s is in fail state.') % bay_id)
|
||||
self.LOG.error(_LE('Bay %s is in fail state.'), bay_id)
|
||||
raise exceptions.ServerFault(
|
||||
"Got into an error condition: %s for %s" %
|
||||
(model.status, bay_id))
|
||||
else:
|
||||
return False
|
||||
except exceptions.NotFound:
|
||||
self.LOG.warning(_LW('Bay %s is not found.') % bay_id)
|
||||
self.LOG.warning(_LW('Bay %s is not found.'), bay_id)
|
||||
return False
|
||||
|
||||
def does_bay_not_exist(self, bay_id):
|
||||
try:
|
||||
self.get_bay(bay_id)
|
||||
except exceptions.NotFound:
|
||||
self.LOG.warning(_LW('Bay %s is not found.') % bay_id)
|
||||
self.LOG.warning(_LW('Bay %s is not found.'), bay_id)
|
||||
return True
|
||||
return False
|
||||
|
|
|
@ -124,7 +124,7 @@ class ClusterClient(client.MagnumClient):
|
|||
lambda: self.does_cluster_exist(cluster_id), 10, 1800)
|
||||
except Exception:
|
||||
# In error state. Clean up the cluster id if desired
|
||||
self.LOG.error(_LE('Cluster %s entered an exception state.') %
|
||||
self.LOG.error(_LE('Cluster %s entered an exception state.'),
|
||||
cluster_id)
|
||||
if delete_on_error:
|
||||
self.LOG.error(_LE('We will attempt to delete clusters now.'))
|
||||
|
@ -141,22 +141,22 @@ class ClusterClient(client.MagnumClient):
|
|||
resp, model = self.get_cluster(cluster_id)
|
||||
if model.status in ['CREATED', 'CREATE_COMPLETE',
|
||||
'ERROR', 'CREATE_FAILED']:
|
||||
self.LOG.info(_LI('Cluster %s succeeded.') % cluster_id)
|
||||
self.LOG.info(_LI('Cluster %s succeeded.'), cluster_id)
|
||||
return True
|
||||
else:
|
||||
return False
|
||||
except exceptions.NotFound:
|
||||
self.LOG.warning(_LW('Cluster %s is not found.') % cluster_id)
|
||||
self.LOG.warning(_LW('Cluster %s is not found.'), cluster_id)
|
||||
return False
|
||||
|
||||
def does_cluster_exist(self, cluster_id):
|
||||
try:
|
||||
resp, model = self.get_cluster(cluster_id)
|
||||
if model.status in ['CREATED', 'CREATE_COMPLETE']:
|
||||
self.LOG.info(_LI('Cluster %s is created.') % cluster_id)
|
||||
self.LOG.info(_LI('Cluster %s is created.'), cluster_id)
|
||||
return True
|
||||
elif model.status in ['ERROR', 'CREATE_FAILED']:
|
||||
self.LOG.error(_LE('Cluster %s is in fail state.') %
|
||||
self.LOG.error(_LE('Cluster %s is in fail state.'),
|
||||
cluster_id)
|
||||
raise exceptions.ServerFault(
|
||||
"Got into an error condition: %s for %s" %
|
||||
|
@ -164,13 +164,13 @@ class ClusterClient(client.MagnumClient):
|
|||
else:
|
||||
return False
|
||||
except exceptions.NotFound:
|
||||
self.LOG.warning(_LW('Cluster %s is not found.') % cluster_id)
|
||||
self.LOG.warning(_LW('Cluster %s is not found.'), cluster_id)
|
||||
return False
|
||||
|
||||
def does_cluster_not_exist(self, cluster_id):
|
||||
try:
|
||||
self.get_cluster(cluster_id)
|
||||
except exceptions.NotFound:
|
||||
self.LOG.warning(_LW('Cluster %s is not found.') % cluster_id)
|
||||
self.LOG.warning(_LW('Cluster %s is not found.'), cluster_id)
|
||||
return True
|
||||
return False
|
||||
|
|
Loading…
Reference in New Issue