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

test: verify proving is resumed after broker crash #11122

Merged
merged 4 commits into from
Feb 20, 2025

Conversation

alexghr
Copy link
Contributor

@alexghr alexghr commented Jan 9, 2025

This test: starts proving an epoch, kills the broker, verifies the epoch is still proven correctly after the broker recovers

This test is currently failing because it needs #10981 first to be merged in.

@alexghr alexghr force-pushed the ag/test-broker-recovery branch from 679201a to 261ef21 Compare February 20, 2025 14:05
@alexghr alexghr requested a review from charlielye as a code owner February 20, 2025 17:09
@alexghr alexghr merged commit e00a86c into master Feb 20, 2025
10 checks passed
@alexghr alexghr deleted the ag/test-broker-recovery branch February 20, 2025 18:14
TomAFrench added a commit that referenced this pull request Feb 20, 2025
* master: (300 commits)
  fix(ci): don't have checks go green immediately (#12168)
  fix: ASSERTS that should throw (#12167)
  fix: retry rm operation in cleanup (#12162)
  chore: Fix linter errors (#12164)
  feat: Barretenberg C++ binary overhaul (#11459)
  fix: call install_hooks in bootstrap (#12159)
  chore: @aztec/stdlib pt. 3: aztec-address out of foundation (#12140)
  test: verify proving is resumed after broker crash (#11122)
  chore(ci3): update ci.md with swc notes (#12147)
  fix: don't try to get bench artifacts on external PR (#12157)
  feat: partial note handling in aztec-nr (#12122)
  fix: external fixes pt 2 (#12153)
  chore: fix message path (#12150)
  chore(ci3): refactor ci3.yml, fix external PR flow (#12037)
  fix: Do not try flushing txs in bot setup if not set (#12144)
  chore: Silence warns on invalid bootnode enr (#12135)
  fix: don't early-out on test fails (#12143)
  feat(avm): deduplicating event emitters (#12137)
  chore: @aztec/stdlib pt.2 -> remove @aztec/types (#12133)
  test: kill prover node and see it recover (#11118)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Prover][Test] Broker failure: system must recover if the ProvingBroker goes down
2 participants