Merge lp:~1chb1n/charm-helpers/os-amulet-keystone-ip into lp:charm-helpers

Proposed by Ryan Beisner
Status: Merged
Merged at revision: 641
Proposed branch: lp:~1chb1n/charm-helpers/os-amulet-keystone-ip
Merge into: lp:charm-helpers
Diff against target: 27 lines (+4/-6)
1 file modified
charmhelpers/contrib/openstack/amulet/utils.py (+4/-6)
To merge this branch: bzr merge lp:~1chb1n/charm-helpers/os-amulet-keystone-ip
Reviewer Review Type Date Requested Status
James Page Approve
Review via email: mp+307510@code.launchpad.net

Description of the change

Update OpenStack Amulet helper to get keystone address from sentry info instead of relation data

To post a comment you must log in.
Revision history for this message
James Page (james-page) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'charmhelpers/contrib/openstack/amulet/utils.py'
2--- charmhelpers/contrib/openstack/amulet/utils.py 2016-09-11 03:28:39 +0000
3+++ charmhelpers/contrib/openstack/amulet/utils.py 2016-10-04 04:46:39 +0000
4@@ -306,10 +306,8 @@
5 password, tenant):
6 """Authenticates admin user with cinder."""
7 # NOTE(beisner): cinder python client doesn't accept tokens.
8- service_ip = \
9- keystone_sentry.relation('shared-db',
10- 'mysql:shared-db')['private-address']
11- ept = "http://{}:5000/v2.0".format(service_ip.strip().decode('utf-8'))
12+ keystone_ip = keystone_sentry.info['private-address']
13+ ept = "http://{}:5000/v2.0".format(keystone_ip.strip().decode('utf-8'))
14 return cinder_client.Client(username, password, tenant, ept)
15
16 def authenticate_keystone_admin(self, keystone_sentry, user, password,
17@@ -319,8 +317,8 @@
18 self.log.debug('Authenticating keystone admin...')
19 unit = keystone_sentry
20 if not keystone_ip:
21- keystone_ip = unit.relation('shared-db',
22- 'mysql:shared-db')['private-address']
23+ keystone_ip = keystone_sentry.info['private-address']
24+
25 base_ep = "http://{}:35357".format(keystone_ip.strip().decode('utf-8'))
26 if not api_version or api_version == 2:
27 ep = base_ep + "/v2.0"

Subscribers

People subscribed via source and target branches