]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
qa: support running under non privileged user
authorLoic Dachary <loic@dachary.org>
Tue, 1 Jul 2014 06:42:58 +0000 (08:42 +0200)
committerSage Weil <sage@redhat.com>
Fri, 1 Aug 2014 23:57:49 +0000 (16:57 -0700)
If the test is run against a cluster started with vstart.sh (which is
the case for make check), the --asok-does-not-need-root disables the use
of sudo and allows the test to run without requiring privileged user
permissions.

Signed-off-by: Loic Dachary <loic@dachary.org>
(cherry picked from commit 522174b066044e51a7019bd6cad81117e83c394e)

qa/workunits/cephtool/test.sh
src/test/vstart_wrapped_tests.sh

index d66a601e63de3af731a5892346b4c90405d0c59f..8ee7da8b6dc638633f2b2ad9e79ce0046faff6e2 100755 (executable)
@@ -3,6 +3,7 @@
 set -e
 set -o functrace
 PS4=' ${FUNCNAME[0]}: $LINENO: '
+SUDO=sudo
 
 function get_pg()
 {
@@ -58,7 +59,7 @@ function get_config_value_or_die()
   target=$1
   config_opt=$2
 
-  raw="`sudo ceph daemon $target config get $config_opt 2>/dev/null`"
+  raw="`$SUDO ceph daemon $target config get $config_opt 2>/dev/null`"
   if [[ $? -ne 0 ]]; then
     echo "error obtaining config opt '$config_opt' from '$target': $raw"
     exit 1
@@ -96,13 +97,13 @@ function test_mon_injectargs_SI()
   # Keep in mind that all integer based options (i.e., INT,
   # LONG, U32, U64) will accept SI unit modifiers.
   initial_value=$(get_config_value_or_die "mon.a" "mon_pg_warn_min_objects")
-  sudo ceph daemon mon.a config set mon_pg_warn_min_objects 10
+  $SUDO ceph daemon mon.a config set mon_pg_warn_min_objects 10
   expect_config_value "mon.a" "mon_pg_warn_min_objects" 10
-  sudo ceph daemon mon.a config set mon_pg_warn_min_objects 10K
+  $SUDO ceph daemon mon.a config set mon_pg_warn_min_objects 10K
   expect_config_value "mon.a" "mon_pg_warn_min_objects" 10240
-  sudo ceph daemon mon.a config set mon_pg_warn_min_objects 1G
+  $SUDO ceph daemon mon.a config set mon_pg_warn_min_objects 1G
   expect_config_value "mon.a" "mon_pg_warn_min_objects" 1073741824
-  sudo ceph daemon mon.a config set mon_pg_warn_min_objects 10F > $TMPFILE || true
+  $SUDO ceph daemon mon.a config set mon_pg_warn_min_objects 10F > $TMPFILE || true
   check_response "'10F': (22) Invalid argument"
   # now test with injectargs
   ceph tell mon.a injectargs '--mon_pg_warn_min_objects 10'
@@ -112,7 +113,7 @@ function test_mon_injectargs_SI()
   ceph tell mon.a injectargs '--mon_pg_warn_min_objects 1G'
   expect_config_value "mon.a" "mon_pg_warn_min_objects" 1073741824
   expect_false ceph injectargs mon.a '--mon_pg_warn_min_objects 10F'
-  sudo ceph daemon mon.a config set mon_pg_warn_min_objects $initial_value
+  $SUDO ceph daemon mon.a config set mon_pg_warn_min_objects $initial_value
 }
 
 function test_tiering()
@@ -796,6 +797,9 @@ while [[ $# -gt 0 ]]; do
     "-l" )
       do_list=1
       ;;
+    "--asok-does-not-need-root" )
+      SUDO=""
+      ;;
     "-t" )
       shift
       if [[ -z "$1" ]]; then
index 71148af74c642cf535d904464816ba69e77d06de..d80863f5d22258f953a10334bfae476c4a858bc7 100755 (executable)
@@ -25,6 +25,6 @@ while read line ; do
   fi
   echo "================ STOP ================="  
 done <<EOF
-../qa/workunits/cephtool/test.sh
+../qa/workunits/cephtool/test.sh --asok-does-not-need-root
 EOF
 exit $code