Skip to content

feat: add sequencer-only benchmark role#188

Draft
niran wants to merge 1 commit into
mainfrom
niran/sequencer-only-roles
Draft

feat: add sequencer-only benchmark role#188
niran wants to merge 1 commit into
mainfrom
niran/sequencer-only-roles

Conversation

@niran
Copy link
Copy Markdown
Contributor

@niran niran commented May 19, 2026

Summary

  • Add benchmark-level roles config while preserving the default sequencer+validator behavior.
  • Support roles: [sequencer] to skip validator datadir setup, execution, log export, and validator metric requirements.
  • Make validator metrics optional in run metadata.
  • Document the sequencer-only snapshot/load-test use case.

Tests

  • go test ./...

@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented May 19, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/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

@niran niran marked this pull request as draft May 19, 2026 19:03
@niran niran changed the title Add sequencer-only benchmark role feat: add sequencer-only benchmark role May 21, 2026
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