]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
qa/cephfs: set omit_sudo to False for wait_until_mounted() in...
authorRishabh Dave <ridave@redhat.com>
Mon, 6 Jul 2020 17:41:27 +0000 (23:11 +0530)
committerRishabh Dave <ridave@redhat.com>
Tue, 7 Jul 2020 04:15:28 +0000 (09:45 +0530)
fuse_mount.py. This isn't critical at all to vstart_runner.py runs but
this patch must dramatically reduce the time it takes in case the
command fails with "permission denied" due to lack of superuser
privileges since in this case the command is re-run 9 more times, each
separated by a sleep for 5 seconds.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
qa/tasks/cephfs/fuse_mount.py

index 70fcaacdef6c0d908d03fc1d4e3093b434f1348b..5fc6d8ae67bdee400ba6feecd61416dc89655f1d 100644 (file)
@@ -241,13 +241,15 @@ class FuseMount(CephFSMount):
 
         self.mounted = True
 
-        # Now that we're mounted, set permissions so that the rest of the test will have
-        # unrestricted access to the filesystem mount.
+        # Now that we're mounted, set permissions so that the rest of the test
+        # will have unrestricted access to the filesystem mount.
         for retry in range(10):
             try:
                 stderr = StringIO()
-                self.client_remote.run(args=['sudo', 'chmod', '1777', self.mountpoint],
-                                       timeout=(15*60), cwd=self.test_dir, stderr=stderr)
+                self.client_remote.run(args=['sudo', 'chmod', '1777',
+                                             self.mountpoint],
+                                       timeout=(15*60), cwd=self.test_dir,
+                                       stderr=stderr, omit_sudo=False)
                 break
             except run.CommandFailedError:
                 stderr = stderr.getvalue()