[libvirt] [PATCH v6 12/13] tests: Add test for failure when vxhs_tls=0
Peter Krempa
pkrempa at redhat.com
Thu Aug 31 14:17:35 UTC 2017
On Wed, Aug 30, 2017 at 18:46:12 -0400, John Ferlan wrote:
> From: Ashish Mittal <Ashish.Mittal at veritas.com>
>
> Add a test case to verify setting vxhs_tls=0 in qemu.conf disables
> TLS for VxHS disks followed by running the same test after enabling
> for the success case.
>
> Signed-off-by: Ashish Mittal <Ashish.Mittal at veritas.com>
> Signed-off-by: John Ferlan <jferlan at redhat.com>
> ---
>
> This is essentially the same v5 patch8 - had to clean up the input
> XML a bit though.
>
> I'm not a fan of this change - I think we should just follow what
> chardev does and if someone has vxhs_tls = 0 and haveTLS = true,
> then we just clear the haveTLS and move on. I think one way or
> another things will fail in any case.
Me neither. NACK
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170831/bdc09deb/attachment-0001.sig>
More information about the libvir-list
mailing list