----- Original Message -----
From: "Eli Mesika" <emesika(a)redhat.com>
To: "Gilad Chaplik" <gchaplik(a)redhat.com>
Cc: "Michal Skrivanek" <michal.skrivanek(a)redhat.com>, devel(a)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(a)redhat.com>
> To: "Michal Skrivanek" <michal.skrivanek(a)redhat.com>
> 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>
> > To: "Alon Bar-Lev" <alonbl(a)redhat.com>, "Yair
Zaslavsky"
> > <yzaslavs(a)redhat.com>
> > 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> wrote:
> >
> > >
> > >
> > > ----- Original Message -----
> > >> From: "Yair Zaslavsky" <yzaslavs(a)redhat.com>
> > >> To: "Alon Bar-Lev" <alonbl(a)redhat.com>
> > >> Cc: "Eli Mesika" <emesika(a)redhat.com>,
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>
> > >>> To: "Eli Mesika" <emesika(a)redhat.com>
> > >>> 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>
> > >>>> To: "Alon Bar-Lev" <alonbl(a)redhat.com>
> > >>>> Cc: devel(a)ovirt.org, "Barak Azulay"
<bazulay(a)redhat.com>, "Oved
> > >>>> Ourfalli"
> > >>>> <oourfali(a)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(a)redhat.com>
> > >>>>> To: "Yair Zaslavsky"
<yzaslavs(a)redhat.com>
> > >>>>> 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>
> > >>>>>> 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?
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(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
>