<div dir="ltr"><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Nov 15, 2018 at 12:36 AM Andrea Bolognani <<a href="mailto:abologna@redhat.com">abologna@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, 2018-11-14 at 15:19 +0800, Han Han wrote:<br>
> When listing snapshot with VIR_DOMAIN_SNAPSHOT_LIST_NO_METADATA, it<br>
> always returns 0 or no snapshot. Because we never implement funtions<br>
> to list no-metadata snapshot in virDomainSnapshotObjListGetNames():<br>
> <br>
>     if ((data.flags & VIR_DOMAIN_SNAPSHOT_FILTERS_METADATA) ==<br>
>         VIR_DOMAIN_SNAPSHOT_LIST_NO_METADATA)<br>
>         return 0;<br>
> <br>
> Add notes for that flag.<br>
> <br>
> Please update the comment and man page of that flag when no-metadata<br>
> snapshot list is implemented in the future.<br>
<br>
I could be missing some information, but from a quick look at the<br>
commit message and the patch it looks to me like you're documenting<br>
a known limitation instead of, you know, addressing it :)<br>
<br></blockquote><div>Bug filed as : <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1650419">https://bugzilla.redhat.com/show_bug.cgi?id=1650419</a> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
If you are able to fix the issue yourself, then please do so;<br>
otherwise, filing a bug seems like it would be a more appropriate<br>
course of action.<br>
<br>
-- <br>
Andrea Bolognani / Red Hat / Virtualization<br>
<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr">Best regards,</div><div dir="ltr">-----------------------------------<br></div><div dir="ltr">Han Han<br>Quality Engineer<br>Redhat.<br><br>Email: <a href="mailto:hhan@redhat.com" target="_blank">hhan@redhat.com</a><br>Phone: +861065339333<br></div></div></div></div></div></div></div></div>