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.

Sign up for GitHub

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

Jump to bottom

Verify that connection-manager timeouts are enforced #3330

Closed
coot opened this issue Sep 9, 2021 · 0 comments · Fixed by #3532
Closed

Verify that connection-manager timeouts are enforced #3330

coot opened this issue Sep 9, 2021 · 0 comments · Fixed by #3532
Assignees
@bolt12
Labels
connection-manager Issues / PRs related to connection-manager testing

Comments

@coot
Copy link
Contributor

coot commented Sep 9, 2021 •
edited
Loading

All the connection manager states with a timeout (^\tau) should be verified.
Once a connection is found in such a state we should verify that it does not
stay longer in that state than specified.

This kind of properties can be nicely expressed using the Signal framework.
@coot coot added networking testing connection-manager Issues / PRs related to connection-manager labels Sep 9, 2021
@coot coot mentioned this issue Sep 9, 2021
Closed
@coot coot added the peer2peer label Sep 9, 2021
@coot coot mentioned this issue Oct 28, 2021
Closed
@bolt12 bolt12 linked a pull request Dec 7, 2021 that will close this issue
Added prop_timeouts_enforced #3532
Merged
@coot coot moved this to In Progress in Ouroboros Network Jan 11, 2022
@bolt12 bolt12 mentioned this issue Jan 13, 2022
Closed
@iohk-bors iohk-bors bot closed this as completed in #3532 Feb 3, 2022
Repository owner moved this from In Progress to Done in Ouroboros Network Feb 3, 2022
@coot coot mentioned this issue Feb 16, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bolt12 bolt12

Labels
connection-manager Issues / PRs related to connection-manager testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added prop_timeouts_enforced IntersectMBO/ouroboros-network
2 participants
@coot @bolt12