Re: [PATCH v2 0/6] Move secret object command line building helpers

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

 




On 05/31/2016 06:39 PM, John Ferlan wrote:
> v1: http://www.redhat.com/archives/libvir-list/2016-May/msg02017.html
> 
> Changes in v2:
> 
> Patch 1 - Move function to a new src/util/virqemu.c with adjustments
> to other affected areas
> 
> Patches 2-4 - Similarly adjust the flow/name of the patches with an
> adjustment so that patch 2 is just the new code in virqemu.c and patch 3
> shows how the master secret code would use the same function to build
> it's command line.  Patch 4 just follows suit...


Based on the ongoing discussion in v1:

http://www.redhat.com/archives/libvir-list/2016-June/msg00014.html


I'll drop patches 2-4 (although I may add one to use
virJSONValueObjectCreate rather than the current open code in
qemuBuildMasterKeyCommandLine).

John

> 
> Patch 5 was reviewed/ACK'd - this patch just shows the changes to remove
> the "virSecretPtr secret = NULL;" and "virObjectUnref(secret);"
> 
> Patch 6 is unchanged
> 
> John Ferlan (6):
>   qemu: Move and rename qemuBuildObjectCommandlineFromJSON
>   util: Introduce virQEMUBuildSecretObjectProps
>   qemu: Use virQEMUBuildSecretObjectProps to build secret objects
>   qemu: Rework secinfo command line building
>   storage: Use virSecretGetSecretString
>   secret: Move virStorageSecretType to secret_util and rename
> 
>  cfg.mk                                             |   2 +-
>  po/POTFILES.in                                     |   1 +
>  src/Makefile.am                                    |   2 +
>  src/libvirt_private.syms                           |   5 +
>  src/libxl/libxl_conf.c                             |   2 +-
>  src/qemu/qemu_command.c                            | 219 +++++----------------
>  src/qemu/qemu_command.h                            |   4 -
>  src/qemu/qemu_domain.c                             |   4 +-
>  src/secret/secret_util.c                           |  18 +-
>  src/secret/secret_util.h                           |  22 ++-
>  src/storage/storage_backend_iscsi.c                |  55 +-----
>  src/storage/storage_backend_rbd.c                  |  49 +----
>  src/util/virqemu.c                                 | 211 ++++++++++++++++++++
>  src/util/virqemu.h                                 |  42 ++++
>  src/util/virstoragefile.c                          |  33 ++--
>  src/util/virstoragefile.h                          |  17 +-
>  tests/qemuargv2xmltest.c                           |   4 +-
>  tests/qemucommandutiltest.c                        |   9 +-
>  ...muxml2argv-disk-drive-network-rbd-auth-AES.args |   3 +-
>  .../qemuxml2argvdata/qemuxml2argv-master-key.args  |   3 +-
>  .../qemuxml2argvdata/qemuxml2argv-name-escape.args |   3 +-
>  21 files changed, 380 insertions(+), 328 deletions(-)
>  create mode 100644 src/util/virqemu.c
>  create mode 100644 src/util/virqemu.h
> 

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]