]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
qa/vstart_runner: update vstart_runner.LocalRemote.sh
authorRishabh Dave <ridave@gmail.com>
Fri, 13 Mar 2020 07:03:50 +0000 (07:03 +0000)
committerNathan Cutler <ncutler@suse.com>
Tue, 16 Jun 2020 13:27:54 +0000 (15:27 +0200)
Commit 9f6c764f10f replaces remote.run calls by remote.sh without
updating the definition of vstart_runner.LocalRemote.sh which breaks the
cephfs tests when executed locally.

Fixes: https://tracker.ceph.com/issues/44579
Signed-off-by: Rishabh Dave <ridave@redhat.com>
(cherry picked from commit 2cc0ee709c36eabe03311a00b72295da468bccf4)

Conflicts:
qa/tasks/vstart_runner.py
- trivial

qa/tasks/vstart_runner.py

index 27a2c33b12273b737e8f41d215f71bb51553582f..cf6aaa123d55485ed5faf5bfbeedb96525e8bf59 100644 (file)
@@ -308,10 +308,22 @@ class LocalRemote(object):
 
         return proc
 
-    def sh(self, command, log_limit=1024, cwd=None, env=None):
+    # XXX: for compatibility keep this method same teuthology.orchestra.remote.sh
+    def sh(self, script, **kwargs):
+        """
+        Shortcut for run method.
+
+        Usage:
+            my_name = remote.sh('whoami')
+            remote_date = remote.sh('date')
+        """
+        if 'stdout' not in kwargs:
+            kwargs['stdout'] = BytesIO()
+        if 'args' not in kwargs:
+            kwargs['args'] = script
+        proc = self.run(**kwargs)
+        return proc.stdout.getvalue()
 
-        return misc.sh(command=command, log_limit=log_limit, cwd=cwd,
-                        env=env)
 
 class LocalDaemon(object):
     def __init__(self, daemon_type, daemon_id):