]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commit
qa/vstart_runner: don't pass sudo for ceph-fuse command
authorRishabh Dave <ridave@redhat.com>
Tue, 9 Aug 2022 08:54:37 +0000 (14:24 +0530)
committerRishabh Dave <ridave@redhat.com>
Mon, 4 Sep 2023 17:08:29 +0000 (22:38 +0530)
commit3125eb20b9dbd0df7bb030691c6103374e9f6457
tree1b50abbed0e509e558606c815f5e17ccf3a3209e
parent1414bb3e0e2a6b3a5021314e74148b44e9923223
qa/vstart_runner: don't pass sudo for ceph-fuse command

Since we pass option "--client_die_on_failed_dentry_invalidate=false"
to ceph-fuse commands issued by vstart_runner.py, passing "sudo" in
ceph-fuse command arguments is unnecessary.

This removes the error message (see below) from vstart_runner.py output
that informs the users that "sudo" was removed from command arguments.
This message is redundant and even misleading as the option above
is passed to ceph-fuse command.

This will have no functional changes on ceph-fuse mount command issued
by vstart_runner.py since FuseMount._run_mount_cmd() did not pass
"omit_sudo=False" (and since default value for omit_sudo in method
LocalRemote.run() of vstart_runner.py is true, vstart_runner.py removes
"sudo" from ceph-fuse command arguments of before execution).

The error message -

DEBUG:__main__:"sudo" was omitted from the following cmd args before execution and logging using function overriding; check vstart_runner.py for more details.
DEBUG:__main__:> sudo ./bin/ceph-fuse /tmp/tmp8o4s_6md/mnt.0 --id 0 --client_mountpoint=/ --client_die_on_failed_dentry_invalidate=false

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