public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: Cyril Hrubis <chrubis@suse.cz>
To: Alexey Kodanev <alexey.kodanev@oracle.com>
Cc: vasily.isaenko@oracle.com, ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH 19/23] network/virt: add test-case 02 to virt_lib.sh
Date: Tue, 11 Aug 2015 20:32:36 +0200	[thread overview]
Message-ID: <20150811183236.GF24227@rei.suse.de> (raw)
In-Reply-To: <1436424468-11756-20-git-send-email-alexey.kodanev@oracle.com>

Hi!
> diff --git a/testcases/network/virt/ipvlan01.sh b/testcases/network/virt/ipvlan01.sh
> index 706c1d8..156de78 100755
> --- a/testcases/network/virt/ipvlan01.sh
> +++ b/testcases/network/virt/ipvlan01.sh
> @@ -16,8 +16,8 @@
>  #
>  # Author: Alexey Kodanev <alexey.kodanev@oracle.com>
>  #
> -# Test-case 1: Local test, check if we can create and then delete ipvlan
> -#              interface 500 times.
> +# Local test, check if we can create and then delete ipvlan
> +# interface 1000 times.
>  
>  TCID=ipvlan01
>  TST_TOTAL=2
> @@ -27,19 +27,8 @@ virt_type="ipvlan"
>  . test_net.sh
>  . virt_lib.sh
>  
> -modes="l2 l3"
> +options="mode l2,mode l3"
>  
> -start_id=1
> -virt_count=500
> -
> -for m in $modes; do
> -	tst_resm TINFO "add $virt_type with mode '$m'"
> -
> -	virt_check_cmd virt_add ltp_v0 mode $m || m=""
> -
> -	virt_add_delete_test "mode $m"
> -
> -	start_id=$(($start_id + $virt_count))
> -done
> +virt_test_02 "$options"
>  
>  tst_exit
> diff --git a/testcases/network/virt/macvlan01.sh b/testcases/network/virt/macvlan01.sh
> index 35febd0..402edde 100755
> --- a/testcases/network/virt/macvlan01.sh
> +++ b/testcases/network/virt/macvlan01.sh
> @@ -16,8 +16,8 @@
>  #
>  # Author: Alexey Kodanev <alexey.kodanev@oracle.com>
>  #
> -# Test-case 1: Local test, check if we can create and then delete macvlan
> -#              interface 500 times.
> +# Local test, check if we can create and then delete macvlan
> +# interface 2000 times.
>  
>  TCID=macvlan01
>  TST_TOTAL=4
> @@ -27,19 +27,8 @@ virt_type="macvlan"
>  . test_net.sh
>  . virt_lib.sh
>  
> -modes="private vepa bridge passthru"
> +options="mode private,mode vepa,mode bridge,mode passthru"
>  
> -start_id=1
> -virt_count=500
> -
> -for m in $modes; do
> -	tst_resm TINFO "add $virt_type with mode '$m'"
> -
> -	virt_check_cmd virt_add ltp_v0 mode $m || m=""
> -
> -	virt_add_delete_test "mode $m"
> -
> -	start_id=$(($start_id + $virt_count))
> -done
> +virt_test_02 "$options"
>  
>  tst_exit
> diff --git a/testcases/network/virt/macvtap01.sh b/testcases/network/virt/macvtap01.sh
> index 8bb3636..3408c94 100755
> --- a/testcases/network/virt/macvtap01.sh
> +++ b/testcases/network/virt/macvtap01.sh
> @@ -16,8 +16,8 @@
>  #
>  # Author: Alexey Kodanev <alexey.kodanev@oracle.com>
>  #
> -# Test-case 1: Local test, check if we can create and then delete macvtap
> -#              interface 500 times.
> +# Local test, check if we can create and then delete macvtap
> +# interface 2000 times.
>  
>  TCID=macvtap01
>  TST_TOTAL=4
> @@ -27,19 +27,8 @@ virt_type="macvtap"
>  . test_net.sh
>  . virt_lib.sh
>  
> -modes="private vepa bridge passthru"
> +options="mode private,mode vepa,mode bridge,mode passthru"
>  
> -start_id=1
> -virt_count=500
> -
> -for m in $modes; do
> -	tst_resm TINFO "add $virt_type with mode '$m'"
> -
> -	virt_check_cmd virt_add ltp_v0 mode $m || m=""
> -
> -	virt_add_delete_test "mode $m"
> -
> -	start_id=$(($start_id + $virt_count))
> -done
> +virt_test_02 "$options"
>  
>  tst_exit
> diff --git a/testcases/network/virt/virt_lib.sh b/testcases/network/virt/virt_lib.sh
> index 2ccba43..d79f7bc 100644
> --- a/testcases/network/virt/virt_lib.sh
> +++ b/testcases/network/virt/virt_lib.sh
> @@ -289,6 +289,34 @@ virt_test_01()
>  	done
>  }
>  
> +# Check if we can create then delete virtual interface n times.
> +# virt_test_02 [OPTIONS]
> +# OPTIONS - different options separated by comma.
> +virt_test_02()
> +{
> +	start_id=1
> +	virt_count=500
> +
> +	local opts=${1:-""}
> +	local n=0
> +
> +	while ((1)); do

while true; here as well

> +		n=$((n + 1))
> +		p="$(echo $opts | cut -d',' -f$n)"
> +		if [ -z "$p" -a $n -gt 1 ]; then
> +			break
> +		fi
> +
> +		tst_resm TINFO "add and then delete $virt_type with '$p'"
> +
> +		virt_check_cmd virt_add ltp_v0 $p || p=""
> +
> +		virt_add_delete_test "$p"
> +
> +		start_id=$(($start_id + $virt_count))
> +	done
> +}
> +
>  tst_require_root
>  
>  case "$virt_type" in
> diff --git a/testcases/network/virt/vlan02.sh b/testcases/network/virt/vlan02.sh
> index 0fb35a3..60addba 100755
> --- a/testcases/network/virt/vlan02.sh
> +++ b/testcases/network/virt/vlan02.sh
> @@ -16,8 +16,8 @@
>  #
>  # Author: Alexey Kodanev <alexey.kodanev@oracle.com>
>  #
> -# Test-case 1: Local test, check if we can create and then delete VLAN
> -#              interface 4095 times.
> +# Local test, check if we can create and then delete VLAN
> +# interface 4095 times.
>  #
>  
>  TCID=vlan02
> diff --git a/testcases/network/virt/vxlan02.sh b/testcases/network/virt/vxlan02.sh
> index 1470495..0b3a21c 100755
> --- a/testcases/network/virt/vxlan02.sh
> +++ b/testcases/network/virt/vxlan02.sh
> @@ -16,8 +16,8 @@
>  #
>  # Author: Alexey Kodanev <alexey.kodanev@oracle.com>
>  #
> -# Test-case 1: Local test, check if we can create and then delete VXLAN
> -#              interface 5000 times.
> +# Local test, check if we can create and then delete VXLAN
> +# interface 5000 times.
>  #
>  
>  TCID=vxlan02
> -- 
> 1.7.1
> 
> 
> ------------------------------------------------------------------------------
> Don't Limit Your Business. Reach for the Cloud.
> GigeNET's Cloud Solutions provide you with the tools and support that
> you need to offload your IT needs and focus on growing your business.
> Configured For All Businesses. Start Your Cloud Today.
> https://www.gigenetcloud.com/
> _______________________________________________
> Ltp-list mailing list
> Ltp-list@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/ltp-list

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  reply	other threads:[~2015-08-11 18:33 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-09  6:47 [LTP] [PATCH 00/23] Add vlan, ipvlan, gre, macvlan interfaces Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 01/23] network/vxlan: make names more abstract, use 'virt' Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 02/23] network/vxlan: rename vxlan dir and library to 'virt' Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 03/23] network/virt_lib.sh: add virt_add() and virt_add_rhost() Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 04/23] network/virt_lib.sh: use ROD_SILENT in cleanup Alexey Kodanev
2015-08-11 18:00   ` Cyril Hrubis
2015-07-09  6:47 ` [LTP] [PATCH 05/23] network/virt_lib.sh: fix regexpr " Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 06/23] network/virt/vlan01: add new test-case Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 07/23] network/virt/vlan02: " Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 08/23] network/virt: rename virt_max to virt_count Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 09/23] network/virt/vlan03: add new test-case Alexey Kodanev
2015-08-11 18:12   ` Cyril Hrubis
     [not found]     ` <55CB2A43.8060808@oracle.com>
