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

From: "Livnat Peer" <lpeer(a)redhat.com&gt; To: "Yaniv Kaul" <ykaul(a)redhat.com&gt; Cc: engine-devel(a)ovirt.org Sent: Tuesday, February 14, 2012 1:00:35 PM Subject: Re: [Engine-devel] Autorecovery feature plan for review On 14/02/12 11:45, Yaniv Kaul wrote: > 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&quot;&lt;yzaslavs(a)redhat.com&gt; >>>> 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 don't have objections either way. Laszlo - let's update the wiki to upgrade by default to true, if we'll get good reason why not to upgrade to true then we can open it again for discussion.

>>>>>> >>>>>> 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 _______________________________________________ Engine-devel mailing list Engine-devel(a)ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel