Question: helm - CRDs for first installation #5386
-
|
I read the doc https://book.kubebuilder.io/plugins/available/helm-v2-alpha.html and However by doing that a simple It will fail with the issue saying CRDs are missing So should I copy the CRD one level up so it is installed the first time and then it will also be update because they are under template. I will be happy to add in the documentation what's the recommended way Shouldn't it create something like or should the cli do it But even with that it won't work I would get another issue |
Beta Was this translation helpful? Give feedback.
Replies: 6 comments 2 replies
-
|
Hi @emirot 👋 Thanks for the report — and also for taking time to check the docs and suggest improvements. About the Helm chart layout (CRDs under templates)This is intentional. The Helm layout we generate follows the design adopted by Kubebuilder for this plugin: We keep CRDs under Important note about CRs (Custom Resources)A key Helm limitation is ordering:
Because of that:
Why I think your error is not caused by the CRD folder locationGiven the tests above, it is more likely one of these is happening:
Quick checks (so we can confirm)Could you please share:
Also, can you confirm:
With that info, we can analyse better what is happing and try to help you out. Proof that the current layout works today (tests + CI)We have good coverage to ensure this scenario works end-to-end: ✅ E2E tests: we mock a project, generate
✅ GitHub Actions: we continuously validate the Helm samples + tutorial projects:
Common Users feedbacks when they understand how Helm handle CRDs and why it is not good for Operators
Thanks again! |
Beta Was this translation helpful? Give feedback.
-
|
Hey @camilamacedo86, Thanks for your detailed answer. Here is the project: https://github.com/argoproj-labs/gitops-promoter/tree/main What I did Then (after fixing #5387) manually So my question is what is the preferred way? From your answer looks like there is two, helm hooks or applying the manifest before. For the later should It be and run helm install ? but kubectl would fail because the .yaml file contains I believe users won't be able to do |
Beta Was this translation helpful? Give feedback.
-
|
@camilamacedo86 Sorry about the confusion, what I have done is easy to reproduce So my question is what is the preferred way of installing. Is that failure expected?
I am not sure to understand
|
Beta Was this translation helpful? Give feedback.
-
|
See that in the extras you have the CRs. Important note about CRs (Custom Resources)A key Helm limitation is ordering:
Because of that:
I will push some fixes that will solve your use case and when those get merged we do a new release. |
Beta Was this translation helpful? Give feedback.
-
|
Hi @emirot See;
Then, for discussion new features:
So, I am marking this one as sorted out and closing it. |
Beta Was this translation helpful? Give feedback.
-
|
Hi @emirot You can test it out with the latest release; https://github.com/kubernetes-sigs/kubebuilder/releases/tag/v4.11.1 The issue is fixed. |
Beta Was this translation helpful? Give feedback.

Hi @emirot
You can test it out with the latest release; https://github.com/kubernetes-sigs/kubebuilder/releases/tag/v4.11.1
The issue is fixed.