[libvirt] [PATCH] docs: Add some changes to news.xml for this release

Martin Kletzander posted 1 patch 6 years, 6 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/83c004008e6d3d02227515da4c5d303313971d5b.1506696563.git.mkletzan@redhat.com
docs/news.xml | 81 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 81 insertions(+)
[libvirt] [PATCH] docs: Add some changes to news.xml for this release
Posted by Martin Kletzander 6 years, 6 months ago
Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
---
 docs/news.xml | 81 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 81 insertions(+)

diff --git a/docs/news.xml b/docs/news.xml
index aab812b259cf..c9e7e4b9d1e4 100644
--- a/docs/news.xml
+++ b/docs/news.xml
@@ -35,6 +35,26 @@
 <libvirt>
   <release version="v3.8.0" date="unreleased">
     <section title="New features">
+      <change>
+        <summary>
+          qemu: Added support for cold-(un)plug of watchdog devices
+        </summary>
+      </change>
+      <change>
+        <summary>
+          qemu: Added support for setting IP address os usernet interfaces
+        </summary>
+      </change>
+      <change>
+        <summary>
+          qemu: Added support for Veritas Hyperscale (VxHS) block devices
+        </summary>
+      </change>
+      <change>
+        <summary>
+          storage: Added new events for pool-build and pool-delete
+        </summary>
+      </change>
     </section>
     <section title="Improvements">
       <change>
@@ -59,8 +79,69 @@
             kernel-forward-plane-offload).
         </description>
       </change>
+      <change>
+        <summary>
+          New CPU models for AMD and Intel
+        </summary>
+        <description>
+          AMD EPYC and Intel Skylake-Server CPU models were added together with
+          their features
+        </description>
+      </change>
+      <change>
+        <summary>
+          Improve long waiting when saving a domain
+        </summary>
+        <description>
+          While waiting for a write to disk to be finished, e.g. during save,
+          even simple operations like <code>virsh list</code> would be blocking
+          due to domain lock.  This is now resolved by unlocking the domain
+          in places where it is not needed.
+        </description>
+      </change>
     </section>
     <section title="Bug fixes">
+      <change>
+        <summary>
+          Proper units are now used in virsh manpage for dom(mem)stats
+        </summary>
+        <description>
+          Previously the documentation used multiples of 1000, but now it is
+          fixe to use multiples of 1024.
+        </description>
+      </change>
+      <change>
+        <summary>
+          qemu: Fix error reporting when disk attachment fails
+        </summary>
+        <description>
+          There was a possibility for the actual error to be overridden or
+          cleared during the rollback.
+        </description>
+      </change>
+      <change>
+        <summary>
+          qemu: Fix assignment of graphics ports after daemon restart
+        </summary>
+        <description>
+          This could be seen with newer kernels that have bug regarding
+          SO_REUSEADDR.  After libvirtd was restarted it could assign already
+          used address to new guests which would make them fail to start.  This
+          is fixed by marking used ports unavailable when reconnecting to
+          running QEMU domains.
+        </description>
+      </change>
+      <change>
+        <summary>
+          Fix message decoding which was caused very strange bug
+        </summary>
+        <description>
+          When parsing an RPC message with file descriptors was interrupted and
+          had to restart, the offset of the payload was calculated badly causing
+          strange issues like not being able to find a domain that was not
+          requested.
+        </description>
+      </change>
     </section>
   </release>
   <release version="v3.7.0" date="2017-09-04">
-- 
2.14.2

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] docs: Add some changes to news.xml for this release
Posted by Andrea Bolognani 6 years, 6 months ago
On Fri, 2017-09-29 at 16:49 +0200, Martin Kletzander wrote:
> +        <description>
> +          While waiting for a write to disk to be finished, e.g. during save,
> +          even simple operations like <code>virsh list</code> would be blocking
> +          due to domain lock.  This is now resolved by unlocking the domain
> +          in places where it is not needed.

s/  / /g

here and further below. I know you prefer double-space-after-period,
but we are consistently using a single space in this file :)

> +        <description>
> +          Previously the documentation used multiples of 1000, but now it is
> +          fixe to use multiples of 1024.

s/fixe/fixed/

> +      <change>
> +        <summary>
> +          Fix message decoding which was caused very strange bug

s/caused/causing/
s/bug/bugs/


With these issues fixed,

  Reviewed-by: Andrea Bolognani <abologna@redhat.com>

Thanks for once again picking up the slack :)

-- 
Andrea Bolognani / Red Hat / Virtualization

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] docs: Add some changes to news.xml for this release
Posted by Martin Kletzander 6 years, 6 months ago
On Mon, Oct 02, 2017 at 10:07:01AM +0200, Andrea Bolognani wrote:
>On Fri, 2017-09-29 at 16:49 +0200, Martin Kletzander wrote:
>> +        <description>
>> +          While waiting for a write to disk to be finished, e.g. during save,
>> +          even simple operations like <code>virsh list</code> would be blocking
>> +          due to domain lock.  This is now resolved by unlocking the domain
>> +          in places where it is not needed.
>
>s/  / /g
>
>here and further below. I know you prefer double-space-after-period,
>but we are consistently using a single space in this file :)
>

Sure, it is a muscle memory at this point.  Anyway:

  :%s_\.  \([A-Z]\)_. \1_gc

showed one already existing line that I fixed as a part of this.

>> +        <description>
>> +          Previously the documentation used multiples of 1000, but now it is
>> +          fixe to use multiples of 1024.
>
>s/fixe/fixed/
>
>> +      <change>
>> +        <summary>
>> +          Fix message decoding which was caused very strange bug
>
>s/caused/causing/
>s/bug/bugs/
>

I believe it was a bug, so I added the indefinite article instead of this last change.

>
>With these issues fixed,
>
>  Reviewed-by: Andrea Bolognani <abologna@redhat.com>
>
>Thanks for once again picking up the slack :)
>

Sure, you could also cut me some slack with the double spaces since this is
fixed width style file ;)

Pushed now, thanks.

>--
>Andrea Bolognani / Red Hat / Virtualization
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list