ec2 credentials uses project id when listing

Bug #1292337 reported by Steve Martinelli
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-openstackclient
Fix Released
Undecided
Chris Johnson

Bug Description

When listing ec2 credentials, project_id is used instead of tenant_id

openstack@plwdevstack:/opt/stack/devstack$ openstack ec2 credentials create
+-----------+----------------------------------+
| Field | Value |
+-----------+----------------------------------+
| access | 487f5721a29049e2aaff1161f61e2e68 |
| secret | b66c8f8967534bcba820c94f64aa49f9 |
| tenant_id | e32775574ea64a14a89e0666f1105901 |
| trust_id | None |
| user_id | 37fd174bf672427fb4b0921db236d606 |
+-----------+----------------------------------+
openstack@plwdevstack:/opt/stack/devstack$ openstack ec2 credentials list
+----------------------------------+----------------------------------+------------+----------------------------------+
| Access | Secret | Project ID | User ID |
+----------------------------------+----------------------------------+------------+----------------------------------+
| 487f5721a29049e2aaff1161f61e2e68 | b66c8f8967534bcba820c94f64aa49f9 | | 37fd174bf672427fb4b0921db236d606 |
| cc896d1dc4664300a61c5d0f9cc03002 | b29bf4cfbc8f478fa70485b6ac908877 | | 37fd174bf672427fb4b0921db236d606 |
+----------------------------------+----------------------------------+------------+----------------------------------+
openstack@plwdevstack:/opt/stack/devstack$ keystone ec2-credentials-list
+--------+----------------------------------+----------------------------------+
| tenant | access | secret |
+--------+----------------------------------+----------------------------------+
| admin | 487f5721a29049e2aaff1161f61e2e68 | b66c8f8967534bcba820c94f64aa49f9 |
| admin | cc896d1dc4664300a61c5d0f9cc03002 | b29bf4cfbc8f478fa70485b6ac908877 |
+--------+----------------------------------+----------------------------------+

Revision history for this message
Terry Howe (thowe-g) wrote :

So, you are saying the header on the list should be "Tenant ID" rather than "Project ID", correct.

Unrelated, but I'm kind of surprised the command isn't

    os credentials create --type ec2

Revision history for this message
Steve Martinelli (stevemar) wrote :

I think the header should stay Project Id, but we should send it the tenant id.

Reference:
https://github.com/openstack/python-openstackclient/blob/master/openstackclient/identity/v2_0/user.py#L169:L182

Changed in python-openstackclient:
assignee: nobody → Chris Johnson (wchrisjohnson)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-openstackclient (master)

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

Changed in python-openstackclient:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-openstackclient (master)

Reviewed: https://review.openstack.org/82125
Committed: https://git.openstack.org/cgit/openstack/python-openstackclient/commit/?id=8117256d7dc1464e86382db5c480d33b01131854
Submitter: Jenkins
Branch: master

commit 8117256d7dc1464e86382db5c480d33b01131854
Author: Chris Johnson <email address hidden>
Date: Fri Mar 21 11:16:25 2014 -0400

    Correct display of project/tenant id on display of credentials

    This change corrects the display of ec2 credentails within the
    ListEC2Creds method. Added explicit headers and corrected data listt o
    specify tenant_id instead of project id.

    Change-Id: I2ea579082bee800d774f202bdc38e2d546e57e77
    Closes-Bug: #1292337

Changed in python-openstackclient:
status: In Progress → Fix Committed
Dean Troyer (dtroyer)
Changed in python-openstackclient:
milestone: none → m4
Dean Troyer (dtroyer)
Changed in python-openstackclient:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.