Re: [AUTOTEST] print login command and change some timeout values

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



----- "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

[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux