[ovirt-devel] Correct way to run internal actions from commands

[Gilad Chaplik]

----- Original Message -----
> From: "Yair Zaslavsky" <yzaslavs at redhat.com>
> To: "Gilad Chaplik" <gchaplik at redhat.com>
> Cc: devel at ovirt.org
> Sent: Thursday, June 19, 2014 11:22:49 AM
> Subject: Re: [ovirt-devel] Correct way to run internal  actions from commands
> 
> 
> 
> ----- Original Message -----
> > From: "Gilad Chaplik" <gchaplik at redhat.com>
> > To: "Yair Zaslavsky" <yzaslavs at redhat.com>
> > Cc: devel at ovirt.org
> > Sent: Thursday, June 19, 2014 11:14:35 AM
> > Subject: Re: [ovirt-devel] Correct way to run internal  actions from
> > commands
> > 
> > We should enforce that in code level: getBackend() should be private in
> > CommandBase, and won't need any convention.
> 
> Using getBackend() is also a "convention".

so you're adding another one :-)

how should I enforce not using getBackend().runXXX in reviews? I've probably need to remember your email...

> 
> 
> > 
> > Thanks,
> > Gilad.
> > 
> > 
> > ----- Original Message -----
> > > From: "Yair Zaslavsky" <yzaslavs at redhat.com>
> > > To: devel at ovirt.org
> > > Sent: Wednesday, June 18, 2014 8:44:41 AM
> > > Subject: [ovirt-devel] Correct way to run internal  actions from commands
> > > 
> > > Hi all,
> > > Thanks to the collaboration of some of the maintainers and others I have
> > > managed to merge a patch which gives the ability to run internal commands
> > > the following way:
> > > 
> > > If you're writing a code in a class that extends CommandBase, please
> > > follow
> > > the following instructions:
> > > 
> > > Instead of Backend.getInstance().runInternalAction (or MultipleActions)
> > > you
> > > should use
> > > 
> > > runInternalAction
> > > 
> > > More patches will be added for running internal queries , and probably
> > > endAction as well.
> > > 
> > > 
> > > Thanks,
> > > Yair
> > > 
> > > 
> > > _______________________________________________
> > > Devel mailing list
> > > Devel at ovirt.org
> > > http://lists.ovirt.org/mailman/listinfo/devel
> > > 
> > 
>