From: Zack Cerza <zack@cerza.org>
Date: Thu, 5 Sep 2013 21:41:27 +0000 (-0500)
Subject: Put sudo in front of adjust-ulimits if it's used
X-Git-Tag: 1.1.0~1899^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=91fd0413cc7c22edfbdfd9a09b240d96b31a3d8a;p=teuthology.git

Put sudo in front of adjust-ulimits if it's used
---

diff --git a/teuthology/task/ceph.py b/teuthology/task/ceph.py
index d89162618..6e3b08aed 100644
--- a/teuthology/task/ceph.py
+++ b/teuthology/task/ceph.py
@@ -507,10 +507,10 @@ def cluster(ctx, config):
                     '-p',
                     '/var/lib/ceph/mds/ceph-{id}'.format(id=id_),
                     run.Raw('&&'),
+                    'sudo',
                     '{tdir}/adjust-ulimits'.format(tdir=testdir),
                     'ceph-coverage',
                     coverage_dir,
-                    'sudo',
                     'ceph-authtool',
                     '--create-keyring',
                     '--gen-key',
@@ -610,11 +610,11 @@ def cluster(ctx, config):
         for id_ in teuthology.roles_of_type(roles_for_host, 'osd'):
             remote.run(
                 args=[
+                    'sudo',
                     'MALLOC_CHECK_=3',
                     '{tdir}/adjust-ulimits'.format(tdir=testdir),
                     'ceph-coverage',
                     coverage_dir,
-                    'sudo',
                     'ceph-osd',
                     '--mkfs',
                     '--mkkey',
@@ -695,10 +695,10 @@ def cluster(ctx, config):
                 )
             remote.run(
                 args=[
+                    'sudo',
                     '{tdir}/adjust-ulimits'.format(tdir=testdir),
                     'ceph-coverage',
                     coverage_dir,
-                    'sudo',
                     'ceph-mon',
                     '--mkfs',
                     '-i', id_,
@@ -874,10 +874,10 @@ def run_daemon(ctx, config, type_):
                 num_active += 1
 
             run_cmd = [
+                'sudo',
                 '{tdir}/adjust-ulimits'.format(tdir=testdir),
                 'ceph-coverage',
                 coverage_dir,
-                'sudo',
                 '{tdir}/daemon-helper'.format(tdir=testdir),
                 daemon_signal,
                 ]
diff --git a/teuthology/task/ceph_client.py b/teuthology/task/ceph_client.py
index fd4f8cb2e..d0cbbb5fe 100644
--- a/teuthology/task/ceph_client.py
+++ b/teuthology/task/ceph_client.py
@@ -15,10 +15,10 @@ def create_keyring(ctx):
             client_keyring = '/etc/ceph/ceph.client.{id}.keyring'.format(id=id_)
             remote.run(
                 args=[
+                    'sudo',
                     '{tdir}/adjust-ulimits'.format(tdir=testdir),
                     'ceph-coverage',
                     coverage_dir,
-                    'sudo',
                     'ceph-authtool',
                     '--create-keyring',
                     '--gen-key',
diff --git a/teuthology/task/rbd.py b/teuthology/task/rbd.py
index d107c6f47..6e1bd71ae 100644
--- a/teuthology/task/rbd.py
+++ b/teuthology/task/rbd.py
@@ -418,10 +418,10 @@ def run_xfstests_one_client(ctx, role, properties):
         # readlink -f <path> in order to get their canonical
         # pathname (so it matches what the kernel remembers).
         args = [
+            '/usr/bin/sudo',
             '{tdir}/adjust-ulimits'.format(tdir=testdir),
             'ceph-coverage',
             '{tdir}/archive/coverage'.format(tdir=testdir),
-            '/usr/bin/sudo',
             'TESTDIR={tdir}'.format(tdir=testdir),
             '/bin/bash',
             test_path,