fix(postgresql container): fix container permissions at startup#189
fix(postgresql container): fix container permissions at startup#189xmnlab merged 9 commits intothegraphnetwork:mainfrom
Conversation
| - ../.env | ||
| ports: | ||
| - ${POSTGRES_PORT}:${POSTGRES_PORT} | ||
| user: "${HOST_UID:-1000}:${HOST_GID:-1000}" |
There was a problem hiding this comment.
any reason for removing this line?
There was a problem hiding this comment.
postgres user has been properly configured with correct ID and GUI and the container initiates with this user here
There was a problem hiding this comment.
oh, that is right .. sorry .. I missed that it was about the postgres service XD
good catch!
| POSTGRES_USER: ${POSTGRES_USER} | ||
| PGDATA: /var/lib/postgresql/data/pgdata | ||
| volumes: | ||
| - ${POSTGRES_CONFIG_FILE_HOST}:/etc/postgresql/postgresql.conf:ro |
There was a problem hiding this comment.
maybe you need to add this variable to .env.tpl and to CI:
https://github.com/thegraphnetwork/EpiGraphHub/pull/189/checks#step:11:1180
|
| - ../.env | ||
| ports: | ||
| - ${POSTGRES_PORT}:${POSTGRES_PORT} | ||
| user: "${HOST_UID:-1000}:${HOST_GID:-1000}" |
There was a problem hiding this comment.
oh, that is right .. sorry .. I missed that it was about the postgres service XD
good catch!
| run: | | ||
| export HOST_UID=$(id -u) | ||
| export HOST_GID=$(id -g) | ||
| export POSTGRES_HOST_UID=$(id -u) |
There was a problem hiding this comment.
prod has a different user for postgres
|
in general, LGTM! just check the error on CI, please |
|
Done |
2 participants
fix #187