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

Re: [PATCH v1 1/4] qemu: revert latest pSeries NVDIMM design changes



On Thu, 2020-09-17 at 17:34 -0300, Daniel Henrique Barboza wrote:
> On 9/17/20 7:48 AM, Andrea Bolognani wrote:
> > A few tweaks to the commit message, though:
> > 
> >    * keep the order of commits consistent to the one they were merged
> >      in, which more specifically means putting 2d93cbdea9d1 at the
> >      very top of the list;
> > 
> >    * lose the commas after the various commit hashes - they break
> >      convenient double-click selection in most terminals;
> > 
> >    * indent the commit subject by two spaces for better readability.
> 
> Ok!
> 
> > The commit is also missing your S-o-b, and as you know that's a
> > blocker for merging.
> 
> Ouch. reverting + squashing took its toll ....

Since we're approaching the freeze period and don't want the revert
to miss the deadline, resulting in another libvirt release
implementing the problematic behavior, I have gone ahead and pushed
this single patch after applying all the tweaks I mentioned above and
that you had confirmed you were okay with anyway.

I've also posted an update to the release notes covering the revert:

  https://www.redhat.com/archives/libvir-list/2020-September/msg01105.html

This is the exact same wording I had suggested in [1], which you were
also okay with, minus the part about reflecting the updated value in
the domain XML, since that's obviously not implemented yet.


[1] https://www.redhat.com/archives/libvir-list/2020-September/msg01020.html
-- 
Andrea Bolognani / Red Hat / Virtualization


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