Volumes, volume snapshots, instance snaphots and keypairs all show cross-tenant info when logged in as admin

This has been around for a while, but it certainly doesn't make sense from a UX point of view.

This should be moved to a blueprint.

Registered: https://blueprints.launchpad.net/horizon/+spec/admin-tenant-volumes-snapshots

Feel free to update description if that doesn't make sense.

Also at https://blueprints.launchpad.net/nova/+spec/differentiate-admin

@Thierry: This is not a "wishlist" item. This is a serious flaw in the way Nova presents information and behaves differently in some places if an admin flag is present.

To put this in perspective, here is the banner we are having to add to Horizon to warn users of the broken behavior:

http://i.imgur.com/eS2sV.png

The blueprint you linked is marked Medium priority so I updated this bug to reflect that.

Fix proposed to branch: master
Review: https://review.openstack.org/10750

Reviewed: https://review.openstack.org/10750
Committed: http://github.com/openstack/cinder/commit/a1b4bd5e86cd865869308c976c6aebf9a4ad23a8
Submitter: Jenkins
Branch: master

commit a1b4bd5e86cd865869308c976c6aebf9a4ad23a8
Author: jakedahn <email address hidden>
Date: Tue Jul 31 16:35:26 2012 -0700

    Admin users should be restricted from seeing all
    volumes by default.

      * Now to view all volumes across all tenants you need
        to include the all_tenants=1 GET param in your api request.
      * Fixes remaining issues blocking bug #967882

    Change-Id: Ie9d74e9c09fa0c192ab6257b5fb02d65b593cbfb

Fix proposed to branch: master
Review: https://review.openstack.org/10799

Fix proposed to branch: master
Review: https://review.openstack.org/10855

Reviewed: https://review.openstack.org/10855
Committed: http://github.com/openstack/cinder/commit/43626d293739dbe06c7fe26845d17c2e580e62dd
Submitter: Jenkins
Branch: master

commit 43626d293739dbe06c7fe26845d17c2e580e62dd
Author: jakedahn <email address hidden>
Date: Sun Aug 5 14:32:07 2012 -0700

    Admin users should be restricted from seeing all
    snapshots by default.

      * Now to view all snapshots across all tenants you need
        to include the all_tenants=1 GET param in your api request.
      * Fixes remaining issues blocking bug #967882

    Change-Id: I2a8338d77badc70201bb315198183f2091df43fb

Fix proposed to branch: master
Review: https://review.openstack.org/11107

Reviewed: https://review.openstack.org/11107
Committed: http://github.com/openstack/nova/commit/8077486b3c15012f4dbf270cd8c9fa3f48cb3d36
Submitter: Jenkins
Branch: master

commit 8077486b3c15012f4dbf270cd8c9fa3f48cb3d36
Author: jakedahn <email address hidden>
Date: Thu Aug 9 14:28:28 2012 -0700

    Default behavior should restrict admins to tenant for volumes.

      * NOTE: This is a port from cinder to nova volumes
      * Now to view all volumes or volume snapshots across
        all tenants you need to include the all_tenants=1
        GET param in your api request.
      * Fixes remaining issues blocking bug #967882

    Change-Id: I7fe15e792b62e59973c7faa2cf1c52929ae5864f

Needs confirmation that the downstream fixes take care of the horizon issues. If so, we can resolve this bug by removing the warning banner from the project dashboard.

The only thing that has yet to be merged is the novaclient patch, which can be found here: https://review.openstack.org/#/c/11113/, not sure why it hasn't been pushed through - probably just need to nag a few people.

But the changes for Cinder, Nova Volumes, Cinderclient, and an issue that popped up with horizon have all been merged.

So, this is fixed for all the items listed here, except it's not fixed for security groups, which weren't listed here... I've opened a new bug to track that since this one is complete as written. New bug here: https://bugs.launchpad.net/nova/+bug/1046054

Great work on getting all these fixed, though!

Fix proposed to branch: stable/essex
Review: https://review.openstack.org/13750