Merge lp://qastaging/~niedbalski/charm-helpers/lp1667478-2 into lp://qastaging/charm-helpers
Proposed by
Jorge Niedbalski
Status: | Merged |
---|---|
Merged at revision: | 699 |
Proposed branch: | lp://qastaging/~niedbalski/charm-helpers/lp1667478-2 |
Merge into: | lp://qastaging/charm-helpers |
Diff against target: |
199 lines (+93/-23) 2 files modified
charmhelpers/contrib/openstack/keystone.py (+45/-11) tests/contrib/openstack/test_keystone_utils.py (+48/-12) |
To merge this branch: | bzr merge lp://qastaging/~niedbalski/charm-helpers/lp1667478-2 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Shane Peters (community) | Approve | ||
Felipe Reyes (community) | Approve | ||
Review via email: mp+318505@code.qastaging.launchpad.net |
Description of the change
This commit adds the following changes:
1) Support to load a keystone manager from a identityservice context.
2) The service_exists function ensures that service_type and service_name are
separated terms to search by.
To post a comment you must log in.
LGTM! thanks jorge