Skip to content

Add version to gloas events#587

Merged
nflaig merged 2 commits intomasterfrom
nflaig/version-events
Mar 17, 2026
Merged

Add version to gloas events#587
nflaig merged 2 commits intomasterfrom
nflaig/version-events

Conversation

@nflaig
Copy link
Copy Markdown
Member

@nflaig nflaig commented Mar 11, 2026

All events that emit data based on spec containers should be wrapped by {"version": "fork_name", "data": { ... }} so we can easily change them across forks and make it easier for consumers as they can just look at the version field and know the format/schema based on the spec container for that fork. This change overall makes the events more forward compatible.

@nflaig nflaig added the Gloas api's needed in Gloas fork. label Mar 11, 2026
Copy link
Copy Markdown
Contributor

@rolfyone rolfyone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@michaelsproul michaelsproul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

I've been meaning to open an issue about updating the head event in a backwards-compatible way. The dependent root fields are wrong since Fulu

rolfyone added a commit to rolfyone/teku that referenced this pull request Mar 12, 2026
Implements change in
ethereum/beacon-APIs#587

If any devnet0 watchers are looking at these, this is non-compatible.
rolfyone added a commit to Consensys/teku that referenced this pull request Mar 17, 2026
Implements change in
ethereum/beacon-APIs#587

If any devnet0 watchers are looking at these, this is non-compatible.
@nflaig nflaig merged commit b4abce4 into master Mar 17, 2026
3 checks passed
@nflaig nflaig deleted the nflaig/version-events branch March 17, 2026 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Gloas api's needed in Gloas fork.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants