Move piglit to run last and reboot the machine prior.

Move piglit to run last and reboot the machine prior.

Change-Id: I4d1be86b96ebe5a7cd62b5ee50eff3a47e1eac06

BUG=none
TEST=none

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=018d378

[ericli, 2011-04-22 22:18:50]

http://codereview.chromium.org/6900029/diff/1/server/site_tests/suites/contro...
File server/site_tests/suites/control.regression (right):

http://codereview.chromium.org/6900029/diff/1/server/site_tests/suites/contro...
server/site_tests/suites/control.regression:146: # Run security suite.
so I think security test should be moved above , before line 130.

[DaleCurtis, 2011-04-22 22:25:06]

http://codereview.chromium.org/6900029/diff/1/server/site_tests/suites/contro...
File server/site_tests/suites/control.regression (right):

http://codereview.chromium.org/6900029/diff/1/server/site_tests/suites/contro...
server/site_tests/suites/control.regression:146: # Run security suite.
Can you chain a job.run_control inside a job.parallel_on_machines ?

On 2011/04/22 22:18:50, ericli wrote:
> so I think security test should be moved above , before line 130.

[ilja, 2011-04-22 22:45:13]

Thanks for changing this! I am not qualified to review though.

On 2011/04/22 22:25:06, dalec wrote:
>
http://codereview.chromium.org/6900029/diff/1/server/site_tests/suites/contro...
> File server/site_tests/suites/control.regression (right):
> 
>
http://codereview.chromium.org/6900029/diff/1/server/site_tests/suites/contro...
> server/site_tests/suites/control.regression:146: # Run security suite.
> Can you chain a job.run_control inside a job.parallel_on_machines ?
> 
> On 2011/04/22 22:18:50, ericli wrote:
> > so I think security test should be moved above , before line 130.

[ericli, 2011-04-23 01:13:19]

http://codereview.chromium.org/6900029/diff/1/server/site_tests/suites/contro...
File server/site_tests/suites/control.regression (right):

http://codereview.chromium.org/6900029/diff/1/server/site_tests/suites/contro...
server/site_tests/suites/control.regression:146: # Run security suite.
You r right.

I take it back,
run_control(security) actually happens even before run_assigned_tests().

LGTM

On 2011/04/22 22:25:06, dalec wrote:
> Can you chain a job.run_control inside a job.parallel_on_machines ?
> 
> On 2011/04/22 22:18:50, ericli wrote:
> > so I think security test should be moved above , before line 130.
>