-
-
Notifications
You must be signed in to change notification settings - Fork 12
Don't delete CRDs when upgrading #405
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
Conversation
See https://stackable-workspace.slack.com/archives/C031BTE7ZGC/p1683797224359929 This should also be cherry-picked to `main`
✅ Deploy Preview for stackable-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
LGTM 👌
Should I update the backport PR, or will we handle that as part of the 23.4.1 release process? @adwk67 |
@nightkr Yes please! The 23.4.1 won't make any automatic changes, so everything should be cherry-picked into branch 23.4 first, before we tag and build everything. |
Ok. |
See https://stackable-workspace.slack.com/archives/C031BTE7ZGC/p1683797224359929
This replaces #402 (because I was dumb and should have done the PR against
main
first), and should be cherry-picked back into 23.4.