geerlingguy / ansible-role-varnish

Ansible Role - Varnish HTTP accelerator
https://galaxy.ansible.com/geerlingguy/varnish/
MIT License
80 stars 88 forks source link

FIX #36: Add config to enable PID file for varnishd #37

Closed csegarra closed 8 years ago

csegarra commented 8 years ago

This commit add two variables varnish_pidfile and varnish_pidfile_path to enable the use of a PID file and to define its path.

It also modifies two templates, adding the options to SysV and systemd start scripts.

This pull request includes the commit of my previous Pull Request, because both modifies the template templates/varnish.service.j2.

csegarra commented 8 years ago

I have merged both variables into one. Now Varnish uses a PID file by default.

Thanks!

geerlingguy commented 8 years ago

Moving commits to https://github.com/geerlingguy/ansible-role-varnish/pull/41