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(cluster): added support for enablePDB option #431

Merged
merged 3 commits into from
Dec 7, 2024

Conversation

mbugeia
Copy link
Contributor

@mbugeia mbugeia commented Oct 30, 2024

Add support for enablePDB configuration option

Fix #395

@itay-grudev
Copy link
Collaborator

@mbugeia Could you add the option to the non default configuration test that verifies that options are passed correctly?

https://github.com/cloudnative-pg/charts/tree/main/charts/cluster/test/postgresql-cluster-configuration

You'll need to set it to false in the cluster values and add the option in the assertion.

@itay-grudev itay-grudev added the chart( cluster ) Related to the cluster chart label Nov 29, 2024
@mbugeia
Copy link
Contributor Author

mbugeia commented Nov 29, 2024

@itay-grudev Thanks for the review, I've added the test. Unfortunatly I'm not sure if it works as I don't know how to run the tests locally 🙈

@itay-grudev itay-grudev merged commit 665396d into cloudnative-pg:main Dec 7, 2024
5 checks passed
yremmet pushed a commit to yremmet/charts that referenced this pull request Dec 9, 2024
hapeho pushed a commit to hapeho/cloudnative-pg-charts that referenced this pull request Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart( cluster ) Related to the cluster chart
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inplement option to set EnablePDB
2 participants