openstack/watcher-tempest-plugin
Code Issues Proposed changes

Merge "Audit and actionplan have different status"

Browse Source
This commit is contained in:
Zuul 2018-08-03 08:26:41 +00:00 committed by Gerrit Code Review
commit 8b78066462
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
watcher_tempest_plugin/tests/scenario/base.py
View File

@ -43,7 +43,9 @@ class BaseInfraOptimScenarioTest(manager.ScenarioTest):
# States where the object is waiting for some event to perform a transition # States where the object is waiting for some event to perform a transition
IDLE_STATES = ('RECOMMENDED', 'FAILED', 'SUCCEEDED', 'CANCELLED') IDLE_STATES = ('RECOMMENDED', 'FAILED', 'SUCCEEDED', 'CANCELLED')
# States where the object can only be DELETED (end of its life-cycle) # States where the object can only be DELETED (end of its life-cycle)
FINISHED_STATES = ('FAILED', 'SUCCEEDED', 'CANCELLED', 'SUPERSEDED') AUDIT_FINISHED_STATES = ('FAILED', 'SUCCEEDED', 'CANCELLED', 'SUSPENDED')
# States where the object can only be DELETED (end of its life-cycle)
AP_FINISHED_STATES = ('FAILED', 'SUCCEEDED', 'CANCELLED', 'SUPERSEDED')
@classmethod @classmethod
def setup_credentials(cls): def setup_credentials(cls):
@ -84,7 +86,7 @@ class BaseInfraOptimScenarioTest(manager.ScenarioTest):
@classmethod @classmethod
def _are_all_action_plans_finished(cls): def _are_all_action_plans_finished(cls):
_, action_plans = cls.client.list_action_plans() _, action_plans = cls.client.list_action_plans()
return all([ap['state'] in cls.FINISHED_STATES return all([ap['state'] in cls.AP_FINISHED_STATES
for ap in action_plans['action_plans']]) for ap in action_plans['action_plans']])
def wait_for_all_action_plans_to_finish(self): def wait_for_all_action_plans_to_finish(self):
@ -278,7 +280,7 @@ class BaseInfraOptimScenarioTest(manager.ScenarioTest):
@classmethod @classmethod
def has_audit_finished(cls, audit_uuid): def has_audit_finished(cls, audit_uuid):
_, audit = cls.client.show_audit(audit_uuid) _, audit = cls.client.show_audit(audit_uuid)
return audit.get('state') in cls.FINISHED_STATES return audit.get('state') in cls.AUDIT_FINISHED_STATES
# ### ACTION PLANS ### # # ### ACTION PLANS ### #
@ -293,14 +295,12 @@ class BaseInfraOptimScenarioTest(manager.ScenarioTest):
def has_action_plan_finished(self, action_plan_uuid): def has_action_plan_finished(self, action_plan_uuid):
_, action_plan = self.client.show_action_plan(action_plan_uuid) _, action_plan = self.client.show_action_plan(action_plan_uuid)
return action_plan.get('state') in ('FAILED', 'SUCCEEDED', 'CANCELLED', return action_plan.get('state') in self.AP_FINISHED_STATES
'SUPERSEDED')
def has_action_plans_finished(self): def has_action_plans_finished(self):
_, action_plans = self.client.list_action_plans() _, action_plans = self.client.list_action_plans()
for ap in action_plans['action_plans']: for ap in action_plans['action_plans']:
_, action_plan = self.client.show_action_plan(ap['uuid']) _, action_plan = self.client.show_action_plan(ap['uuid'])
if action_plan.get('state') not in ('FAILED', 'SUCCEEDED', if action_plan.get('state') not in self.AP_FINISHED_STATES:
'CANCELLED', 'SUPERSEDED'):
return False return False
return True return True