From: Zack Cerza <zack@cerza.org>
Date: Tue, 20 Aug 2013 21:25:25 +0000 (-0500)
Subject: Add get_test_user(), because hardcoding is bad
X-Git-Tag: 1.1.0~1957^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F43%2Fhead;p=teuthology.git

Add get_test_user(), because hardcoding is bad
---

diff --git a/teuthology/misc.py b/teuthology/misc.py
index b74a2bb7d..337d0115f 100644
--- a/teuthology/misc.py
+++ b/teuthology/misc.py
@@ -86,14 +86,23 @@ def get_testdir(ctx):
                     user=get_user()[0:2],
                     stamp=stamp)
 
+
+def get_test_user(ctx):
+    """
+    :returns: str -- the user to run tests as on remote hosts
+    """
+    return ctx.teuthology_config.get('test_user', 'ubuntu')
+
+
 def get_testdir_base(ctx):
     if 'test_path' in ctx.teuthology_config:
         return ctx.teuthology_config['test_path']
-    owner_user = ctx.owner.split('@')[0]
+    test_user = get_test_user(ctx)
     # FIXME this ideally should use os.path.expanduser() in the future, in case
     # $HOME isn't /home/$USER - e.g. on a Mac. However, since we're executing
     # this on the server side, it won't work properly.
-    return ctx.teuthology_config.get('base_test_dir', '/home/%s/cephtest' % owner_user)
+    return ctx.teuthology_config.get('base_test_dir', '/home/%s/cephtest' %
+                                     test_user)
 
 def get_ceph_binary_url(package=None,
                         branch=None, tag=None, sha1=None, dist=None,