feat: integrate ibc x 0.52

[julienrbrt]

Closes: #4387

[Pantani]

Is the app wiring still not working for IBC?

[julienrbrt]

Is the app wiring still not working for IBC?

Nope, they haven't prioritized it yet :/ -> cosmos/ibc-go#3560

[julienrbrt]

Using cosmos/ibc-go#7882 branch until this gets merged to main.

[gjermundgaraba]

Is the app wiring still not working for IBC?

Nope, they haven't prioritized it yet :/ -> cosmos/ibc-go#3560

We have prioritized it and tried: cosmos/ibc-go#6216

But the wiring system doesn't support our structure very well, so it is iceboxed at the moment. Hopefully we can revisit this, because we obviously would prefer to have app wiring in our own test images as well to reduce maintenance.

[gjermundgaraba]

Using cosmos/ibc-go#7882 branch until this gets merged to main.

Merged now :)

[julienrbrt]

Is the app wiring still not working for IBC?

Nope, they haven't prioritized it yet :/ -> cosmos/ibc-go#3560

We have prioritized it and tried: cosmos/ibc-go#6216

But the wiring system doesn't support our structure very well, so it is iceboxed at the moment. Hopefully we can revisit this, because we obviously would prefer to have app wiring in our own test images as well to reduce maintenance.

That'd be awesome. As you can see the wiring without IBC adopting app wiring isn't the best: https://github.com/ignite/example/blob/master/app/ibc.go#L41-L42 & https://github.com/noble-assets/noble/blob/2592437/legacy.go#L45.
It isn't bad, the mixed way it supported, but it isn't that intuitive. You could already, migrate your simapp to something to using runtime as well (like does ignite and noble), so sdk upgrades would be less painful in your simapp.

[Pantani]

The relayer tests are still broken. Maybe there is something wrong with the IBC implementation

[julienrbrt]

The relayer tests are still broken. Maybe there is something wrong with the IBC implementation

They are broken in v28 too, so I didn't pay that much attention to them. We can check that in a follow up.

[Pantani]

I didn't try an IBC transfer manually, but the test shows this error: [BLOG] panic: send-ibc-post flag redefined: timeout-timestamp

[julienrbrt]

I didn't try an IBC transfer manually, but the test shows this error: [BLOG] panic: send-ibc-post flag redefined: timeout-timestamp

Yeah that's what I need to fix upstream