[libvirt] [PATCH] news: Document recent snapshot topological flag

Eric Blake posted 1 patch 5 days ago
Test syntax-check passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20190314011004.3693-1-eblake@redhat.com
docs/news.xml | 12 ++++++++++++
1 file changed, 12 insertions(+)

[libvirt] [PATCH] news: Document recent snapshot topological flag

Posted by Eric Blake 5 days ago
A new API flag is news-worthy.

Signed-off-by: Eric Blake <eblake@redhat.com>
---

I'll probably push this one as trivial in 24 hours, if it does not
get a review sooner.

 docs/news.xml | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/docs/news.xml b/docs/news.xml
index 896700fa97..1973a0096d 100644
--- a/docs/news.xml
+++ b/docs/news.xml
@@ -82,6 +82,18 @@
           chooses.
         </description>
       </change>
+      <change>
+        <summary>
+          snapshots: Topologically-sorted listing
+        </summary>
+        <description>
+          A new flag VIR_DOMAIN_SNAPSHOT_LIST_TOPOLOGICAL is available
+          for the various snapshot listing APIs such as
+          virDomainSnapshotListAll(). For drivers that support the
+          flag, the listed snapshots are guaranteed to be sorted such
+          that parents occur before children.
+        </description>
+      </change>
     </section>
     <section title="Improvements">
     </section>
-- 
2.20.1

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

Re: [libvirt] [PATCH] news: Document recent snapshot topological flag

Posted by Martin Kletzander 4 days ago
On Wed, Mar 13, 2019 at 08:10:04PM -0500, Eric Blake wrote:
>A new API flag is news-worthy.
>
>Signed-off-by: Eric Blake <eblake@redhat.com>
>---
>
>I'll probably push this one as trivial in 24 hours, if it does not
>get a review sooner.
>
> docs/news.xml | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
>diff --git a/docs/news.xml b/docs/news.xml
>index 896700fa97..1973a0096d 100644
>--- a/docs/news.xml
>+++ b/docs/news.xml
>@@ -82,6 +82,18 @@
>           chooses.
>         </description>
>       </change>
>+      <change>
>+        <summary>
>+          snapshots: Topologically-sorted listing

"snapshots: Add support for Topological sorting" gives me a feel of optionality, but both are fine.  Either way,

Reviewed-by: Martin Kletzander <mkletzan@redhat.com>
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list