Don't list every tenant in ec2 credentials form

Bug #982602 reported by Cole Robinson on 2012-04-15
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Undecided
Unassigned

Bug Description

Currently horizon seems to list every available tenant in the ec2 and openstack cred download forms. Seems like a good idea to limit those lists only to tenants associated with the logged in user.

Cole Robinson (crobinso) wrote :

This 'openstack' creds screen also blows up for a non-admin user due to the way the tenant lists are fetched, so this change would fix that issue as well.

Andy Chong (andycjw) on 2012-04-18
Changed in horizon:
status: New → Confirmed
assignee: nobody → Andy Chong (andycjw)
Andy Chong (andycjw) wrote :

horizon only list all available tenant associated with logged in user, I can't reproduce the problem reported here.
even for admin user it will only list projects associated to it, it's the same for non-admin as well

Changed in horizon:
status: Confirmed → New
status: New → Incomplete
assignee: Andy Chong (andycjw) → nobody
Cole Robinson (crobinso) wrote :

Andy, not sure why you can't see it, but it's definitely an issue with fedora essex packages. Could be a local problem I guess.

Either way, I submitted a patch that fixes it for me:

https://review.openstack.org/#/c/6580/

Devin Carlen (devcamcar) on 2012-04-23
tags: added: essex-backport-potential
Devin Carlen (devcamcar) on 2012-05-01
Changed in horizon:
milestone: none → folsom-1
Devin Carlen (devcamcar) wrote :

Cole, your review expired: https://review.openstack.org/#/c/6580/

Can you address Gabriel's comments and resubmit?

Gabriel Hurley (gabriel-hurley) wrote :

Bumping from milestone due to inactivity.

Changed in horizon:
milestone: folsom-1 → folsom-2
Thierry Carrez (ttx) on 2012-06-26
Changed in horizon:
milestone: folsom-2 → none
Mark McLoughlin (markmc) on 2012-07-24
no longer affects: horizon/essex
Launchpad Janitor (janitor) wrote :

[Expired for OpenStack Dashboard (Horizon) because there has been no activity for 60 days.]

Changed in horizon:
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers