quota-usage lists information with wrong tenant-id

[Tushar Bankar]

cinder quota-usage wrong-tenantid

cinder returns with 200 Ok with usage of quota.

But it is expected 404 Not Found

[Tushar Bankar (tushar-bankar)]

Response:

 cinder quota-usage wrong-tenantid
+-----------+--------+----------+-------+
| Type | In_use | Reserved | Limit |
+-----------+--------+----------+-------+
| gigabytes | 0 | 0 | 1000 |
| snapshots | 0 | 0 | 10 |
| volumes | 0 | 0 | 10 |
+-----------+--------+----------+-------+

[John Griffith (john-griffith)]

Same opinion as 1307475, 1307486. Should fix these up but certainly not show-stoppers for RC

[yatin (yatinkarel)]

The api does not even validate the uuid, if tenant-id is passed in non-uuid format, it does not raise an InvlidUUID exception.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Change abandoned by Mike Perez (<email address hidden>) on branch: master
Review: https://review.openstack.org/133148
Reason: no activity. also see https://review.openstack.org/#/c/143645/

[Huang Zhiteng (zhiteng-huang)]

Is it really necessary to verify project id for read only operations like this?

[OpenStack Infra (hudson-openstack)]

Change abandoned by Anton Arefiev (<email address hidden>) on branch: master
Review: https://review.openstack.org/143645
Reason: More general approach is needed here

[Varun (varun-utagikar)]

Hey

We have a Kilo deployment. I can see that 'cinder quota-usage' still shows empty 'in-use' column but i can see right values in database.

Should I file a new bug or please let me know if this issue is being sorted out in any existing bug.

Varun

[Michal Dulko (michal-dulko-f)]

@Varun: It seems like the issue you're describing isn't really related to this bugreport.

[Yuriy Nesenenko (ynesenenko)]

The same incorrect results appeared after the execution of the following commands.
$ cinder quota-usage invalid-tenantid
$ cinder quota-show invalid_tenantid
$ cinder quota-defaults invalid_tenantid
+----------------------+-------+
| Property | Value |
+----------------------+-------+
| backup_gigabytes | 1000 |
| backups | 10 |
| gigabytes | 1000 |
| gigabytes_ceph | -1 |
| per_volume_gigabytes | -1 |
| snapshots | 10 |
| snapshots_ceph | -1 |
| volumes | 10 |
| volumes_ceph | -1 |
+----------------------+-------+

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Change abandoned by Yuriy Nesenenko (<email address hidden>) on branch: master
Review: https://review.openstack.org/315673