diff options
| author | Michael Kerrin <michael.kerrin@hp.com> | 2013-04-24 13:44:34 +0000 |
|---|---|---|
| committer | Michael Kerrin <michael.kerrin@hp.com> | 2013-04-24 13:55:46 +0000 |
| commit | 813c7a488c07926d235684f45b6afcaf5f91ace5 (patch) | |
| tree | cf78f1d18a03437a652175b502b4a98415816a0c /openstack | |
| parent | da901232e9bbcf32ac6a86dc62fdd5fbf9bfb9c9 (diff) | |
| download | oslo-813c7a488c07926d235684f45b6afcaf5f91ace5.tar.gz oslo-813c7a488c07926d235684f45b6afcaf5f91ace5.tar.xz oslo-813c7a488c07926d235684f45b6afcaf5f91ace5.zip | |
oslo logging tries to run chmod on file
It should not be the responsibility of oslo to manage log files
like this.
Change-Id: I8397a8799a08acd1f0c43de57f3b39b9389f1cd8
Diffstat (limited to 'openstack')
| -rw-r--r-- | openstack/common/log.py | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/openstack/common/log.py b/openstack/common/log.py index fe25c76..c29308c 100644 --- a/openstack/common/log.py +++ b/openstack/common/log.py @@ -37,7 +37,6 @@ import logging import logging.config import logging.handlers import os -import stat import sys import traceback @@ -104,10 +103,7 @@ logging_cli_opts = [ generic_log_opts = [ cfg.BoolOpt('use_stderr', default=True, - help='Log output to standard error'), - cfg.StrOpt('logfile_mode', - default='0644', - help='Default file mode used when creating log files'), + help='Log output to standard error') ] log_opts = [ @@ -399,11 +395,6 @@ def _setup_logging_from_conf(): filelog = logging.handlers.WatchedFileHandler(logpath) log_root.addHandler(filelog) - mode = int(CONF.logfile_mode, 8) - st = os.stat(logpath) - if st.st_mode != (stat.S_IFREG | mode): - os.chmod(logpath, mode) - if CONF.use_stderr: streamlog = ColorHandler() log_root.addHandler(streamlog) |
