[ovirt-devel] Small reminder about 0000_config.sql

[Eli Mesika]

----- Original Message -----
> From: "Gilad Chaplik" <gchaplik at redhat.com>
> To: "Michal Skrivanek" <michal.skrivanek at redhat.com>
> Cc: devel at ovirt.org
> Sent: Friday, April 18, 2014 12:59:48 PM
> Subject: Re: [ovirt-devel] Small reminder about 0000_config.sql
> 
> ----- Original Message -----
> > From: "Michal Skrivanek" <michal.skrivanek at redhat.com>
> > To: "Alon Bar-Lev" <alonbl at redhat.com>, "Yair Zaslavsky"
> > <yzaslavs at redhat.com>
> > Cc: devel at ovirt.org
> > Sent: Friday, April 18, 2014 11:05:48 AM
> > Subject: Re: [ovirt-devel] Small reminder about 0000_config.sql
> > 
> > 
> > On Apr 18, 2014, at 09:10 , Alon Bar-Lev <alonbl at redhat.com> wrote:
> > 
> > > 
> > > 
> > > ----- Original Message -----
> > >> From: "Yair Zaslavsky" <yzaslavs at redhat.com>
> > >> To: "Alon Bar-Lev" <alonbl at redhat.com>
> > >> Cc: "Eli Mesika" <emesika at redhat.com>, devel at ovirt.org
> > >> Sent: Friday, April 18, 2014 4:57:54 AM
> > >> Subject: Re: [ovirt-devel] Small reminder about 0000_config.sql
> > >> 
> > >> 
> > >> 
> > >> ----- Original Message -----
> > >>> From: "Alon Bar-Lev" <alonbl at redhat.com>
> > >>> To: "Eli Mesika" <emesika at redhat.com>
> > >>> Cc: devel at ovirt.org
> > >>> Sent: Friday, April 18, 2014 1:19:41 AM
> > >>> Subject: Re: [ovirt-devel] Small reminder about 0000_config.sql
> > >>> 
> > >>> 
> > >>> 
> > >>> ----- Original Message -----
> > >>>> From: "Eli Mesika" <emesika at redhat.com>
> > >>>> To: "Alon Bar-Lev" <alonbl at redhat.com>
> > >>>> Cc: devel at ovirt.org, "Barak Azulay" <bazulay at redhat.com>, "Oved
> > >>>> Ourfalli"
> > >>>> <oourfali at redhat.com>
> > >>>> Sent: Friday, April 18, 2014 12:06:38 AM
> > >>>> Subject: Re: [ovirt-devel] Small reminder about 0000_config.sql
> > >>>> 
> > >>>> 
> > >>>> 
> > >>>> ----- Original Message -----
> > >>>>> From: "Alon Bar-Lev" <alonbl at redhat.com>
> > >>>>> To: "Yair Zaslavsky" <yzaslavs at redhat.com>
> > >>>>> Cc: devel at ovirt.org
> > >>>>> Sent: Thursday, April 17, 2014 10:45:40 PM
> > >>>>> Subject: Re: [ovirt-devel] Small reminder about 0000_config.sql
> > >>>>> 
> > >>>>> 
> > >>>>> 
> > >>>>> ----- Original Message -----
> > >>>>>> From: "Yair Zaslavsky" <yzaslavs at redhat.com>
> > >>>>>> To: devel at ovirt.org
> > >>>>>> Sent: Thursday, April 17, 2014 12:43:37 PM
> > >>>>>> Subject: [ovirt-devel] Small reminder about 0000_config.sql
> > >>>>>> 
> > >>>>>> Hi all, and happy holidays.
> > >>>>>> I would like to give a small reminder about the contents of
> > >>>>>> 0000_config.sql
> > >>>>>> -
> > >>>>>> We are trying to keep an alphabetical order when it comes to the
> > >>>>>> option
> > >>>>>> names
> > >>>>>> (the keys).
> 
> I think we should order/maintain the options by topic (network, storage,
> console, etc.), we can order alphabetically within topics but IMO pointless.
> Does someone really lookup alphabetically?

Maybe use a 3 letter prefix for subject and then the sorting makes sense ?

> 
> > >>>>> 
> > >>>>> We really need to remove this mechanism of config and hold within
> > >>>>> database
> > >>>>> only non default values.
> > >>>>> Default values should go into properties file or even into code.
> > >>>>> We need someone to take ownership of this ask, I will be happy to
> > >>>>> instruct.
> > >>>> 
> > >>>> Alon, I will be glad to own that (CCing Barak & Oved) , can you please
> > >>>> open
> > >>>> a
> > >>>> oVirt RFE on that ?
> > >>> 
> > >>> Thanks!
> > >>> 
> > >>> Done[1]
> > >>> 
> > >>> [1] https://bugzilla.redhat.com/show_bug.cgi?id=1089093
> > >> 
> > >> Bare in mind default values already exist in code (maybe need to be
> > >> revisited, but that's something else) at ConfigValues.
> > 
> > I'd be more than happy to see all those going away. Many of them are
> > outdate
> > and plain wrong.
> > 
> > > 
> > > engine-config cannot set a value to a "default" that exists only in
> > > ConfigValues.
> > > So in principal you are right, but the entire solution does not support
> > > this.
> > > 
> > >> 
> > >> 
> > >>> 
> > >>>> 
> > >>>>> 
> > >>>>>> 
> > >>>>>> Thanks,
> > >>>>>> 
> > >>>>>> Yair
> > >>>>>> _______________________________________________
> > >>>>>> Devel mailing list
> > >>>>>> Devel at ovirt.org
> > >>>>>> http://lists.ovirt.org/mailman/listinfo/devel
> > >>>>>> 
> > >>>>> _______________________________________________
> > >>>>> Devel mailing list
> > >>>>> Devel at ovirt.org
> > >>>>> http://lists.ovirt.org/mailman/listinfo/devel
> > >>>>> 
> > >>>> 
> > >>> _______________________________________________
> > >>> Devel mailing list
> > >>> Devel at ovirt.org
> > >>> http://lists.ovirt.org/mailman/listinfo/devel
> > >>> 
> > >> 
> > > _______________________________________________
> > > Devel mailing list
> > > Devel at ovirt.org
> > > http://lists.ovirt.org/mailman/listinfo/devel
> > 
> > _______________________________________________
> > Devel mailing list
> > Devel at ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/devel
> > 
> _______________________________________________
> Devel mailing list
> Devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel
>