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
=== modified file 'charmhelpers/contrib/amulet/utils.py'
--- charmhelpers/contrib/amulet/utils.py 2016-10-13 13:28:00 +0000
+++ charmhelpers/contrib/amulet/utils.py 2016-12-12 13:30:30 +0000
@@ -148,7 +148,8 @@
148148
149 for service_name in services_list:149 for service_name in services_list:
150 if (self.ubuntu_releases.index(release) >= systemd_switch or150 if (self.ubuntu_releases.index(release) >= systemd_switch or
151 service_name in ['rabbitmq-server', 'apache2']):151 service_name in ['rabbitmq-server', 'apache2',
152 'memcached']):
152 # init is systemd (or regular sysv)153 # init is systemd (or regular sysv)
153 cmd = 'sudo service {} status'.format(service_name)154 cmd = 'sudo service {} status'.format(service_name)
154 output, code = sentry_unit.run(cmd)155 output, code = sentry_unit.run(cmd)
155156
=== modified file 'charmhelpers/contrib/openstack/amulet/utils.py'
--- charmhelpers/contrib/openstack/amulet/utils.py 2016-12-07 10:34:35 +0000
+++ charmhelpers/contrib/openstack/amulet/utils.py 2016-12-12 13:30:30 +0000
@@ -1177,9 +1177,14 @@
1177 '/etc/memcached.conf')1177 '/etc/memcached.conf')
1178 contents = self.file_contents_safe(sentry_unit, '/etc/memcached.conf',1178 contents = self.file_contents_safe(sentry_unit, '/etc/memcached.conf',
1179 fatal=True)1179 fatal=True)
1180 ubuntu_release, _ = self.run_cmd_unit(sentry_unit, 'lsb_release -cs')
1181 if ubuntu_release <= 'trusty':
1182 memcache_listen_addr = 'ip6-localhost'
1183 else:
1184 memcache_listen_addr = '::1'
1180 expected = {1185 expected = {
1181 '-p': '11211',1186 '-p': '11211',
1182 '-l': '::1'}1187 '-l': memcache_listen_addr}
1183 found = []1188 found = []
1184 for key, value in expected.items():1189 for key, value in expected.items():
1185 for line in contents.split('\n'):1190 for line in contents.split('\n'):

Subscribers

People subscribed via source and target branches