From cf9779eda379e3db4a954fe17e559af386cf0c43 Mon Sep 17 00:00:00 2001 From: Ihar Hrachyshka Date: Wed, 12 Apr 2017 11:05:01 -0700 Subject: [PATCH] Don't set use_stderr to False for tests Since the option is False by default in the oslo.log library, this is effectively a no-op. Also it may confuse readers that try to understand how logging is set up for tests. TrivialFix Change-Id: I091566519912251547c703139b38daea2979a93d --- neutron/tests/base.py | 1 - neutron/tests/unit/__init__.py | 5 ----- 2 files changed, 6 deletions(-) diff --git a/neutron/tests/base.py b/neutron/tests/base.py index c7263d13ad1..b26512fbe32 100644 --- a/neutron/tests/base.py +++ b/neutron/tests/base.py @@ -82,7 +82,6 @@ def setup_test_logging(config_opts, log_dir, log_file_path_template): log_file = sanitize_log_path( os.path.join(log_dir, log_file_path_template)) config_opts.set_override('log_file', log_file) - config_opts.set_override('use_stderr', False) config.setup_logging() diff --git a/neutron/tests/unit/__init__.py b/neutron/tests/unit/__init__.py index b6e11d08343..f679bebd006 100644 --- a/neutron/tests/unit/__init__.py +++ b/neutron/tests/unit/__init__.py @@ -13,12 +13,7 @@ # License for the specific language governing permissions and limitations # under the License. -from oslo_config import cfg - from neutron.common import eventlet_utils -cfg.CONF.use_stderr = False - - eventlet_utils.monkey_patch()