X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=logger.py;h=8abd8fd10bb8150b77bd3f1dbcf1021a3c86e0e0;hb=refs%2Fheads%2Fplanetlab-4_0-branch;hp=3411df50a8e4bd0b35521ca3c4c1a4890875601a;hpb=aac3e5d7c3443d6e1cb33525aefad35be5fe077a;p=nodemanager.git

diff --git a/logger.py b/logger.py
index 3411df5..8abd8fd 100644
--- a/logger.py
+++ b/logger.py
@@ -1,25 +1,27 @@
-import fcntl
-import os
+"""A very simple logger that tries to be concurrency-safe."""
+
+import os, sys
 import subprocess
 import time
 import traceback
 
-from config import LOG_FILE
 
+LOG_FILE = '/var/log/nm'
 
 def log(msg):
     """Write <msg> to the log file."""
-    # the next three lines ought to be an atomic operation but aren't
-    fd = os.open(LOG_FILE, os.O_WRONLY | os.O_CREAT | os.O_APPEND, 0600)
-    flags = fcntl.fcntl(fd, fcntl.F_GETFD)
-    fcntl.fcntl(fd, fcntl.F_SETFD, flags | fcntl.FD_CLOEXEC)
-    if not msg.endswith('\n'): msg += '\n'
-    os.write(fd, '%s: %s' % (time.asctime(time.gmtime()), msg))
-    os.close(fd)
+    try:
+        fd = os.open(LOG_FILE, os.O_WRONLY | os.O_CREAT | os.O_APPEND, 0600)
+        if not msg.endswith('\n'): msg += '\n'
+        os.write(fd, '%s: %s' % (time.asctime(time.gmtime()), msg))
+        os.close(fd)
+    except OSError:
+        sys.stderr.write(msg)
+        sys.stderr.flush()
 
 def log_call(*args):
     log('running command %s' % ' '.join(args))
-    try: subprocess.call(args)
+    try: subprocess.call(args, close_fds=True)
     except: log_exc()
 
 def log_exc():