The branch, master has been updated
       via  c2dd325 test that build farm locks are > 0 size
      from  d405260 Add an easy way to got back to the error list

http://gitweb.samba.org/?p=build-farm.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit c2dd3258882427afeaf3e1f7eb59bc13d30441e7
Author: Andrew Bartlett <abart...@samba.org>
Date:   Thu May 17 11:56:35 2012 +1000

    test that build farm locks are > 0 size
    
    This does make the locks less atomic, but disk full situations are more 
common.
    
    When the disk fills up, then the zero length file is created, and the host
    drops out of the build farm.
    
    Andrew Bartlett

-----------------------------------------------------------------------

Summary of changes:
 build_test.fns |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/build_test.fns b/build_test.fns
index 392180a..2358524 100644
--- a/build_test.fns
+++ b/build_test.fns
@@ -196,7 +196,9 @@ lock_file() {
                        return 0
                fi
 
-               if test -f "$lckf"; then
+               # We need to assert that the file is > 0 size, as otherwise we 
never
+                # recover from disk full situations
+               if test -f "$lckf" && test -s "$lckf"; then
                        test x$machine = x$host || {
                                echo "lock file $lckf is valid for other 
machine $machine"
                                return 1


-- 
build.samba.org

Reply via email to