Skip to content

Conversation

@GetTuh
Copy link
Contributor

@GetTuh GetTuh commented Apr 19, 2024

- What does it implement / fix / improve? Why?

Armv7 support , since Linuxserver's image is not supporting armhf anymore
This mariadb image is recently updated and running on linux alpine.

- Are the changes related to an existing issue?

no

Acceptance Criteria:

  • [ x] Features and enhancements must be fully implemented so that they can be released at any time without additional work
  • [ x] Automated unit and/or acceptance tests are mandatory to ensure the changes work as expected and to reduce repetitive manual work
  • [ x] Frontend components must be responsive to work and look properly on phones, tablets, and desktop computers; you must have tested them on all major browsers and different devices
  • [ x] Documentation and translation updates should be provided if needed
  • [ x] In case you submit database-related changes, they must be tested and compatible with SQLite 3 and MariaDB 10.5.12+

@lastzero
Copy link
Member

Thank you very much! Did you check if the names of the environment variables are the same? We currently use the following for the official MariaDB image:

environment:
      MARIADB_AUTO_UPGRADE: "1"
      MARIADB_INITDB_SKIP_TZINFO: "1"
      MARIADB_DATABASE: "photoprism"
      MARIADB_USER: "photoprism"
      MARIADB_PASSWORD: "insecure"
      MARIADB_ROOT_PASSWORD: "insecure"

@GetTuh
Copy link
Contributor Author

GetTuh commented Apr 20, 2024

The names are the same as before for linuxsystem's but different than the one you posted:

  image: yobasystems/alpine-mariadb
  environment:
    MYSQL_ROOT_PASSWORD: hguyFtgfR4r9R4r76
    MYSQL_DATABASE: wordpressdb
    MYSQL_USER: wordpressuser
    MYSQL_PASSWORD: hguyFt6S95dgfR4ryb

I've had the server running since yesterday without problems.

@lastzero
Copy link
Member

I see... it's a shame that they don't support the same variables as the original image, especially MARIADB_AUTO_UPGRADE, as that could avoid upgrade issues and confusion among inexperienced users:

@lastzero lastzero added the docker Docker Images, Build Scripts, Config & Deployment Examples label Apr 20, 2024
@lastzero lastzero merged commit 609bdab into photoprism:develop Apr 20, 2024
@lastzero lastzero changed the title Update docker-compose for armv7 MariaDB: Change image name in ARMv7 docker compose example Apr 20, 2024
@lastzero lastzero changed the title MariaDB: Change image name in ARMv7 docker compose example MariaDB: Change image name in Docker Compose config example for ARMv7 Apr 20, 2024
lastzero added a commit that referenced this pull request Apr 20, 2024
Signed-off-by: Michael Mayer <michael@photoprism.app>
lastzero added a commit that referenced this pull request Apr 21, 2024
Signed-off-by: Michael Mayer <michael@photoprism.app>
lastzero added a commit to photoprism/photoprism-docs that referenced this pull request Apr 21, 2024
@lastzero
Copy link
Member

@GetTuh Note that I changed the mount point of the data volume in the config example after checking the image documentation (the linuxserver image used /config instead of the default /var/lib/mysql):

If you are using /config with the new image, we strongly recommend that you create a backup dump (as described in our documentation) and then reinitialize the MariaDB service with the new settings.

@graciousgrey graciousgrey added the released Available in the stable release label May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Docker Images, Build Scripts, Config & Deployment Examples released Available in the stable release

Projects

Status: Release 🌈

Development

Successfully merging this pull request may close these issues.

3 participants