devops-coop / ansible-haproxy

Installs and configure HAProxy
Apache License 2.0
96 stars 97 forks source link

ansible removes newlines #105

Closed fishman closed 6 years ago

fishman commented 6 years ago

ansible removes newlines which causes tcp-request configs to overlap with backends

benwebber commented 6 years ago

Thanks for contributing, Reza! :tada:

Unfortunately we cannot accept this PR because pending changes to the role obsolete these templates (#107).

You can read a description of the pending changes here:

https://github.com/devops-coop/ansible-haproxy/blob/3ef2eb9f3d22ff27a1043c689b3819c87ab44ffa/README.md

The pending changes dramatically simplify the role configuration and handle whitespace much better than the current role.

Cheers!