Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
For historical reasons we had some directives specified in the server.tpl.php
, but since now recommended Dotdeb Nginx build (for Debian Squeeze) includes some of them in the main /etc/nginx/nginx.conf
config file, it breaks things with fatal errors and requires removing them manually from /etc/nginx/nginx.conf
, so we need to sync our config to avoid this.
Comments
Comment #1
omega8cc CreditAttribution: omega8cc commentedFixed in commits:
6.x-1.x - http://drupalcode.org/project/provision.git/commit/abaa275
6.x-2.x - http://drupalcode.org/project/provision.git/commit/c4eed04
Comment #3
mikl CreditAttribution: mikl commentedThis issue and it associated commits does a lot more than it says on the tin.
For one, it quadruples the value of the worker_connections setting, a change that is probably only beneficial for the largest Ægir/Barracuda deployments. For the rest of us, it is probably a small performance penalty and causes increased RAM usage.
Worse still, the change also introduces a `use epoll;` statement. This is a Linux-specific option. There’s no telling what nginx will do if you try to load epoll on a platform that doesn’t support it, but it’s probably not a something beneficial.
I’d recommend that the `use epoll` statement be removed soonest, and I think you should consider whether the quadrupling of worker_connections is the right setting for Aegir core.
Comment #4
omega8cc CreditAttribution: omega8cc commentedYou have missed something. The included here (and updated)
nginx.conf
file is an *example* only. Provision will not overwrite your own/etc/nginx/nginx.conf
file. It is included only for reference but not used anywhere. It has been included as a part of this commit because of the context with changes introduced in theserver.tpl.php
template.