<div dir="ltr">Will add omitempty, thank you.<div><br></div><div>Also I will add there storage volume.</div><div><br></div><div>Thank you!</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 5, 2017 at 5:48 PM, Daniel P. Berrange <span dir="ltr"><<a href="mailto:berrange@redhat.com" target="_blank">berrange@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Thu, Jan 05, 2017 at 05:25:09PM +0100, Alexey Slaykovsky wrote:<br>
> Signed-off-by: Alexey Slaykovsky <<a href="mailto:aslaikov@redhat.com">aslaikov@redhat.com</a>><br>
> ---<br>
>  storage.go      | 114 +++++++++++++++++++++++++++<br>
>  storage_test.go | 240 ++++++++++++++++++++++++++++++<wbr>++++++++++++++++++++++++++<br>
>  2 files changed, 354 insertions(+)<br>
>  create mode 100644 storage.go<br>
>  create mode 100644 storage_test.go<br>
><br>
> diff --git a/storage.go b/storage.go<br>
> new file mode 100644<br>
> index 0000000..b45f58e<br>
> --- /dev/null<br>
> +++ b/storage.go<br>
> @@ -0,0 +1,114 @@<br>
> +/*<br>
> + * This file is part of the libvirt-go-xml project<br>
> + *<br>
> + * Permission is hereby granted, free of charge, to any person obtaining a copy<br>
> + * of this software and associated documentation files (the "Software"), to deal<br>
> + * in the Software without restriction, including without limitation the rights<br>
> + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell<br>
> + * copies of the Software, and to permit persons to whom the Software is<br>
> + * furnished to do so, subject to the following conditions:<br>
> + *<br>
> + * The above copyright notice and this permission notice shall be included in<br>
> + * all copies or substantial portions of the Software.<br>
> + *<br>
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR<br>
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,<br>
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE<br>
> + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER<br>
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,<br>
> + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN<br>
> + * THE SOFTWARE.<br>
> + *<br>
> + * Copyright (C) 2017 Red Hat, Inc.<br>
> + *<br>
> + */<br>
> +<br>
> +package libvirtxml<br>
> +<br>
> +import "encoding/xml"<br>
> +<br>
> +type StoragePoolTargetPermissions struct {<br>
> +     Owner string `xml:"owner"`<br>
> +     Group string `xml:"group"`<br>
> +     Mode  string `xml:"mode"`<br>
> +     Label string `xml:"label"`<br>
<br>
</div></div>We need omitempty on all of these, since it is valid to leave any<br>
of them unset when defining a new storage pool.<br>
<br>
The rest of it looks good though.<br>
<br>
Regards,<br>
Daniel<br>
<span class="HOEnZb"><font color="#888888">--<br>
|: <a href="http://berrange.com" rel="noreferrer" target="_blank">http://berrange.com</a>      -o-    <a href="http://www.flickr.com/photos/dberrange/" rel="noreferrer" target="_blank">http://www.flickr.com/photos/<wbr>dberrange/</a> :|<br>
|: <a href="http://libvirt.org" rel="noreferrer" target="_blank">http://libvirt.org</a>              -o-             <a href="http://virt-manager.org" rel="noreferrer" target="_blank">http://virt-manager.org</a> :|<br>
|: <a href="http://entangle-photo.org" rel="noreferrer" target="_blank">http://entangle-photo.org</a>       -o-    <a href="http://search.cpan.org/~danberr/" rel="noreferrer" target="_blank">http://search.cpan.org/~<wbr>danberr/</a> :|<br>
</font></span></blockquote></div><br></div>