[libvirt] [PATCH] virsh: tell if domain is updated in dominfo

Osier Yang jyang at redhat.com
Thu Nov 18 07:45:44 UTC 2010


This patch is based on following 4 patches:

https://www.redhat.com/archives/libvir-list/2010-November/msg00661.html
https://www.redhat.com/archives/libvir-list/2010-November/msg00662.html
https://www.redhat.com/archives/libvir-list/2010-November/msg00663.html
https://www.redhat.com/archives/libvir-list/2010-November/msg00664.html

These 4 patches are still not ACKed, could anyone help review it? Thanks

- Osier

于 2010年11月18日 15:26, Osier Yang 写道:
> * tools/virsh.c (introduce new information 'Updated' for 'dominfo',
> standsfor if a domain is updated via operations like 'update-device',
> 'update-disk', etc.)
> ---
>   tools/virsh.c |    7 +++++++
>   1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/tools/virsh.c b/tools/virsh.c
> index ae88cc0..d260443 100644
> --- a/tools/virsh.c
> +++ b/tools/virsh.c
> @@ -1992,6 +1992,7 @@ cmdDominfo(vshControl *ctl, const vshCmd *cmd)
>       virSecurityModel secmodel;
>       virSecurityLabel seclabel;
>       int persistent = 0;
> +    int updated = -1;
>       int ret = TRUE, autostart;
>       unsigned int id;
>       char *str, uuid[VIR_UUID_STRING_BUFLEN];
> @@ -2053,6 +2054,12 @@ cmdDominfo(vshControl *ctl, const vshCmd *cmd)
>       else
>           vshPrint(ctl, "%-15s %s\n", _("Persistent:"), persistent ? _("yes") : _("no"));
>
> +    updated = virDomainIsUpdated(dom);
> +    if (updated<  0)
> +        vshPrint(ctl, "%-15s %s\n", _("Updated:"), _("unknown"));
> +    else
> +        vshPrint(ctl, "%-15s %s\n", _("Updated:"), updated ? _("yes") : _("no"));
> +
>       /* Check and display whether the domain autostarts or not */
>       if (!virDomainGetAutostart(dom,&autostart)) {
>           vshPrint(ctl, "%-15s %s\n", _("Autostart:"),
> --
> 1.7.3.2
>
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list




More information about the libvir-list mailing list