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

Downgrade Cassandra version for CI #629

Merged
merged 1 commit into from
Jan 13, 2023
Merged

Conversation

Gor027
Copy link
Contributor

@Gor027 Gor027 commented Jan 13, 2023

The cassandra tests were run on the latest version of Cassandra 4.1.0 and most probably the version contains some bugs as some tests were flaky. This PR pins version 4.0.7 for creating a 3-node Cassandra cluster, which solves the flakiness problem, however, the root cause of the problem is still unknown.
Additionally, memory flags HEAP_NEWSIZE=512M and MAX_HEAP_SIZE=2048M are added for each node creation as the GitHub Actions Linux virtual machine has a RAM limit of 7GB.

Fixes: #620

Pre-review checklist

  • I 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 added appropriate Fixes: annotations to PR description.

The cassandra tests were run on the latest version of Cassandra 4.1.0
and most probably the version contains some bugs as some tests were
flaky. This PR pins version 4.0.7 for creating a 3-node Cassandra
cluster, which solves the flakiness problem, however, the root cause of
the problem is still unknown.
Additionally, memory flags `HEAP_NEWSIZE=512M` and `MAX_HEAP_SIZE=2048M`
are added for each node creation as the GitHub Actions Linux virtual
machine has a RAM limit of 7GB.
@Gor027 Gor027 requested a review from piodul January 13, 2023 17:19
@cvybhu cvybhu merged commit 0e139c9 into scylladb:main Jan 13, 2023
cvybhu added a commit to cvybhu/scylla-rust-driver that referenced this pull request 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: scylladb#620
Refs: scylladb#629

Fixes: scylladb#633

Signed-off-by: Jan Ciolek <jan.ciolek@scylladb.com>
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 Cassandra tests flakiness
2 participants