Skip to content

Conversation

ggivo
Copy link
Collaborator

@ggivo ggivo commented Aug 19, 2025

Disable the health checks by default in FailoverIntegrationTest to avoid interfering with CB tests

Error:  Tests run: 7, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 1.297 s <<< FAILURE! -- in redis.clients.jedis.failover.FailoverIntegrationTest
Error:  redis.clients.jedis.failover.FailoverIntegrationTest.testManualFailoverInflightCommandsWithErrorsPropagateError -- Time elapsed: 0.035 s <<< ERROR!
redis.clients.jedis.exceptions.JedisValidationException: cluster:localhost:29380 failed to connect. Please check configuration and try again.
	at redis.clients.jedis.providers.MultiClusterPooledConnectionProvider.validateTargetConnection(MultiClusterPooledConnectionProvider.java:481)
	at redis.clients.jedis.providers.MultiClusterPooledConnectionProvider.setActiveCluster(MultiClusterPooledConnectionProvider.java:526)
	at redis.clients.jedis.providers.MultiClusterPooledConnectionProvider.setActiveCluster(MultiClusterPooledConnectionProvider.java:494)
	at redis.clients.jedis.failover.FailoverIntegrationTest.testManualFailoverInflightCommandsWithErrorsPropagateError(FailoverIntegrationTest.java:217)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at java.util.ArrayList.forEach(ArrayList.java:1259)
	at java.util.ArrayList.forEach(ArrayList.java:1259)
Caused by: redis.clients.jedis.exceptions.JedisConnectionException: Cluster is not healthy
	at redis.clients.jedis.providers.MultiClusterPooledConnectionProvider$Cluster.getConnection(MultiClusterPooledConnectionProvider.java:653)
	at redis.clients.jedis.providers.MultiClusterPooledConnectionProvider.validateTargetConnection(MultiClusterPooledConnectionProvider.java:471)
	... 6 more
@ggivo ggivo changed the base branch from master to feature/automatic-failover August 19, 2025 10:54
@ggivo ggivo changed the title Attempt to fix flaky test [automatic failover] Attempt to fix flaky test Aug 19, 2025
@uglide
Copy link
Contributor

uglide commented Aug 22, 2025

run scenario tests

@uglide
Copy link
Contributor

uglide commented Aug 22, 2025

Testcase Errors Failures Skipped Total
redis.clients.jedis.scenario.ActiveActiveFailoverTest 0 1 0 1
redis.clients.jedis.scenario.ClusterTopologyRefreshTest 0 0 0 1
redis.clients.jedis.scenario.ConnectionInterruptionTest 0 0 0 4

---- Details for maintainers

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.

2 participants