Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-31 Thread Daniel P. Berrange
On Tue, Oct 31, 2017 at 07:04:35AM +0100, Michal Privoznik wrote: > On 10/30/2017 11:12 PM, Jim Fehlig wrote: > > On 10/30/2017 03:31 PM, Jim Fehlig wrote: > >> On 10/03/2017 07:53 AM, Daniel P. Berrange wrote: > >>> On Tue, Oct 03, 2017 at 02:11:44PM +0200, Martin Kletzander wrote: > On Tue,

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-31 Thread Michal Privoznik
On 10/30/2017 11:12 PM, Jim Fehlig wrote: > On 10/30/2017 03:31 PM, Jim Fehlig wrote: >> On 10/03/2017 07:53 AM, Daniel P. Berrange wrote: >>> On Tue, Oct 03, 2017 at 02:11:44PM +0200, Martin Kletzander wrote: On Tue, Oct 03, 2017 at 12:58:59PM +0200, Michal Privoznik wrote: >

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-30 Thread Jim Fehlig
On 10/30/2017 03:31 PM, Jim Fehlig wrote: On 10/03/2017 07:53 AM, Daniel P. Berrange wrote: On Tue, Oct 03, 2017 at 02:11:44PM +0200, Martin Kletzander wrote: On Tue, Oct 03, 2017 at 12:58:59PM +0200, Michal Privoznik wrote: https://bugzilla.redhat.com/show_bug.cgi?id=1434451 It comes handy

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-30 Thread Jim Fehlig
On 10/03/2017 07:53 AM, Daniel P. Berrange wrote: On Tue, Oct 03, 2017 at 02:11:44PM +0200, Martin Kletzander wrote: On Tue, Oct 03, 2017 at 12:58:59PM +0200, Michal Privoznik wrote: https://bugzilla.redhat.com/show_bug.cgi?id=1434451 It comes handy for management application to be able to

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-05 Thread Daniel P. Berrange
On Thu, Oct 05, 2017 at 01:31:02PM +0200, Michal Privoznik wrote: > On 10/05/2017 01:01 PM, Pavel Hrdina wrote: > > On Thu, Oct 05, 2017 at 10:40:01AM +0100, Daniel P. Berrange wrote: > >> On Thu, Oct 05, 2017 at 11:27:29AM +0200, Martin Kletzander wrote: > >>> On Thu, Oct 05, 2017 at 10:44:29AM

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-05 Thread Daniel P. Berrange
On Thu, Oct 05, 2017 at 01:26:54PM +0200, Peter Krempa wrote: > On Thu, Oct 05, 2017 at 10:40:01 +0100, Daniel Berrange wrote: > > On Thu, Oct 05, 2017 at 11:27:29AM +0200, Martin Kletzander wrote: > > > On Thu, Oct 05, 2017 at 10:44:29AM +0200, Michal Privoznik wrote: > > > > On 10/05/2017 10:10

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-05 Thread Michal Privoznik
On 10/05/2017 01:01 PM, Pavel Hrdina wrote: > On Thu, Oct 05, 2017 at 10:40:01AM +0100, Daniel P. Berrange wrote: >> On Thu, Oct 05, 2017 at 11:27:29AM +0200, Martin Kletzander wrote: >>> On Thu, Oct 05, 2017 at 10:44:29AM +0200, Michal Privoznik wrote: On 10/05/2017 10:10 AM, Daniel P.

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-05 Thread Peter Krempa
On Thu, Oct 05, 2017 at 10:40:01 +0100, Daniel Berrange wrote: > On Thu, Oct 05, 2017 at 11:27:29AM +0200, Martin Kletzander wrote: > > On Thu, Oct 05, 2017 at 10:44:29AM +0200, Michal Privoznik wrote: > > > On 10/05/2017 10:10 AM, Daniel P. Berrange wrote: > > > > On Wed, Oct 04, 2017 at

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-05 Thread Pavel Hrdina
On Thu, Oct 05, 2017 at 10:40:01AM +0100, Daniel P. Berrange wrote: > On Thu, Oct 05, 2017 at 11:27:29AM +0200, Martin Kletzander wrote: > > On Thu, Oct 05, 2017 at 10:44:29AM +0200, Michal Privoznik wrote: > > > On 10/05/2017 10:10 AM, Daniel P. Berrange wrote: > > > > On Wed, Oct 04, 2017 at

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-05 Thread Daniel P. Berrange
On Thu, Oct 05, 2017 at 11:28:35AM +0200, Michal Privoznik wrote: > On 10/05/2017 11:13 AM, Daniel P. Berrange wrote: > > On Thu, Oct 05, 2017 at 10:44:29AM +0200, Michal Privoznik wrote: > >> On 10/05/2017 10:10 AM, Daniel P. Berrange wrote: > >>> On Wed, Oct 04, 2017 at 08:31:36AM +0200, Martin

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-05 Thread Daniel P. Berrange
On Thu, Oct 05, 2017 at 11:27:29AM +0200, Martin Kletzander wrote: > On Thu, Oct 05, 2017 at 10:44:29AM +0200, Michal Privoznik wrote: > > On 10/05/2017 10:10 AM, Daniel P. Berrange wrote: > > > On Wed, Oct 04, 2017 at 08:31:36AM +0200, Martin Kletzander wrote: > > > > On Tue, Oct 03, 2017 at

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-05 Thread Martin Kletzander
On Thu, Oct 05, 2017 at 10:44:29AM +0200, Michal Privoznik wrote: On 10/05/2017 10:10 AM, Daniel P. Berrange wrote: On Wed, Oct 04, 2017 at 08:31:36AM +0200, Martin Kletzander wrote: On Tue, Oct 03, 2017 at 03:10:48PM +0100, Daniel P. Berrange wrote: On Tue, Oct 03, 2017 at 04:03:20PM +0200,

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-05 Thread Michal Privoznik
On 10/05/2017 11:13 AM, Daniel P. Berrange wrote: > On Thu, Oct 05, 2017 at 10:44:29AM +0200, Michal Privoznik wrote: >> On 10/05/2017 10:10 AM, Daniel P. Berrange wrote: >>> On Wed, Oct 04, 2017 at 08:31:36AM +0200, Martin Kletzander wrote: On Tue, Oct 03, 2017 at 03:10:48PM +0100, Daniel P.

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-05 Thread Daniel P. Berrange
On Thu, Oct 05, 2017 at 10:44:29AM +0200, Michal Privoznik wrote: > On 10/05/2017 10:10 AM, Daniel P. Berrange wrote: > > On Wed, Oct 04, 2017 at 08:31:36AM +0200, Martin Kletzander wrote: > >> On Tue, Oct 03, 2017 at 03:10:48PM +0100, Daniel P. Berrange wrote: > >>> On Tue, Oct 03, 2017 at

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-05 Thread Michal Privoznik
On 10/05/2017 10:10 AM, Daniel P. Berrange wrote: > On Wed, Oct 04, 2017 at 08:31:36AM +0200, Martin Kletzander wrote: >> On Tue, Oct 03, 2017 at 03:10:48PM +0100, Daniel P. Berrange wrote: >>> On Tue, Oct 03, 2017 at 04:03:20PM +0200, Martin Kletzander wrote: On Tue, Oct 03, 2017 at

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-05 Thread Daniel P. Berrange
On Wed, Oct 04, 2017 at 08:31:36AM +0200, Martin Kletzander wrote: > On Tue, Oct 03, 2017 at 03:10:48PM +0100, Daniel P. Berrange wrote: > > On Tue, Oct 03, 2017 at 04:03:20PM +0200, Martin Kletzander wrote: > > > On Tue, Oct 03, 2017 at 02:53:46PM +0100, Daniel P. Berrange wrote: > > > > On Tue,

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-04 Thread Martin Kletzander
On Tue, Oct 03, 2017 at 03:10:48PM +0100, Daniel P. Berrange wrote: On Tue, Oct 03, 2017 at 04:03:20PM +0200, Martin Kletzander wrote: On Tue, Oct 03, 2017 at 02:53:46PM +0100, Daniel P. Berrange wrote: > On Tue, Oct 03, 2017 at 02:11:44PM +0200, Martin Kletzander wrote: > > On Tue, Oct 03,

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-03 Thread Daniel P. Berrange
On Tue, Oct 03, 2017 at 04:03:20PM +0200, Martin Kletzander wrote: > On Tue, Oct 03, 2017 at 02:53:46PM +0100, Daniel P. Berrange wrote: > > On Tue, Oct 03, 2017 at 02:11:44PM +0200, Martin Kletzander wrote: > > > On Tue, Oct 03, 2017 at 12:58:59PM +0200, Michal Privoznik wrote: > > > >

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-03 Thread Martin Kletzander
On Tue, Oct 03, 2017 at 02:53:46PM +0100, Daniel P. Berrange wrote: On Tue, Oct 03, 2017 at 02:11:44PM +0200, Martin Kletzander wrote: On Tue, Oct 03, 2017 at 12:58:59PM +0200, Michal Privoznik wrote: > https://bugzilla.redhat.com/show_bug.cgi?id=1434451 > > It comes handy for management

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-03 Thread Daniel P. Berrange
On Tue, Oct 03, 2017 at 02:11:44PM +0200, Martin Kletzander wrote: > On Tue, Oct 03, 2017 at 12:58:59PM +0200, Michal Privoznik wrote: > > https://bugzilla.redhat.com/show_bug.cgi?id=1434451 > > > > It comes handy for management application to be able to have a > > per-device label so that it can

Re: [libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-03 Thread Martin Kletzander
On Tue, Oct 03, 2017 at 12:58:59PM +0200, Michal Privoznik wrote: https://bugzilla.redhat.com/show_bug.cgi?id=1434451 It comes handy for management application to be able to have a per-device label so that it can uniquely identify devices it cares about. The advantage of this approach is that

[libvirt] [PATCH v2 3/3] conf: Allow users to define UUID for devices

2017-10-03 Thread Michal Privoznik
https://bugzilla.redhat.com/show_bug.cgi?id=1434451 It comes handy for management application to be able to have a per-device label so that it can uniquely identify devices it cares about. The advantage of this approach is that we don't have to generate aliases at define time (non trivial amount