Cosmos-sdk: gaiacli query returns error in all usage

[jackzampolin]

Summary of Bug

gaiacli query * is returning error for validators who have already bonded.

Steps to Reproduce

• Join a node to the gaia-9000 testnet
• gaiacli tx create-validator to create your validator
• Run gaiacli query signing-info $(gaiad tendermint show-validator) --chain-id gaia-9000
• See the error:

ERROR: Invalid commit -- insufficient old voting power: got 200, needed 267

---

For Admin Use

• [ ] Not duplicate issue
• [ ] Appropriate labels applied
• [ ] Appropriate contributors tagged
• [ ] Contributor assigned/self-assigned

[zmanian]

This can be overridden via the --trust-node option that disables light client checks.

[alexanderbez]

Mhmmm, this might be tricky. I cannot reproduce this on the latest develop running a localnet. I can, however, reproduce the following when running: gaiacli query signing-info

ERROR: failed to prove merkle proof: expected 0 args, got 1

I'm not sure the two are related though.

[jackzampolin]

Per @cwgoes this error is likely light client verification related and will require a tendermint change.

[alexanderbez]

Which error exactly? Care to shed some more light here @cwgoes ?

[cwgoes]

This one: ERROR: Invalid commit -- insufficient old voting power: got 200, needed 267 (from the original post).

[cwgoes]

Also, we should communicate these kinds of errors better to the CLI user.

[zmanian]

I think the reason why we can't replicated this on local testnets is because the local testnet doesn't have a dynamic validator set.

What would it take have a testnet with a dynamic validator set?

[jackzampolin]

cc @greg-szabo @mircea-c

[cwgoes]

This should be moved to Tendermint, it's not an SDK issue. The relevant error is in types/validator_set.go - https://github.com/tendermint/tendermint/blob/master/types/validator_set.go#L306.

[cwgoes]

Closing in favor of https://github.com/tendermint/tendermint/issues/2862.