2015-08-12 11:21       ` Cyril Hrubis
2015-07-09  6:47 ` [LTP] [PATCH 10/23] network/virt/vlan01.sh: add protocol, loose_binding and hdr options Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 11/23] network/virt/vlan03.sh: " Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 12/23] network/virt: add new macvlan01 test Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 13/23] network/virt: add new ipvlan01 test Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 14/23] network/virt/vxlan01.sh: add parameters Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 15/23] network/virt/vxlan03.sh: add VXLAN-GBP (group policy context) test Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 16/23] network/virt: add macvtap01 test Alexey Kodanev
2015-08-11 18:21   ` Cyril Hrubis
     [not found]     ` <55CB2DA3.1050706@oracle.com>
2015-08-12 11:41       ` Cyril Hrubis
2015-07-09  6:47 ` [LTP] [PATCH 17/23] network/virt: add test-case 01 to virt_lib.sh Alexey Kodanev
2015-08-11 18:24   ` Cyril Hrubis
2015-07-09  6:47 ` [LTP] [PATCH 18/23] network/virt: move checking for command support " Alexey Kodanev
2015-08-11 18:28   ` Cyril Hrubis
     [not found]     ` <55CB31E5.1050808@oracle.com>
2015-08-12 11:49       ` Cyril Hrubis
2015-07-09  6:47 ` [LTP] [PATCH 19/23] network/virt: add test-case 02 " Alexey Kodanev
2015-08-11 18:32   ` Cyril Hrubis [this message]
2015-07-09  6:47 ` [LTP] [PATCH 20/23] network/virt/vxlan03.sh: fix vxlan_compare_netperf() Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 21/23] network/virt: add gre tunnel test Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 22/23] network/virt: adjust virt_threshold parameter Alexey Kodanev
2015-07-09  6:47 ` [LTP] [PATCH 23/23] network/virt: don't skip when feature options not supported Alexey Kodanev
2015-08-11 18:44 ` [LTP] [PATCH 00/23] Add vlan, ipvlan, gre, macvlan interfaces Cyril Hrubis

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=20150811183236.GF24227@rei.suse.de \
    --to=chrubis@suse.cz \
    --cc=alexey.kodanev@oracle.com \
    --cc=ltp-list@lists.sourceforge.net \
    --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