Skip to content

Add mariadb client config to Dockerfile#1129

Merged
deer-wmde merged 3 commits into
mainfrom
add-mariadb-config-to-dockerfile
May 18, 2026
Merged

Add mariadb client config to Dockerfile#1129
deer-wmde merged 3 commits into
mainfrom
add-mariadb-config-to-dockerfile

Conversation

@outdooracorn
Copy link
Copy Markdown
Member

@outdooracorn outdooracorn commented May 15, 2026

In order to prevent TLS/SSL error: SSL is required, but the server does not support it errors when running php artisan migrate:install as part of the pre-install-job.yaml1

Bug: T426216

This should be in the container image instead of only being mounted in the docker compose dev env

Bug: T426216
@outdooracorn outdooracorn self-assigned this May 15, 2026
@outdooracorn outdooracorn force-pushed the add-mariadb-config-to-dockerfile branch from cf8e1b7 to 37e3b31 Compare May 15, 2026 12:26
@outdooracorn outdooracorn requested a review from dati18 May 15, 2026 14:17
@deer-wmde deer-wmde merged commit 02b92c3 into main May 18, 2026
3 checks passed
@deer-wmde deer-wmde deleted the add-mariadb-config-to-dockerfile branch May 18, 2026 14:28
@outdooracorn
Copy link
Copy Markdown
Member Author

outdooracorn commented May 18, 2026

@deer-wmde FYI we now have to watch out when we do squash and merge as GitHub is really annoying and likes to break the formating of the commit message. In this case the Bug: T426216 line in 02b92c3 is no longer a part of the git trailer and so won't be picked up by the Phabricator bot.

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