[libvirt] [libvirt-perl PATCH] Add support for VIR_SECRET_USAGE_TYPE_PASSPHRASE

Martin Kletzander mkletzan at redhat.com
Mon Jul 4 15:39:03 UTC 2016


On Mon, Jul 04, 2016 at 04:22:32PM +0100, Daniel P. Berrange wrote:
>On Mon, Jul 04, 2016 at 04:49:56PM +0200, Martin Kletzander wrote:
>> On Mon, Jul 04, 2016 at 03:41:16PM +0100, Daniel P. Berrange wrote:
>> > On Mon, Jul 04, 2016 at 04:37:27PM +0200, Martin Kletzander wrote:
>> > > Signed-off-by: Martin Kletzander <mkletzan at redhat.com>
>> > > ---
>> > > This fixes a build for me with upstream libvirt.
>> >
>> > NACK, per my reply to John's original patch, I think this should
>> > not have been added to libvirt API in the first place.
>> >
>>
>> I missed that.  Looking at it now I was wondering why we have special
>> purpose, dedicated secrets to that they are used safely and now we have
>> general-purpose secret.  OK, good to know, I haven't thought of that, I
>> just wanted to fix the test.  Thanks for the info.
>
>Oh, I guess John is not around today, due to 4th July, so we'll have to
>wait for him to return to resolve this.
>
>ACK to pushing your patch now to fix the build, and we'll just revert
>it later if we revert the main libvirt.so patch.
>

OK, pushed now, feel free to revert it when appropriate.  Thanks.

>Regards,
>Daniel
>--
>|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
>|: http://libvirt.org              -o-             http://virt-manager.org :|
>|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
>|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|
>
>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160704/5999cacb/attachment-0001.sig>


More information about the libvir-list mailing list