public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: chrubis@suse.cz
To: Stanislav Kholmanskikh <stanislav.kholmanskikh@oracle.com>
Cc: vasily.isaenko@oracle.com, ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH 3/5] rpc-tirpc-full-test-suite: fixed the cmdline in runtest
Date: Tue, 11 Feb 2014 16:46:00 +0100	[thread overview]
Message-ID: <20140211154600.GC15043@rei.Home> (raw)
In-Reply-To: <1388048842-28373-4-git-send-email-stanislav.kholmanskikh@oracle.com>

Hi!
> To make debugging easier we don't use run_rpc_test.sh but
> use its own test case name for each test case.
> 
> Signed-off-by: Stanislav Kholmanskikh <stanislav.kholmanskikh@oracle.com>
> ---
>  runtest/rpc                                        |    8 +++++++-
>  runtest/stress.part3                               |    8 +++++++-
>  .../rpc/rpc-tirpc-full-test-suite/rpc_ts_run.sh    |   12 +++++++-----
>  3 files changed, 21 insertions(+), 7 deletions(-)
> 
> diff --git a/runtest/rpc b/runtest/rpc
> index c9e5f0d..7c18eda 100644
> --- a/runtest/rpc
> +++ b/runtest/rpc
> @@ -2,7 +2,13 @@
>  #
>  # PLEASE READ THE README FILE IN /rpc BEFORE RUNNING THESE.
>  #
> -run_rpc_tests.sh run_rpc_tests.sh
> +rpc_addrmanagmt_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_addrmanagmt_basic_lib.sh
> +rpc_auth_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_auth_basic_lib.sh
> +rpc_broadc_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_broadc_basic_lib.sh
> +rpc_createdestroy_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_createdestroy_basic_lib.sh
> +rpc_err_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_err_basic_lib.sh
> +rpc_regunreg_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_regunreg_basic_lib.sh
> +rpc_stdcall_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_stdcall_basic_lib.sh

I don't like the long path here, but we cannot do anything about it
unless we install the binaries into the standard localtion.

>  rpc01 rpc01
>  rpcinfo rpcinfo01
>  rup rup01
> diff --git a/runtest/stress.part3 b/runtest/stress.part3
> index bb4807c..57a5106 100644
> --- a/runtest/stress.part3
> +++ b/runtest/stress.part3
> @@ -886,6 +886,12 @@ hangup01 hangup01
>  mc_cmds mc_cmds
>  mc_opts export TCsrc=$LTPROOT/testcases/network/multicast/mc_opts; mc_opts
>  rpc01 export TCbin=$LTPROOT/testcases/network/rpc/basic_tests/rpc01; rpc01
> -run_rpc_tests.sh export TCbin=$LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; run_rpc_tests.sh
> +rpc_addrmanagmt_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_addrmanagmt_basic_lib.sh
> +rpc_auth_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_auth_basic_lib.sh
> +rpc_broadc_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_broadc_basic_lib.sh
> +rpc_createdestroy_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_createdestroy_basic_lib.sh
> +rpc_err_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_err_basic_lib.sh
> +rpc_regunreg_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_regunreg_basic_lib.sh
> +rpc_stdcall_basic_lib	cd $LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite; ./rpc_ts_run.sh -l rpc_stdcall_basic_lib.sh
>  perf_lan perf_lan
>  
> diff --git a/testcases/network/rpc/rpc-tirpc-full-test-suite/rpc_ts_run.sh b/testcases/network/rpc/rpc-tirpc-full-test-suite/rpc_ts_run.sh
> index e3fca59..66be9b8 100755
> --- a/testcases/network/rpc/rpc-tirpc-full-test-suite/rpc_ts_run.sh
> +++ b/testcases/network/rpc/rpc-tirpc-full-test-suite/rpc_ts_run.sh
> @@ -24,15 +24,17 @@
>  # *** INITIALISATION ***
>  # **********************
>  
> +LTPROOT=${LTPROOT:=$(readlink -f ../../../..)}

Why are you redefining the LTPROOT? It should be exported by the ltp-pan
correcty.

>  # Indicates in which directory tests binaries are installed
> -SERVERTSTPACKDIR=/home/lerouzic/benchs/ltp-full-20080930/testcases/network/rpc/rpc-tirpc-full-test-suite/tests_pack
> -CLIENTTSTPACKDIR=/home/lerouzic/benchs/ltp-full-20080930/testcases/network/rpc/rpc-tirpc-full-test-suite/tests_pack
> +SERVERTSTPACKDIR=$LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite/tests_pack
> +CLIENTTSTPACKDIR=$LTPROOT/testcases/network/rpc/rpc-tirpc-full-test-suite/tests_pack
>  # LOCALIP variable is computer IP/name used to run tests scripts
>  # if LOCALIP = SERVERIP = CLIENTIP, that means all tests are in local mode
>  # (run on the same machine)
> -LOCALIP=localhost
> -SERVERIP=localhost
> -CLIENTIP=localhost
> +LOCALIP=`hostname`
> +SERVERIP=`hostname`
> +CLIENTIP=`hostname`
>  SERVERUSER=root
>  CLIENTUSER=root
>  # number of threads for thread tests

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
Android apps run on BlackBerry 10
Introducing the new BlackBerry 10.2.1 Runtime for Android apps.
Now with support for Jelly Bean, Bluetooth, Mapview and more.
Get your Android app in front of a whole new audience.  Start now.
http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  reply	other threads:[~2014-02-11 15:46 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-26  9:07 [LTP] rpc-tirpc-full-test-suite Stanislav Kholmanskikh
2013-12-26  9:07 ` [LTP] [PATCH 1/5] rpc-tirpc-full-test-suite: do not use tst_exit Stanislav Kholmanskikh
2013-12-26  9:07 ` [LTP] [PATCH 2/5] rpc-tirpc-full-test-suite: enabled build of the rpc test cases Stanislav Kholmanskikh
2014-02-11 15:42   ` chrubis
2013-12-26  9:07 ` [LTP] [PATCH 3/5] rpc-tirpc-full-test-suite: fixed the cmdline in runtest Stanislav Kholmanskikh
2014-02-11 15:46   ` chrubis [this message]
2013-12-26  9:07 ` [LTP] [PATCH 4/5] rpc-tirpc-full-test-suite: pass exit value to the wrapper Stanislav Kholmanskikh
2014-02-11 15:54   ` chrubis
2013-12-26  9:07 ` [LTP] [PATCH 5/5] rpc_stdcall_basic_lib: fixed IS_EX match Stanislav Kholmanskikh
2014-02-11 15:10 ` [LTP] rpc-tirpc-full-test-suite chrubis
     [not found]   ` <530C7004.3060102@oracle.com>
2014-02-26 13:47     ` chrubis
     [not found]       ` <530DF8C6.50600@oracle.com>
2014-02-26 15:08         ` chrubis
     [not found]           ` <5317454D.5010505@oracle.com>
2014-03-11 14:02             ` chrubis
     [not found]               ` <531F1FEE.1060001@oracle.com>
2014-03-11 15:08                 ` chrubis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140211154600.GC15043@rei.Home \
    --to=chrubis@suse.cz \
    --cc=ltp-list@lists.sourceforge.net \
    --cc=stanislav.kholmanskikh@oracle.com \
    --cc=vasily.isaenko@oracle.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox