----- Original Message -----
> From: "Doron Fediuck" <dfedi...@redhat.com>
> To: "Vinzenz Feenstra" <eviliss...@redhat.com>
> Cc: vdsm-devel@lists.fedorahosted.org
> Sent: Friday, October 18, 2013 10:17:55 AM
> Subject: Re: [vdsm] API.py | gerrit.ovirt Code Review
>
>
>
> ----- 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.
In general we want to get rid of API.py in favor of subsystem specific classes.
So removing things from API.py and moving them to another files is the
recommended
course of action.
> >
> >
> >
> > --
> > 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
>
_______________________________________________
vdsm-devel mailing list
vdsm-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-devel
