diff options
author | Max Reitz <mreitz@redhat.com> | 2019-02-01 20:29:27 +0100 |
---|---|---|
committer | Max Reitz <mreitz@redhat.com> | 2019-02-25 15:11:27 +0100 |
commit | 97e2f021f844383d85de526ce88667ca34ecd277 (patch) | |
tree | b7164342ae584dc79a49099b55a3a0827e73b578 /tests/qemu-iotests/228 | |
parent | abc521a9aa470421dc9285cafe16ff64f3044ac5 (diff) |
block: Generically refresh runtime options
Instead of having every block driver which implements
bdrv_refresh_filename() copy all of the strong runtime options over to
bs->full_open_options, implement this process generically in
bdrv_refresh_filename().
This patch only adds this new generic implementation, it does not remove
the old functionality. This is done in a follow-up patch.
With this patch, some superfluous information (that should never have
been there) may be removed from some JSON filenames, as can be seen in
the change to iotests 110's and 228's reference outputs.
Signed-off-by: Max Reitz <mreitz@redhat.com>
Message-id: 20190201192935.18394-24-mreitz@redhat.com
Signed-off-by: Max Reitz <mreitz@redhat.com>
Diffstat (limited to 'tests/qemu-iotests/228')
-rwxr-xr-x | tests/qemu-iotests/228 | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/tests/qemu-iotests/228 b/tests/qemu-iotests/228 index 2930f8442c..7a57c41e29 100755 --- a/tests/qemu-iotests/228 +++ b/tests/qemu-iotests/228 @@ -230,9 +230,10 @@ with iotests.FilePath('base.img') as base_img_path, \ overlay='node0') # This should give us the original plain result - # FIXME: Currently, the block layer considers the runtime backing - # file to be different from the image header, which is - # wrong. This is fixed by a future patch. + # FIXME: Currently, it yields a json:{} filename even though it + # only contains a @driver and a @file entry, so a plain + # filename would obviously suffice. This is fixed by a + # future patch. log_node_info(vm.node_info('node0')) |