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

[Gilad Chaplik]

----- Original Message -----
> From: "Eli Mesika" <emesika at redhat.com>
> To: "Gilad Chaplik" <gchaplik at redhat.com>
> Cc: "Michal Skrivanek" <michal.skrivanek at redhat.com>, devel at ovirt.org
> Sent: Tuesday, April 22, 2014 1:11:44 AM
> Subject: Re: [ovirt-devel] Small reminder about 0000_config.sql
> 
> 
> 
> ----- 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 ?

+1 :-)
> 
> > 
> > > >>>>> 
> > > >>>>> 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
> > 
>