]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Code Review / openstack-build / cinder-build.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 30cd678) | patch
Fix backup list all_tenants=0 filtering for admin
authorGorka Eguileor <geguileo@redhat.com>
Tue, 18 Aug 2015 09:29:15 +0000 (11:29 +0200)
committerGorka Eguileor <geguileo@redhat.com>
Tue, 25 Aug 2015 11:01:28 +0000 (13:01 +0200)
commit34101027c6fea35f8acbc0463a7e6c95f6dfeb66
tree1b2a9fa2ea3f7e02c77d799f4aa3be4d24eeb1e6tree | snapshot
parent30cd678b57a5d8d87f06b2dc9a2707e04b1b806ecommit | diff
Fix backup list all_tenants=0 filtering for admin

When a user with admin role lists backups specifying all_tenants=0 in
the request, it will not behave as expected, it will return backups for
all tenants just as if all_tenants=1 had been specified.

This patch fixes this issue and will only return all tenants in backup
listings if all_tenants is set to a value meaning True (1, true, yes,
y).

Change-Id: Ia74bc258a86f6dc48483dfe17bf5a312aec4058d
Closes-Bug: #1485900
cinder/backup/api.py diff | blob | history
cinder/tests/unit/test_backup.py diff | blob | history