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

Install SQLite on the OpenConext phpfpm container #9

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

Conversation

MKodde
Copy link
Member

@MKodde MKodde commented Mar 3, 2022

Sqlite is required to run certain db integration tests. Having a MySQL database in a test env often proves to be overkill so we opted for a lightweight SQLite solution in the User Lifecycle project. The SQLite PHP extension was already compiled into the Alpine PHP image. But SQLite itself had to be installed

@MKodde MKodde force-pushed the maintenance/add-sqlite-to-openconext-phpfpm-dev branch from 1d94523 to cc9bd99 Compare March 3, 2022 07:58
@MKodde MKodde force-pushed the maintenance/add-sqlite-to-openconext-phpfpm-dev branch from cc9bd99 to f6551f5 Compare March 3, 2022 07:59
@MKodde MKodde requested a review from quartje March 7, 2022 09:22
@quartje
Copy link
Contributor

quartje commented Mar 7, 2022

Where is the test located that uses this? If it uses Docker, a Docker mySQL image should be trivial to use, which also creates a situation that looks more like what is used on production

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