[ovirt-devel] Annotating internal commands

Yair Zaslavsky yzaslavs at redhat.com
Wed Jun 25 10:08:35 UTC 2014



----- Original Message -----
> From: "Omer Frenkel" <ofrenkel at redhat.com>
> To: "Yair Zaslavsky" <yzaslavs at redhat.com>
> Cc: devel at ovirt.org
> Sent: Wednesday, June 25, 2014 10:53:16 AM
> Subject: Re: [ovirt-devel] Annotating internal commands
> 
> 
> 
> ----- Original Message -----
> > From: "Yair Zaslavsky" <yzaslavs at redhat.com>
> > To: devel at ovirt.org
> > Sent: Wednesday, June 25, 2014 6:19:35 AM
> > Subject: [ovirt-devel] Annotating internal commands
> > 
> > Hi,
> > Some of you already got or will get emails from me asking if several
> > commands
> > are only internal or not.
> > I basically check usages at rest-api and ui, but I might have missed
> > something.
> > 
> > IMHO, we should continue (until we device not to :) ) to annotate internal
> > commands with the
> > 
> > @InternalCommandAttribute
> > 
> 
> amm, the name of this attribute might be misleading,
> this attribute, afair, just make the infra skip audit log of the command.
> although it makes sense that any command that a user can run will not have
> this, because we want to audit log any action user do,
> but there might be some cases where command that is called only internally
> does need to be logged, and then you cannot use it..
> (i didnt look for examples, don't know if there are any)

Thanks Omer, I'll review once again, If this is the case - the annotation name is somewhat misleading, of we should have an attribute that will mark if to audit log or not.
What do you think?

> 
> > 
> > Thanks,
> > Yair
> > _______________________________________________
> > Devel mailing list
> > Devel at ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/devel
> > 
> 



More information about the Devel mailing list