Closed (fixed)
Project:
Provision
Version:
6.x-1.x-dev
Component:
HTTP Service
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
15 Apr 2012 at 14:16 UTC
Updated:
12 Jun 2014 at 08:41 UTC
Jump to comment: Most recent
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 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
miklThis 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 commentedYou have missed something. The included here (and updated)
nginx.conffile is an *example* only. Provision will not overwrite your own/etc/nginx/nginx.conffile. 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.phptemplate.