openstack
/
neutron
Code Issues Proposed changes

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
This commit is contained in:
Ihar Hrachyshka 2017-04-12 11:05:01 -07:00 committed by Kevin Benton
parent 6da00730ab
commit cf9779eda3
2 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
neutron/tests/base.py
View File

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

5
neutron/tests/unit/__init__.py
View File

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