[LTP] [PATCH v2 1/3] network: merge networktests.sh/networkstress.sh into network.sh
Alexey Kodanev
alexey.kodanev@oracle.com
Tue May 17 15:09:54 CEST 2016
Hi,
On 05/17/2016 03:29 PM, Cyril Hrubis wrote:
> Hi!
>> @@ -4,68 +4,6 @@ TST_TOTAL=1
>> TCID="network_settings"
>> . test_net.sh
> We are working around the checks in test.sh here by defining TST_TOTAL
> and TCID. Maybe it would be better to add the settings to separate
> script sourced by the test_net.sh and here as well so that we don't have
> to...
But if we don't source test.sh we will break some dependencies between
test_net.sh and test.sh,
e.g. variables like this one:
export RHOST_HWADDRS=${RHOST_HWADDRS:-"$(tst_get_hwaddrs rhost)"}
It might call tst_brkm/tst_resm...
BTW, if we already using tst_* functions from test.sh, may be we could
replace network.sh's 'echo' commands there too?
Thanks,
Alexey
>
> Otherwise this version looks fine.
>
More information about the ltp
mailing list