Merge lp:~launchpa6/charm-helpers/fix-lstrip-in-apachesslcontext into lp:charm-helpers

Proposed by John McEleney
Status: Needs review
Proposed branch: lp:~launchpa6/charm-helpers/fix-lstrip-in-apachesslcontext
Merge into: lp:charm-helpers
Diff against target: 11 lines (+1/-1)
1 file modified
charmhelpers/contrib/openstack/context.py (+1/-1)
To merge this branch: bzr merge lp:~launchpa6/charm-helpers/fix-lstrip-in-apachesslcontext
Reviewer Review Type Date Requested Status
charmers Pending
Review via email: mp+289629@code.launchpad.net

Description of the change

Fix for bug #1559988. Use array slice instead of lstrip to determine canonical name.

To post a comment you must log in.

Unmerged revisions

553. By John McEleney

Fix for bug #1559988. Use array slice instead of lstrip to determine canonical name.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'charmhelpers/contrib/openstack/context.py'
2--- charmhelpers/contrib/openstack/context.py 2016-01-29 10:20:29 +0000
3+++ charmhelpers/contrib/openstack/context.py 2016-03-21 11:39:38 +0000
4@@ -755,7 +755,7 @@
5 rdata = relation_get(rid=r_id, unit=unit)
6 for k in rdata:
7 if k.startswith('ssl_key_'):
8- cns.append(k.lstrip('ssl_key_'))
9+ cns.append(k[8:])
10
11 return sorted(list(set(cns)))
12

Subscribers

People subscribed via source and target branches