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

zalan-postgres-health-check container image: do we still need this? #1882

Open
jefflill opened this issue Feb 11, 2024 · 3 comments
Open

zalan-postgres-health-check container image: do we still need this? #1882

jefflill opened this issue Feb 11, 2024 · 3 comments
Assignees

Comments

@jefflill
Copy link
Collaborator

During the Kubernetes v1.29 upgrade, I noticed that we build the zalan-postgres-heralth-check container image. It doesn't look like we reference this anywhere. I'm going to delete this now.

We should follow up to be sure.

@jefflill jefflill assigned jefflill and marcusbooyah and unassigned jefflill Feb 11, 2024
@jefflill jefflill changed the title zalan-postgres-heralth-check container image: do we still need this zalan-postgres-health-check container image: do we still need this Feb 11, 2024
@jefflill
Copy link
Collaborator Author

CLOSING: It looks like later versions of this operator has probably integrated the health check and metrics because these settings are not longer in the Helm chart.

@jefflill jefflill changed the title zalan-postgres-health-check container image: do we still need this zalan-postgres-health-check container image: do we still need this? Feb 11, 2024
@jefflill
Copy link
Collaborator Author

We might need this after all:

zalando/postgres-operator#1703

@jefflill jefflill reopened this Feb 11, 2024
@jefflill
Copy link
Collaborator Author

jefflill commented Feb 12, 2024

...although it looks like we've disabled this functionality:

image

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

No branches or pull requests

2 participants