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

[Gilad Chaplik]

----- 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?

> >>>>> 
> >>>>> 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 outdated
> 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
>