On Mon, Jun 20, 2011 at 8:35 PM, Dale Curtis <[email protected]> wrote:
> The four hour timeout for crash collection is way too long for Chrome
> OS, so we moved this parameter into global config for easier control.

Thanks, applied:

http://autotest.kernel.org/changeset/5433

> Signed-off-by: Dale Curtis <[email protected]>
> ---
>  server/crashcollect.py |    8 +++++++-
>  1 files changed, 7 insertions(+), 1 deletions(-)
>
> diff --git a/server/crashcollect.py b/server/crashcollect.py
> index 6e1de7f..dc582ac 100644
> --- a/server/crashcollect.py
> +++ b/server/crashcollect.py
> @@ -1,5 +1,6 @@
>  import os, time, pickle, logging, shutil
>
> +from autotest_lib.client.common_lib import global_config
>  from autotest_lib.server import utils
>
>
> @@ -33,7 +34,12 @@ def get_crashinfo(host, test_start_time):
>         collect_uncollected_logs(host)
>
>
> -def wait_for_machine_to_recover(host, hours_to_wait=4.0):
> +# Load default for number of hours to wait before giving up on crash
> collection.
> +HOURS_TO_WAIT = global_config.global_config.get_config_value(
> +    'SERVER', 'crash_collection_hours_to_wait', type=float, default=4.0)
> +
> +
> +def wait_for_machine_to_recover(host, hours_to_wait=HOURS_TO_WAIT):
>     """Wait for a machine (possibly down) to become accessible again.
>
>     @param host: A RemoteHost instance to wait on
> --
> 1.7.3.1
>
> _______________________________________________
> Autotest mailing list
> [email protected]
> http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
>
>



-- 
Lucas
_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to