Remove _show_resource in mistral

Mistral client (version) > 1 provides ability to directly use
_show_resource in resource, therefore remove unrequired
_show_resource overwrite method in mistral.
Depends-On: I52746d2ef98598fad20df3732991e1b56d234e9a

Change-Id: Idf0043bb1dc9be4bdf3540ae108d5eb225e8f562
This commit is contained in:
ricolin 2016-06-09 09:33:35 +08:00
parent 909c842e3d
commit dfbd237ce3
4 changed files with 6 additions and 14 deletions

View File

@ -141,13 +141,6 @@ class CronTrigger(resource.Resource):
elif name == self.REMAINING_EXECUTIONS:
return trigger.remaining_executions
# TODO(tlashchova): remove this method when mistralclient>1.0.0 is used.
def _show_resource(self):
cron_trigger = self.client().cron_triggers.get(self.resource_id)
if hasattr(cron_trigger, 'to_dict'):
super(CronTrigger, self)._show_resource()
return cron_trigger._data
def get_live_state(self, resource_properties):
# Currently mistral just deletes cron trigger that was executed
# (i.e. remaining execution is reached zero). In this case we can't

View File

@ -599,13 +599,6 @@ class Workflow(signal_responder.SignalResponder,
elif name == self.ALARM_URL and self.resource_id is not None:
return six.text_type(self._get_ec2_signed_url())
# TODO(tlashchova): remove this method when mistralclient>1.0.0 is used.
def _show_resource(self):
workflow = self.client().workflows.get(self.resource_id)
if hasattr(workflow, 'to_dict'):
super(Workflow, self)._show_resource()
return workflow._data
def resource_mapping():
return {

View File

@ -44,6 +44,9 @@ class FakeCronTrigger(object):
self.remaining_executions = 3
self._data = {'trigger': 'info'}
def to_dict(self):
return self._data
class MistralCronTriggerTest(common.HeatTestCase):

View File

@ -314,6 +314,9 @@ class FakeWorkflow(object):
self.name = name
self._data = {'workflow': 'info'}
def to_dict(self):
return self._data
class TestMistralWorkflow(common.HeatTestCase):
def setUp(self):