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

Restart based test for divergence #4557

Closed
mhofman opened this issue Feb 15, 2022 · 1 comment
Closed

Restart based test for divergence #4557

mhofman opened this issue Feb 15, 2022 · 1 comment
Assignees
Labels
cosmic-swingset package: cosmic-swingset enhancement New feature or request tooling repo-wide infrastructure vaults_triage DO NOT USE

Comments

@mhofman
Copy link
Member

mhofman commented Feb 15, 2022

What is the Problem Being Solved?

With the addition of #4133 and #4256, we now have an integration test that can detect simple causes of divergence (closing #3444).

However in the past we've encountered other triggers for divergence between nodes, in particular when a node restarts, e.g.:

Description of the Design

A testnet with at least 4 nodes, with loadgen applied, where one node at a time gets randomly restarted. For best results, the loadgen should use a random RPC node which will get restarted.

Security Considerations

None

Test Plan

Probably part of a standalone nightly test.

@mhofman mhofman added enhancement New feature or request cosmic-swingset package: cosmic-swingset tooling repo-wide infrastructure labels Feb 15, 2022
@Tartuffo Tartuffo added this to the Mainnet 1 milestone Mar 23, 2022
@Tartuffo Tartuffo removed this from the Mainnet 1 RC0 milestone Sep 21, 2022
@ivanlei ivanlei added the vaults_triage DO NOT USE label Jan 17, 2023
@ivanlei
Copy link
Contributor

ivanlei commented Jan 24, 2023

From @mhofman in slack --

Yeah the issues about hunting down divergences now have a better story given the improvements I've made to replay tools. I think we no longer need to use the loadgen or instagoric as the means to detect these. We could still use them, but mostly to generate transcripts that could be used as input to the replay tools.

@ivanlei ivanlei closed this as not planned Won't fix, can't repro, duplicate, stale Jan 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cosmic-swingset package: cosmic-swingset enhancement New feature or request tooling repo-wide infrastructure vaults_triage DO NOT USE
Projects
None yet
Development

No branches or pull requests

4 participants