On Mon, Jul 10, 2017 at 11:15:14AM +0200, Thomas Hipp wrote:
Add support for QEMU, and add test code.
Signed-off-by: Thomas Hipp <thipp(a)suse.de>
---
Changes since v1:
- support multiple Args and Envs
---
domain.go | 50 +++++++++++++++++++++++++++++++++-----------------
domain_test.go | 28 ++++++++++++++++++++++++++++
2 files changed, 61 insertions(+), 17 deletions(-)
diff --git a/domain.go b/domain.go
index b9b0f77..8c1fa76 100644
--- a/domain.go
+++ b/domain.go
@@ -590,27 +590,43 @@ type DomainFeatureList struct {
SMM *DomainFeatureState `xml:"smm"`
}
+type DomainQEMUCommandlineArg struct {
+ Value string `xml:"value,attr"`
+}
+
+type DomainQEMUCommandlineEnv struct {
+ Name string `xml:"name,attr"`
+ Value string `xml:"value,attr,omitempty"`
+}
+
+type DomainQEMUCommandline struct {
Add
XMLName xml.Name `xml:"http://libvirt.org/schemas/domain/qemu/1.0
commandline"`
+ Args []DomainQEMUCommandlineArg `xml:"qemu arg"`
And change to `xml:"arg"`
+ Envs []DomainQEMUCommandlineEnv `xml:"qemu env"`
And change to `xml:"arg"`
+}
+
// NB, try to keep the order of fields in this struct
// matching the order of XML elements that libvirt
// will generate when dumping XML.
type Domain struct {
...snip...
+ QEMUCommandline *DomainQEMUCommandline `xml:"qemu
commandline"`
And change to `xml:"commandline"`
}
Then, I think it will generate valid XML that works with libvirt
Regards,
Daniel
--
|:
https://berrange.com -o-
https://www.flickr.com/photos/dberrange :|
|:
https://libvirt.org -o-
https://fstop138.berrange.com :|
|:
https://entangle-photo.org -o-
https://www.instagram.com/dberrange :|