informalsystems / hermes-sdk

Apache License 2.0
9 stars 3 forks source link

Connection OpenTry from Cosmos to Sovereign fails #331

Closed soareschen closed 3 months ago

soareschen commented 4 months ago

When relaying Connection OpenTry from Cosmos to Sovereign, the message fails on Sovereign with error in the form:

Processing IBC core message: Connection(OpenTry(MsgConnectionOpenTry { client_id_on_b: ClientId("07-tendermint-0"), client_state_of_b_on_a: Any { type_url: "/ibc.lightclients.wasm.v1.ClientState", ... }, counterparty: Counterparty { client_id: ClientId("08-wasm-0"), connection_id: Some(ConnectionId("connection-0")), prefix: ibc }, versions_on_a: [Version { identifier: "1", features: ["ORDER_ORDERED", "ORDER_UNORDERED"] }], ... , proofs_height_on_a: Height { revision: 2117181155, height: 29 }, consensus_height_of_b_on_a: Height { revision: 0, height: 8 }, delay_period: 0ns, signer: Signer("cosmos000000000000000000000000000000000000000"), proof_consensus_state_of_b: None, previous_connection_id: "" })) at visible slot number: 10
Tx 0xbd146a6ce7083b6aaa6e7405232720913b17c872efff1cb4d280ba6b84cbad06 was reverted error: ICS02 Client error: unknown client state type: `/ibc.lightclients.wasm.v1.ClientState`

This is probably caused by sovereign-ibc assuming that the SovereignClientState is stored natively on the Cosmos chain. However, in actual it is wrapped inside a WasmClientState on Cosmos.

This is currently being fixed on https://github.com/informalsystems/sovereign-ibc/pull/160. After that, we will need to update the dependencies on sov-rollup-starter, and try running the test again.