OpenE2K/qemu-e2k
13
4
Fork 0
Code Issues 4 Pull Requests Projects 2 Releases Activity

Revert "qapi: fix examples of blockdev-add with qcow2"

Browse Source 
This reverts commit b652293832.

Kevin Wolf NAKed this patch, because:

    'file' is a required member (defined in BlockdevOptionsGenericFormat),
    removing it makes the example invalid. 'data-file' is only an additional
    optional member to be used for external data files (i.e. when the guest
    data is kept separate from the metadata in the .qcow2 file).

However, it had already been merged then.  Revert.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20220930171908.846769-1-armbru@redhat.com>
Reviewed-by: Victor Toso <victortoso@redhat.com>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
This commit is contained in:
Markus Armbruster 2022-09-30 19:19:08 +02:00 committed by Kevin Wolf
parent f1d33f55c4
commit 34f58df51d
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
qapi/block-core.json
View File

@ -1541,8 +1541,8 @@
# -> { "execute": "blockdev-add",
# "arguments": { "driver": "qcow2",
# "node-name": "node1534",
# "data-file": { "driver": "file",
# "filename": "hd1.qcow2" },
# "file": { "driver": "file",
# "filename": "hd1.qcow2" },
# "backing": null } }
#
# <- { "return": {} }
@ -4378,7 +4378,7 @@
# "arguments": {
# "driver": "qcow2",
# "node-name": "test1",
# "data-file": {
# "file": {
# "driver": "file",
# "filename": "test.qcow2"
# }
@ -4395,7 +4395,7 @@
# "cache": {
# "direct": true
# },
# "data-file": {
# "file": {
# "driver": "file",
# "filename": "/tmp/test.qcow2"
# },
@ -4477,7 +4477,7 @@
# "arguments": {
# "driver": "qcow2",
# "node-name": "node0",
# "data-file": {
# "file": {
# "driver": "file",
# "filename": "test.qcow2"
# }