devops-coop / ansible-haproxy

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

allow for only installing haproxy 1.5 #36

Closed jonhatalla closed 9 years ago

jonhatalla commented 9 years ago

As it stands, this PR will only change the folder structure. We manage our own haproxy config with a specialized template, but everything else in your role is great for us.

PierrePaul commented 9 years ago

Good idea. Would you mind if I changed the var to "haproxy_manage_config" [True | False] ?

jonhatalla commented 9 years ago

not at all

jonhatalla commented 9 years ago

just a heads up - this won't start the service if haproxy_manage_config is set to false. This actually should be expected since the default config won't be of much use anyways.

The current tests should still pass as no functionality has changed from their perspective.

jonhatalla commented 9 years ago

i went ahead and updated the variable name

PierrePaul commented 9 years ago

Thank you!