Skip to content
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.

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

feat(postgresql): support multiple versions #207

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

sebastian-de
Copy link
Contributor

@sebastian-de sebastian-de commented Feb 21, 2024

If a container with postgresql_container_name exists, only update PostgreSQL to postgresql_container_version if the given major version matches the one of the existing container.
Otherwise the version of the existing container will stay unchanged.

preparation for https://github.com/famedly/infra-meta/issues/884

@sebastian-de sebastian-de requested a review from a team as a code owner February 21, 2024 09:58
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@sebastian-de sebastian-de force-pushed the sebastian-de/postgres-major-version branch 3 times, most recently from 5083442 to d251387 Compare February 21, 2024 22:19
roles/postgresql/defaults/main.yml Outdated Show resolved Hide resolved
roles/postgresql/tasks/main.yml Outdated Show resolved Hide resolved
roles/postgresql/tasks/main.yml Outdated Show resolved Hide resolved
roles/postgresql/tasks/main.yml Show resolved Hide resolved
@jcgruenhage jcgruenhage self-requested a review February 26, 2024 09:59
Copy link
Contributor

@jcgruenhage jcgruenhage left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, misclicked

@sebastian-de sebastian-de force-pushed the sebastian-de/postgres-major-version branch 3 times, most recently from aa16daf to 0023012 Compare February 26, 2024 13:39
By default the role will now update PostgreSQL to the latest available minor
version, when only a major version is given, or the version of the deployed
container if postgresql_container_version does not match the major version
of that container.
@sebastian-de sebastian-de force-pushed the sebastian-de/postgres-major-version branch from 0023012 to b8af3d5 Compare February 28, 2024 10:52
@sebastian-de sebastian-de merged commit b8af3d5 into main Feb 28, 2024
14 of 22 checks passed
@sebastian-de sebastian-de deleted the sebastian-de/postgres-major-version branch February 28, 2024 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants