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

transport::connection::tests::test_coalescing failed in CI #864

Open
Lorak-mmk opened this issue Nov 21, 2023 · 6 comments · May be fixed by #1188
Open

transport::connection::tests::test_coalescing failed in CI #864

Lorak-mmk opened this issue Nov 21, 2023 · 6 comments · May be fixed by #1188
Assignees

Comments

@Lorak-mmk
Copy link
Collaborator

#863 (comment)

failures:

---- transport::connection::tests::test_coalescing stdout ----
Unique name: test_rust_1700493980_11
thread 'transport::connection::tests::test_coalescing' panicked at scylla/src/transport/connection.rs:2034:74:
called `Result::unwrap()` on an `Err` value: DbError(Unprepared { statement_id: b"i\xb60\x01/\xdc\x02\xb6\xec;\x18\x8e\xaa\xf38\xcf" }, "Prepared query with ID 69b630012fdc02b6ec3b188eaaf338cf not found (either the query was not prepared on this host (maybe the host has been restarted?) or you have prepared too many queries and it has been evicted from the internal cache)")

Strange; I know that we now sometimes prepare statements in case of query as well, but it falls back to execute which automatically reprepares the statement (but just once).

This test failed in CI, but the failure is unrelated to the PR it failed it. Needs to be investigated.

@piodul
Copy link
Collaborator

piodul commented Dec 1, 2023

Fortunately, it's pretty easy to reproduce locally:

  1. Modify the docker-compose command to use the test/cluster/cassandra/docker-compose.yml file,
  2. make up to start the cluster
  3. SCYLLA_URI=172.42.0.2:9042 SCYLLA_URI2=172.42.0.3:9042 SCYLLA_URI3=172.42.0.4:9042 cargo test -- test_coalescing

and it reproduces pretty much every time. What's interesting is that it's not necessary to run all the tests, running just this one test alone is sufficient to reproduce.

@avelanarius avelanarius assigned muzarski and unassigned piodul Feb 23, 2024
@roydahan
Copy link
Collaborator

roydahan commented Jul 1, 2024

Is this test still failing? flakey? or it was removed from CI?

@wprzytula
Copy link
Collaborator

@muzarski
Copy link
Contributor

muzarski commented Jul 31, 2024

@muzarski
Copy link
Contributor

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 a pull request may close this issue.

5 participants