Error decoding '/etc/consul.conf': invalid character ']' looking for beginning of value about ansible-consul

[MrMMorris]

it seems to only happen on the server that is last in the list. The other 2 are fine.

Pinging @Suvitruf who made the change

[echupriyanov]

Quick patch:

eric@iMac-Eric /V/D/W/s/a/c/r/ansible-consul> git diff templates/consul.json.j2 
diff --git a/templates/consul.json.j2 b/templates/consul.json.j2
index 84de291..d8df6a2 100644
--- a/templates/consul.json.j2
+++ b/templates/consul.json.j2
@@ -3,7 +3,7 @@
   "ui_dir": "{{ consul_ui_dir }}",
 {% endif %}
 {% if consul_join_at_start is defined and consul_join_at_start %}
-  "start_join": [{% for host in consul_servers %}{% if host not in ansible_all_ipv4_addresses %}"{{host}}"{% if not loop.last %}, {% endif %}{% endif %}{% endfor %}],
+  "start_join": [{% for host in consul_servers %}"{{host}}"{% if not loop.last %}, {% endif %}{% endfor %}],
 {% endif %}
 {% if consul_disable_remote_exec is defined %}
   "disable_remote_exec": {{ "true" if consul_disable_remote_exec else "false" }},
eric@iMac-Eric /V/D/W/s/a/c/r/ansible-consul> 

Fixes this particular issue

[MrMMorris]

merging spree! thanks @savagegus 😄

[mattfinlayson]

👍

On Thu, Jan 14, 2016 at 1:58 PM, Marcus Morris [email protected]
wrote:

merging spree! thanks @savagegus https://github.com/savagegus [image:
😄]

—
Reply to this email directly or view it on GitHub
#88 (comment)
.