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

On 02/14/2012 09:59 AM, Yair Zaslavsky wrote: > 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. Why? Why not improve their user experience and provide them with such feature? Current behaviour sucks - as your system admin.

>>>>> >>>>> 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. I'm not sure I see the reason a user would want to turn it off, on a per-object basis. If it's in the 'Advanced' settings of a host/storage, fine, but otherwise, it's just another cryptic feature to turn on/off. Y. > >>>> _______________________________________________ >>>> 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 _______________________________________________ Engine-devel mailing list Engine-devel(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel