Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IBC integration #250

Closed
hacheigriega opened this issue Mar 22, 2021 · 3 comments · Fixed by #251
Closed

IBC integration #250

hacheigriega opened this issue Mar 22, 2021 · 3 comments · Fixed by #251

Comments

@hacheigriega
Copy link
Contributor

hacheigriega commented Mar 22, 2021

Ensure that IBC has been integrated properly.

Issue identified from local testing with CertiK Chain and Regen Ledger

  1. Light client initialization fails despite the success message:
$ rly light init yulei-1 -f
successfully created light client for yulei-1 by trusting endpoint http://localhost:26657...

$ rly chains list
 0: yulei-1              -> key(✔) bal(✔) light(✘) path(✔)
 1: regen-1              -> key(✔) bal(✔) light(✘) path(✔)
  1. Relayer fails to link two chains:
$ rly tx link demo -d -o 3s

I[2021-03-22|13:32:41.416] - [yulei-1] -> creating client on yulei-1 for regen-1 header-height{767} trust-period(330h0m0s)
I[2021-03-22|13:32:41.421] Error: querying clients on yulei-1 failed: unknown query path: unknown request
Error: unrecognized message route: ibc; message index: 0: unknown request: invalid request

References:

@hacheigriega
Copy link
Contributor Author

hacheigriega commented Mar 22, 2021

Submitted a PR for the first problem: cosmos/relayer#465

@jim380
Copy link
Contributor

jim380 commented Mar 24, 2021

@hacheigriega very nice catch HY. 1 would be hard to catch if you were linking two live chains I suppose.

2 is exactly when I encountered when attempting to link our chain and regen's testnet.

@hacheigriega
Copy link
Contributor Author

Thanks @jim380 . 2 is now also fixed but now recipient’s balances are not updated after a transfer despite no errors.

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 a pull request may close this issue.

2 participants