CiscoCloud / haproxy-consul

Dynamic haproxy configuration using consul
Apache License 2.0
168 stars 85 forks source link

Don't generate empty backends #15

Closed BrianHicks closed 8 years ago

BrianHicks commented 9 years ago

This PR makes it so that consul-template doe not generate invalid configs by generating empty backends and referring (via ACLs) to backends that aren't generated.

This still should be tested in a production workload, but my testing locally no longer generates bad configs for suspended Marathon apps.

Fixes #14

stevendborrelli commented 8 years ago

This bit us on one of the clusters. Can we merge this and update the haproxy-consul container in the docker hub?

BrianHicks commented 8 years ago

If it looks good to you, it works for me!