----- "sudhir kumar" <smalikphy@xxxxxxxxx> wrote:
> On Fri, Jul 24, 2009 at 5:54 PM, Michael Goldish<mgoldish@xxxxxxxxxx>
> wrote:
> >
> > ----- "sudhir kumar" <smalikphy@xxxxxxxxx> wrote:
> >
> >> This patch does two small things.
> >> 1. Prints the guest login command to debug messages.
> >
> > Why do we want to do that?
> I do not see any harm in that. We are logging "trying to login". If
> sometimes login fail we can check by manually typing the same command
> and see what went wrong. That print statement has helped me in past
> quite a number of times.
OK, no problem. What do you think about printing the login command and
the "trying to login" message on the same line, like:
Trying to login: ssh root@localhost 5000
or
Trying to login with command: ssh root@localhost 5000
or
Trying to login (ssh root@localhost 5000)
or something like that.
If you don't like any of these options, the patch is OK as it is.
I just thought it would be a good idea to keep the output short,
because "Trying to login" is displayed repeatedly during boot so
it can produce a lot of (not so interesting) output.
> >> 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.
>
> Yes please,
> the test failed very near to the sshd daemon was about to run. So
> that
> shows 120 seconds is not sufficient. The host was not at all loaded
> and is a pretty high end machine.
> Thanks.
OK, I agree. I guess I run fast guests most of the time, and some guests
can take much longer to boot.
> >> Signed-off-by: Sudhir Kumar <skumar@xxxxxxxxxxxxxxxxxx>
> >>
> >> 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
> >
>
>
>
> --
> Sudhir Kumar
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
