Setting auto mode by default for worker_processes parameter

[deimosfr]

Hi,

It would be nice to set the auto mode if nothing is set as described in the official documentation (http://nginx.org/en/docs/ngx_core_module.html#worker_processes). If you don't want to use auto because it requires a certain version, you may use this instead:

worker_processes {{ nginx_workers_processes | default(ansible_processor_cores) }};

What do you think about it ?

[jdauphant]

Hi @deimosfr ,
Thanks for the pull request, it's a good idea but what's the difference with the actual setting ?

nginx_worker_processes: "{{ ansible_processor_vcpus }}" # in defaults/main.yml

Normally ansible_processor_vcpus is more accurate than ansible_processor_cores in case of multithreading. ( https://github.com/ansible/ansible/blob/215359084b0a246cf55b1220790cb3e601531bc9/lib/ansible/module_utils/facts.py#L638 )

[deimosfr]

Sorry for the mistake I didn't catch this one ;-).

However, the 'auto' mode may be better. Didn't looked at the Nginx code yet to know which solution is the best as they are not very clear in the documentation.

[jdauphant]

No problem