-
Notifications
You must be signed in to change notification settings - Fork 8
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
[CI] Delete validator pod if stuck during E2E test #415
Conversation
The CI will now also run the e2e tests on devnet, which increases the time it takes to complete all CI checks. If you just created a pull request, you might need to push another commit to produce a container image DevNet can utilize to spin up infrastructure. You can use |
The image is going to be pushed after the next commit. You can use |
* feat/proof-validation: Offline review of 406 [LocalNet] Fix/localnet regenesis (#424) [CI] Delete validator pod if stuck during E2E test (#415) Fixing reletive link in documentation [Docs] Add code review guidelines & developer tips (#426) [Quick change] Bump default gas limit (#431) chore: Fix unit tests and ring client removal consideration chore: Remove pubkey client chore: Add missing change requests chore: Address review change requests Update pull_request_template.md Update pull_request_template.md [Cleanup] Wrap errors consistently via error types. (#422) [LocalNet] Allow to scale up actors (#414) [Test] Replace time waiting with message submission waiting (#418) Remove accidentally added config files (#420) fix: Ignore externally initiated transactions (#407) [Bug] Address go-routines leak (#410)
Summary
Human Summary
Delete stuck validator Pod during E2E test.
AI Summary
reviewpad:summary
Issue
DevNets now use a new helm chart that provisions validators as StatefulSets. As a result, if such validator had unsuccessful deployment, it won't be upgraded to the newer version due to StatefulSet uptime guarantees. We need to kill that pod, so StatefulSet controller creates a new one with correct image.
Type of change
Select one or more:
Testing
make go_develop_and_test
make test_e2e
devnet-test-e2e
label to the PR. This is VERY expensive, only do it after all the reviews are complete.Sanity Checklist