Re: [Engine-devel] Autorecovery feature plan for review

On 02/14/2012 09:48 AM, Omer Frenkel wrote: > > ----- Original Message ----- >> From: "Yair Zaslavsky"<yzaslavs(a)redhat.com> >> To: engine-devel(a)ovirt.org >> Sent: Tuesday, February 14, 2012 9:20:10 AM >> Subject: Re: [Engine-devel] Autorecovery feature plan for review >> >> On 02/14/2012 08:59 AM, Itamar Heim wrote: >>> On 02/14/2012 08:57 AM, Livnat Peer wrote: >>>> On 14/02/12 05:56, Itamar Heim wrote: >>>>> On 02/13/2012 12:32 PM, Laszlo Hornyak wrote: >>>>>> Hi, >>>>>> >>>>>> Please review the plan document for autorecovery. >>>>>> http://www.ovirt.org/wiki/Features/Autorecovery >>>>> why would we disable auto recovery by default? it sounds like the >>>>> preferred behavior? >>>>> >>>> I think that by default Laszlo meant in the upgrade process to >>>> maintain >>>> current behavior.

>>>> >>>> I agree that for new entities the default should be true. >>> i think the only combination which will allow this is for db to >>> default >>> to false and code to default to true for this property? >> Why can't we during upgrade process set to all existing entities in >> DB >> the value to false, but still have the column defined as "default >> true"? > why all the trouble? i think this field should be mandatory as any other field, > user has to specify it during the entity creation, right where he provide the name and any other field for the new entity. Fine by me.

>>> _______________________________________________ >>> Engine-devel mailing list >>> Engine-devel(a)ovirt.org >>> http://lists.ovirt.org/mailman/listinfo/engine-devel >> _______________________________________________ >> Engine-devel mailing list >> Engine-devel(a)ovirt.org >> http://lists.ovirt.org/mailman/listinfo/engine-devel >> _______________________________________________ Engine-devel mailing list Engine-devel(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel