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

[Flaky Test] AZAffinity Read Strategy Test failing randomly #3027

Open
prateek-kumar-improving opened this issue Jan 30, 2025 · 1 comment
Open
Assignees
Labels
bug Something isn't working Flaky-tests Flaky-tests node Node.js wrapper

Comments

@prateek-kumar-improving
Copy link
Collaborator

prateek-kumar-improving commented Jan 30, 2025

Test Name

GlideClusterClient › AZAffinity Read Strategy Tests › should route GET commands to all replicas with the same AZ using protocol 1

Test Location

No response

Failure Permlink

https://github.com/valkey-io/valkey-glide/actions/runs/13040478467/job/36381014352?pr=2835

Frequency

No response

Steps to Reproduce

System Information

No response

Language and Version

No response

Engine Version

No response

Logs

CI pipeline link: https://github.com/valkey-io/valkey-glide/actions/runs/13040478467/job/36381014352?pr=2835
CI pipeline output:
GlideClusterClient › AZAffinity Read Strategy Tests › should route GET commands to all replicas with the same AZ using protocol 1

expect(received).toBe(expected) // Object.is equality

Expected: 12
Received: 8

  2289 |                     }).length;
  2290 |
> 2291 |                     expect(matching_entries_count).toBe(n_replicas); // Should expect 12 as the cluster was created with 3 primary and 4 replicas, totalling 12 replica nodes
       |                                                    ^
  2292 |                 } finally {
  2293 |                     // Cleanup
  2294 |                     await client_for_config_set?.configSet(

  at tests/GlideClusterClient.test.ts:2291:52
  at fulfilled (tests/GlideClusterClient.test.ts:8:58)

Screenshots

No response

Glide Version

No response

Expected Behavior

No response

Actual Behavior

No response

Possible Fixes

No response

@prateek-kumar-improving prateek-kumar-improving added the bug Something isn't working label Jan 30, 2025
@prateek-kumar-improving prateek-kumar-improving changed the title [Flaky Test] GlideClusterClient › AZAffinity Read Strategy Tests › should route GET commands to all replicas with the same AZ using protocol 1 [Flaky Test] AZAffinity Read Strategy Test failing randomly Jan 30, 2025
@Muhammad-awawdi-amazon
Copy link
Collaborator

We've dealt with this before and increased timeouts. this relates to issue #2946
I'll take a look at it next week

@Muhammad-awawdi-amazon Muhammad-awawdi-amazon added node Node.js wrapper Flaky-tests Flaky-tests labels Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Flaky-tests Flaky-tests node Node.js wrapper
Projects
None yet
Development

No branches or pull requests

2 participants