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

Converted base image to php-fpm #90

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

Conversation

errolsancaktar
Copy link

Ran into issues running this in kubernetes and had to rebuild off of different base. Changed to php-fpm alpine image and fixed a couple small things with syntax changes in php8

@patrickdk77
Copy link
Collaborator

But what was the issue? and does this under previous bug fixes.
It was already build ontop of alpine base to start with.

@errolsancaktar
Copy link
Author

So specifically the issue was around how php7 fpm works with hugepages i believe. On your version fpm would always crash and never stay running. When i went to recreate your docker image it was a private base image so i recreated it with a standard one and set up the dockerfile to move the code and configs to the appropriate locations.

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

Successfully merging this pull request may close these issues.

2 participants