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

@@ -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()))

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()

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')

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.

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.

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.

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 = """
 ---

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.

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.

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')

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')

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.

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.

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.

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"

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')

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')

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'

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.

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.

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')

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.

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')

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')

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')

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.

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.

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.

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.

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):

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):

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):

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):

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):

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 = """

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):

tools/sync_db.py

@@ -22,7 +22,6 @@ from mistral.services import workflows
 
 
 CONF = cfg.CONF
-LOG = logging.getLogger(__name__)
 
 
 def main():