]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
qa/vstart_runner.py: add a note about omitting sudo
authorRishabh Dave <ridave@redhat.com>
Mon, 25 Apr 2022 07:48:47 +0000 (13:18 +0530)
committerRishabh Dave <ridave@redhat.com>
Mon, 2 May 2022 14:55:19 +0000 (20:25 +0530)
About the commit date: this commit got dropped from the patch series
during some PR branch update but is added back now.

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

index 2fa8bbe6be358723306bc6597deb591b874d1758..341fb036a3c2a5c916715fe1230b94ab5c2851c4 100644 (file)
@@ -425,6 +425,8 @@ ceph-coverage() {
         # We'll let sudo be a part of command even omit flag says otherwise in
         # cases of commands which can normally be ran only by root.
         last_arg = args[args.rfind(' ') + 1 : ]
+        # XXX: should sudo be omitted/allowed by default in cases similar to
+        # that of "exec sudo" as well?
         if 'sudo' in args:
             for x in ('passwd', 'chown'):
                 if x == first_arg or x == last_arg or f' {x} ' in args: