On Fri, Nov 09, 2012 at 05:06:05AM -0500, Eli Mesika wrote: > > > ----- Original Message ----- > > From: "Itamar Heim" <ih...@redhat.com> > > To: "Eli Mesika" <emes...@redhat.com> > > Cc: "engine-devel" <engine-devel@ovirt.org>, "Michael Pasternak" > > <mpast...@redhat.com>, "Simon Grinberg" > > <sgrin...@redhat.com>, "Dan Kenigsberg" <dan...@redhat.com> > > Sent: Friday, November 9, 2012 12:02:37 PM > > Subject: Re: [Engine-devel] [Design for 3.2 RFE] Improving proxy selection > > algorithm for Power Management operations > > > > On 11/09/2012 10:52 AM, Eli Mesika wrote: > > > > >> > > > >> > > FenceWrapper > > >> > > > >> >i understand danken suggested going this way, rather than than > > >> >another > > >> >instance of vdsm. > > >> >is vdsm only calling these scripts today and all logic is in > > >> >engine, > > >> >or > > >> >does vdsm has any logic in wrapping these scripts (not a blocker > > >> >to > > >> >doing FenceWrapper, just worth extracting that logic from vdsm to > > >> >such a > > >> >script, then using it in both. i hope answer is 'no logic'...) > > > vdsm has some logic that maps between the call passed to it from > > > engine and the actual parameters generated for the script. > > > AFAIK, this logic only "builds" the correct arguments for the > > > command according to the agent type > > > > > > > can we extract it to an external wrapper? > > I'd hate to fix bugs/changes twice for this. > > I'll check it with danken on SUN
Saggi has had a nascent attempt to factor the little logic we have out http://gerrit.ovirt.org/#/c/7190/7/vdsm/API.py AFAIR there's nothing there beyond: - log everything but passwords, - build the input stream, - run the script - convert its return code and there's also killing dormant scripts on vdsm exist (which I find not important at all). Dan. _______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel