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

Gilad Chaplik gchaplik at redhat.com
Thu Jun 19 09:43:25 UTC 2014


----- Original Message -----
> From: "Alon Bar-Lev" <alonbl at redhat.com>
> To: "Gilad Chaplik" <gchaplik at redhat.com>
> Cc: "Yair Zaslavsky" <yzaslavs at redhat.com>, devel at ovirt.org
> Sent: Thursday, June 19, 2014 11:41:37 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:33:19 AM
> > Subject: Re: [ovirt-devel] Correct way to run internal actions from
> > commands
> > 
> > ----- 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...
> 
> Soon you will probably different signature for the helper runXXX() and the
> getBackend().runXXX()[1], as we are adding command context inheritance, to
> enable proper security context, correlation id inheritance and remove the
> ThreadLocal usage.
> 
> So it will be easier to remember... :)

don't complain about the quality :-), that's what I've found:
https://www.youtube.com/watch?v=Qj7WyfceJ58

Thanks, 
Gilad.

> 
> [1] http://gerrit.ovirt.org/#/c/28829/
> 
> > 
> > > 
> > > 
> > > > 
> > > > 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
> > > > > 
> > > > 
> > > 
> > _______________________________________________
> > Devel mailing list
> > Devel at ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/devel
> > 
> 



More information about the Devel mailing list