Merge lp:~gnuoy/charm-helpers/memcache-test-fixes into lp:charm-helpers

Proposed by Liam Young
Status: Merged
Merged at revision: 667
Proposed branch: lp:~gnuoy/charm-helpers/memcache-test-fixes
Merge into: lp:charm-helpers
Diff against target: 33 lines (+8/-2)
2 files modified
charmhelpers/contrib/amulet/utils.py (+2/-1)
charmhelpers/contrib/openstack/amulet/utils.py (+6/-1)
To merge this branch: bzr merge lp:~gnuoy/charm-helpers/memcache-test-fixes
Reviewer Review Type Date Requested Status
Alex Kavanagh Approve
charmers Pending
Review via email: mp+313014@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alex Kavanagh (ajkavanagh) wrote :

LGTM.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'charmhelpers/contrib/amulet/utils.py'
2--- charmhelpers/contrib/amulet/utils.py 2016-10-13 13:28:00 +0000
3+++ charmhelpers/contrib/amulet/utils.py 2016-12-12 13:30:30 +0000
4@@ -148,7 +148,8 @@
5
6 for service_name in services_list:
7 if (self.ubuntu_releases.index(release) >= systemd_switch or
8- service_name in ['rabbitmq-server', 'apache2']):
9+ service_name in ['rabbitmq-server', 'apache2',
10+ 'memcached']):
11 # init is systemd (or regular sysv)
12 cmd = 'sudo service {} status'.format(service_name)
13 output, code = sentry_unit.run(cmd)
14
15=== modified file 'charmhelpers/contrib/openstack/amulet/utils.py'
16--- charmhelpers/contrib/openstack/amulet/utils.py 2016-12-07 10:34:35 +0000
17+++ charmhelpers/contrib/openstack/amulet/utils.py 2016-12-12 13:30:30 +0000
18@@ -1177,9 +1177,14 @@
19 '/etc/memcached.conf')
20 contents = self.file_contents_safe(sentry_unit, '/etc/memcached.conf',
21 fatal=True)
22+ ubuntu_release, _ = self.run_cmd_unit(sentry_unit, 'lsb_release -cs')
23+ if ubuntu_release <= 'trusty':
24+ memcache_listen_addr = 'ip6-localhost'
25+ else:
26+ memcache_listen_addr = '::1'
27 expected = {
28 '-p': '11211',
29- '-l': '::1'}
30+ '-l': memcache_listen_addr}
31 found = []
32 for key, value in expected.items():
33 for line in contents.split('\n'):

Subscribers

People subscribed via source and target branches