[libvirt] [glib PATCH V6] Add bindings for virDomainSnapshotCreateXML()

[Zeeshan Ali (Khattak)]

On Fri, Jul 13, 2012 at 9:45 AM, Jovanka Gulicoska
<jovanka.gulicoska at gmail.com> wrote:
> ---
>  libvirt-gobject/libvirt-gobject-domain.c |   42 ++++++++++++++++++++++++++++++
>  libvirt-gobject/libvirt-gobject-domain.h |    6 +++++
>  libvirt-gobject/libvirt-gobject.sym      |    1 +
>  3 files changed, 49 insertions(+)
>
> diff --git a/libvirt-gobject/libvirt-gobject-domain.c b/libvirt-gobject/libvirt-gobject-domain.c
> index 088cd33..ce81971 100644
> --- a/libvirt-gobject/libvirt-gobject-domain.c
> +++ b/libvirt-gobject/libvirt-gobject-domain.c
> @@ -1123,3 +1123,45 @@ GList *gvir_domain_get_devices(GVirDomain *domain,
>
>      return g_list_reverse (ret);
>  }
> +
> +/**
> + * gvir_domain_snapshot_create_xml:
> + * @dom: the domain
> + * @custom_conf: (allow-none): configuration of snapshot or NULL
> + * @flags: the flags
> + * @err: (allow-none):Place-holder for error or NULL
> + *
> + * Returns: (transfer full): snapshot of domain. The returned object should be
> + * unreffed when no longer needed
> + */
> +GVirConfigDomainSnapshot *gvir_domain_snapshot_create_xml
> +                                      (GVirDomain *dom,
> +                                       GVirConfigDomainSnapshot *custom_conf,
> +                                       guint flags,
> +                                       GError **err)
> +{
> +    GVirDomainPrivate *priv;
> +    virDomainSnapshot *snapshot;
> +    GVirConfigDomainSnapshot *conf_snapshot;
> +    gchar *custom_xml = NULL;
> +
> +    g_return_val_if_fail(GVIR_IS_DOMAIN(dom), FALSE);
> +    g_return_val_if_fail(err == NULL || *err == NULL, NULL);
> +
> +    priv = dom->priv;
> +
> +    if (custom_conf != NULL)
> +        custom_xml = gvir_config_object_to_xml(GVIR_CONFIG_OBJECT(custom_conf));
> +
> +    if (!(snapshot = virDomainSnapshotCreateXML(priv->handle, custom_xml, flags))) {
> +        gvir_set_error_literal(err, GVIR_DOMAIN_ERROR,
> +                               0,
> +                              "Unable to create snapshot of domain");
> +        return NULL;
> +    }
> +
> +    conf_snapshot = gvir_config_domain_snapshot_new_from_xml(custom_xml, err);

You seem to be completely ignoring the snapshot returned by
virDomainSnapshotCreateXML. The returned virDomainSnapshotPtr maps to
GVirDomainSnapshot, not GVirConfigDomainSnapshot. You need to:

1. Get the config xml used by virDomainSnapshotPtr using
virDomainSnapshotGetXMLDesc.
2. Create GVirConfigDomainSnapshot out of that xml string.
3. Create the GVirDomainSnapshot with this GVirConfigDomainSnapshot
using g_object_new().
4 Return this GVirDomainSnapshot.

> +
> +    g_free(custom_xml);
> +    return conf_snapshot;
> +}
> diff --git a/libvirt-gobject/libvirt-gobject-domain.h b/libvirt-gobject/libvirt-gobject-domain.h
> index 87b94f4..8c4d28e 100644
> --- a/libvirt-gobject/libvirt-gobject-domain.h
> +++ b/libvirt-gobject/libvirt-gobject-domain.h
> @@ -202,6 +202,12 @@ gboolean gvir_domain_get_saved(GVirDomain *dom);
>  GList *gvir_domain_get_devices(GVirDomain *domain,
>                                 GError **err);
>
> +GVirConfigDomainSnapshot *gvir_domain_snapshot_create_xml
> +                                      (GVirDomain *dom,
> +                                       GVirConfigDomainSnapshot *custom_conf,
> +                                       guint flags,
> +                                       GError **err);
> +
>  G_END_DECLS
>
>  #endif /* __LIBVIRT_GOBJECT_DOMAIN_H__ */
> diff --git a/libvirt-gobject/libvirt-gobject.sym b/libvirt-gobject/libvirt-gobject.sym
> index 94e441a..045e1b5 100644
> --- a/libvirt-gobject/libvirt-gobject.sym
> +++ b/libvirt-gobject/libvirt-gobject.sym
> @@ -75,6 +75,7 @@ LIBVIRT_GOBJECT_0.0.8 {
>         gvir_domain_get_persistent;
>         gvir_domain_get_saved;
>         gvir_domain_screenshot;
> +       gvir_domain_snapshot_create_xml;
>
>         gvir_domain_snapshot_get_type;
>         gvir_domain_snapshot_handle_get_type;
> --
> 1.7.10.4
>
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list



-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124