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

bug: flaky peer-exchange compliance test #1477

Closed
danisharora099 opened this issue Aug 14, 2023 · 3 comments
Closed

bug: flaky peer-exchange compliance test #1477

danisharora099 opened this issue Aug 14, 2023 · 3 comments
Assignees
Labels
test Issue related to the test suite with no expected consequence to production code

Comments

@danisharora099
Copy link
Collaborator

This is a bug report

Problem

The peer-exchange compliance test within js-waku has been inconsistent and flaky on its runs:
it was seen to be flakily passing/failing based on local/CI envs without any reproducible/deterministic way.
Recently during some deps upgrade, this flaky test started to consistently fail in the CI while inconsistently passing in the local env.
Certain attempts like #1468 were made to resolve this.

Proposed Solutions

To be figured out

Notes

Failing peer-exchange compliance test might be related to libp2p/js-libp2p#1846

investigation led to a conclusion that peer:update/peer:identify might not be get triggered within the CI
(#1472 (comment))

@danisharora099
Copy link
Collaborator Author

danisharora099 commented Aug 14, 2023

Solved with #1478

But will keep this issue open to discuss:

we have added a simple timeout to the dial which waits for the discovery class to have initialised; while this might work, there are theoretically still chances for race conditions

@weboko weboko added the test Issue related to the test suite with no expected consequence to production code label Aug 31, 2023
@weboko
Copy link
Collaborator

weboko commented Aug 31, 2023

Next steps:

  • do profiling and try to get it happen locally;
  • check logs step by step

@danisharora099
Copy link
Collaborator Author

we've done a few improvements to remove flakiness like retries, and the said issue has not been reproduced since #1478 was merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Issue related to the test suite with no expected consequence to production code
Projects
Archived in project
Development

No branches or pull requests

2 participants