use malloc for default storage & update systemd service param

geerlingguy / ansible-role-varnish

Ansible Role - Varnish HTTP accelerator

https://galaxy.ansible.com/geerlingguy/varnish/

MIT License

81 stars 89 forks source link

use malloc for default storage & update systemd service param #30

Closed jdowning closed 8 years ago

geerlingguy commented 8 years ago

This seems to have been fixed in a different PR or commit; see https://github.com/geerlingguy/ansible-role-varnish/commit/74d5acb1af1dc183d431d884780b4a7ceaf01e42

jdowning commented 8 years ago

🆒 However, I would say the default value should be varnish_storage: "malloc,256M". What are your thoughts?

geerlingguy commented 8 years ago

@justindowning - Can you create a new issue for that? We can discuss further there, as I don't want to make the change without good justification. I remember file storage was Varnish's default, at least as of 3.x, and I would like to keep using whatever Varnish's default is moving forward (haven't had time to check what the default is on a fresh install of 4.1 or 5.0 yet).