----- "sudhir kumar" <smalik...@gmail.com> wrote:
> This patch does two small things.
> 1. Prints the guest login command to debug messages.
Why do we want to do that?
> 2. Changes the guest login timeout to 240 seconds. I see the timeout
> for
> *.wait_for() functions in boot test is 240 seconds, while in reboot is
> 120
> seconds which causes the test to fail. We might have missed it by
> mistake.
> 240 seconds is a reasonable timeout duration. This patch fixes that.
Using the same timeout value everywhere makes sense, but it surprises me
that tests are failing because 120 isn't enough. It sounds like the host
has to be heavily loaded for the boot to take longer than 2 minutes. But
if it happened to you then let's increase the timeout.
> Signed-off-by: Sudhir Kumar <sku...@linux.vnet.ibm.com>
>
> Index: autotest/client/tests/kvm/kvm_utils.py
> ===================================================================
> --- autotest.orig/client/tests/kvm/kvm_utils.py
> +++ autotest/client/tests/kvm/kvm_utils.py
> @@ -637,6 +637,7 @@ def remote_login(command, password, prom
> password_prompt_count = 0
>
> logging.debug("Trying to login...")
> + logging.debug("Guest login Command: %s" % command)
>
> while True:
> (match, text) = sub.read_until_last_line_matches(
> Index: autotest/client/tests/kvm/kvm_tests.py
> ===================================================================
> --- autotest.orig/client/tests/kvm/kvm_tests.py
> +++ autotest/client/tests/kvm/kvm_tests.py
> @@ -48,7 +48,7 @@ def run_boot(test, params, env):
>
> logging.info("Guest is down; waiting for it to go up
> again...")
>
> - session = kvm_utils.wait_for(vm.ssh_login, 120, 0, 2)
> + session = kvm_utils.wait_for(vm.ssh_login, 240, 0, 2)
> if not session:
> raise error.TestFail("Could not log into guest after
> reboot")
>
> @@ -88,7 +88,7 @@ def run_shutdown(test, params, env):
>
> logging.info("Shutdown command sent; waiting for guest to go
> down...")
>
> - if not kvm_utils.wait_for(vm.is_dead, 120, 0, 1):
> + if not kvm_utils.wait_for(vm.is_dead, 240, 0, 1):
> raise error.TestFail("Guest refuses to go down")
>
> logging.info("Guest is down")
> @@ -445,7 +445,7 @@ def run_yum_update(test, params, env):
>
> logging.info("Logging into guest...")
>
> - session = kvm_utils.wait_for(vm.ssh_login, 120, 0, 2)
> + session = kvm_utils.wait_for(vm.ssh_login, 240, 0, 2)
> if not session:
> message = "Could not log into guest"
> logging.error(message)
>
>
>
> --
> Sudhir Kumar
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
