[gnome-continuous-yocto/gnomeostree-3.28-rocko: 7437/8267] scripts/runqemu: Don't print error messages about tap file locks



commit 06280ec2e329bbc7413dc162177d1da642608c05
Author: Richard Purdie <richard purdie linuxfoundation org>
Date:   Tue Aug 29 17:04:38 2017 +0100

    scripts/runqemu: Don't print error messages about tap file locks
    
    Errors like:
    runqemu - ERROR - Acquiring lockfile /tmp/qemu-tap-locks/tap0.lock failed: [Errno 11] Resource 
temporarily unavailable
    
    are not really fatal errors. Change these to info messages instead
    so people look later in the log for the real errors.
    
    (From OE-Core rev: fac12de72bda1e864e71538be07d6c6f6e987498)
    
    Signed-off-by: Richard Purdie <richard purdie linuxfoundation org>

 scripts/runqemu |   10 +++++++---
 1 files changed, 7 insertions(+), 3 deletions(-)
---
diff --git a/scripts/runqemu b/scripts/runqemu
index 1131324..df76270 100755
--- a/scripts/runqemu
+++ b/scripts/runqemu
@@ -232,13 +232,17 @@ class BaseConfig(object):
         self.mac_tap = "52:54:00:12:34:"
         self.mac_slirp = "52:54:00:12:35:"
 
-    def acquire_lock(self):
+    def acquire_lock(self, error=True):
         logger.debug("Acquiring lockfile %s..." % self.lock)
         try:
             self.lock_descriptor = open(self.lock, 'w')
             fcntl.flock(self.lock_descriptor, fcntl.LOCK_EX|fcntl.LOCK_NB)
         except Exception as e:
-            logger.error("Acquiring lockfile %s failed: %s" % (self.lock, e))
+            msg = "Acquiring lockfile %s failed: %s" % (self.lock, e)
+            if error:
+                logger.error(msg)
+            else:
+                logger.info(msg)
             if self.lock_descriptor:
                 self.lock_descriptor.close()
             return False
@@ -927,7 +931,7 @@ class BaseConfig(object):
                 logger.info('Found %s.skip, skipping %s' % (lockfile, p))
                 continue
             self.lock = lockfile + '.lock'
-            if self.acquire_lock():
+            if self.acquire_lock(error=False):
                 tap = p
                 logger.info("Using preconfigured tap device %s" % tap)
                 logger.info("If this is not intended, touch %s.skip to make runqemu skip %s." %(lockfile, 
tap))


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]