Re: [PATCH go-xml v2] Add support for QEMU

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxx>
> ---
> 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 :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux