Add igp testing to cosmos e2e

[misko9]

No description provided.

[KyleMoser]

Looks good. Only thing I don't agree with is changing the payment from the Coin to the amount in the PayForGas events. IMO it was better as-is, it permanently records what payment was made (amount+denom). I could see a future where payments are supported in other denoms, or the payment denom changes for some reason.

That said, I don't feel that strongly about it, so I'm marking this approved. Just curious why that change is important.

[misko9]

Looks good. Only thing I don't agree with is changing the payment from the Coin to the amount in the PayForGas events. IMO it was better as-is, it permanently records what payment was made (amount+denom). I could see a future where payments are supported in other denoms, or the payment denom changes for some reason.

That said, I don't feel that strongly about it, so I'm marking this approved. Just curious why that change is important.

Thanks @KyleMoser, I agree that it would be nice to be able to support multiple denoms, but hyperlane's agents don't support that yet. The relayer expects the payment event attribute to be an integer. We could add another event attribute for the denom, but it wouldn't be used.

[misko9]

I will merge this in after the Val Announce PR since this is a stacked PR.