Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Bedrock WordPress apps #134

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add support for Bedrock WordPress apps #134

wants to merge 5 commits into from

Conversation

singlow
Copy link

@singlow singlow commented May 14, 2014

Bedrock is a Composer based WordPress setup. These changes
allow for it as a framework type if specified by name. I've
included an nginx config that supports WordPress style
rewrites.

Jacob Williams added 2 commits May 14, 2014 12:13
Bedrock is a Composer based WordPress setup. These changes
allow for it as a framework type if specified by name. I've
included an nginx config that supports WordPress style
rewrites.
@retlehs
Copy link

retlehs commented Jan 2, 2015

👍

@timcv
Copy link

timcv commented Feb 10, 2015

This looks awesome!

@@ -196,7 +196,7 @@ pm = static
; forget to tweak pm.* to fit your needs.
; Note: Used when pm is set to 'static', 'dynamic' or 'ondemand'
; Note: This value is mandatory.
pm.max_children = 8
pm.max_children = 3
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why changing this ?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was something i should have done in a branch to save memory on the my server that has many apps running on only 4 cores. I committed a revert of this for master.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am using this with dokku on a server with ~10 apps and the memory usage is a bit much with 8 php processes per app - epecially since I do not have 8 cores. I have moved this customization into another branch.

Jacob Williams added 2 commits February 10, 2015 09:57
This should have been done in a branch.

This reverts commit fb7490d.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants