Re: [libvirt] [PATCHv2 2/4] network: change location of network state xml files

2014-04-24 Thread Eric Blake
On 04/24/2014 04:30 AM, Laine Stump wrote: >>> >> ACK; however... Your call if you want to wait for virDirRead() from: >> >> http://www.redhat.com/archives/libvir-list/2014-April/msg00745.html >> >> Then refactor your for (;;) loop to use it... > > I think I would rather push it as-is, then send

Re: [libvirt] [PATCHv2 2/4] network: change location of network state xml files

2014-04-24 Thread Laine Stump
On 04/23/2014 09:52 PM, John Ferlan wrote: > > On 04/17/2014 07:43 AM, Laine Stump wrote: >> For some reason these have been stored in /var/lib, although other >> drivers (e.g. qemu and lxc) store their state files in /var/run. >> >> It's much nicer to store state files in /var/run because it is >>

Re: [libvirt] [PATCHv2 2/4] network: change location of network state xml files

2014-04-23 Thread John Ferlan
On 04/17/2014 07:43 AM, Laine Stump wrote: > For some reason these have been stored in /var/lib, although other > drivers (e.g. qemu and lxc) store their state files in /var/run. > > It's much nicer to store state files in /var/run because it is > automatically cleared out when the system reboot

[libvirt] [PATCHv2 2/4] network: change location of network state xml files

2014-04-17 Thread Laine Stump
For some reason these have been stored in /var/lib, although other drivers (e.g. qemu and lxc) store their state files in /var/run. It's much nicer to store state files in /var/run because it is automatically cleared out when the system reboots. We can then use existence of the state file as a con