-
Notifications
You must be signed in to change notification settings - Fork 65
scripts: adding scripts to get logs and create consensus reports for specific height #12468
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
base: main-v0.14.1-committer
Are you sure you want to change the base?
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
51596c3 to
3ccc69c
Compare
44a6c72 to
203e3f3
Compare
lev-starkware
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lev-starkware resolved 2 discussions.
Reviewable status: 0 of 2 files reviewed, all discussions resolved (waiting on @TzahiTaub).
203e3f3 to
aba265d
Compare
lev-starkware
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lev-starkware resolved 1 discussion.
Reviewable status: 0 of 2 files reviewed, all discussions resolved (waiting on @TzahiTaub).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
aba265d to
2232394
Compare
lev-starkware
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lev-starkware resolved 2 discussions.
Reviewable status: 0 of 2 files reviewed, all discussions resolved (waiting on @TzahiTaub).

Note
Medium Risk
New operational scripts execute
gcloudsubprocesses and implement non-trivial log parsing/correlation; mistakes or missing config (e.g., expectedenv_map.yaml) could lead to incomplete/incorrect reports or runtime failures, but production code paths are unaffected.Overview
Adds a new
generate_consensus_report_v1_2.pyscript that parses GCP Logs JSON exports for a given block height and emits a canonical plain-text consensus report, including per-round timing/metadata, per-node vote extraction, validation failures with evidence, and resource/weights lookup correlated by namespace and timestamps.Adds a companion
generate_logs_and_report.pyCLI that buildsgcloud logging readqueries (with optional auto/near/custom time windows), downloads the relevant logs for a height, and optionally invokes the report generator to produce a.txtreport (including temp-file handling when no JSON output path is provided).Written by Cursor Bugbot for commit 2232394. This will update automatically on new commits. Configure here.