test_show_quota_usage uses admin tenant name, it should be admin tenant id

Bug #1321617 reported by Harshada Mangesh Kakad
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tempest
Fix Released
Undecided
Harshada Mangesh Kakad

Bug Description

Test case related to quota usage (test_show_quota_usage) uses tenant name, ideally it should use tenant id as quota requires UUID of tenant and not tenant name.

Changed in tempest:
assignee: nobody → Harshada Mangesh Kakad (harshada-kakad)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tempest (master)

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

Changed in tempest:
status: New → In Progress
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tempest (master)

Reviewed: https://review.openstack.org/95087
Committed: https://git.openstack.org/cgit/openstack/tempest/commit/?id=7519a52761a7ad4ea6db87b66aaaad826b1fbcfc
Submitter: Jenkins
Branch: master

commit 7519a52761a7ad4ea6db87b66aaaad826b1fbcfc
Author: Harshada Mangesh Kakad <email address hidden>
Date: Wed Nov 5 22:52:58 2014 -0800

    quota testcases use tenant_name should use tenant_id

    Test case related to quota usage (test_show_quota_usage,
    test_cinder_quota_defaults and test_cinder_quota_show)
    uses tenant name, ideally it should use tenant id as quota
    requires UUID of tenant and not tenant name.

    Cinder quota-show ideally requires tenant_id to show quota.
    As there is bug in cinder (BUG: 1307475) , that if we give
    non-existent tenant_id still cinder show the quota and
    returns 200 (OK). Hence, these testcases should use
    tenant_id and not tenant_name.

    Change-Id: Icab25afd482b34290bc74be2ba420f6b016dde18
    Closes-Bug: #1321617

Changed in tempest:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tempest (master)

Change abandoned by Sean Dague (<email address hidden>) on branch: master
Review: https://review.openstack.org/132954
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers