Re: [libvirt] [PATCH 2/2] qemu: Validate the domain after marking the current domain as transient

2017-03-20 Thread Marc Hartmayer
On Wed, Mar 08, 2017 at 10:41 AM +0100, "Daniel P. Berrange" wrote: > On Wed, Mar 08, 2017 at 10:32:32AM +0100, Marc Hartmayer wrote: >> On Mon, Feb 27, 2017 at 11:20 AM +0100, "Daniel P. Berrange" >> wrote: >> > On Mon, Feb 27, 2017 at 10:59:39AM

Re: [libvirt] [PATCH 2/2] qemu: Validate the domain after marking the current domain as transient

2017-03-08 Thread Daniel P. Berrange
On Wed, Mar 08, 2017 at 10:32:32AM +0100, Marc Hartmayer wrote: > On Mon, Feb 27, 2017 at 11:20 AM +0100, "Daniel P. Berrange" > wrote: > > On Mon, Feb 27, 2017 at 10:59:39AM +0100, Marc Hartmayer wrote: > >> On Thu, Feb 23, 2017 at 05:36 PM +0100, "Daniel P. Berrange" > >>

Re: [libvirt] [PATCH 2/2] qemu: Validate the domain after marking the current domain as transient

2017-02-27 Thread Daniel P. Berrange
On Mon, Feb 27, 2017 at 10:59:39AM +0100, Marc Hartmayer wrote: > On Thu, Feb 23, 2017 at 05:36 PM +0100, "Daniel P. Berrange" > wrote: > > On Thu, Feb 23, 2017 at 05:22:44PM +0100, Marc Hartmayer wrote: > >> On Thu, Feb 23, 2017 at 03:33 PM +0100, Michal Privoznik > >>

Re: [libvirt] [PATCH 2/2] qemu: Validate the domain after marking the current domain as transient

2017-02-27 Thread Marc Hartmayer
On Thu, Feb 23, 2017 at 05:36 PM +0100, "Daniel P. Berrange" wrote: > On Thu, Feb 23, 2017 at 05:22:44PM +0100, Marc Hartmayer wrote: >> On Thu, Feb 23, 2017 at 03:33 PM +0100, Michal Privoznik >> wrote: >> > On 02/23/2017 10:44 AM, Marc Hartmayer

Re: [libvirt] [PATCH 2/2] qemu: Validate the domain after marking the current domain as transient

2017-02-23 Thread Peter Krempa
On Thu, Feb 23, 2017 at 17:22:44 +0100, Marc Hartmayer wrote: > On Thu, Feb 23, 2017 at 03:33 PM +0100, Michal Privoznik > wrote: > > On 02/23/2017 10:44 AM, Marc Hartmayer wrote: > >> Validate the domain that actually will be started. It's semantically > >> more clear and

Re: [libvirt] [PATCH 2/2] qemu: Validate the domain after marking the current domain as transient

2017-02-23 Thread Daniel P. Berrange
On Thu, Feb 23, 2017 at 05:22:44PM +0100, Marc Hartmayer wrote: > On Thu, Feb 23, 2017 at 03:33 PM +0100, Michal Privoznik > wrote: > > On 02/23/2017 10:44 AM, Marc Hartmayer wrote: > >> Validate the domain that actually will be started. It's semantically > >> more clear and

Re: [libvirt] [PATCH 2/2] qemu: Validate the domain after marking the current domain as transient

2017-02-23 Thread Marc Hartmayer
On Thu, Feb 23, 2017 at 03:33 PM +0100, Michal Privoznik wrote: > On 02/23/2017 10:44 AM, Marc Hartmayer wrote: >> Validate the domain that actually will be started. It's semantically >> more clear and also it can detect failures that may have happened in >>

Re: [libvirt] [PATCH 2/2] qemu: Validate the domain after marking the current domain as transient

2017-02-23 Thread Michal Privoznik
On 02/23/2017 10:44 AM, Marc Hartmayer wrote: > Validate the domain that actually will be started. It's semantically > more clear and also it can detect failures that may have happened in > virDomainObjSetDefTransient(). > > Signed-off-by: Marc Hartmayer >

[libvirt] [PATCH 2/2] qemu: Validate the domain after marking the current domain as transient

2017-02-23 Thread Marc Hartmayer
Validate the domain that actually will be started. It's semantically more clear and also it can detect failures that may have happened in virDomainObjSetDefTransient(). Signed-off-by: Marc Hartmayer Reviewed-by: Bjoern Walk Reviewed-by: