Comment 2 for bug 1671220

Revision history for this message
prashkre (prashkre) wrote :

wangxiyuan, I have reproduced this issue with user as admin in proj1. If we look at lines [0], [1] same 'filters' object is being used at both places and this 'filters' object is updated on call from [0] to [2] by deleting 'all_tenants' entry from it. So when a call is made from [1] to [3] in the next flow, it doesn't have 'all_tenants' in filters even though user is admin then execution goes to else block at [4].

[0] https://github.com/openstack/cinder/blob/master/cinder/api/contrib/consistencygroups.py#L138
[1] https://github.com/openstack/cinder/blob/master/cinder/api/contrib/consistencygroups.py#L142

[2] https://github.com/openstack/cinder/blob/master/cinder/consistencygroup/api.py#L711
[3] https://github.com/openstack/cinder/blob/master/cinder/group/api.py#L760
[4] https://github.com/openstack/cinder/blob/master/cinder/group/api.py#L774