You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am not entirely sure if last remaining tests have anything to do with the SDK 0.52 upgrade, unless it's the relayer that is somehow bugging out over some message changing.
But it would be great to get these fixed before we merge in feat/ibc-eureka so that we can test compatibility properly with only the Eureka changes (and so that we can stay confident in any patches we may need to backport).
There are two main areas that are breaking:
Interchain Accounts Queries
The failing test is TestInterchainAccountsQuery and seems to only fail when Chain A is running main and Chain B is running older versions (v7, v8, etc).
=== NAME TestTransferChannelUpgradesTestSuite/TestChannelUpgrade_WithFeeMiddleware_CrossingHello_Succeeds
upgradesv2_test.go:343:
Error Trace: /Users/gg/code/ibc-go/e2e/tests/transfer/upgradesv2_test.go:343
Error: Received unexpected error:
github.com/cosmos/ibc-go/v9/modules/apps/29-fee/types.MetadataFromVersion
/Users/gg/code/ibc-go/modules/apps/29-fee/types/metadata.go:11
github.com/cosmos/ibc-go/e2e/tests/transfer.(*TransferChannelUpgradesTestSuite).TestChannelUpgrade_WithFeeMiddleware_CrossingHello_Succeeds.func6
/Users/gg/code/ibc-go/e2e/tests/transfer/upgradesv2_test.go:342
failed to unmarshal metadata from version: ics20-2: invalid ICS29 middleware version
The two failing tests are TestChannelUpgrade_WithFeeMiddleware_CrossingHello_Succeeds and TestChannelUpgrade_WithFeeMiddleware_Succeeds, but it is not entirely clear if this is version related, since most of them seem to run fine (see the jobs below).
=== NAME TestInterchainAccountsQueryTestSuite/TestInterchainAccountsQuery
query_test.go:147:
Error Trace: /home/runner/work/ibc-go/ibc-go/e2e/tests/interchain_accounts/query_test.go:147
Error: "[]" should have 1 item(s), but has 0
Test: TestInterchainAccountsQueryTestSuite/TestInterchainAccountsQuery
Don't have more to contribute re: the cause, but it seems like we're failing at unmarshalling an empty ack so might be worth figuring out what failed upstream. Perhaps the receive failed (ie the tx succeeded but maybe the app callback failed) and if we see the error returned on OnRecvPacket (which if it exists seems to pass through these tests) then we can narrow down the cause easier compared to what is getting returned right now
I am not entirely sure if last remaining tests have anything to do with the SDK 0.52 upgrade, unless it's the relayer that is somehow bugging out over some message changing.
But it would be great to get these fixed before we merge in
feat/ibc-eureka
so that we can test compatibility properly with only the Eureka changes (and so that we can stay confident in any patches we may need to backport).There are two main areas that are breaking:
Interchain Accounts Queries
The failing test is
TestInterchainAccountsQuery
and seems to only fail when Chain A is runningmain
and Chain B is running older versions (v7, v8, etc).Links to failing jobs:
Channel upgrades with fee middleware
The two failing tests are TestChannelUpgrade_WithFeeMiddleware_CrossingHello_Succeeds and TestChannelUpgrade_WithFeeMiddleware_Succeeds, but it is not entirely clear if this is version related, since most of them seem to run fine (see the jobs below).
Links to failing jobs:
The text was updated successfully, but these errors were encountered: