On 05/22/2018 10:04 AM, Peter Krempa wrote: > This applies on top of the text monitor cleanup. See explanation in 3/4 > for justification. > > Peter Krempa (4): > tests: qemublock: Switch to qcow2+luks in test files > tests: qemu: Modernize/remove qcow2 encryption from tests not related > to storage > qemu: domain: Forbid storage with old QCOW2 encryption > qemu: Remove code for setting up disk passphrases > This would be nice, but based on this series: https://www.redhat.com/archives/libvir-list/2018-May/msg01268.html I believe there are quite a few more tests/files to modify/delete in order to remove qcow[2] from the source tree. There's also the formatstorageencryption and formatsecret documentation that would need updating. Based only on the effort from the above series to convert/consume a non encrypted image to result in a qcow[2] encrypted image - I assume conversion of qcow[2] images is not a simple exercise. Not sure whether anyone really uses qcow[2] encryption anymore in the wild, but just telling them they have to convert (without providing a shred of details as to what that entails isn't very friendly. Also not sure it's possible to just convert to using LUKS since at one time at least usage required having code/tests inside a "# ifdef WITH_GNUTLS" (something that can be seen in the diffs from tests/qemuxml2argvtest.c in patch 3). John > src/qemu/qemu_domain.c | 20 ++-- > src/qemu/qemu_monitor.c | 13 --- > src/qemu/qemu_monitor.h | 4 - > src/qemu/qemu_monitor_json.c | 28 ------ > src/qemu/qemu_monitor_json.h | 4 - > src/qemu/qemu_process.c | 103 --------------------- > .../file-qcow2-backing-chain-encryption.json | 2 +- > .../file-qcow2-backing-chain-encryption.xml | 2 +- > ...etwork-qcow2-backing-chain-encryption_auth.json | 2 +- > ...network-qcow2-backing-chain-encryption_auth.xml | 2 +- > tests/qemumonitorjsontest.c | 2 - > tests/qemuxml2argvdata/encrypted-disk-usage.args | 8 +- > tests/qemuxml2argvdata/encrypted-disk-usage.xml | 2 +- > tests/qemuxml2argvdata/encrypted-disk.args | 8 +- > tests/qemuxml2argvdata/encrypted-disk.xml | 2 +- > tests/qemuxml2argvdata/interface-server.xml | 3 - > tests/qemuxml2argvdata/user-aliases.args | 8 +- > tests/qemuxml2argvdata/user-aliases.xml | 2 +- > tests/qemuxml2argvtest.c | 7 +- > tests/qemuxml2xmloutdata/encrypted-disk.xml | 2 +- > tests/qemuxml2xmloutdata/interface-server.xml | 3 - > tests/qemuxml2xmltest.c | 6 +- > 22 files changed, 50 insertions(+), 183 deletions(-) > -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
