[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