LGTM, thanks

On Wed, Jul 30, 2014 at 2:49 PM, 'Hrvoje Ribicic' via ganeti-devel <
ganeti-devel@googlegroups.com> wrote:

> From: "Jose A. Lopes" <jabolo...@google.com>
>
> ... which means that a domain is in the queue behind other domains
> waiting to be scheduled.  The domain is runnable but it is not
> currently running.
>
> Signed-off-by: Jose A. Lopes <jabolo...@google.com>
> Reviewed-by: Hrvoje Ribicic <r...@google.com>
>
> Cherry-picked from 30350df5fc6b0cda9be12189864ebcca89d8a9ea.
>
> Signed-off-by: Hrvoje Ribicic <r...@google.com>
> ---
>  lib/hypervisor/hv_xen.py | 8 +++++++-
>  1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/lib/hypervisor/hv_xen.py b/lib/hypervisor/hv_xen.py
> index 326fccd..df6a2dc 100644
> --- a/lib/hypervisor/hv_xen.py
> +++ b/lib/hypervisor/hv_xen.py
> @@ -172,10 +172,16 @@ def _GetInstanceList(fn, include_node, _timeout=5):
>  #
>  # An instance is running if it is in the following Xen states:
>  # running, blocked, or paused.
> +#
> +# A state of nothing '------' means that the domain is runnable but it
> +# is not currently running.  That means it is in the queue behind
> +# other domains waiting to be scheduled to run.
> +#
> http://old-list-archives.xenproject.org/xen-users/2007-06/msg00849.html
>  def _IsInstanceRunning(instance_info):
>    return instance_info == "r-----" \
>        or instance_info == "-b----" \
> -      or instance_info == "--p---"
> +      or instance_info == "--p---" \
> +      or instance_info == "------"
>
>
>  def _IsInstanceShutdown(instance_info):
> --
> 2.0.0.526.g5318336
>
>


-- 
Helga Velroyen | Software Engineer | hel...@google.com |

Google Germany GmbH
Dienerstr. 12
80331 München

Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschäftsführer: Graham Law, Christine Elizabeth Flores

Reply via email to