Since qemu 2.9 the options changed from a monolithic string into fine
grained options for the json pseudo-protocol object.
---
src/util/virstoragefile.c | 28 ++++++++++++++++++++++++----
tests/virstoragetest.c | 11 +++++++++++
2 files changed, 35 insertions(+), 4 deletions(-)
diff --git a/src/util/virstoragefile.c b/src/util/virstoragefile.c
index dc06bec94..629908b7a 100644
--- a/src/util/virstoragefile.c
+++ b/src/util/virstoragefile.c
@@ -3057,6 +3057,8 @@ virStorageSourceParseBackingJSONSheepdog(virStorageSourcePtr src,
int opaque ATTRIBUTE_UNUSED)
{
const char *filename;
+ const char *vdi = virJSONValueObjectGetString(json, "vdi");
+ virJSONValuePtr server = virJSONValueObjectGetObject(json, "server");
/* legacy URI based syntax passed via 'filename' option */
if ((filename = virJSONValueObjectGetString(json, "filename"))) {
@@ -3065,13 +3067,31 @@ virStorageSourceParseBackingJSONSheepdog(virStorageSourcePtr src,
VIR_STORAGE_NET_PROTOCOL_SHEEPDOG);
/* libvirt doesn't implement a parser for the legacy non-URI syntax */
+ virReportError(VIR_ERR_INVALID_ARG, "%s",
+ _("missing sheepdog URI in JSON backing volume definition"));
+ return -1;
}
- /* Sheepdog currently supports only URI and legacy syntax passed in as filename */
- virReportError(VIR_ERR_INVALID_ARG, "%s",
- _("missing sheepdog URI in JSON backing volume definition"));
+ src->type = VIR_STORAGE_TYPE_NETWORK;
+ src->protocol = VIR_STORAGE_NET_PROTOCOL_SHEEPDOG;
- return -1;
+ if (!vdi) {
+ virReportError(VIR_ERR_INVALID_ARG, "%s", _("missing sheepdog vdi name"));
+ return -1;
+ }
+
+ if (VIR_STRDUP(src->path, vdi) < 0)
+ return -1;
+
+ if (VIR_ALLOC(src->hosts) < 0)
+ return -1;
+
+ src->nhosts = 1;
+
+ if (virStorageSourceParseBackingJSONSocketAddress(src->hosts, server) < 0)
+ return -1;
+
+ return 0;
}
diff --git a/tests/virstoragetest.c b/tests/virstoragetest.c
index 7abd30c55..cbde02523 100644
--- a/tests/virstoragetest.c
+++ b/tests/virstoragetest.c
@@ -1553,6 +1553,17 @@ mymain(void)
"<source protocol='iscsi' name='iqn.2016-12.com.virttest:emulated-iscsi-noauth.target/6'>\n"
" <host name='test.org'/>\n"
"</source>\n");
+ TEST_BACKING_PARSE("json:{\"file\":{\"driver\":\"sheepdog\","
+ "\"transport\":\"tcp\","
+ "\"vdi\":\"test\","
+ "\"server\":{ \"type\":\"tcp\","
+ "\"host\":\"example.com\""
+ "}"
+ "}"
+ "}",
+ "<source protocol='sheepdog' name='test'>\n"
+ " <host name='example.com'/>\n"
+ "</source>\n");
cleanup:
/* Final cleanup */
--
2.12.2
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list