Interchain Accounts / lupercalia

[faddat]

This PR will bring ICA to Juno, as well as:

• wasmd v0.24.0
• ibc-go
• an upgrade handler folder to better organize code
• go 1.18
• tighter linting

This PR should be applied after a governance "Software Upgrade" proposal to juno passes.

The governance "Software upgrade" proposal should not be made until this code has been merged to Juno's main branch.

[faddat]

Awesome @nghuyenthevinh2000!

Ok, now we just add the ica stuff which is here:

https://github.com/cosmos/ibc-go/blob/main/testing/simapp/app.go

[faddat]

we should also use wasmd from the same commit (nearly) as Osmosis -- but not totally the same unless we put osmo's fast iavl on juno, which... actually makes a lot of sense.

Yeah lets use this one:

osmosis-labs/wasmd#3

[faddat]

I am making a cosmos-sdk fork with @pomvn's fast iavl patch only, so we don't depend on osmosis-sdk

[faddat]

https://github.com/notional-labs/cosmos-sdk/tree/v0.45.1-junofast

tfw when you wrote the same thing twice

https://github.com/notional-labs/cosmos-sdk/tree/refs/tags/v0.45.1-allfast

we should use allfast...

[giansalex]

we can keep this pull request only for ibc-go v3 - ICA, and move iavl and other forks to other pull requests.
I also heard that iavl fork throws app_hash errors in osmosis, or does it have another cause?

[faddat]

no one really knows about those apphash errors, we can take fast iavl out

[vuong177]

@faddat, i think we should remove iavl from wasmd

[faddat]

yes you're right and thanks very much vuong.

It was kind of all or nothing with fast iavl.

[faddat]

@vuong177 @nghuyenthevinh2000

Hey guys, this needs to be reworked. I'll click around a little, and see how much is needed.

[faddat]

Hey, I am going to close this in favor of #200

While writing that, it became apparent that it might as well be this.