[Piuparts-commits] rev 310 - in trunk: . debian

[Holger Levsen]

Author: holger
Date: 2009-04-03 18:35:44 +0000 (Fri, 03 Apr 2009)
New Revision: 310

Modified:
   trunk/debian/changelog
   trunk/piuparts.py
Log:
add "FAIL: " to logging output for seven more failure types, so that it
becomes easier to group failure causes.

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog	2009-04-02 18:27:12 UTC (rev 309)
+++ trunk/debian/changelog	2009-04-03 18:35:44 UTC (rev 310)
@@ -14,6 +14,8 @@
     - add an optional parameter to panic() method to specify what exit status
       to use. Also thanks to Andres.
     - add advice how to read the logfile to the top of the logfiles produced.
+    - add "FAIL: " to logging output for seven more failure types, so that it
+      becomes easier to group failure causes.
   * piupartslib/packagesdb.py:
     - change the test whether a package is testable to check whether the 
       package is of priority "required", and not whether it's "Essential".

Modified: trunk/piuparts.py
===================================================================
--- trunk/piuparts.py	2009-04-02 18:27:12 UTC (rev 309)
+++ trunk/piuparts.py	2009-04-03 18:35:44 UTC (rev 310)
@@ -881,7 +881,7 @@
         (status, output) = run(["lsof", "-w", "+D", self.name], ignore_errors=True)
         count = len(output.split("\n")) - 1
         if count > 0:
-            logging.error("Processes are running inside chroot:\n%s" % 
+            logging.error("FAIL: Processes are running inside chroot:\n%s" % 
                           indent_string(output))
             panic()
 
@@ -937,7 +937,7 @@
                         target = "<unknown>"
                     broken.append("%s -> %s" % (name, target))
         if broken:
-            logging.error("Broken symlinks:\n%s" % 
+            logging.error("FAIL: Broken symlinks:\n%s" % 
                           indent_string("\n".join(broken)))
 	    panic()
         else:
@@ -985,7 +985,7 @@
             (retval, output) = self.run([file])
             if output:
                 failed = True
-                logging.error("Cron file %s has output with package removed" % file)
+                logging.error("FAIL: Cron file %s has output with package removed" % file)
 
         if failed:
             panic()
@@ -1254,7 +1254,7 @@
             f = file(tf)
             broken = False
             for l in f:
-                logging.error("Broken symlink: " + l)
+                logging.error("FAIL: Broken symlink: " + l)
                 broken = True
             if broken: panic()
             logging.debug("No broken symlinks found.")
@@ -1455,15 +1455,15 @@
 
     ok = True
     if new:
-        logging.error("Package purging left files on system:\n" +
+        logging.error("FAIL: Package purging left files on system:\n" +
                        file_list(new, file_owners))
         ok = False
     if removed:
-        logging.error("After purging files have disappeared:\n" +
+        logging.error("FAIL: After purging files have disappeared:\n" +
                       file_list(removed, file_owners))
         ok = False
     if modified:
-        logging.error("After purging files have been modified:\n" +
+        logging.error("FAIL: After purging files have been modified:\n" +
                       file_list(modified, file_owners))
         ok = False