Skip to content

track slot as part of fork choice debug API#4565

Merged
etan-status merged 3 commits intounstablefrom
dev/etan/fc-debugslot
Jan 31, 2023
Merged

track slot as part of fork choice debug API#4565
etan-status merged 3 commits intounstablefrom
dev/etan/fc-debugslot

Conversation

@etan-status
Copy link
Contributor

Extends fork choice state to also track slot numbers to improve accuracy of /eth/v1/debug/fork_choice endpoint. Autoenable this API on devnet, and disable some extra checks on devnet to aid focused testing efforts. Align fork choice pruning logic with API based on checkpoints vs root.

Extends fork choice state to also track slot numbers to improve accuracy
of `/eth/v1/debug/fork_choice` endpoint. Autoenable this API on devnet,
and disable some extra checks on devnet to aid focused testing efforts.
Align fork choice pruning logic with API based on checkpoints vs root.
@tersec
Copy link
Contributor

tersec commented Jan 30, 2023

macos-amd64 failing due to same issue as in #4569 -- seems to be/have been a Homebrew outage. It doesn't look like the GitHub Actions CI actually requires gnu-getopt brew, as that PR passes even without it, but perhaps it is required.

@github-actions
Copy link

Unit Test Results

         6 files   -        3       710 suites   - 355   19m 47s ⏱️ - 13m 51s
  3 547 tests ±       0    3 310 ✔️ ±       0  237 💤 ±  0  0 ±0 
10 256 runs   - 5 130  10 014 ✔️  - 5 107  242 💤  - 23  0 ±0 

Results for commit d747ea4. ± Comparison against base commit b2f415c.

@etan-status etan-status enabled auto-merge (squash) January 31, 2023 11:39
@etan-status etan-status merged commit ea6a6b1 into unstable Jan 31, 2023
@etan-status etan-status deleted the dev/etan/fc-debugslot branch January 31, 2023 12:35
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.

2 participants