Skip to content

ci: remove load test benchmark workflow#176

Merged
kmchicoine merged 8 commits intomainfrom
remove-load-test-ci
May 1, 2026
Merged

ci: remove load test benchmark workflow#176
kmchicoine merged 8 commits intomainfrom
remove-load-test-ci

Conversation

@kmchicoine
Copy link
Copy Markdown
Contributor

Summary

  • Removes the load test benchmark GitHub Action workflow that ran on PRs and manual dispatch

- ThroughputChart: D3 dual-axis line chart (TPS left, gas/s right) with
  hover tooltip; positioned by elapsed_secs to handle irregular sample
  spacing
- ConfigCard: grouped grid showing load shape, target, funding, repro
  seed, and workload mix; omits null fields rather than rendering '\u2014'
- formatEthFromWeiString: BigInt-based wei\u2192ETH for u128 strings that
  exceed Number.MAX_SAFE_INTEGER
- Fix top_failure_reasons schema mismatch: API returns [reason, count]
  tuples, not {reason, count} objects
- New config and throughput_timeseries fields are optional on
  LoadTestResult; sections are gated on presence so older runs that
  predate these fields still render cleanly
@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented May 1, 2026

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@kmchicoine kmchicoine enabled auto-merge (squash) May 1, 2026 20:54
@kmchicoine kmchicoine merged commit a371c25 into main May 1, 2026
25 of 26 checks passed
@kmchicoine kmchicoine deleted the remove-load-test-ci branch May 1, 2026 21:27
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.

3 participants