Remove dependency to autotest_run.sh

Remove the dependency to autotest_run.sh so I can get rid of it in next CL.

Change-Id: I1032932b696929fccf97f911c555ec5a24f589d4

BUG=9291
TEST=Run storageFio test since it need prebuild test and deps.


w/wo emerge autotest 
w/wo cros_workon 
in/outside of chroot. 

and all its combinations.

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=983fdb5

[kmixter1, 2010-11-17 23:59:15]

http://codereview.chromium.org/5188002/diff/1/run_remote_tests.sh
File run_remote_tests.sh (right):

http://codereview.chromium.org/5188002/diff/1/run_remote_tests.sh#newcode274
run_remote_tests.sh:274: export AUTOSERV_TEST_ARGS="${FLAGS_args}"
nit: I'd make these local and lowercase now that they don't need to be exported
through the environment.

http://codereview.chromium.org/5188002/diff/1/run_remote_tests.sh#newcode281
run_remote_tests.sh:281: LOGNAME=${USER} ./server/autoserv ${AUTOSERV_ARGS}
${AUTOSERV_TEST_ARGS}
I don't think you can pass AUTOSERV_TEST_ARGS this way.  You may want to test
this with a wifi network test which uses those (will need to ask sleffer for
temporary access to the cell).  See the -a addition in the else clause.  You may
want to pull that code up above the if [ build ] statement.

[ericli, 2010-11-18 01:09:59]

PTAL.

Comments have been addressed and I also include to remove autotest_run.sh in
this CL.

[kmixter1, 2010-11-18 01:54:38]

LGTM

Make sure to keep an eye on PFQ shortly after you submit since it uses the code
you modified.

http://codereview.chromium.org/5188002/diff/7001/run_remote_tests.sh
File run_remote_tests.sh (right):

http://codereview.chromium.org/5188002/diff/7001/run_remote_tests.sh#newcode276
run_remote_tests.sh:276: autoserv_test_args="-a \"${autoserv_test_args}\""
nit: You may want to just roll this directly into autoserv_args and avoid
introducing autoserv_test_args.

[ericli, 2010-11-18 18:26:29]

Ken,
I will leave it as it is in this CL and will change it in my next CL.

Pushing now.

On 2010/11/18 01:54:38, kmixter1 wrote:
> LGTM
> 
> Make sure to keep an eye on PFQ shortly after you submit since it uses the
code
> you modified.
> 
> http://codereview.chromium.org/5188002/diff/7001/run_remote_tests.sh
> File run_remote_tests.sh (right):
> 
>
http://codereview.chromium.org/5188002/diff/7001/run_remote_tests.sh#newcode276
> run_remote_tests.sh:276: autoserv_test_args="-a \"${autoserv_test_args}\""
> nit: You may want to just roll this directly into autoserv_args and avoid
> introducing autoserv_test_args.