[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