Skip to content
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

Tests need to verify that everything works with older mgmt-api versions also #201

Closed
burmanm opened this issue Oct 13, 2021 · 1 comment · Fixed by #209
Closed

Tests need to verify that everything works with older mgmt-api versions also #201

burmanm opened this issue Oct 13, 2021 · 1 comment · Fixed by #209
Labels
bug Something isn't working

Comments

@burmanm
Copy link
Contributor

burmanm commented Oct 13, 2021

What happened?
As seen in PR #176 , if we had only tested this with a new version of management-apis, all the tests would pass. But as soon as user would use an older version things would break completely and the cass-operator is completely unusable. We need to verify that new features that use newer features from management-api also work with older serverImages.

@burmanm burmanm added the bug Something isn't working label Oct 13, 2021
@jsanda
Copy link
Contributor

jsanda commented Oct 13, 2021

It's not clear to me that we need to do this. Don't get me wrong. It would certainly be good to do, but I question whether it is necessary. Alternatively would it be sufficient to simply document version compatibility?

@jdonenine What are your thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants