@@ -23,10 +23,6 @@ VIRT_PERF_THRESHOLD_MIN=160
do_test()
{
- if [ -z $ip_local -o -z $ip_remote ]; then
- tst_brk TBROK "you must specify IP address"
- fi
-
tst_res TINFO "the same VNI must work"
# VNI is 24 bits long, so max value, which is not reserved, is 0xFFFFFE
vxlan_setup_subnet_$vxlan_dst_addr "id 0xFFFFFE" "id 0xFFFFFE"
@@ -61,11 +61,9 @@ virt_lib_setup()
TST_NEEDS_ROOT=1
. tst_net.sh
-ip_local=$(tst_ipaddr)
ip_virt_local="$(TST_IPV6= tst_ipaddr_un)"
ip6_virt_local="$(TST_IPV6=6 tst_ipaddr_un)"
-ip_remote=$(tst_ipaddr rhost)
ip_virt_remote="$(TST_IPV6= tst_ipaddr_un rhost)"
ip6_virt_remote="$(TST_IPV6=6 tst_ipaddr_un rhost)"
@@ -376,10 +374,6 @@ virt_gre_setup()
[ "$TST_IPV6" ] && virt_type="ip6gre"
virt_lib_setup
- if [ -z $ip_local -o -z $ip_remote ]; then
- tst_brk TBROK "you must specify IP address"
- fi
-
tst_res TINFO "test $virt_type"
virt_setup "local $(tst_ipaddr) remote $(tst_ipaddr rhost) dev $(tst_iface)" \
"local $(tst_ipaddr rhost) remote $(tst_ipaddr) dev $(tst_iface rhost)"
@@ -24,18 +24,10 @@ TST_NEEDS_TMPDIR=1
TST_TEST_DATA=",$p0 $lb0 $rh1,$p1 $lb1 $rh1"
TST_TEST_DATA_IFS=","
TST_TESTFUNC=do_test
-TST_SETUP=do_setup
+TST_SETUP=virt_lib_setup
TST_CLEANUP=virt_cleanup
. virt_lib.sh
-do_setup()
-{
- if [ -z $ip_local -o -z $ip_remote ]; then
- tst_brk TBROK "you must specify IP address"
- fi
- virt_lib_setup
-}
-
do_test()
{
virt_check_cmd virt_add ltp_v0 id 0 $2 || return
@@ -36,10 +36,6 @@ VIRT_PERF_THRESHOLD_MIN=160
do_test()
{
- if [ -z $ip_local -o -z $ip_remote ]; then
- tst_brk TBROK "you must specify IP address"
- fi
-
virt_check_cmd virt_add ltp_v0 id 0 $2 || return
tst_res TINFO "the same VNI must work"
@@ -20,10 +20,6 @@ VIRT_PERF_THRESHOLD_MIN=160
do_test()
{
- if [ -z $ip_local -o -z $ip_remote ]; then
- tst_brk TBROK "you must specify IP address"
- fi
-
virt_check_cmd virt_add ltp_v0 id 0 $2 || return
tst_res TINFO "the same VNI must work"
+ now unused $ip_local and $ip_remote variables. These checks were originally (before f275363d7) in tests which use virt_compare_netperf also with variables. It should be safe to expect tst_ipaddr() output to be safe even for SSH based setup ("two host" configuration) due various checks in tst_net.sh. Signed-off-by: Petr Vorel <pvorel@suse.cz> --- testcases/network/virt/geneve01.sh | 4 ---- testcases/network/virt/virt_lib.sh | 6 ------ testcases/network/virt/vlan03.sh | 10 +--------- testcases/network/virt/vxlan03.sh | 4 ---- testcases/network/virt/vxlan04.sh | 4 ---- 5 files changed, 1 insertion(+), 27 deletions(-)