]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
qa: export DIFF_LENGTH env parameter to print the full git diff 50196/head
authorXiubo Li <xiubli@redhat.com>
Tue, 21 Feb 2023 05:20:50 +0000 (13:20 +0800)
committerXiubo Li <xiubli@redhat.com>
Wed, 22 Feb 2023 06:15:14 +0000 (14:15 +0800)
From the xfstests-dev's README doc set DIFF_LENGTH to 0 will print
the full diff instead of the default 10.

This will be very helpful to debug the test failures, or we will
lose many valuable debug infomation.

Signed-off-by: Xiubo Li <xiubli@redhat.com>
qa/tasks/cephfs/test_acls.py
qa/tasks/cephfs/test_fscrypt.py

index 6e19e55c3731c26a56b6d72ace8e4c974e920435..a03f15693c2bc34f2eb4958159610621e9dec196 100644 (file)
@@ -27,8 +27,8 @@ class TestACLs(XFSTestsDev):
         # failure on our own (since this command doesn't set right error code
         # and error message in some cases) and print custom log messages
         # accordingly.
-        proc = self.mount_a.client_remote.run(args=['sudo', './check',
-            'generic/099'], cwd=self.xfstests_repo_path, stdout=StringIO(),
+        proc = self.mount_a.client_remote.run(args=['sudo', 'env', 'DIFF_LENGTH=0',
+            './check', 'generic/099'], cwd=self.xfstests_repo_path, stdout=StringIO(),
             stderr=StringIO(), timeout=30, check_status=False,omit_sudo=False,
             label='running tests for ACLs from xfstests-dev')
 
index 3af9df3cb255c1b715b613578c3425af1b4fd546..11dd2038f92888e4c4a60e1b0fc9e7f4fd740d7f 100644 (file)
@@ -32,8 +32,8 @@ class TestFscrypt(XFSTestsDev):
         # failure on our own (since this command doesn't set right error code
         # and error message in some cases) and print custom log messages
         # accordingly.
-        proc = self.mount_a.client_remote.run(args=['sudo', './check',
-            '-g', 'encrypt'], cwd=self.xfstests_repo_path, stdout=StringIO(),
+        proc = self.mount_a.client_remote.run(args=['sudo', 'env', 'DIFF_LENGTH=0',
+            './check', '-g', 'encrypt'], cwd=self.xfstests_repo_path, stdout=StringIO(),
             stderr=StringIO(), timeout=900, check_status=False, omit_sudo=False,
             label='running tests for encrypt from xfstests-dev')
 
@@ -59,8 +59,8 @@ class TestFscrypt(XFSTestsDev):
         # failure on our own (since this command doesn't set right error code
         # and error message in some cases) and print custom log messages
         # accordingly. This will take a long time and set the timeout to 3 hours.
-        proc = self.mount_a.client_remote.run(args=['sudo', './check',
-            '-g', 'quick', '-E', './ceph.exclude'], cwd=self.xfstests_repo_path,
+        proc = self.mount_a.client_remote.run(args=['sudo', 'env', 'DIFF_LENGTH=0',
+            './check', '-g', 'quick', '-E', './ceph.exclude'], cwd=self.xfstests_repo_path,
             stdout=StringIO(), stderr=StringIO(), timeout=10800, check_status=False,
             omit_sudo=False, label='running tests for dummy_encryption from xfstests-dev')