Closed p-offtermatt closed 3 months ago
.github/workflows/nightly-e2e.yml (2)
`296-311`: The configuration for the `active-set-changes-test` job is consistent with other test jobs in terms of Go version and setup steps. --- `331-332`: Correctly added `active-set-changes-test` to the dependency list for the `nightly-test-fail` job.
Does it make sense to also add this test in the nightly tests?
Ah, great idea! I was thinking it's probably not right to run it on every commit, but nightly seems good
LGTM. Great work!
Description
Closes: #XXXX
The test case uses the default config, except it sets the MaxValidators to 2.
Run the test locally by running
These are the steps to reproduce the chain halt:
The chain halt will make the test time out. You can look at the provider nodes logs by running
and they should include this:
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues.
I have...
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.
I have...
Summary by CodeRabbit
New Features
Chores