It helps to understand the original intent. But from an API point of view (or new user point of view) it is confusing.
I don't know if this is consistent with other commands. Is this the only nova command where you have to indicate via an option, whether you want to operate as admin or not? (And if that's really what is desired, maybe something like 'as_admin' would be clearer, but then you'd still need something to indicate "from all tenants".). The policy/rules stuff should take care of whether you have access to a command/feature or not. 'all tenants' to me, means *all tenants*. And 'tenant' means a specific tenant. It seems like specifying both is conflicting (do you want one tenant, or all tenants?) I'd actually prefer 'tenant=*' instead of 'all tenants' but I digress ;) --ruby -----Original Message----- From: Chris Behrens <cbehr...@codestud.com> Reply-To: OpenStack Development Mailing List <openstack-dev@lists.openstack.org> Date: Monday, June 17, 2013 1:12 PM To: OpenStack Development Mailing List <openstack-dev@lists.openstack.org> Subject: Re: [openstack-dev] Nova list with all-tenants flag > > >The original intent behind 'all_tenants=1' was whether or not to include >all instances for all tenants in the filtering process. Basically it >means "Should I operate as admin or not?". I still view it this way, and >would prefer it still mean this (although > it might be more clear with a different name). > > >This means that if you're an adminŠ you're not treated as one UNTIL you >specify all_tenants=1. So technically I think specifying a 'tenant_id' >filter should raise or do whatever it does for a normal user. I don't >think specifying a 'tenant_id' filter > should mean that we automatically turn 'all_tenants' on. > > >I don't see the bug deal in needing to remember to use all_tenants=1 so >that all instances will be filtered, not just the instances belong to the >admin user. > > >My opinion, > > >- Chris > > > > >On Jun 16, 2013, at 11:09 PM, Aarti Mahesh Kriplani ><aarti.kripl...@rackspace.com> wrote: > > >Hello all, > > >There have been a few contrasting views on this bug ><https://bugs.launchpad.net/nova/+bug/1185290><https://bugs.launchpad.net/ >nova/+bug/1185290>. >I would really like some views on how we want to use the all-tenenats >flag going ahead. >Comments/Suggestions? > > >Thanks, >Aarti >_______________________________________________ >OpenStack-dev mailing list >OpenStack-dev@lists.openstack.org >http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev > > > > _______________________________________________ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
