[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v7 2/3] block: add support for encryption secret
Daniel P. Berrange
Re: [Qemu-block] [PATCH v7 2/3] block: add support for encryption secrets in block I/O tests
Wed, 30 Mar 2016 16:53:46 +0100
On Wed, Mar 30, 2016 at 05:51:16PM +0200, Max Reitz wrote:
> On 30.03.2016 15:53, Daniel P. Berrange wrote:
> > The LUKS block driver tests will require the ability to specify
> > encryption secrets with block devices. This requires using the
> > --object argument to qemu-img/qemu-io to create a 'secret'
> > object.
> > When the IMGKEYSECRET env variable is set, it provides the
> > password to be associated with a secret called 'keysec0'
> > Signed-off-by: Daniel P. Berrange <address@hidden>
> > ---
> > tests/qemu-iotests/common | 1 +
> > tests/qemu-iotests/common.config | 6 ++++++
> > tests/qemu-iotests/common.filter | 3 ++-
> > tests/qemu-iotests/common.rc | 16 +++++++++++++---
> > 4 files changed, 22 insertions(+), 4 deletions(-)
> Is there a reason why you didn't implement the same logic in
> _qemu_img_wrapper as in _qemu_io_wrapper?
Mostly because the --image-opts syntax isn't used with
the 'create' command for qemu-img. I guess I could have
trie to detect which command was being run and set the
> This works, but it appears a bit overcomplicated to me.
> Reviewed-by: Max Reitz <address@hidden>
|: 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 :|