----- Original Message -----
> From: "Vinzenz Feenstra" <eviliss...@redhat.com>
> To: vdsm-devel@lists.fedorahosted.org
> Sent: Thursday, October 17, 2013 1:22:48 PM
> Subject: Re: [vdsm] API.py | gerrit.ovirt Code Review
> 
> On 10/17/2013 08:43 AM, Doron Fediuck wrote:
> > http://gerrit.ovirt.org/#/c/20126/4/vdsm/API.py
> >
> > Dan,
> > just a general design question.
> >
> > The above will report the HA score to the engine.
> > I suspect that in the next versions we'll extend the
> > HA integration for other operations, such as shutting down HA.
> >
> > So going forward I think we'll need something like vdsm/momIF.py
> > to stabilize this integration.
> >
> > What do you think?
> 
> I think if you already know that you'll be extending this, it'd be nicer
> to already start adding this to a new module where you can keep
> everything together related to this. Rather than extending bits all over
> the place and having everywhere these conditional imports.
> 
> 
> 
> --
> Regards,
> 
> Vinzenz Feenstra | Senior Software Engineer
> RedHat Engineering Virtualization R & D
> Phone: +420 532 294 625
> IRC: vfeenstr or evilissimo
> 
> Better technology. Faster innovation. Powered by community collaboration.
> See how it works at redhat.com
> 

The idea is to keep what we no have, and when need to extend
we'll replace the import with an interface the same way mom
has.
The only question here is design wise, if such an interface
will make sense.
_______________________________________________
vdsm-devel mailing list
vdsm-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel

Reply via email to