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

Introduce a php-fpm 8.1 container #12

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Introduce a php-fpm 8.1 container #12

wants to merge 3 commits into from

Conversation

MKodde
Copy link
Member

@MKodde MKodde commented Nov 22, 2022

For SP Dashboard, the upgrade to Symfony 5 forces us to use a more modern PHP version. We could satisfy that by simply going to PHP 7.3.

Note that this is experimental. And should fit the SPD design philosophy. But will probably not be suitable for use in other Openconext projects (as they run on a more conventional virtual machine based architecture). Whereas SPD runs in a container.

@MKodde MKodde requested a review from quartje November 22, 2022 12:39
@MKodde MKodde force-pushed the feature/php-fpm-81 branch 2 times, most recently from dfd6ad9 to 915b6a4 Compare November 22, 2022 12:56
@quartje
Copy link
Contributor

quartje commented Nov 22, 2022

Nice warp into the feature. However, I'd like to retain the possibility to create php 7 containers.
I will add some commits to make that happen

This container (dev version) does not install apcu and apcu-bc.
@MKodde MKodde force-pushed the feature/php-fpm-81 branch from 5d4d42f to d602022 Compare January 4, 2023 08:23
- Enable the PHP Mysql extention
- Use composer 2 instead of 1
@MKodde MKodde force-pushed the feature/php-fpm-81 branch from d602022 to 60bdd13 Compare January 4, 2023 08:28
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