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

Update variables to MariaDB #1243

Merged
merged 6 commits into from
Oct 14, 2024
Merged

Update variables to MariaDB #1243

merged 6 commits into from
Oct 14, 2024

Conversation

drcursor
Copy link
Contributor

  • Added comment directing people. to the example docker compose file.
  • Changed variables to MARIADB as we are using the MARIADB container.
  • Fixed small typo (missing "=")

gantoine and others added 6 commits September 4, 2024 20:10
…te-dev-instructions-for-arch-linux

chore(README): Update dev instructions for Arch Linux users
Switched to MARIADB variables as we are using the MARIADB container
Added comment directing people. to the example docker compose file. Changed variables to MARIADB as we are using the MARIADB container
@gantoine gantoine changed the base branch from release to master October 14, 2024 18:41
@gantoine gantoine merged commit 4dd82a9 into rommapp:master Oct 14, 2024
2 checks passed
@adamantike
Copy link
Collaborator

We will need to revert this change. As can be seen here, the LinuxServer's MariaDB image only supports MYSQL_ prefixed environment variables.

Considering that the official MariaDB image supports both MARIADB_ and MYSQL_ prefixes, we can go back to MYSQL_ temporarily, until LinuxServer image gets support for this new naming convention.

adamantike added a commit that referenced this pull request Nov 6, 2024
This reverts changes introduced in #1243.

The LinuxServer's MariaDB image only supports `MYSQL_` prefixed
environment variables.
Considering that the official MariaDB image supports both `MARIADB_` and
`MYSQL_` prefixes, we can go back to `MYSQL_` temporarily, until
LinuxServer image gets support for this new naming convention.
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.

4 participants