[libvirt] [PATCH] net: merge virNetworkGetDHCPLeases and virNetworkGetDHCPLeasesForMAC

Daniel Veillard veillard at redhat.com
Mon Jun 30 03:26:22 UTC 2014


On Fri, Jun 27, 2014 at 09:45:09AM +0200, Peter Krempa wrote:
> On 06/26/14 17:29, Ján Tomko wrote:
> > On 06/26/2014 04:51 PM, Peter Krempa wrote:
> >> Instead of maintaining two very similar APIs, add the "@mac" parameter
> >> to virNetworkGetDHCPLeases and kill virNetworkGetDHCPLeasesForMAC. Both
> >> of those functions would return data the same way, so making @mac an
> >> optional filter simplifies a lot of stuff.
[...]
> > ACK with the debug message added and virsh functionality fixed.
> > 
> > Jan
> > 
> 
> I've resolved all your comments and pushed this patch.
> 
> Thanks

  Not finished, this now break the libvirt-python build which expects
virNetworkGetDHCPLeasesForMAC to be in the exported set of function,
so a bit of cleanup is needed there, preferably before the release :-)

Daniel

-- 
Daniel Veillard      | Open Source and Standards, Red Hat
veillard at redhat.com  | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | virtualization library  http://libvirt.org/




More information about the libvir-list mailing list