openstack/openstack-ansible-memcached_server
Code Issues Proposed changes

Increase default connection count to 4096

Browse Source 
The current default is 1024 however this is not sufficent for most
deployments and it will cause a lot of issues if it does hit the
limit as the server will stop responding.

The overhead for this is very low and other deployment tooling
usually sets this to 8192[1], so 4096 is even very conservative.

[1]: https://github.com/saz/puppet-memcached/blob/bb9d55a2/manifests/init.pp#L26

Change-Id: I604f96a9431076e1f630d2045eac4c88a92935c1
This commit is contained in:
Mohammed Naser 2018-06-13 12:35:35 -04:00
parent 6b94d5a134
commit d784ea1846
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
defaults/main.yml
View File

@ -48,7 +48,7 @@ memcached_memory: "{{ base_memcached_memory | int // 4 if base_memcached_memory
memcached_port: 11211
memcached_listen: "127.0.0.1"
memcached_log: /var/log/memcached/memcached.log
memcached_connections: 1024
memcached_connections: 4096
memcached_threads: 4
memcached_file_limits: "{{ memcached_connections | int + 1024 }}"

2
tests/test.yml
View File

@ -42,7 +42,7 @@
- "'-p 11211' in memcached_proc.stdout"
- "'-u {{ memcached_user }}' in memcached_proc.stdout"
- "'-l 127.0.0.1' in memcached_proc.stdout"
- "'-c 1024' in memcached_proc.stdout"
- "'-c 4096' in memcached_proc.stdout"
- "'-t 4' in memcached_proc.stdout"
- "'STAT pid' in memcached_stats.stdout"
vars_files: