Skip to content

fix: add params module to pacific-1 genesis#5

Merged
bdchatham merged 2 commits intomainfrom
fix/pacific1-genesis-params
Apr 14, 2026
Merged

fix: add params module to pacific-1 genesis#5
bdchatham merged 2 commits intomainfrom
fix/pacific1-genesis-params

Conversation

@bdchatham
Copy link
Copy Markdown
Collaborator

@bdchatham bdchatham commented Apr 14, 2026

Summary

  • Populates the params module in pacific-1's embedded genesis (was null)
  • Fixes v6.4.1 panic on cold start: cosmos gas multiplier numerator can not be 0

Context

When a new node attempts to block sync from genesis using seid v6.4.1, InitGenesis calls SetCosmosGasParams which validates that the numerator and denominator are non-zero. With "params": null in the genesis, the proto-unmarshaled CosmosGasParams has zero values, causing a panic.

Values are sourced from on-chain state on a running pacific-1 node:

$ seid query params cosmosgasparams
cosmos_gas_multiplier_denominator: "1"
cosmos_gas_multiplier_numerator: "1"

$ seid query params feesparams
global_minimum_gas_prices:
- amount: "0.020000000000000000"
  denom: usei

Test plan

  • go test ./... passes
  • Archive node cold start from genesis with v6.4.1 no longer panics

🤖 Generated with Claude Code

bdchatham and others added 2 commits April 14, 2026 10:31
The params module genesis was null, causing seid v6.4.1 to panic
during InitGenesis with "cosmos gas multiplier numerator can not be 0".
The zero-value CosmosGasParams fails validation in SetCosmosGasParams.

Populates the params module with on-chain values queried from a running
pacific-1 node:
- cosmosGasMultiplierNumerator: 1
- cosmosGasMultiplierDenominator: 1
- globalMinimumGasPrices: 0.02 usei

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Same issue as pacific-1: the params module was either null or missing
cosmosGasParams, causing v6.4.1 to panic on cold start.

On-chain values are identical across all three networks:
- cosmosGasMultiplierNumerator: 1
- cosmosGasMultiplierDenominator: 1
- globalMinimumGasPrices: 0.02 usei

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@bdchatham bdchatham merged commit 6b3762e into main Apr 14, 2026
2 checks passed
bdchatham added a commit to sei-protocol/seictl that referenced this pull request Apr 14, 2026
## Summary

- Bumps sei-config from v0.0.9 to v0.0.12
- Picks up the genesis params fix for all three networks
(sei-protocol/sei-config#5)

## Context

The `configure-genesis` sidecar task writes the embedded genesis from
sei-config. Without the params module populated, seid v6.4.1 panics
during `InitGenesis` with "cosmos gas multiplier numerator can not be
0". This blocks archive nodes from block syncing from genesis.

## Test plan
- [ ] CI passes
- [ ] Archive node cold start from genesis no longer panics

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

1 participant