commit d81dc75a3c56928c430217ee315aa14fc0ddbca1 Author: Renat Akhmerov Date: Mon Oct 5 16:15:13 2020 +0700 Remove the module mistral/actions/action_factory.py * This module was always a weird entity in the system having just one function that essentially creates a dynamic class. It was created just because we didn't understand where else to put this function. But now after the action provider refactoring we don't need it anymore. Action instantiation is now a responsibility of action descriptor classes. Change-Id: Ic4b6a9a7ca2784a892d2998359edb220ff8c8911 diff --git a/doc/source/user/wf_lang_v2.rst b/doc/source/user/wf_lang_v2.rst index d075703..1217910 100644 --- a/doc/source/user/wf_lang_v2.rst +++ b/doc/source/user/wf_lang_v2.rst @@ -1595,8 +1595,8 @@ Task publish result (partial to keep the documentation short): "type": "ACTION", "workflow_execution_id": "c0a4d2ff-0127-4826-8370-0570ef8cad80", "state": "ERROR", - "state_info": "Failed to run action [action_ex_id=bcb04b28-6d50-458e-9b7e-a45a5ff1ca01, action_cls='', attributes='{}', params='{}']\n Fail action expected exception.", - "result": "Failed to run action [action_ex_id=bcb04b28-6d50-458e-9b7e-a45a5ff1ca01, action_cls='', attributes='{}', params='{}']\n Fail action expected exception.", + "state_info": "Failed to run action [action_ex_id=bcb04b28-6d50-458e-9b7e-a45a5ff1ca01, action_cls='', attributes='{}', params='{}']\n Fail action expected exception.", + "result": "Failed to run action [action_ex_id=bcb04b28-6d50-458e-9b7e-a45a5ff1ca01, action_cls='', attributes='{}', params='{}']\n Fail action expected exception.", "published": {}, "spec": { "action": "std.fail", diff --git a/mistral/actions/action_factory.py b/mistral/actions/action_factory.py deleted file mode 100644 index b4a188c..0000000 --- a/mistral/actions/action_factory.py +++ /dev/null @@ -1,29 +0,0 @@ -# Copyright 2014 - Mirantis, Inc. -# -# Licensed under the Apache License, Version 2.0 (the "License"); -# you may not use this file except in compliance with the License. -# You may obtain a copy of the License at -# -# http://www.apache.org/licenses/LICENSE-2.0 -# -# Unless required by applicable law or agreed to in writing, software -# distributed under the License is distributed on an "AS IS" BASIS, -# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. -# See the License for the specific language governing permissions and -# limitations under the License. - -# TODO(rakhmerov): the module needs to be deleted after we add action providers -from oslo_utils import importutils - - -def construct_action_class(action_class_str, attributes): - # Rebuild action class and restore attributes. - action_class = importutils.import_class(action_class_str) - - unique_action_class = type( - action_class.__name__, - (action_class,), - attributes - ) - - return unique_action_class