-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat(pvc): split PVCs for database and storage #1010
base: split-deployment
Are you sure you want to change the base?
Conversation
/build_test |
|
/build_test |
|
@ebaron is this because I renamed the CRD property in v1beta2 compared to what v1beta1 had? |
I'm not sure if this is causing the test failure, but I don't think we can break API by renaming an existing property in v1beta2: I think the proper way would be to deprecate the old property, but still use it as a fallback if the new properties aren't given.
|
/build_test |
|
|
Welcome to Cryostat! 👋
Before contributing, make sure you have:
main
branch[chore, ci, docs, feat, fix, test]
git commit -S -m "YOUR_COMMIT_MESSAGE"
Fixes: #1004
Description of the change:
This change adds allows the users to provide...
Motivation for the change:
This change is helpful because users may want to...
How to manually test: