commit 3ae247fdceac0e2c7bb6160f9ffefc4ad5d8da29
Author: Jamie Lennox <email address hidden>
Date: Sun May 31 13:37:50 2015 +1000
Set tenant options on parsed namespace
Because of the way OSC registers all plugins together we end up
with os-tenant-X parameters being saved to the project-X attribute after
parsing. If you are using the v2 plugins directly then they and os-client-config
expect the tenant_X values and will assuming no scoping information if
they are not present.
Validating options for scope will also fail in this situation, not just
because the resultant auth dictionary is missing the tenant-X
attributes, but because OSC validates that either project or domain
scope information is present.
Fix this by just always setting the v2 parameters if the v3 parameters
are present. This will have no effect on the generic or v3 case but fix
the v2 case.
Expand validation to include the tenant options so it knows that v2
plugins are scoped.
Reviewed: https:/ /review. openstack. org/187033 /git.openstack. org/cgit/ openstack/ python- openstackclient /commit/ ?id=3ae247fdcea c0e2c7bb6160f9f fefc4ad5d8da29
Committed: https:/
Submitter: Jenkins
Branch: master
commit 3ae247fdceac0e2 c7bb6160f9ffefc 4ad5d8da29
Author: Jamie Lennox <email address hidden>
Date: Sun May 31 13:37:50 2015 +1000
Set tenant options on parsed namespace
Because of the way OSC registers all plugins together we end up
with os-tenant-X parameters being saved to the project-X attribute after
parsing. If you are using the v2 plugins directly then they and os-client-config
expect the tenant_X values and will assuming no scoping information if
they are not present.
Validating options for scope will also fail in this situation, not just
because the resultant auth dictionary is missing the tenant-X
attributes, but because OSC validates that either project or domain
scope information is present.
Fix this by just always setting the v2 parameters if the v3 parameters
are present. This will have no effect on the generic or v3 case but fix
the v2 case.
Expand validation to include the tenant options so it knows that v2
plugins are scoped.
Change-Id: I8cab3e423663f8 01cbf2d83106c67 1bddc58d7e6
Closes-Bug: #1460369