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

Helm value enable_crd_registration vs. Documentation #2695

Open
fgleixner opened this issue Jul 17, 2024 · 0 comments
Open

Helm value enable_crd_registration vs. Documentation #2695

fgleixner opened this issue Jul 17, 2024 · 0 comments

Comments

@fgleixner
Copy link

Reading the values.yaml from the Helm chart:

configGeneral:
  # the deployment should create/update the CRDs
  enable_crd_registration: true

makes me think, that a Helm upgrade would upgrade the CRDs.

But in the documentation, i read:
https://postgres-operator.readthedocs.io/en/latest/administrator/#upgrading-the-operator

When using helm, be aware that installing the new chart will not update the Postgresql and OperatorConfiguration CRD. Make sure to update them before with the provided manifests in the crds folder. Otherwise, you might face errors about new Postgres manifest or configuration options being unknown to the CRD schema validation.

In fact, the helm upgrade upgrades the CRDs, but for me it is not clear if this is a safe upgrade path. If it is, documentation should be updated, if not, a warning in the Helm values.yaml would be good.

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

1 participant