-
Notifications
You must be signed in to change notification settings - Fork 118
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
docs: upgrade Keptn to Helm from a manifest installation #2270
Conversation
✅ Deploy Preview for keptn-lifecycle-toolkit ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the best way to support this would be providing a CRD-less manifest for each old installation. This way it could simply be: kubectl delete -f keptn-vX.Y.Z-crdless.yaml; helm install ...
This way the CRD definition will live and the CR won't be lost, but all the rest is cleaned up. I don't know, however, how Helm handles the CRD again since are already there 🤔
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Co-authored-by: Meg McRoberts <meg.mcroberts@dynatrace.com> Signed-off-by: odubajDT <93584209+odubajDT@users.noreply.github.com>
Co-authored-by: Meg McRoberts <meg.mcroberts@dynatrace.com> Signed-off-by: odubajDT <93584209+odubajDT@users.noreply.github.com>
If we provide a CRD-less chart than users need to update CRDs manually via manifest, and they are stuck forever in this 😢 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm thinking that we should pull the backup info into its own page in the "Operate" section then link to that from here. But let's merge this PR first then I can do that in a separate PR.
Signed-off-by: odubajDT <93584209+odubajDT@users.noreply.github.com>
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Fixes: #1176