From 66104191d182aedd7b76f4a1b839c448f97c3ffe Mon Sep 17 00:00:00 2001 From: Brian Coca Date: Tue, 26 Jan 2016 11:22:27 -0500 Subject: [PATCH] fixed permissions check for ansible.log fixes #13990 --- lib/ansible/utils/display.py | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/lib/ansible/utils/display.py b/lib/ansible/utils/display.py index 3703c15540b..ba927237599 100644 --- a/lib/ansible/utils/display.py +++ b/lib/ansible/utils/display.py @@ -48,18 +48,17 @@ except NameError: # These are module level as we currently fork and serialize the whole process and locks in the objects don't play well with that debug_lock = Lock() +logger = None #TODO: make this a logging callback instead if C.DEFAULT_LOG_PATH: path = C.DEFAULT_LOG_PATH - if (os.path.exists(path) and not os.access(path, os.W_OK)) or not os.access(os.path.dirname(path), os.W_OK): - print("[WARNING]: log file at %s is not writeable, aborting\n" % path, file=sys.stderr) - - logging.basicConfig(filename=path, level=logging.DEBUG, format='%(asctime)s %(name)s %(message)s') - mypid = str(os.getpid()) - user = getpass.getuser() - logger = logging.getLogger("p=%s u=%s | " % (mypid, user)) -else: - logger = None + if (os.path.exists(path) and os.access(path, os.W_OK)) or os.access(os.path.dirname(path), os.W_OK): + logging.basicConfig(filename=path, level=logging.DEBUG, format='%(asctime)s %(name)s %(message)s') + mypid = str(os.getpid()) + user = getpass.getuser() + logger = logging.getLogger("p=%s u=%s | " % (mypid, user)) + else: + print("[WARNING]: log file at %s is not writeable and we cannot create it, aborting\n" % path, file=sys.stderr) class Display: