Skip to content

prepare postgreslet for operator v2.0.1 upgrade - #622

Open
TLINDEN wants to merge 33 commits into
mainfrom
upgrade-operator-v2
Open

prepare postgreslet for operator v2.0.1 upgrade#622
TLINDEN wants to merge 33 commits into
mainfrom
upgrade-operator-v2

Conversation

@TLINDEN

@TLINDEN TLINDEN commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator
  • add new flag enable-pod-topology-spread-constraint
  • keep tsc webhook supported though, in case someone keeps using the old operator
  • update default postgres operator image to v2.0.1

- add new flag enable-pod-topology-spread-constraint
- turn off enable-pod-topology-spread-constraint-webhook if operator
  >= v2.x
- keep tsc webhook supported though, in case someone keeps using the
  old operator
@TLINDEN
TLINDEN marked this pull request as draft August 3, 2026 07:22
@TLINDEN

TLINDEN commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator Author

PR can't be merged unless the zalando operator API has been upgraded to v2 and has been implemented here.

Thomas von Dein and others added 4 commits August 3, 2026 10:14
Upstream did not update the module version when releasing v2.
Pinning the commit from [the upstream release](https://github.com/zalando/postgres-operator/releases/tag/v2.0.1)
instead.

Upstream work: [issue](zalando/postgres-operator#3155) [PR](zalando/postgres-operator#3156)

Signed-off-by: Benjamin Ritter <benjamin.ritter@x-cellent.com>
Signed-off-by: Benjamin Ritter <benjamin.ritter@x-cellent.com>
Signed-off-by: Benjamin Ritter <benjamin.ritter@x-cellent.com>
@l0wl3vel l0wl3vel mentioned this pull request Aug 3, 2026
@TLINDEN
TLINDEN marked this pull request as ready for review August 6, 2026 10:55
@TLINDEN
TLINDEN requested a review from l0wl3vel August 6, 2026 10:56
@TLINDEN TLINDEN self-assigned this Aug 6, 2026
@TLINDEN TLINDEN added the dependencies Pull requests that update a dependency file label Aug 6, 2026
Comment thread main.go Outdated
Comment thread main.go Outdated
Comment thread .github/workflows/pull_request.yaml
Comment thread api/v1/postgres_types.go Outdated
Comment thread api/v1/postgres_types.go Outdated
Comment thread api/v1/postgres_types.go Outdated
Comment thread api/v1/postgres_types.go
Comment thread external/svc-postgres-operator.yaml Outdated
Comment thread pkg/operatormanager/operatormanager.go Outdated
Comment thread controllers/postgres_controller.go Outdated
Comment thread controllers/postgres_controller.go Outdated
Comment thread api/v1/postgres_types.go
Comment thread external/svc-postgres-operator.yaml Outdated
Comment thread .github/workflows/pull_request.yaml
Comment thread api/v1/postgres_types.go

@l0wl3vel l0wl3vel left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Remove the changes for kubernetes_use_configmaps, then we are good.

fixed the missing seccomp Profile: b646b7e

lgtm otherwise.

Comment thread pkg/operatormanager/operatormanager.go Outdated
Comment thread external/svc-postgres-operator.yaml
@TLINDEN

TLINDEN commented Aug 17, 2026

Copy link
Copy Markdown
Collaborator Author

@l0wl3vel after I merged main, I got theses linter errors. I removed the exhaustruct linter, explicitly adding fields with their default values just to satisfy it, makes not much sense to me. Do you agree?

I also had to revert another change: 5190bbd (it's not supported)

@l0wl3vel

l0wl3vel commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

Removing exhauststruct sounds good to me. Going to cherrypick this into a seperate PR: #627

@l0wl3vel

Copy link
Copy Markdown
Collaborator

https://github.com/zalando/postgres-operator/releases/tag/v2.0.2 is available. With v2 module

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants