[libvirt] [PATCH v3 2/5] storage: Use virSecretGetSecretString
Peter Krempa
pkrempa at redhat.com
Mon Jun 6 07:32:32 UTC 2016
On Fri, Jun 03, 2016 at 06:52:50 -0400, John Ferlan wrote:
> Rather than inline code secret lookup for rbd/iscsi, use the common function.
>
> Signed-off-by: John Ferlan <jferlan at redhat.com>
> ---
> src/Makefile.am | 1 +
> src/storage/storage_backend_iscsi.c | 50 +++++--------------------------------
> src/storage/storage_backend_rbd.c | 48 +++--------------------------------
> 3 files changed, 10 insertions(+), 89 deletions(-)
>
> diff --git a/src/Makefile.am b/src/Makefile.am
> index f3c9a14..019242b 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -1615,6 +1615,7 @@ libvirt_driver_storage_impl_la_SOURCES =
> libvirt_driver_storage_impl_la_CFLAGS = \
> -I$(srcdir)/access \
> -I$(srcdir)/conf \
> + -I$(srcdir)/secret \
Similarly to my complaint in 5/5 this is breaking the boundaries between
the storage driver and the secret driver. This is not the first case of
this though so we just need to thoroughly check that the appropriate
bits are linked in to the storage driver.
I'll need to check this.
Otherwise looks good.
Peter
More information about the libvir-list
mailing list