[Qemu-devel] [PATCH v3 0/2] migration: fixes to handling tls-hostname/tls-creds

Daniel P. Berrange posted 2 patches 7 years, 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170302161921.10181-1-berrange@redhat.com
Test checkpatch passed
Test docker passed
Test s390x passed
migration/migration.c | 22 ++++++++++++++++------
qapi-schema.json      |  8 +++++++-
2 files changed, 23 insertions(+), 7 deletions(-)
[Qemu-devel] [PATCH v3 0/2] migration: fixes to handling tls-hostname/tls-creds
Posted by Daniel P. Berrange 7 years, 1 month ago
The need for these two patches was identified during implementation  of
TLS encrypted migration in libvirt.

Changed in v3:

 - Add to qapi-schema.json docs

Daniel P. Berrange (2):
  migration: allow clearing migration string parameters
  migration: always report tls-creds & tls-hostname migrate parameters

 migration/migration.c | 22 ++++++++++++++++------
 qapi-schema.json      |  8 +++++++-
 2 files changed, 23 insertions(+), 7 deletions(-)

-- 
2.9.3


Re: [Qemu-devel] [PATCH for-2.9 v3 0/2] migration: fixes to handling tls-hostname/tls-creds
Posted by Eric Blake 7 years, 1 month ago
On 03/02/2017 10:19 AM, Daniel P. Berrange wrote:
> The need for these two patches was identified during implementation  of
> TLS encrypted migration in libvirt.
> 
> Changed in v3:
> 
>  - Add to qapi-schema.json docs
> 
> Daniel P. Berrange (2):
>   migration: allow clearing migration string parameters
>   migration: always report tls-creds & tls-hostname migrate parameters
> 
>  migration/migration.c | 22 ++++++++++++++++------
>  qapi-schema.json      |  8 +++++++-
>  2 files changed, 23 insertions(+), 7 deletions(-)

The discoverability of whether "" works is not visible in introspection,
but reporting "" on output helps.  For the series:

Reviewed-by: Eric Blake <eblake@redhat.com>

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org