-
Notifications
You must be signed in to change notification settings - Fork 79
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
Support hcd #1319
Support hcd #1319
Conversation
return float64(256) | ||
} | ||
if kc.Spec.Cassandra.ServerType.IsCassandra() { |
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.
Isn't this breaking backwards compatibility? 16 is the default for OSS versions and all previous deployments would use that.
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.
you're right, and I shouldn't have touched this anyway here.
@@ -10258,7 +10258,7 @@ spec: | |||
ServerVersion is the Cassandra or DSE version. The following versions are supported: | |||
- Cassandra: 3.11.X, 4.X.X and 5.X.X | |||
- DSE: 6.8.X, 7.x.x | |||
pattern: (6\.8\.\d+)|(3\.11\.\d+)|(4\.\d+\.\d+)|(5\.\d+\.\d+)|(7\.\d+\.\d+) | |||
- HCD: 1.0.x |
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.
suggestion: We shouldn't remove pattern entirely, it should be at least something like \d+\.\d+\.\d+
. We're not supporting versions like 6.9
that would auto-update to newest point-release for example.
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.
fair enough 👍
@burmanm , I addressed your comments. Could you review again? |
Quality Gate passedIssues Measures |
What this PR does:
Adds support for HCD as server type and remove the regex checks for server versions which already exist in cass-operator.
Fixes #1320
Checklist