Skip to content
This repository has been archived by the owner on Mar 19, 2024. It is now read-only.

Ensure PHP-FPM maps to the port we specify in the configs. #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

conatus
Copy link
Contributor

@conatus conatus commented Dec 28, 2013

In the PHP setup we tell nginx to pass to PHP-FPM running on port 127.0.0.1:9000.

This ensures the file is created and set.

@conatus
Copy link
Contributor Author

conatus commented Dec 29, 2013

This doesn't appear to be the case on Ubuntu 13.04 and 13.10 from testing with Vagrant. I've always had to jimmy this pool.d file manually to allow this - it defaults as far as I can see to the socket version. Hence this change.

@vpetersson
Copy link
Owner

Interesting. I have not tested it on 13.04 and 13.10.

Actually, from a performance perspective, using a socket is supposed to be better.

Yet, we could simply make it a configuration option that allows you to either use socket or TCP/IP.

Alternatively, if we want to be really fancy, we can make the default value different depending on if it's 12.04 or 13.04/13.10.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants