Skip to content

Commit e5e0ec5

Browse files
committed
ci: enable ErrorsRequestTimeouts metrics test
To be quite honest, I thought we'll be able to enable more tests, but: 1. StatsConnections Requires `cass_cluster_set_num_threads_io`, `cass_cluster_set_core_connections_per_host` and `cass_cluster_set_constant_reconnect`. 2. ErrorsConnectionTimeouts Requires `cass_cluster_set_core_connections_per_host`. 3. SpeculativeExecutionRequests Requires `cass_session_get_speculative_execution_metrics`. 4. Requests This one is interesting. It turns out that cpp-driver stores latency stats as microseconds, while rust-driver stores them as milliseconds. Because of that, the mean and median latency is rounded to 0 (at least for my machine). The test expects them to be greater than 0, which makes sense assuming the driver collects the stats with microsecond precision. I'm not sure how to address this one. Is there any way to force higher >=1ms latencies in the test?
1 parent f23e98b commit e5e0ec5

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

Makefile

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ SCYLLA_TEST_FILTER := $(subst ${SPACE},${EMPTY},ClusterTests.*\
2525
:LoggingTests.*\
2626
:PreparedMetadataTests.*\
2727
:UseKeyspaceCaseSensitiveTests.*\
28+
:MetricsTests.Integration_Cassandra_ErrorsRequestTimeouts\
2829
:-PreparedTests.Integration_Cassandra_PreparedIDUnchangedDuringReprepare\
2930
:HeartbeatTests.Integration_Cassandra_HeartbeatFailed\
3031
:ControlConnectionTests.Integration_Cassandra_TopologyChange\
@@ -64,6 +65,7 @@ CASSANDRA_TEST_FILTER := $(subst ${SPACE},${EMPTY},ClusterTests.*\
6465
:LoggingTests.*\
6566
:PreparedMetadataTests.*\
6667
:UseKeyspaceCaseSensitiveTests.*\
68+
:MetricsTests.Integration_Cassandra_ErrorsRequestTimeouts\
6769
:-PreparedTests.Integration_Cassandra_PreparedIDUnchangedDuringReprepare\
6870
:PreparedTests.Integration_Cassandra_FailFastWhenPreparedIDChangesDuringReprepare\
6971
:HeartbeatTests.Integration_Cassandra_HeartbeatFailed\

0 commit comments

Comments
 (0)