Re: [libvirt] [PATCH] Allow address lease query with inactive domain

2016-09-08 Thread Amador Pahim
On Thu, Sep 8, 2016 at 2:46 AM, Laine Stump wrote: > On 09/07/2016 05:57 AM, Amador Pahim wrote: >> >> On Wed, Sep 7, 2016 at 11:41 AM, Daniel P. Berrange >> wrote: >>> >>> On Wed, Sep 07, 2016 at 11:18:58AM +0200, Amador Pahim wrote: The

Re: [libvirt] [PATCH] Allow address lease query with inactive domain

2016-09-07 Thread Laine Stump
On 09/07/2016 05:57 AM, Amador Pahim wrote: On Wed, Sep 7, 2016 at 11:41 AM, Daniel P. Berrange wrote: On Wed, Sep 07, 2016 at 11:18:58AM +0200, Amador Pahim wrote: The DomainInterfaceAddresses, when using the source type LEASE, does not need the domain to be running. It

Re: [libvirt] [PATCH] Allow address lease query with inactive domain

2016-09-07 Thread Amador Pahim
On Wed, Sep 7, 2016 at 11:41 AM, Daniel P. Berrange wrote: > On Wed, Sep 07, 2016 at 11:18:58AM +0200, Amador Pahim wrote: >> The DomainInterfaceAddresses, when using the source type >> LEASE, does not need the domain to be running. It only checks the >> dhcp lease file and

Re: [libvirt] [PATCH] Allow address lease query with inactive domain

2016-09-07 Thread Daniel P. Berrange
On Wed, Sep 07, 2016 at 11:18:58AM +0200, Amador Pahim wrote: > The DomainInterfaceAddresses, when using the source type > LEASE, does not need the domain to be running. It only checks the > dhcp lease file and gets the address information from a valid lease, > if any. > > This patch removes the

[libvirt] [PATCH] Allow address lease query with inactive domain

2016-09-07 Thread Amador Pahim
The DomainInterfaceAddresses, when using the source type LEASE, does not need the domain to be running. It only checks the dhcp lease file and gets the address information from a valid lease, if any. This patch removes the virDomainObjIsActive(vm) check from the LEASE query type on both qemu and