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

fix(ingress) disable duplicate CRD installation #1052

Closed
wants to merge 1 commit into from

Conversation

NissesSenap
Copy link

@NissesSenap NissesSenap commented Apr 10, 2024

What this PR does / why we need it:

Remove

Which issue this PR fixes

Special notes for your reviewer:

Checklist

  • PR is based off the current tip of the main branch.
  • Changes are documented under the "Unreleased" header in CHANGELOG.md
  • New or modified sections of values.yaml are documented in the README.md
  • Commits follow the Kong commit message guidelines

@NissesSenap NissesSenap requested a review from a team as a code owner April 10, 2024 13:01
@CLAassistant
Copy link

CLAassistant commented Apr 10, 2024

CLA assistant check
All committers have signed the CLA.

@NissesSenap
Copy link
Author

NissesSenap commented Apr 10, 2024

I don't think this will work, due to templates for CRD don't work in helm 3 if I understand it correctly https://helm.sh/docs/topics/charts/#custom-resource-definitions-crds
We would need to do skipCRDs on the subcharts, but that isn't possible helm/helm#10540

The issue have popped up a number of times in helm, but no luck. So I'm unsure if there is any way around this. If anyone have a better idea on how to solve it please share.

@NissesSenap
Copy link
Author

This won't change anything, so I will close this

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

Successfully merging this pull request may close these issues.

kong ingress duplicate CRD installation
2 participants