Merge lp:~aisrael/charms/trusty/memcached/noipv6 into lp:charms/trusty/memcached

[Adam Israel]

The previous revision synced charm-helpers to enable detection of (or lack thereof) ipv6 networking. This merge proposal adds the soft_fail=True param to ufw.enable() so that, if ipv6 is not available, the installation continues using only ipv4 instead of failing with `charmhelpers.contrib.network.ufw.UFWIPv6Error: IPv6 firewall support broken`.

[Charles Butler (lazypower)]

Greetings Adam,

This MP appears to have gone a bit stale with the charm and does not apply cleanly anymore. Can you give this another look and re-open once you've validated this is still an applicable fix?

Thanks!

review: Needs Fixing

[Kevin W Monroe (kwmonroe)]

This may be moot. memcached now has an allow-ufw-ip6-softfail config option:

http://bazaar.launchpad.net/~charmers/charms/trusty/memcached/trunk/revision/67#config.yaml

Setting that to True when deploying with the local provider should allow for successful deployments to lxc.

Unmerged revisions

65. By Adam Israel on 2015-02-13

Turn on ufw's soft_fail, so install doesn't fail if ipv6 isn't available

=== modified file 'hooks/memcached_hooks.py'
--- hooks/memcached_hooks.py	2014-12-19 18:24:29 +0000
+++ hooks/memcached_hooks.py	2015-02-13 19:57:49 +0000
@@ -53,7 +53,7 @@
     with open(ETC_DEFAULT_MEMCACHED, 'w') as f:
         f.write('ENABLE_MEMCACHED=yes\n')
 
-    ufw.enable()
+    ufw.enable(soft_fail=True)
     ufw.service('ssh', 'open')