[libvirt] [PATCH 7/7] network: include plugged interface XML in "plugged" network hook
Eric Blake
eblake at redhat.com
Mon Feb 24 19:46:20 UTC 2014
On 02/24/2014 12:27 PM, Laine Stump wrote:
>>>
>> ACK although the indentation of XML we're passing to the hook script seems off:
>
> Right. That's what the last paragraph of the commit log message is about
> - fixing that indentation would require a much more invasive change that
> would touch all the other device parsing functions, which could turn any
> potential backport into a real headache, so I chose to not fix it in
> this series.
Correct - whitespace cleanups in a separate commit, especially in order
to ease backport efforts, is a good tradeoff to make.
>> This is not a show stopper for me. I wonder if we should push these patches now, even during the freeze as this is very closely related to the network hooks - one of the main features in this release.
>
> In a way it is a bugfix for that feature (since the functionality of the
> "plugged" hook is severely limited without it). I would feel more
> comfortable about pushing it, though, if danpb took a look at the commit
> log message for patch 5/7 and gave his okay on the change in the XML. My
> opinion is that I should have done it this way to begin with, but Dan
> has much better insight than I do on what is and isn't good for
> management applications.
I agree with getting Dan's opinion, but you have my ACK for including
this in time for 1.2.2 as a bug fix of the hook feature.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140224/8db6f0b0/attachment-0001.sig>
More information about the libvir-list
mailing list