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.GetTransferChannel should return correct Transfer Channel #1029

Open
joelsmith-2019 opened this issue Mar 19, 2024 · 1 comment
Open

Comments

@joelsmith-2019
Copy link

joelsmith-2019 commented Mar 19, 2024

Current Behavior

In my own fork of Interchaintest, I have created a test that transfers funds from Cosmos (A) -> Osmosis (B) -> Juno (C). I am running into inconsistent results using ibc.GetTransferChannel when trying to receive the channel between B -> A and B -> C.

Reference to fork & test: Interchaintest

Test 1: (PASSES)
ibc_swap_err_1

Test 2: (FAILS)
ibc_swap_err_2

From the screenshots above, you can see that test 1 created the correct channels between all chains. However, after running the test again, the same channel is now used for B -> A and B -> C. This is resulting in the funds being sent back to chain A instead of Chain C. (It might take multiple runs of the test to see both scenarios.)

FYI, I am running this test on Windows inside of WSL2.

Desired Behavior

I should get consistent results using ibc.GetTransferChannel when trying to receive the channel between B -> A and B -> C.

@jonathanpberger jonathanpberger changed the title Incorrect Transfer Channel Returned from ibc.GetTransferChannel Incorrect Transfer Channel Returned from ibc.GetTransferChannel Jul 11, 2024
@jonathanpberger jonathanpberger changed the title Incorrect Transfer Channel Returned from ibc.GetTransferChannel ibc.GetTransferChannel should return correct Transfer Channel Jul 11, 2024
@Reecepbcups
Copy link
Member

I have not been able to re-produce this on my native linux based machine.

11 deterministic runs of using main 7b7fea1

    multi_hop_send_test.go:135: AB Channel transfer channel-0
    multi_hop_send_test.go:136: BA Channel transfer channel-0
    multi_hop_send_test.go:137: BC Channel transfer channel-1
    multi_hop_send_test.go:138: CB Channel transfer channel-0

Let me know if you still get this on main using relayer v2.5.2 within WSL

@Reecepbcups Reecepbcups self-assigned this Aug 13, 2024
@Reecepbcups Reecepbcups removed their assignment Aug 23, 2024
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

No branches or pull requests

2 participants