mattermost / mattermost/docker

Post mattermost-docker migration minor issue with docker container health

Open
#59 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Shell
Stars
533
Forks
284
PR merge metrics
No merged PRs in 30d

Description

Hey Mattermost team, great work!

I was on mattermost-docker up until now. I had used the early version of the upgrade script to get my postgres instance onto version 13.2 and all has been well.

I saw the movement towards this repo as the main docker repo and so I wanted to migrate over. I upgraded my mattermost-docker instance up to version 5.39, the latest, and that started up just fine.

Then rysnc'd my volumes directory from mattermost-docker to this repos volumes directory. I set up tthe .env witth all the old settings for the postgres username, database name, and password. The postgres database started okay, but the new mattermost docker was having errors about my postgres user not being able to authenticate of course causing mattermost to not start up.

I noticed a small discrepancy in the way the database directories were set between the 2 repos.

The new docker repo (this one) had the path being passed into the container like this:
./volumes/db/var/lib/postgresql/data:/var/lib/postgresql/data

However, with the upgrade script I used from the other repo on the mass thread about the postgres migrations has it set like this:
./volumes/db/13/data:/var/lib/postgresql/data

So, I ended up just updating my .env file from this new repo to have this instead:
POSTGRES_DATA_PATH=./volumes/db/13/data

So now the containers both start up, and it appears to be working just fine. However, the docker container for mattermost reports that it's state is "unhealthy".

Admittedly postgres is not my strong suite when it comes to databases. Should I do it differently somehow? Maybe copy the data from ./volumes/db/13/data to ./volumes/db/var/lib/postgresql/data?

Again, everything is working and I can't tell why the container reports as unhealthy, thus why I call it a minor issue. I'm just concerned there might be underlying issues I'm not seeing yet.

Any suggestions, concerns, thoughts?

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the .env setting POSTGRES_DATA_PATH and compare the two volume paths described: ./volumes/db/13/data and ./volumes/db/var/lib/postgresql/data. Check the Mattermost container's reported health status alongside the PostgreSQL container after migration; done means the correct data path is established and the health status is explained or corrected.

Written by the indexing model from the issue text.

Assessment

Tech stack
docker, postgresql
Domain
databases, devops
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.