Skip to content

fix(presets/genesis-chain): opt validators into sidecar signing via operatorKeyring#185

Closed
bdchatham wants to merge 1 commit into
mainfrom
fix/genesis-chain-preset-operator-keyring
Closed

fix(presets/genesis-chain): opt validators into sidecar signing via operatorKeyring#185
bdchatham wants to merge 1 commit into
mainfrom
fix/genesis-chain-preset-operator-keyring

Conversation

@bdchatham
Copy link
Copy Markdown
Contributor

Summary

  • Add `operatorKeyring: {}` to the bench validator template in the `genesis-chain` preset. Without it, sei-k8s-controller emits no `SEI_KEYRING_BACKEND` on the sidecar, the Cosmos SDK can't open any keyring, and gov-vote / gov-software-upgrade tasks fail their preconditions.
  • With the empty block, the controller wires the sidecar with `SEI_KEYRING_BACKEND=test` + `SEI_KEYRING_DIR=$SEI_HOME/keyring-test` — exactly where the gentx task writes the validator key during the genesis ceremony. The sidecar reads it back and gov-vote / gov-software-upgrade unlock for nightly major-upgrade workflows.

Depends on

Test plan

  • `go test ./nodedeployment/...` passes (preset is YAML; tests load it as-is).
  • End-to-end: nightly major-upgrade workflow on harbor cluster signs gov-vote + gov-software-upgrade tasks successfully after both this PR and the controller PR land.

🤖 Generated with Claude Code

…peratorKeyring

Adds operatorKeyring: {} to the bench validator template. Without it,
sei-k8s-controller emits no SEI_KEYRING_BACKEND env on the sidecar, so the
Cosmos SDK can't open any keyring and gov-vote / gov-software-upgrade tasks
fail their preconditions.

With the empty block, the controller wires the sidecar with
SEI_KEYRING_BACKEND=test + SEI_KEYRING_DIR=\$SEI_HOME/keyring-test —
exactly where the gentx task writes the validator key during the genesis
ceremony. The sidecar reads it back, gov-vote / gov-software-upgrade
unlock for nightly major-upgrade workflows.

Depends on sei-k8s-controller#296.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@cursor
Copy link
Copy Markdown

cursor Bot commented May 20, 2026

You have used all Bugbot PR reviews included in your free trial for your GitHub account on this workspace.

To continue using Bugbot reviews, enable Bugbot for your team in the Cursor dashboard.

@bdchatham
Copy link
Copy Markdown
Contributor Author

Superseded — the preset opt-in is no longer needed. Per cross-review (sei-protocol/sei-k8s-controller#296), the controller now defaults validators to a test-backend keyring rooted at $SEI_HOME (the SDK appends keyring-test/), removing the need for an explicit operatorKeyring: {} block on the preset. Replaced by #186 (rename gentx uid to node_admin) which fixes the actual blocker (key uid mismatch between gentx and the sidecar's signing flow).

@bdchatham bdchatham closed this May 20, 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.

1 participant