Skip to content

Conversation

@SupernaviX
Copy link
Collaborator

@SupernaviX SupernaviX commented Nov 13, 2025

Description

Fixes several issues I ran into when trying to replace upstream-chain-fetcher with peer-network-interface.

  • Add the consensus module to the non-mainnet omnibus configurations. Without this, nothing was listening to their messages, so the system wasn't doing much.
  • PeerNetworkInterface wasn't consistently setting new_epoch to true correctly. It was checking if the current slot was the first slot in an epoch, but that is not a reliable detection method because there's no guarantee a block will always be created in that slot. Switch to computing this by tracking the last epoch we reported instead.
  • Update parameter_state to download and bundle preview genesis config values. This lets us run more of the system against preview values.
  • Fix an issue in utxo_state where the module assumed it never processed block 0.

How was this tested?

I ran omnibus with both the sanchonet and preview configurations. It doesn't make it past epoch 2 on either, but it makes it farther than before.

Checklist

  • My code builds and passes local tests
  • I added/updated tests for my changes, where applicable
  • I updated documentation (if applicable)
  • CI is green for this PR

Impact / Side effects

The omnibus preview/sanchonet configurations now actually do something, instead of just spewing blocks into nothingness.

Reviewer notes / Areas to focus

@SupernaviX SupernaviX marked this pull request as ready for review November 14, 2025 02:16
@SupernaviX SupernaviX merged commit 5210273 into main Nov 17, 2025
2 checks passed
@SupernaviX SupernaviX deleted the sg/preview-sanchonet-fixes branch November 17, 2025 14:46
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