Re: [libvirt] [PATCH 3/6] conf: Introduce infrastructure to add config validation to define time

2016-05-19 Thread Cole Robinson
On 05/18/2016 08:43 AM, Pavel Hrdina wrote: > On Tue, May 17, 2016 at 04:25:39PM +0200, Peter Krempa wrote: >> Recently there were a few patches adding checks to the post parse >> callback infrastructure that rejected previously accepted configuration. >> This unfortunately was not acceptable since

Re: [libvirt] [PATCH 3/6] conf: Introduce infrastructure to add config validation to define time

2016-05-18 Thread Pavel Hrdina
On Tue, May 17, 2016 at 04:25:39PM +0200, Peter Krempa wrote: > Recently there were a few patches adding checks to the post parse > callback infrastructure that rejected previously accepted configuration. > This unfortunately was not acceptable since we would fail to load the > configs from the dis

Re: [libvirt] [PATCH 3/6] conf: Introduce infrastructure to add config validation to define time

2016-05-17 Thread Peter Krempa
On Tue, May 17, 2016 at 23:04:19 -0600, Jim Fehlig wrote: > On 05/17/2016 08:25 AM, Peter Krempa wrote: > > Recently there were a few patches adding checks to the post parse > > callback infrastructure that rejected previously accepted configuration. > > This unfortunately was not acceptable since

Re: [libvirt] [PATCH 3/6] conf: Introduce infrastructure to add config validation to define time

2016-05-17 Thread Jim Fehlig
On 05/17/2016 08:25 AM, Peter Krempa wrote: > Recently there were a few patches adding checks to the post parse > callback infrastructure that rejected previously accepted configuration. > This unfortunately was not acceptable since we would fail to load the > configs from the disk and the VMs woul

Re: [libvirt] [PATCH 3/6] conf: Introduce infrastructure to add config validation to define time

2016-05-17 Thread John Ferlan
On 05/17/2016 10:25 AM, Peter Krempa wrote: > Recently there were a few patches adding checks to the post parse > callback infrastructure that rejected previously accepted configuration. > This unfortunately was not acceptable since we would fail to load the > configs from the disk and the VMs wo

[libvirt] [PATCH 3/6] conf: Introduce infrastructure to add config validation to define time

2016-05-17 Thread Peter Krempa
Recently there were a few patches adding checks to the post parse callback infrastructure that rejected previously accepted configuration. This unfortunately was not acceptable since we would fail to load the configs from the disk and the VMs would vanish. This patch adds helpers which are called