Remove symlink for PostgreSQL data directory in Alpine images#1394
Merged
tianon merged 2 commits intodocker-library:masterfrom Jan 29, 2026
Merged
Remove symlink for PostgreSQL data directory in Alpine images#1394tianon merged 2 commits intodocker-library:masterfrom
tianon merged 2 commits intodocker-library:masterfrom
Conversation
Member
|
Great catch! Honestly kind of hilarious that we missed it in #1372 This needs to be updated in the templating (as seen in the failing CI); would you be willing to update with that change/fix or do you need us to take over from here? |
Removed symbolic link creation for PGDATA in Dockerfile.
Contributor
Author
|
Ah totally forgot about the template. Fixed that as well. CI should now be passing! |
docker-library-bot
added a commit
to docker-library-bot/official-images
that referenced
this pull request
Jan 29, 2026
Changes: - docker-library/postgres@ec0cb67: Remove symlink for PostgreSQL data directory in Alpine images (docker-library/postgres#1394)
tianon
pushed a commit
to docker-library/official-images
that referenced
this pull request
Jan 29, 2026
Changes: - docker-library/postgres@ec0cb67: Remove symlink for PostgreSQL data directory in Alpine images (docker-library/postgres#1394) Co-authored-by: Docker Library Bot <doi+docker-library-bot@docker.com>
keeganwitt
pushed a commit
to keeganwitt/docker-library-official-images
that referenced
this pull request
Feb 3, 2026
Changes: - docker-library/postgres@ec0cb67: Remove symlink for PostgreSQL data directory in Alpine images (docker-library/postgres#1394) Co-authored-by: Docker Library Bot <doi+docker-library-bot@docker.com>
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
iobruno
added a commit
to iobruno/data-catalog-labs
that referenced
this pull request
Mar 5, 2026
* Fixes vol-metabase-db to mount on `/var/lib/postgresql` instead of `/var/lib/postgresql` for postgres:18-alpine since docker-library/postgres#1394 * Fixes healthcheck cmd
iobruno
added a commit
to iobruno/data-catalog-labs
that referenced
this pull request
Mar 5, 2026
* Fixes vol-airflow-metastore-localexec and vol-airflow-metastore-celeryexec to mount on `/var/lib/postgresql` for postgres:18-alpine since docker-library/postgres#1394 * Fixes healthcheck cmd * Adds extra ENV_VARS on compose.celery.yaml to match compose.local.yaml * Removes `ingest-db`
iobruno
added a commit
to iobruno/data-catalog-labs
that referenced
this pull request
Mar 5, 2026
* Fixes vol-datahub-pgdata and conduktor-metastore to mount on `/var/lib/postgresql` instead of `/var/lib/postgresql` for postgres:18-alpine since docker-library/postgres#1394 * Fixes healthcheck cmd
Merged
4 tasks
iobruno
added a commit
to iobruno/data-catalog-labs
that referenced
this pull request
Mar 5, 2026
) * Fix Postgres data mount points for **airflow**, **datahub**, **conduktor**, and **metabase** to use `/var/lib/postgresql` for `postgres:18-alpine` (per docker-library/postgres#1394) * Fix healthcheck commands across Postgres services * Update db credentials for conduktor-console * Enforce naming the airflow image `apache/airflow:2.10.5-python3.11-celery-custom` * Dry up running instructions on airflow/README
iobruno
added a commit
to iobruno/data-engineering-labs
that referenced
this pull request
Mar 5, 2026
…or postgres:18-alpine+ Since docker-library/postgres#1394, the default PGDATA changed from /var/lib/postgresql/data to /var/lib/postgresql/data2. Mounting at /var/lib/postgresql avoids breaking on this change. Also use $$POSTGRES_USER env var in healthcheck instead of hardcoded username. Ref: docker-library/postgres#1394 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
iobruno
added a commit
to iobruno/data-engineering-labs
that referenced
this pull request
Mar 5, 2026
…or postgres:18-alpine+ Update mount point from /var/lib/postgresql/data to /var/lib/postgresql and use $$POSTGRES_USER in pg_isready healthchecks to dynamically match each service's configured user. This fixes compatibility with postgres:18-alpine and later, following the PGDATA relocation introduced in docker-library/postgres#1394. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
iobruno
added a commit
to iobruno/data-engineering-labs
that referenced
this pull request
Mar 5, 2026
…service Update volume mount to /var/lib/postgresql to avoid nested data dir issues introduced in postgres:18-alpine and later (docker-library/postgres#1394), and use $$POSTGRES_USER env var in healthcheck for consistency. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
iobruno
added a commit
to iobruno/data-engineering-labs
that referenced
this pull request
Mar 5, 2026
…ose files Change volume mount from /var/lib/postgresql/data to /var/lib/postgresql and use $$POSTGRES_USER env var in pg_isready healthcheck instead of hardcoded username across all compose files. Fix for postgres:18-alpine and later versions since docker-library/postgres#1394 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
iobruno
added a commit
to iobruno/data-engineering-labs
that referenced
this pull request
Mar 5, 2026
Fix for postgres:18-alpine and later versions, since the default data directory changed from /var/lib/postgresql/data to /var/lib/postgresql. See: docker-library/postgres#1394 Also use $$POSTGRES_USER env var in healthcheck instead of hardcoded username. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This was referenced Mar 5, 2026
iobruno
added a commit
to iobruno/data-engineering-labs
that referenced
this pull request
Mar 5, 2026
Fix volume mount point from /var/lib/postgresql/data to /var/lib/postgresql and use $$POSTGRES_USER env var in pg_isready healthcheck for both ingest-db and prefect-metastore services. This addresses a breaking change in postgres:18-alpine and later versions introduced by docker-library/postgres#1394. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2 tasks
iobruno
added a commit
to iobruno/data-catalog-labs
that referenced
this pull request
Mar 5, 2026
) * Fix Postgres data mount points for **airflow**, **datahub**, **conduktor**, and **metabase** to use `/var/lib/postgresql` for `postgres:18-alpine` (This addresses a breaking change introduced in [docker-library/postgres#1394](docker-library/postgres#1394)) * Fix healthcheck commands across Postgres services * Update db credentials for conduktor-console * Enforce naming the airflow image `apache/airflow:2.10.5-python3.11-celery-custom` * Dry up running instructions on airflow/README
2 tasks
iobruno
added a commit
to iobruno/data-catalog-labs
that referenced
this pull request
Mar 5, 2026
) * Fix Postgres data mount points for **airflow**, **datahub**, **conduktor**, and **metabase** to use `/var/lib/postgresql` for `postgres:18-alpine` (This addresses a breaking change introduced in [docker-library/postgres#1394](docker-library/postgres#1394)) * Fix healthcheck commands across Postgres services * Update db credentials for conduktor-console * Enforce naming the airflow image `apache/airflow:2.10.5-python3.11-celery-custom` * Dry up running instructions on airflow/README
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removed symbolic link on
/var/lib/postgresql/datato retain backwards compatiblity as already done for Debian images in #1372.This resolves #1393.