Skip to content

feat: initial postgresql-bitnami image (with ARM) #606

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.

Sign up for GitHub

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

Jump to bottom
Merged
thesuperzapper merged 2 commits into from
Jun 14, 2022
Merged

feat: initial postgresql-bitnami image (with ARM) #606

thesuperzapper merged 2 commits into from
Jun 14, 2022

Conversation

@thesuperzapper
Copy link
Member

@thesuperzapper thesuperzapper commented Jun 14, 2022

What issues does your PR fix?

What does your PR do?

  • Adds an initial postgresql-bitnami image:
    • includes postgres 11.16, pgAudit 1.3.4, and nss_wrapper 1.1.11
    • uses very similar scripts to bitnami/postgresql, so compatibility is maintained with the old stable/postgresql helm chart (we currently still use stable/postgresql, see stop using embedded stable/postgres chart #303)
    • based on alpine (rather than Debian like bitnami/postgresql)
    • cross-compiles for amd64 and arm64 (different from bitnami/postgresql, which only builds for amd64)

@thesuperzapper thesuperzapper requested a review from gsemet as a code owner June 14, 2022 08:01
@thesuperzapper thesuperzapper changed the title feat: initial postgresql-bitnami image feat: initial postgresql-bitnami image (with ARM) Jun 14, 2022
@thesuperzapper
Copy link
Member Author

thesuperzapper commented Jun 14, 2022

In the interest of the ARM builds finishing before the heat death of the universe, I have added instructions on how to build the arm64 image locally, and cache these layers to our ci/images/postgresql-bitnami/11/alpine package.

Layers in will be pulled from the ghcr.io/airflow-helm/ci/images/postgresql-bitnami/11/alpine cache during test/release pipelines, because we have set --cache-from=type=registry,ref=ghcr.io/airflow-helm/ci/images/postgresql-bitnami/11/alpine in our buildx.

@thesuperzapper thesuperzapper merged commit 7bb9cfe into airflow-helm:main Jun 14, 2022
@thesuperzapper thesuperzapper deleted the postgresql-bitnami branch June 14, 2022 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gsemet gsemet Awaiting requested review from gsemet gsemet is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@thesuperzapper