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

From: "Gilad Chaplik" <gchaplik(a)redhat.com&gt; To: "Michal Skrivanek" <michal.skrivanek(a)redhat.com&gt; Cc: devel(a)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(a)redhat.com&gt; > To: "Alon Bar-Lev" <alonbl(a)redhat.com&gt;, "Yair Zaslavsky" > <yzaslavs(a)redhat.com&gt; > Cc: devel(a)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(a)redhat.com&gt; wrote: > > > > > > > ----- Original Message ----- > >> From: "Yair Zaslavsky" <yzaslavs(a)redhat.com&gt; > >> To: "Alon Bar-Lev" <alonbl(a)redhat.com&gt; > >> Cc: "Eli Mesika" <emesika(a)redhat.com&gt;, devel(a)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(a)redhat.com&gt; > >>> To: "Eli Mesika" <emesika(a)redhat.com&gt; > >>> Cc: devel(a)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(a)redhat.com&gt; > >>>> To: "Alon Bar-Lev" <alonbl(a)redhat.com&gt; > >>>> Cc: devel(a)ovirt.org, "Barak Azulay" <bazulay(a)redhat.com&gt;, "Oved > >>>> Ourfalli" > >>>> <oourfali(a)redhat.com&gt; > >>>> 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(a)redhat.com&gt; > >>>>> To: "Yair Zaslavsky" <yzaslavs(a)redhat.com&gt; > >>>>> Cc: devel(a)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(a)redhat.com&gt; > >>>>>> To: devel(a)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 > 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(a)ovirt.org > >>>>>> http://lists.ovirt.org/mailman/listinfo/devel > >>>>>> > >>>>> _______________________________________________ > >>>>> Devel mailing list > >>>>> Devel(a)ovirt.org > >>>>> http://lists.ovirt.org/mailman/listinfo/devel > >>>>> > >>>> > >>> _______________________________________________ > >>> Devel mailing list > >>> Devel(a)ovirt.org > >>> http://lists.ovirt.org/mailman/listinfo/devel > >>> > >> > > _______________________________________________ > > Devel mailing list > > Devel(a)ovirt.org > > http://lists.ovirt.org/mailman/listinfo/devel > > _______________________________________________ > Devel mailing list > Devel(a)ovirt.org > http://lists.ovirt.org/mailman/listinfo/devel > _______________________________________________ Devel mailing list Devel(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/devel