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

Bump Go/C* versions #1813

Closed
wants to merge 2 commits into from
Closed

Conversation

absurdfarce
Copy link
Contributor

@absurdfarce absurdfarce commented Sep 5, 2024

In prep for 1.7.0 release.

Ambition here is to match the "Sunsetting Model" that was in place with gocql.

Test run shows this passes all existing tests.

@smiklosovic
Copy link
Contributor

we are contemplating about doing this in #1812

Would you mind to take it there?

@absurdfarce
Copy link
Contributor Author

absurdfarce commented Sep 5, 2024

@smiklosovic I actually think this should be handled independently. We should upgrade the Go versions for 1.7.0 and I'd rather not bring any additional changes along with that if we can avoid it. Ideally 1.7.0 would basically be what was in 1.6.0 along with a few changes related to the release/ASF conversion.

go: [ '1.19', '1.20' ]
cassandra_version: [ '4.0.8', '4.1.1' ]
go: [ '1.22', '1.23' ]
cassandra_version: [ '4.0.13', '4.1.6' ]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

4.1.6 and 4.0.13 are the latest 4.1.x and 4.0.x as of this writing. But now I'm wondering if we shouldn't be aiming for 4.1.x & 5.0.0 instead.

Trouble is stock ccm has issues with 5.0.0... and supporting a new major version of Cassandra probably needs a ticket (or perhaps even tickets) all to itself.

@absurdfarce
Copy link
Contributor Author

Closing in favor of #1825

@absurdfarce absurdfarce deleted the 170-test-updates branch September 20, 2024 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants