Merge "Remove mistral from the expand_roles deploy workflow"

tripleoclient/tests/v1/overcloud_node/fakes.py

@@ -14,8 +14,12 @@
 #
 
 import mock
+
 from osc_lib.tests import utils
 
+from tripleoclient import plugin
+from tripleoclient.tests import fakes
+
 
 class FakeClientWrapper(object):
 
@@ -48,4 +52,7 @@ class TestOvercloudNode(utils.TestCommand):
 
         self.app.client_manager.baremetal = mock.Mock()
         self.app.client_manager.workflow_engine = mock.Mock()
-        self.app.client_manager.tripleoclient = FakeClientWrapper()
+        tc = self.app.client_manager.tripleoclient = FakeClientWrapper()
+        tc.create_mistral_context = plugin.ClientWrapper(
+            instance=fakes.FakeInstanceData
+        ).create_mistral_context

tripleoclient/tests/v1/overcloud_node/test_overcloud_node.py

@@ -1359,21 +1359,6 @@ class TestUnprovisionNode(fakes.TestOvercloudNode):
                                             argslist, verifylist)
             self.cmd.take_action(parsed_args)
 
-        self.workflow.executions.create.assert_called_once_with(
-            'tripleo.baremetal_deploy.v1.undeploy_roles',
-            workflow_input={
-                'plan': 'overcloud',
-                'roles': [{
-                    'name': 'Unprovisioned',
-                    'count': 0,
-                    'instances': [
-                        {'hostname': u'compute-0', 'provisioned': False},
-                        {'hostname': u'controller-0', 'provisioned': False}
-                    ]
-                }]
-            }
-        )
-
     def test_ok_all(self):
         rv = mock.Mock()
         rv.output = json.dumps({
@@ -1407,20 +1392,3 @@ class TestUnprovisionNode(fakes.TestOvercloudNode):
             parsed_args = self.check_parser(self.cmd,
                                             argslist, verifylist)
             self.cmd.take_action(parsed_args)
-
-        self.workflow.executions.create.assert_called_once_with(
-            'tripleo.baremetal_deploy.v1.undeploy_roles',
-            workflow_input={
-                'plan': 'overcloud',
-                'roles': [{
-                    'name': 'Unprovisioned',
-                    'count': 0,
-                    'instances': [
-                        {'hostname': u'compute-0', 'provisioned': False},
-                        {'hostname': u'controller-0', 'provisioned': False},
-                        {'hostname': u'compute-1', 'provisioned': False},
-                        {'hostname': u'controller-1', 'provisioned': False}
-                    ]
-                }]
-            }
-        )

tripleoclient/workflows/baremetal.py

@@ -17,6 +17,7 @@ from __future__ import print_function
 import six
 
 from tripleo_common.actions import baremetal
+from tripleo_common.actions import baremetal_deploy
 
 from tripleoclient import exceptions
 from tripleoclient.workflows import base
@@ -613,11 +614,10 @@ def undeploy_roles(clients, **workflow_input):
 
 
 def expand_roles(clients, roles, stackname, provisioned):
-    workflow_client = clients.workflow_engine
-    return base.call_action(
-        workflow_client,
-        'tripleo.baremetal_deploy.expand_roles',
+    context = clients.tripleoclient.create_mistral_context()
+    expand_roles = baremetal_deploy.ExpandRolesAction(
         roles=roles,
         stackname=stackname,
         provisioned=provisioned
     )
+    return expand_roles.run(context=context)