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

cassandra/docker-compose: go back to using the latest cassandra version #850

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

cvybhu
Copy link
Contributor

@cvybhu cvybhu commented Oct 19, 2023

In January the tests started failing on Cassandra 4.1, so we pinned the cassandra version to 4.0.7, which didn't have these problems. 4.0.7 is the version that we've been using in the CI since then.

In the meantime there were new releases of Cassandra, at the moment the latest is 4.1.3. Let's try to go back to the latest version, maybe the problems fixed themselves during the last nine months.

Refs: #620
Refs: #629

Fixes: #633

Pre-review checklist

  • have split my patch into logically separate commits.
  • All commit messages clearly explain what they change and why.
  • I added relevant tests for new features and bug fixes.
  • All commits compile, pass static checks and pass test.
  • PR description sums up the changes and reasons why they should be introduced.
  • I have provided docstrings for the public items that I want to introduce.
  • I have adjusted the documentation in ./docs/source/.
  • I added appropriate Fixes: annotations to PR description.

In January the tests started failing on Cassandra `4.1`, so we pinned the cassandra
version to `4.0.7`, which didn't have these problems. `4.0.7` is the version that we've
been using in the CI since then.

In the meantime there were new releases of `Cassandra`, at the moment the latest is `4.1.3`.
Let's try to go back to the latest version, maybe the problems fixed themselves during the last nine months.

Refs: scylladb#620
Refs: scylladb#629

Fixes: scylladb#633

Signed-off-by: Jan Ciolek <[email protected]>
@cvybhu
Copy link
Contributor Author

cvybhu commented Oct 19, 2023

I will rebase this pull request a few times to rerun the CI and make sure that everything works with Cassandra 4.1.3

@cvybhu
Copy link
Contributor Author

cvybhu commented Nov 1, 2023

I ran the cassandra workflow 10 times and it succeeded every time. It looks like everything works fine with the latest cassandra.

image

You can view the runs here :https://github.com/scylladb/scylla-rust-driver/actions/runs/6576484163
Click on image to expand the list.

@cvybhu cvybhu requested a review from Lorak-mmk November 1, 2023 08:32
@Lorak-mmk Lorak-mmk merged commit a5d5345 into scylladb:main Nov 2, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Investigate why tests fail with the newest Cassandra and bring it back in the CI
2 participants