From 2d58a9fd7e5b43c59da6d6ebb6f5e5e9bc8e66fe Mon Sep 17 00:00:00 2001 From: caoyue Date: Thu, 14 Jan 2016 20:33:10 +0800 Subject: [PATCH] Remove unused logging import it's obviously the code was copied from other place, let's make it perfect. Change-Id: Iebac30fe1240fe0c1f2849120e9566397ec73538 --- mistral/engine/utils.py | 4 ---- mistral/tests/unit/actions/test_action_manager.py | 5 ----- mistral/tests/unit/engine/test_action_context.py | 3 --- mistral/tests/unit/engine/test_action_defaults.py | 2 -- mistral/tests/unit/engine/test_adhoc_actions.py | 2 -- mistral/tests/unit/engine/test_commands.py | 2 -- mistral/tests/unit/engine/test_cron_trigger.py | 2 -- mistral/tests/unit/engine/test_dataflow.py | 2 -- mistral/tests/unit/engine/test_default_engine.py | 2 -- mistral/tests/unit/engine/test_direct_workflow.py | 3 --- .../tests/unit/engine/test_direct_workflow_with_cycles.py | 3 --- mistral/tests/unit/engine/test_environment.py | 2 -- mistral/tests/unit/engine/test_error_result.py | 2 -- .../unit/engine/test_execution_fields_size_limitation.py | 2 -- mistral/tests/unit/engine/test_javascript_action.py | 4 ++-- mistral/tests/unit/engine/test_join.py | 2 -- mistral/tests/unit/engine/test_noop_task.py | 2 -- mistral/tests/unit/engine/test_policies.py | 4 ++-- mistral/tests/unit/engine/test_race_condition.py | 2 -- mistral/tests/unit/engine/test_reverse_workflow.py | 2 -- mistral/tests/unit/engine/test_state_info.py | 3 +-- mistral/tests/unit/engine/test_subworkflows.py | 2 -- mistral/tests/unit/engine/test_task_defaults.py | 2 -- mistral/tests/unit/engine/test_with_items.py | 2 -- mistral/tests/unit/engine/test_workflow_variables.py | 3 --- mistral/tests/unit/engine/test_yaql_functions.py | 2 -- mistral/tests/unit/services/test_action_service.py | 2 -- mistral/tests/unit/services/test_workbook_service.py | 2 -- mistral/tests/unit/services/test_workflow_service.py | 2 -- mistral/tests/unit/workbook/v2/base.py | 4 ---- mistral/tests/unit/workbook/v2/test_actions.py | 5 ----- mistral/tests/unit/workbook/v2/test_workbook.py | 4 ---- mistral/tests/unit/workbook/v2/test_workflows.py | 3 --- mistral/tests/unit/workflow/test_direct_workflow.py | 3 --- mistral/tests/unit/workflow/test_reverse_workflow.py | 4 ---- mistral/tests/unit/workflow/test_states.py | 4 ---- tools/sync_db.py | 1 - 37 files changed, 5 insertions(+), 95 deletions(-) diff --git a/mistral/engine/utils.py b/mistral/engine/utils.py index c8dda7dd..8d910c78 100644 --- a/mistral/engine/utils.py +++ b/mistral/engine/utils.py @@ -16,14 +16,10 @@ import copy import six -from oslo_log import log as logging - from mistral.db.v2 import api as db_api from mistral import exceptions as exc from mistral import utils -LOG = logging.getLogger(__name__) - def validate_input(definition, input, spec=None): input_param_names = copy.deepcopy(list((input or {}).keys())) diff --git a/mistral/tests/unit/actions/test_action_manager.py b/mistral/tests/unit/actions/test_action_manager.py index cc0bcc3e..989ceb80 100644 --- a/mistral/tests/unit/actions/test_action_manager.py +++ b/mistral/tests/unit/actions/test_action_manager.py @@ -14,16 +14,11 @@ # See the License for the specific language governing permissions and # limitations under the License. -from oslo_log import log as logging - from mistral.actions import std_actions as std from mistral.services import action_manager as a_m from mistral.tests.unit import base -LOG = logging.getLogger(__name__) - - class ActionManagerTest(base.DbTestCase): def test_register_standard_actions(self): action_list = a_m.get_registered_actions() diff --git a/mistral/tests/unit/engine/test_action_context.py b/mistral/tests/unit/engine/test_action_context.py index b5049773..9db2d82d 100644 --- a/mistral/tests/unit/engine/test_action_context.py +++ b/mistral/tests/unit/engine/test_action_context.py @@ -14,7 +14,6 @@ import mock from oslo_config import cfg -from oslo_log import log as logging import requests from mistral.actions import std_actions @@ -25,8 +24,6 @@ from mistral.tests.unit.engine import base from mistral.workflow import states -LOG = logging.getLogger(__name__) - # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. cfg.CONF.set_default('auth_enable', False, group='pecan') diff --git a/mistral/tests/unit/engine/test_action_defaults.py b/mistral/tests/unit/engine/test_action_defaults.py index 49131047..9869fb8d 100644 --- a/mistral/tests/unit/engine/test_action_defaults.py +++ b/mistral/tests/unit/engine/test_action_defaults.py @@ -14,7 +14,6 @@ import mock from oslo_config import cfg -from oslo_log import log as logging import requests import six import testtools @@ -26,7 +25,6 @@ from mistral.tests.unit import base as test_base from mistral.tests.unit.engine import base from mistral.workflow import states -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/engine/test_adhoc_actions.py b/mistral/tests/unit/engine/test_adhoc_actions.py index 2d27670a..938ae4cc 100644 --- a/mistral/tests/unit/engine/test_adhoc_actions.py +++ b/mistral/tests/unit/engine/test_adhoc_actions.py @@ -13,14 +13,12 @@ # limitations under the License. from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral.services import workbooks as wb_service from mistral.tests.unit.engine import base from mistral.workflow import states -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/engine/test_commands.py b/mistral/tests/unit/engine/test_commands.py index 2b50f462..0eac3149 100644 --- a/mistral/tests/unit/engine/test_commands.py +++ b/mistral/tests/unit/engine/test_commands.py @@ -13,14 +13,12 @@ # limitations under the License. from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral.services import workbooks as wb_service from mistral.tests.unit.engine import base from mistral.workflow import states -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/engine/test_cron_trigger.py b/mistral/tests/unit/engine/test_cron_trigger.py index e79a3d84..43c101d4 100644 --- a/mistral/tests/unit/engine/test_cron_trigger.py +++ b/mistral/tests/unit/engine/test_cron_trigger.py @@ -15,7 +15,6 @@ import datetime import mock from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral.services import periodic @@ -24,7 +23,6 @@ from mistral.services import triggers from mistral.services import workflows from mistral.tests.unit.engine import base -LOG = logging.getLogger(__name__) WORKFLOW_LIST = """ --- diff --git a/mistral/tests/unit/engine/test_dataflow.py b/mistral/tests/unit/engine/test_dataflow.py index 7c9301ec..4db442e5 100644 --- a/mistral/tests/unit/engine/test_dataflow.py +++ b/mistral/tests/unit/engine/test_dataflow.py @@ -16,7 +16,6 @@ import mock from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral.db.v2.sqlalchemy import models @@ -26,7 +25,6 @@ from mistral.tests.unit.engine import base as engine_test_base from mistral.workflow import data_flow from mistral.workflow import states -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/engine/test_default_engine.py b/mistral/tests/unit/engine/test_default_engine.py index be25da61..72a08d4c 100644 --- a/mistral/tests/unit/engine/test_default_engine.py +++ b/mistral/tests/unit/engine/test_default_engine.py @@ -18,7 +18,6 @@ import uuid import mock from oslo_config import cfg -from oslo_log import log as logging from oslo_messaging.rpc import client as rpc_client from mistral.db.v2 import api as db_api @@ -31,7 +30,6 @@ from mistral.tests.unit.engine import base as eng_test_base from mistral.workflow import states from mistral.workflow import utils as wf_utils -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/engine/test_direct_workflow.py b/mistral/tests/unit/engine/test_direct_workflow.py index b1a7545f..acffcd9b 100644 --- a/mistral/tests/unit/engine/test_direct_workflow.py +++ b/mistral/tests/unit/engine/test_direct_workflow.py @@ -14,7 +14,6 @@ # limitations under the License. from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral import exceptions as exc @@ -24,8 +23,6 @@ from mistral.workflow import states from mistral.workflow import utils as wf_utils -LOG = logging.getLogger(__name__) - # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. cfg.CONF.set_default('auth_enable', False, group='pecan') diff --git a/mistral/tests/unit/engine/test_direct_workflow_with_cycles.py b/mistral/tests/unit/engine/test_direct_workflow_with_cycles.py index e8ec3ab6..75bca80c 100644 --- a/mistral/tests/unit/engine/test_direct_workflow_with_cycles.py +++ b/mistral/tests/unit/engine/test_direct_workflow_with_cycles.py @@ -13,7 +13,6 @@ # limitations under the License. from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral.services import workflows as wf_service @@ -22,8 +21,6 @@ from mistral.workflow import data_flow from mistral.workflow import states -LOG = logging.getLogger(__name__) - # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. cfg.CONF.set_default('auth_enable', False, group='pecan') diff --git a/mistral/tests/unit/engine/test_environment.py b/mistral/tests/unit/engine/test_environment.py index 9a404211..bf5587fd 100644 --- a/mistral/tests/unit/engine/test_environment.py +++ b/mistral/tests/unit/engine/test_environment.py @@ -14,7 +14,6 @@ import mock from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral.engine import default_executor @@ -22,7 +21,6 @@ from mistral.engine import rpc from mistral.services import workbooks as wb_service from mistral.tests.unit.engine import base -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/engine/test_error_result.py b/mistral/tests/unit/engine/test_error_result.py index 4e44b988..3ce9e4f6 100644 --- a/mistral/tests/unit/engine/test_error_result.py +++ b/mistral/tests/unit/engine/test_error_result.py @@ -13,7 +13,6 @@ # limitations under the License. from oslo_config import cfg -from oslo_log import log as logging from mistral.actions import base as actions_base from mistral.db.v2 import api as db_api @@ -24,7 +23,6 @@ from mistral.workflow import data_flow from mistral.workflow import states from mistral.workflow import utils as wf_utils -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/engine/test_execution_fields_size_limitation.py b/mistral/tests/unit/engine/test_execution_fields_size_limitation.py index a0c9b374..60832774 100644 --- a/mistral/tests/unit/engine/test_execution_fields_size_limitation.py +++ b/mistral/tests/unit/engine/test_execution_fields_size_limitation.py @@ -13,7 +13,6 @@ # limitations under the License. from oslo_config import cfg -from oslo_log import log as logging import testtools from mistral.actions import base as actions_base @@ -25,7 +24,6 @@ from mistral.tests.unit.engine import base from mistral.workflow import states from mistral.workflow import utils as wf_utils -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/engine/test_javascript_action.py b/mistral/tests/unit/engine/test_javascript_action.py index 2878429d..ec648e14 100644 --- a/mistral/tests/unit/engine/test_javascript_action.py +++ b/mistral/tests/unit/engine/test_javascript_action.py @@ -14,7 +14,6 @@ import mock from oslo_config import cfg -from oslo_log import log as logging import testtools from mistral.db.v2 import api as db_api @@ -23,11 +22,12 @@ from mistral.tests.unit.engine import base from mistral.utils import javascript from mistral.workflow import states -LOG = logging.getLogger(__name__) + # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. cfg.CONF.set_default('auth_enable', False, group='pecan') + WORKBOOK = """ --- version: "2.0" diff --git a/mistral/tests/unit/engine/test_join.py b/mistral/tests/unit/engine/test_join.py index 1691937f..ae64cdf8 100644 --- a/mistral/tests/unit/engine/test_join.py +++ b/mistral/tests/unit/engine/test_join.py @@ -13,7 +13,6 @@ # limitations under the License. from oslo_config import cfg -from oslo_log import log as logging import testtools from mistral.db.v2 import api as db_api @@ -22,7 +21,6 @@ from mistral.tests.unit.engine import base from mistral.workflow import states -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. cfg.CONF.set_default('auth_enable', False, group='pecan') diff --git a/mistral/tests/unit/engine/test_noop_task.py b/mistral/tests/unit/engine/test_noop_task.py index 4d50f88b..c48bbf50 100644 --- a/mistral/tests/unit/engine/test_noop_task.py +++ b/mistral/tests/unit/engine/test_noop_task.py @@ -13,14 +13,12 @@ # limitations under the License. from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral.services import workflows as wf_service from mistral.tests.unit.engine import base from mistral.workflow import states -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. cfg.CONF.set_default('auth_enable', False, group='pecan') diff --git a/mistral/tests/unit/engine/test_policies.py b/mistral/tests/unit/engine/test_policies.py index 75a2c6ec..e2b6eb35 100644 --- a/mistral/tests/unit/engine/test_policies.py +++ b/mistral/tests/unit/engine/test_policies.py @@ -14,7 +14,6 @@ import mock from oslo_config import cfg -from oslo_log import log as logging from mistral.actions import std_actions from mistral.db.v2 import api as db_api @@ -26,11 +25,12 @@ from mistral.tests.unit.engine import base from mistral.workbook import parser as spec_parser from mistral.workflow import states -LOG = logging.getLogger(__name__) + # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. cfg.CONF.set_default('auth_enable', False, group='pecan') + WORKBOOK = """ --- version: '2.0' diff --git a/mistral/tests/unit/engine/test_race_condition.py b/mistral/tests/unit/engine/test_race_condition.py index 036b2af5..dd18edcd 100644 --- a/mistral/tests/unit/engine/test_race_condition.py +++ b/mistral/tests/unit/engine/test_race_condition.py @@ -15,7 +15,6 @@ from eventlet import corolocal from eventlet import semaphore from oslo_config import cfg -from oslo_log import log as logging import testtools from mistral.actions import base as action_base @@ -25,7 +24,6 @@ from mistral.tests.unit import base as test_base from mistral.tests.unit.engine import base from mistral.workflow import states -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/engine/test_reverse_workflow.py b/mistral/tests/unit/engine/test_reverse_workflow.py index 1c52cc14..31080571 100644 --- a/mistral/tests/unit/engine/test_reverse_workflow.py +++ b/mistral/tests/unit/engine/test_reverse_workflow.py @@ -13,7 +13,6 @@ # limitations under the License. from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral import exceptions as exc @@ -22,7 +21,6 @@ from mistral.services import workflows as wf_service from mistral.tests.unit.engine import base from mistral.workflow import states -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/engine/test_state_info.py b/mistral/tests/unit/engine/test_state_info.py index d2d1eac6..9561f860 100644 --- a/mistral/tests/unit/engine/test_state_info.py +++ b/mistral/tests/unit/engine/test_state_info.py @@ -16,7 +16,6 @@ import mock from oslo_config import cfg -from oslo_log import log as logging from mistral.actions import std_actions from mistral.db.v2 import api as db_api @@ -25,7 +24,7 @@ from mistral.services import workflows as wf_service from mistral.tests.unit.engine import base from mistral.workflow import states -LOG = logging.getLogger(__name__) + # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. cfg.CONF.set_default('auth_enable', False, group='pecan') diff --git a/mistral/tests/unit/engine/test_subworkflows.py b/mistral/tests/unit/engine/test_subworkflows.py index 1c6a7341..5c7413f4 100644 --- a/mistral/tests/unit/engine/test_subworkflows.py +++ b/mistral/tests/unit/engine/test_subworkflows.py @@ -15,7 +15,6 @@ import mock from oslo_config import cfg -from oslo_log import log as logging from mistral.actions import std_actions from mistral import context as auth_context @@ -25,7 +24,6 @@ from mistral.services import workbooks as wb_service from mistral.tests.unit.engine import base from mistral.workflow import states -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/engine/test_task_defaults.py b/mistral/tests/unit/engine/test_task_defaults.py index 42bb0696..589d246e 100644 --- a/mistral/tests/unit/engine/test_task_defaults.py +++ b/mistral/tests/unit/engine/test_task_defaults.py @@ -14,7 +14,6 @@ import datetime as dt from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral.services import scheduler @@ -23,7 +22,6 @@ from mistral.tests.unit.engine import base from mistral.workflow import states -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. cfg.CONF.set_default('auth_enable', False, group='pecan') diff --git a/mistral/tests/unit/engine/test_with_items.py b/mistral/tests/unit/engine/test_with_items.py index 70764646..356a05f6 100644 --- a/mistral/tests/unit/engine/test_with_items.py +++ b/mistral/tests/unit/engine/test_with_items.py @@ -14,7 +14,6 @@ import copy from oslo_config import cfg -from oslo_log import log as logging from mistral.actions import base as action_base from mistral.db.v2 import api as db_api @@ -30,7 +29,6 @@ from mistral.workflow import utils as wf_utils # TODO(nmakhotkin) Need to write more tests. -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. cfg.CONF.set_default('auth_enable', False, group='pecan') diff --git a/mistral/tests/unit/engine/test_workflow_variables.py b/mistral/tests/unit/engine/test_workflow_variables.py index bff0a78a..9398f83e 100644 --- a/mistral/tests/unit/engine/test_workflow_variables.py +++ b/mistral/tests/unit/engine/test_workflow_variables.py @@ -13,7 +13,6 @@ # limitations under the License. from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral.services import workflows as wf_service @@ -21,8 +20,6 @@ from mistral.tests.unit.engine import base from mistral.workflow import states -LOG = logging.getLogger(__name__) - # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. cfg.CONF.set_default('auth_enable', False, group='pecan') diff --git a/mistral/tests/unit/engine/test_yaql_functions.py b/mistral/tests/unit/engine/test_yaql_functions.py index df03580d..4ccf7f91 100644 --- a/mistral/tests/unit/engine/test_yaql_functions.py +++ b/mistral/tests/unit/engine/test_yaql_functions.py @@ -13,14 +13,12 @@ # limitations under the License. from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral.services import workflows as wf_service from mistral.tests.unit.engine import base as engine_test_base from mistral.workflow import states -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/services/test_action_service.py b/mistral/tests/unit/services/test_action_service.py index 4d37a506..bd7db568 100644 --- a/mistral/tests/unit/services/test_action_service.py +++ b/mistral/tests/unit/services/test_action_service.py @@ -13,7 +13,6 @@ # limitations under the License. from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral.services import actions as action_service @@ -21,7 +20,6 @@ from mistral.tests.unit import base from mistral import utils from mistral.workbook import parser as spec_parser -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/services/test_workbook_service.py b/mistral/tests/unit/services/test_workbook_service.py index 42f2b382..aa2059bd 100644 --- a/mistral/tests/unit/services/test_workbook_service.py +++ b/mistral/tests/unit/services/test_workbook_service.py @@ -13,14 +13,12 @@ # limitations under the License. from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral.services import workbooks as wb_service from mistral.tests.unit import base from mistral.workbook import parser as spec_parser -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/services/test_workflow_service.py b/mistral/tests/unit/services/test_workflow_service.py index fb55c03d..5c6812cd 100644 --- a/mistral/tests/unit/services/test_workflow_service.py +++ b/mistral/tests/unit/services/test_workflow_service.py @@ -16,7 +16,6 @@ import copy from oslo_config import cfg -from oslo_log import log as logging from mistral.db.v2.sqlalchemy import api as db_api from mistral import exceptions as exc @@ -26,7 +25,6 @@ from mistral import utils from mistral.workbook import parser as spec_parser from mistral.workflow import states -LOG = logging.getLogger(__name__) # Use the set_default method to set value otherwise in certain test cases # the change in value is not permanent. diff --git a/mistral/tests/unit/workbook/v2/base.py b/mistral/tests/unit/workbook/v2/base.py index eb244f5e..6c62208a 100644 --- a/mistral/tests/unit/workbook/v2/base.py +++ b/mistral/tests/unit/workbook/v2/base.py @@ -14,7 +14,6 @@ import copy -from oslo_log import log as logging import yaml from mistral import exceptions as exc @@ -23,9 +22,6 @@ from mistral import utils from mistral.workbook import parser as spec_parser -LOG = logging.getLogger(__name__) - - class WorkflowSpecValidationTestCase(base.BaseTest): def __init__(self, *args, **kwargs): diff --git a/mistral/tests/unit/workbook/v2/test_actions.py b/mistral/tests/unit/workbook/v2/test_actions.py index bd86bc47..4e0c5612 100644 --- a/mistral/tests/unit/workbook/v2/test_actions.py +++ b/mistral/tests/unit/workbook/v2/test_actions.py @@ -14,15 +14,10 @@ import copy -from oslo_log import log as logging - from mistral.tests.unit.workbook.v2 import base from mistral import utils -LOG = logging.getLogger(__name__) - - class ActionSpecValidation(base.WorkbookSpecValidationTestCase): def test_base_required(self): diff --git a/mistral/tests/unit/workbook/v2/test_workbook.py b/mistral/tests/unit/workbook/v2/test_workbook.py index dd6e3d25..78d084b9 100644 --- a/mistral/tests/unit/workbook/v2/test_workbook.py +++ b/mistral/tests/unit/workbook/v2/test_workbook.py @@ -15,16 +15,12 @@ import copy -from oslo_log import log as logging import yaml from mistral import exceptions as exc from mistral.tests.unit.workbook.v2 import base -LOG = logging.getLogger(__name__) - - class WorkbookSpecValidation(base.WorkbookSpecValidationTestCase): def test_build_valid_workbook_spec(self): diff --git a/mistral/tests/unit/workbook/v2/test_workflows.py b/mistral/tests/unit/workbook/v2/test_workflows.py index 82eb56f5..70cc6c49 100644 --- a/mistral/tests/unit/workbook/v2/test_workflows.py +++ b/mistral/tests/unit/workbook/v2/test_workflows.py @@ -14,15 +14,12 @@ import copy -from oslo_log import log as logging import yaml from mistral import exceptions as exc from mistral.tests.unit.workbook.v2 import base from mistral import utils -LOG = logging.getLogger(__name__) - class WorkflowSpecValidation(base.WorkflowSpecValidationTestCase): def test_workflow_types(self): diff --git a/mistral/tests/unit/workflow/test_direct_workflow.py b/mistral/tests/unit/workflow/test_direct_workflow.py index c19d6373..8e864a0e 100644 --- a/mistral/tests/unit/workflow/test_direct_workflow.py +++ b/mistral/tests/unit/workflow/test_direct_workflow.py @@ -13,7 +13,6 @@ # limitations under the License. import mock -from oslo_log import log as logging from mistral.db.v2 import api as db_api from mistral.db.v2.sqlalchemy import models @@ -23,8 +22,6 @@ from mistral.workbook import parser as spec_parser from mistral.workflow import direct_workflow as d_wf from mistral.workflow import states -LOG = logging.getLogger(__name__) - class DirectWorkflowControllerTest(base.DbTestCase): def _prepare_test(self, wf_text): diff --git a/mistral/tests/unit/workflow/test_reverse_workflow.py b/mistral/tests/unit/workflow/test_reverse_workflow.py index 5643dff0..abd42956 100644 --- a/mistral/tests/unit/workflow/test_reverse_workflow.py +++ b/mistral/tests/unit/workflow/test_reverse_workflow.py @@ -12,8 +12,6 @@ # See the License for the specific language governing permissions and # limitations under the License. -from oslo_log import log as logging - from mistral.db.v2.sqlalchemy import models from mistral import exceptions as exc from mistral.tests.unit import base @@ -22,8 +20,6 @@ from mistral.workflow import reverse_workflow as reverse_wf from mistral.workflow import states -LOG = logging.getLogger(__name__) - # TODO(rakhmerov): This workflow is too simple. Add more complicated one. WB = """ diff --git a/mistral/tests/unit/workflow/test_states.py b/mistral/tests/unit/workflow/test_states.py index f234f7b7..d0f7068a 100644 --- a/mistral/tests/unit/workflow/test_states.py +++ b/mistral/tests/unit/workflow/test_states.py @@ -14,13 +14,9 @@ # See the License for the specific language governing permissions and # limitations under the License. -from oslo_log import log as logging - from mistral.tests.unit import base from mistral.workflow import states as s -LOG = logging.getLogger(__name__) - class StatesModuleTest(base.BaseTest): def test_is_valid_transition(self): diff --git a/tools/sync_db.py b/tools/sync_db.py index 76b16be9..7e70d3c8 100644 --- a/tools/sync_db.py +++ b/tools/sync_db.py @@ -22,7 +22,6 @@ from mistral.services import workflows CONF = cfg.CONF -LOG = logging.getLogger(__name__) def main():