Skip to content

core/app: fork-aware slot fraction timings - #4676

Open
KaloyanTanev wants to merge 4 commits into
gloasfrom
kalo/gloas-proposer-deadline
Open

core/app: fork-aware slot fraction timings#4676
KaloyanTanev wants to merge 4 commits into
gloasfrom
kalo/gloas-proposer-deadline

Conversation

@KaloyanTanev

Copy link
Copy Markdown
Collaborator

Make the remaining thirds-based intra-slot timings fork-aware for the gloas quarter-slot schedule: the proposer/randao duty deadline now follows the attestation due offset (4s pre-gloas, 3s post) via newSlotOffsetFunc, and the SSE block/block-gossip lateness thresholds resolve the same offset per slot instead of a hardcoded slotDuration/3. Also refit sse_head_delay histogram buckets with quarter-slot boundaries (3s/9s, keeping 4s so existing dashboard queries are unaffected) and reword the thirds-era metric help texts.

category: feature
ticket: #4324

@KaloyanTanev KaloyanTanev self-assigned this Sep 1, 2026
@github-actions github-actions Bot added the branch-invalid PR raised against invalid branch. Not a main or release branch. label Sep 1, 2026
@KaloyanTanev KaloyanTanev mentioned this pull request Sep 1, 2026
33 tasks
@sonarqubecloud

sonarqubecloud Bot commented Sep 1, 2026

Copy link
Copy Markdown

@codecov

codecov Bot commented Sep 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 68.75000% with 5 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (gloas@c02d596). Learn more about missing BASE report.

Files with missing lines Patch % Lines
app/sse/listener.go 72.72% 2 Missing and 1 partial ⚠️
core/deadline.go 60.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##             gloas    #4676   +/-   ##
========================================
  Coverage         ?   58.81%           
========================================
  Files            ?      246           
  Lines            ?    34896           
  Branches         ?        0           
========================================
  Hits             ?    20525           
  Misses           ?    11811           
  Partials         ?     2560           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch-invalid PR raised against invalid branch. Not a main or release branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant