Skip to content

Bump spilo and target version for PostgreSQL 15 #2139

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
merged 9 commits into from
Jan 4, 2023
Merged

Bump spilo and target version for PostgreSQL 15 #2139

merged 9 commits into from
Jan 4, 2023

Conversation

yoshihikoueno
Copy link
Contributor

@yoshihikoueno yoshihikoueno commented Dec 15, 2022 •
edited by FxKu
Loading

Hi,

This PR adds support for PostgreSQL 15 and drops support for 9.5 and 9.6.
As for Spilo image, I've set spilo-cdp-15:2.1-p253 for now, but I'm willing to change it to non-CDP one once it gets released.

Also, I chose not to touch under ./e2e as there's currently no spilo image for it as far as I know.

  • provide offical spilo-15 release image
  • provide e2e runner images for 15
  • update target_major_version to 15
  • update README with 15 support
  • bump example manifests to 15
  • include 15 in UI deployment

@yoshihikoueno yoshihikoueno changed the title Feature/support postgresql15 Add support for PostgreSQL 15 Dec 15, 2022
@FxKu
Copy link
Member

FxKu commented Dec 15, 2022

Thanks @yoshihikoueno . Looks like we had the same idea at the same time. Sorry, I will bump Postgres in #2140 .

@FxKu FxKu closed this Dec 15, 2022
@FxKu FxKu reopened this Dec 15, 2022
@FxKu
Copy link
Member

FxKu commented Dec 15, 2022

or maybe there's still a place for this PR because we want to merge #2140 without bumping spilo and manifest examples so that we have an operator that just accepts 15 for now. You can rebase after we have merged.

@FxKu FxKu mentioned this pull request Dec 15, 2022
Merged
6 tasks
@FxKu FxKu changed the title Add support for PostgreSQL 15 Bump spilo and target version for PostgreSQL 15 Dec 15, 2022
@FxKu FxKu added this to the 1.9 milestone Dec 15, 2022
@FxKu
Copy link
Member

FxKu commented Dec 30, 2022 •
edited
Loading

@yoshihikoueno could you update the image to ghcr.io/zalando/spilo-15:2.1-p9 (see newest release)?

@FxKu
Copy link
Member

FxKu commented Dec 30, 2022

For e2e tests please update images to:

SPILO_CURRENT = "registry.opensource.zalan.do/acid/spilo-15-e2e:0.1"
SPILO_LAZY = "registry.opensource.zalan.do/acid/spilo-15-e2e:0.2"

@yoshihikoueno
Copy link
Contributor Author

yoshihikoueno commented Dec 31, 2022 •
edited
Loading

@FxKu
Thank you for informing me of the new images.
I'll switch to them in a few days.

@yoshihikoueno
Copy link
Contributor Author

yoshihikoueno commented Jan 4, 2023 •
edited
Loading

@FxKu
I've bumped the spilo image and also merged the master branch to fix the conflicts.

Currently the e2e test fails at test_node_affinity (ref).
Does this need to be fixed?

FxKu
FxKu reviewed Jan 4, 2023
View reviewed changes
@FxKu
Copy link
Member

FxKu commented Jan 4, 2023

👍

@jopadi
Copy link
Member

jopadi commented Jan 4, 2023

👍

@FxKu FxKu merged commit becf8a4 into zalando:master Jan 4, 2023
@FxKu FxKu mentioned this pull request Dec 18, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FxKu FxKu FxKu left review comments

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

@Jan-M Jan-M Awaiting requested review from Jan-M Jan-M is a code owner

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

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

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

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.9
Development

Successfully merging this pull request may close these issues.
3 participants
@yoshihikoueno @FxKu @jopadi