test-concurrency: Lower lock timeout
authorDan Nicholson <dbn@endlessos.org>
Mon, 19 Apr 2021 16:03:57 +0000 (10:03 -0600)
committerDan Nicholson <dbn@endlessos.org>
Sat, 5 Jun 2021 15:15:34 +0000 (09:15 -0600)
If there's a locking issue in this test, then it's likely not going to
resolve after a few seconds of serializing access. Lower the default 30
second lock timeout to 5 seconds to prevent the test from hanging
unnecessarily.

tests/test-concurrency.py

index e4ce21e9e1c77d81a68fc892971fc19c166ed73d..3679ddb63c9db68554d1ea6049c4877897e6ace7 100755 (executable)
@@ -40,11 +40,15 @@ def mktree(dname, serial=0):
             f.write('{} {} {}\n'.format(dname, serial, v))
 
 subprocess.check_call(['ostree', '--repo=repo', 'init', '--mode=bare'])
-# like the bit in libtest, but let's do it unconditionally since it's simpler,
-# and we don't need xattr coverage for this
 with open('repo/config', 'a') as f:
+    # like the bit in libtest, but let's do it unconditionally since
+    # it's simpler, and we don't need xattr coverage for this
     f.write('disable-xattrs=true\n')
 
+    # Make any locking errors fail quickly instead of blocking the test
+    # for 30 seconds.
+    f.write('lock-timeout-secs=5\n')
+
 def commit(v):
     tdir='tree{}'.format(v)
     cmd = ['ostree', '--repo=repo', 'commit', '--fsync=0', '-b', tdir, '--tree=dir='+tdir]