fix(test): enable REST API for the upgrade chain (TestChainUpgrade)#439
Conversation
TestChainUpgrade resolves the proposal ID and polls upgrade state over the Cosmos gov REST API, which is off by default — so the generic provisioning left port 1317 connection-refused and resolveProposalID polled a dead endpoint. Add the configOverrides the major-upgrade scenario uses (api.rest.enable, tx_index.indexer=kv). Smoke-proven: this is the fix that made TestChainUpgrade PASS live on harbor (the change landed after the suite commit was pushed, so it missed #438). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
PR SummaryLow Risk Overview This PR adds Reviewed by Cursor Bugbot for commit a6e3199. Bugbot is set up for automated code reviews on this repo. Configure here. |
What
TestChainUpgrade(merged in #438) resolves the proposal ID and polls the upgrade state over the Cosmos gov REST API, which is off by default. The generic provisioning omitted the override, so port 1317 was connection-refused andresolveProposalIDpolled a dead endpoint to the deadline. Adds theconfigOverridesthe major-upgrade scenario uses:api.rest.enable=true,tx_index.indexer=kv.Why it missed #438
This fix landed in my working tree after the #438 branch was pushed (the live smoke surfaced the gap), so the smoke-proven binary had it but the merged commit did not. This restores parity between main and the version that passed live.
Test
Smoke-proven: this is exactly the change that made
--- PASS: TestChainUpgrade (319s)on harbor with the realv6.5image pair.go build/vet -tags integration+golangci-lintclean.🤖 Generated with Claude Code