Instance name not shown on attached volume table row

[Tihomir Trifonov]

http://db.tt/invodfcQ

Under Instances & Volumes, if a volume is attached to instance, under "Attachments" the instance is displayed as ID only, and it is not obvious which instance is that, you have to compare instance IDs to understand.

Maybe the "Attachments" column should show something like :

Instance "name" (xxxxxxxx-xxxx-xxxx....) as it id displayed in volume details page.

[Jim Yeh (lemonlatte)]

Can a volume attach on some where other than instance?

If not, the label may change to "Attached Instance". Then, each volumes just display "name" (xxxxxxxx-xxxx-xxxx....). I think it is simple and clear.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/6859
Committed: http://github.com/openstack/horizon/commit/d89d83cf2d96b910170b6e620b6a87540cce705c
Submitter: Jenkins
Branch: master

commit d89d83cf2d96b910170b6e620b6a87540cce705c
Author: Tihomir Trifonov <email address hidden>
Date: Wed Apr 11 15:44:18 2012 +0300

    Added instance name for volume attachment

    Fixes bug 974916.

    The instance is displayed in the same way
    as it is in volume detail page.

    Change-Id: I33ee44befa846767d5556bc56695306951414dc2

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/7729
Committed: http://github.com/openstack/horizon/commit/9f581a80ef61405a5a383273774a70a86abc3030
Submitter: Jenkins
Branch: stable/essex

commit 9f581a80ef61405a5a383273774a70a86abc3030
Author: Tihomir Trifonov <email address hidden>
Date: Wed Apr 11 15:44:18 2012 +0300

    Added instance name for volume attachment

    Fixes bug 974916.

    The instance is displayed in the same way
    as it is in volume detail page.

    Change-Id: I33ee44befa846767d5556bc56695306951414dc2

[Rafi Khardalian (rkhardalian)]

The stable/essex version of this commit should probably be reverted. This commit breaks the instances and volumes page. Volumes are not properly scoped for admin users, where instances are. Thus, if you're opening the page as an admin, viewing instance->volume attachments created by anyone else, you're going to be lacking the data you need via the instances object.

I have it patched but am lacking the time to get it through the review process.