-
Notifications
You must be signed in to change notification settings - Fork 126
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
Add support for the Hermes relayer #396
Add support for the Hermes relayer #396
Conversation
`chown -R "$(stat -c '%u:%g' "$1")" "$2"`, | ||
"_", // Meaningless arg0 for sh -c with positional args. | ||
mountPath, | ||
path.Join(mountPath, relPath), | ||
mountPath, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made this change as the hermes relayer ended up making several files in other places in the home directory and was running into permissions issues. I thought it might be fine if we just set the home directory to be owned by the relayer user. If this is a problem we can figure something else out!
|
||
chain2Chans, err := r.GetChannels(ctx, eRep, chain2.Config().ChainID) | ||
require.NoError(t, err) | ||
require.Equal(t, 1, len(chain2Chans)) | ||
require.Equal(t, "STATE_CLOSED", chain2Chans[0].State) | ||
require.Subset(t, []string{"STATE_CLOSED", "Closed"}, []string{chain2Chans[0].State}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I noticed the go relayer and the hermes relayer return different states, this just accounts for either of them
func (r *DockerRelayer) WriteFileToHomeDir(ctx context.Context, relativePath string, contents []byte) error { | ||
fw := dockerutil.NewFileWriter(r.log, r.client, r.testName) | ||
if err := fw.WriteFile(ctx, r.volumeName, relativePath, contents); err != nil { | ||
return fmt.Errorf("failed to write file: %w", err) | ||
} | ||
return nil | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added this helper function as the hermes relayer required a few different files to be created.
…on/interchaintest into add-hermes-relayer-support
Agree with your points on the assumptions for the interfaces. Glad to see you could work around them until we figure out the correct refactor. And I like what you did with the |
func (c commander) UpdatePath(pathName, homeDir string, filter ibc.ChannelFilter) []string { | ||
// TODO: figure out how to implement this. | ||
panic("implement me") | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know if this is required or not for hermes, the tests are passing without.
package hermes | ||
|
||
import "github.com/strangelove-ventures/interchaintest/v6/ibc" | ||
|
||
var _ ibc.Wallet = &Wallet{} | ||
|
||
type WalletModel struct { | ||
Mnemonic string `json:"mnemonic"` | ||
Address string `json:"address"` | ||
} | ||
|
||
type Wallet struct { | ||
mnemonic string | ||
address string | ||
keyName string | ||
} | ||
|
||
func NewWallet(keyname string, address string, mnemonic string) *Wallet { | ||
return &Wallet{ | ||
mnemonic: mnemonic, | ||
address: address, | ||
keyName: keyname, | ||
} | ||
} | ||
|
||
func (w *Wallet) KeyName() string { | ||
return w.keyName | ||
} | ||
|
||
func (w *Wallet) FormattedAddress() string { | ||
return w.address | ||
} | ||
|
||
// Get mnemonic, only used for relayer wallets | ||
func (w *Wallet) Mnemonic() string { | ||
return w.mnemonic | ||
} | ||
|
||
// Get Address | ||
func (w *Wallet) Address() []byte { | ||
return []byte(w.address) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is essentially the same as the cosmos relayer wallet.
I just realised that the the test matrix seems to always be defaulting to the in memory one (not loading from file). I will look into this tomorrow! |
Hey @jtieri / @misko9 , I'm seeing some strange failures in the conformance tests. I'm seeing them in the both the go relayer and the hermes implementaiton so I'm not 100% sure if I've broken something. It appears as if both relayers are failing here. In this PR I've temporarily hard coded hermes in to run the conformance test CI, but the Please let me know if anything is jumping out about them! (I've converted back to a draft until the CI failures are resolved) |
Looks like all the CI is now passing with hermes plugged into the conformance tests, thanks for the review @seanchen1991 , and thanks for the support @misko9 / @jtieri ❤️ |
* Param proposals, balance inquiries, and IBC transfer error handling (#393) * Extract file writing/copying logic to helpers * Add ParamChangeProposal * Update file naming for param change proposal * Use Code to detect errors in IBC transfers * Add AllBalances and ParamChangeProposal * Add support for the Hermes relayer (#396) * adding some scaffolding for hermes relayer * chore: updating interface to accept create connection options type * chore: wip * wip: adding path map to hermes relayer type * writing mnemonic file * correctly reading toml config * ibc test passing with hermes relayer * learn ibc test passing with hermes relayer * adding parse client and connection output * adding hermes test cases * remove unused types * undid import change * reverted import change * reverted some unintentional changes * fix linting error * adding hermes to conformance matrix * adding default value for matrix file for CI * pass absolute value for matrix file * removed extra part of path * adding hermes to Labels function * add capabilities for hermes * temporarily strip down number of tests to verify hermes relayer * fixing conformance tests * fixing channel tests in TestRelayerSetup * revert to go rly to test * bump hermes version * extract json response correctly * extract json result from stdout * correct channel parsing json stdout * set field ClearOnStart to true * switch back to go relayer as default * add hermes to the default relayers list * Update version of ibc-go from v6 to v7 from merge * Bump sdk and ibc-go --------- Co-authored-by: bigs <[email protected]> Co-authored-by: Cian Hatton <[email protected]>
* Param proposals, balance inquiries, and IBC transfer error handling (#393) * Extract file writing/copying logic to helpers * Add ParamChangeProposal * Update file naming for param change proposal * Use Code to detect errors in IBC transfers * Add AllBalances and ParamChangeProposal * Add support for the Hermes relayer (#396) * adding some scaffolding for hermes relayer * chore: updating interface to accept create connection options type * chore: wip * wip: adding path map to hermes relayer type * writing mnemonic file * correctly reading toml config * ibc test passing with hermes relayer * learn ibc test passing with hermes relayer * adding parse client and connection output * adding hermes test cases * remove unused types * undid import change * reverted import change * reverted some unintentional changes * fix linting error * adding hermes to conformance matrix * adding default value for matrix file for CI * pass absolute value for matrix file * removed extra part of path * adding hermes to Labels function * add capabilities for hermes * temporarily strip down number of tests to verify hermes relayer * fixing conformance tests * fixing channel tests in TestRelayerSetup * revert to go rly to test * bump hermes version * extract json response correctly * extract json result from stdout * correct channel parsing json stdout * set field ClearOnStart to true * switch back to go relayer as default * add hermes to the default relayers list * Update version of ibc-go from v6 to v7 from merge * Bump sdk and ibc-go --------- Co-authored-by: bigs <[email protected]> Co-authored-by: Cian Hatton <[email protected]>
* chore: update to 0.47 * bump go * updates * updates * feedback * Fix BroadcastSync for cosmos broadcaster (#360) * Fix BroadcastSync for cosmos broadcaster * Fixing docstring * public grpc, register tmlightclient codecs as no longer by default, attempt base64 decode on key if string value doesn't match for backwards compatibility Add test for IBC with chains pre/post SDK 47 upgrade * Bump ibc-go to latest commit * deps: upgrade to ibc-go/v7 and bump ibctest go mod (#372) * bumping ibc-go to v7 branch * bumping ibctest go mod to v7 * chore: align User interface with CosmosWallet and fix issue with broadcast mode sync (#388) * chore: v0.47 branch merge and bumps (#406) * Param proposals, balance inquiries, and IBC transfer error handling (#393) * Extract file writing/copying logic to helpers * Add ParamChangeProposal * Update file naming for param change proposal * Use Code to detect errors in IBC transfers * Add AllBalances and ParamChangeProposal * Add support for the Hermes relayer (#396) * adding some scaffolding for hermes relayer * chore: updating interface to accept create connection options type * chore: wip * wip: adding path map to hermes relayer type * writing mnemonic file * correctly reading toml config * ibc test passing with hermes relayer * learn ibc test passing with hermes relayer * adding parse client and connection output * adding hermes test cases * remove unused types * undid import change * reverted import change * reverted some unintentional changes * fix linting error * adding hermes to conformance matrix * adding default value for matrix file for CI * pass absolute value for matrix file * removed extra part of path * adding hermes to Labels function * add capabilities for hermes * temporarily strip down number of tests to verify hermes relayer * fixing conformance tests * fixing channel tests in TestRelayerSetup * revert to go rly to test * bump hermes version * extract json response correctly * extract json result from stdout * correct channel parsing json stdout * set field ClearOnStart to true * switch back to go relayer as default * add hermes to the default relayers list * Update version of ibc-go from v6 to v7 from merge * Bump sdk and ibc-go --------- Co-authored-by: bigs <[email protected]> Co-authored-by: Cian Hatton <[email protected]> * deps: bump SDK to v0.47-rc3 (#414) Co-authored-by: Carlos Rodriguez <[email protected]> * Consolidate flush command into single command (#417) * Consolidate flush command into single command * Updates for hermes * Bump to rly default with consolidated flush * Flush with channelID since hermes requires it * feat: add ReadFile method for reading files from docker fs (#423) Expose a ReadFile method on ChainNode so that you can read files from the docker filesystem within test cases that exist in downstream repos. --------- Co-authored-by: Justin Tieri <[email protected]> * Add support for genesis sub commands (#384) * Add support for genesis sub commands * Remove logging and add some basic documentation for UsingNewGenesisCommand * Remove UsingNewGenesisCommand from chainspec + add test --------- Co-authored-by: Julien Robert <[email protected]> Co-authored-by: Cian Hatton <[email protected]> Co-authored-by: Damian Nolan <[email protected]> Co-authored-by: Gjermund Garaba <[email protected]> Co-authored-by: bigs <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Justin Tieri <[email protected]>
* Param proposals, balance inquiries, and IBC transfer error handling (#393) * Extract file writing/copying logic to helpers * Add ParamChangeProposal * Update file naming for param change proposal * Use Code to detect errors in IBC transfers * Add AllBalances and ParamChangeProposal * Add support for the Hermes relayer (#396) * adding some scaffolding for hermes relayer * chore: updating interface to accept create connection options type * chore: wip * wip: adding path map to hermes relayer type * writing mnemonic file * correctly reading toml config * ibc test passing with hermes relayer * learn ibc test passing with hermes relayer * adding parse client and connection output * adding hermes test cases * remove unused types * undid import change * reverted import change * reverted some unintentional changes * fix linting error * adding hermes to conformance matrix * adding default value for matrix file for CI * pass absolute value for matrix file * removed extra part of path * adding hermes to Labels function * add capabilities for hermes * temporarily strip down number of tests to verify hermes relayer * fixing conformance tests * fixing channel tests in TestRelayerSetup * revert to go rly to test * bump hermes version * extract json response correctly * extract json result from stdout * correct channel parsing json stdout * set field ClearOnStart to true * switch back to go relayer as default * add hermes to the default relayers list * update penumbra chain spin-up (#409) * Create .github/dependabot.yml Start dependabot. * Bump github.com/BurntSushi/toml from 1.2.0 to 1.2.1 (#432) Bumps [github.com/BurntSushi/toml](https://github.com/BurntSushi/toml) from 1.2.0 to 1.2.1. - [Release notes](https://github.com/BurntSushi/toml/releases) - [Commits](BurntSushi/toml@v1.2.0...v1.2.1) --- updated-dependencies: - dependency-name: github.com/BurntSushi/toml dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump modernc.org/sqlite from 1.17.3 to 1.21.0 (#433) --- updated-dependencies: - dependency-name: modernc.org/sqlite dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump leonsteinhaeuser/project-beta-automations from 2.0.1 to 2.1.0 (#428) Bumps [leonsteinhaeuser/project-beta-automations](https://github.com/leonsteinhaeuser/project-beta-automations) from 2.0.1 to 2.1.0. - [Release notes](https://github.com/leonsteinhaeuser/project-beta-automations/releases) - [Commits](leonsteinhaeuser/project-beta-automations@v2.0.1...v2.1.0) --- updated-dependencies: - dependency-name: leonsteinhaeuser/project-beta-automations dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dan Kanefsky <[email protected]> * Bump golangci/golangci-lint-action from 2 to 3 (#429) Bumps [golangci/golangci-lint-action](https://github.com/golangci/golangci-lint-action) from 2 to 3. - [Release notes](https://github.com/golangci/golangci-lint-action/releases) - [Commits](golangci/golangci-lint-action@v2...v3) --- updated-dependencies: - dependency-name: golangci/golangci-lint-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dan Kanefsky <[email protected]> * Bump actions/checkout from 1 to 3 (#430) Bumps [actions/checkout](https://github.com/actions/checkout) from 1 to 3. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v1...v3) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dan Kanefsky <[email protected]> * Bump github.com/gdamore/tcell/v2 (#434) Bumps [github.com/gdamore/tcell/v2](https://github.com/gdamore/tcell) from 2.4.1-0.20210905002822-f057f0a857a1 to 2.6.0. - [Release notes](https://github.com/gdamore/tcell/releases) - [Commits](https://github.com/gdamore/tcell/commits/v2.6.0) --- updated-dependencies: - dependency-name: github.com/gdamore/tcell/v2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dan Kanefsky <[email protected]> * Bump github.com/avast/retry-go/v4 from 4.0.4 to 4.3.3 (#437) Bumps [github.com/avast/retry-go/v4](https://github.com/avast/retry-go) from 4.0.4 to 4.3.3. - [Release notes](https://github.com/avast/retry-go/releases) - [Commits](avast/retry-go@4.0.4...4.3.3) --- updated-dependencies: - dependency-name: github.com/avast/retry-go/v4 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump github.com/stretchr/testify from 1.8.1 to 1.8.2 (#436) Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.8.1 to 1.8.2. - [Release notes](https://github.com/stretchr/testify/releases) - [Commits](stretchr/testify@v1.8.1...v1.8.2) --- updated-dependencies: - dependency-name: github.com/stretchr/testify dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * Bump go.uber.org/zap from 1.21.0 to 1.24.0 (#439) Bumps [go.uber.org/zap](https://github.com/uber-go/zap) from 1.21.0 to 1.24.0. - [Release notes](https://github.com/uber-go/zap/releases) - [Changelog](https://github.com/uber-go/zap/blob/master/CHANGELOG.md) - [Commits](uber-go/zap@v1.21.0...v1.24.0) --- updated-dependencies: - dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dan Kanefsky <[email protected]> * Cosmos SDK v0.47 (#359) * chore: update to 0.47 * bump go * updates * updates * feedback * Fix BroadcastSync for cosmos broadcaster (#360) * Fix BroadcastSync for cosmos broadcaster * Fixing docstring * public grpc, register tmlightclient codecs as no longer by default, attempt base64 decode on key if string value doesn't match for backwards compatibility Add test for IBC with chains pre/post SDK 47 upgrade * Bump ibc-go to latest commit * deps: upgrade to ibc-go/v7 and bump ibctest go mod (#372) * bumping ibc-go to v7 branch * bumping ibctest go mod to v7 * chore: align User interface with CosmosWallet and fix issue with broadcast mode sync (#388) * chore: v0.47 branch merge and bumps (#406) * Param proposals, balance inquiries, and IBC transfer error handling (#393) * Extract file writing/copying logic to helpers * Add ParamChangeProposal * Update file naming for param change proposal * Use Code to detect errors in IBC transfers * Add AllBalances and ParamChangeProposal * Add support for the Hermes relayer (#396) * adding some scaffolding for hermes relayer * chore: updating interface to accept create connection options type * chore: wip * wip: adding path map to hermes relayer type * writing mnemonic file * correctly reading toml config * ibc test passing with hermes relayer * learn ibc test passing with hermes relayer * adding parse client and connection output * adding hermes test cases * remove unused types * undid import change * reverted import change * reverted some unintentional changes * fix linting error * adding hermes to conformance matrix * adding default value for matrix file for CI * pass absolute value for matrix file * removed extra part of path * adding hermes to Labels function * add capabilities for hermes * temporarily strip down number of tests to verify hermes relayer * fixing conformance tests * fixing channel tests in TestRelayerSetup * revert to go rly to test * bump hermes version * extract json response correctly * extract json result from stdout * correct channel parsing json stdout * set field ClearOnStart to true * switch back to go relayer as default * add hermes to the default relayers list * Update version of ibc-go from v6 to v7 from merge * Bump sdk and ibc-go --------- Co-authored-by: bigs <[email protected]> Co-authored-by: Cian Hatton <[email protected]> * deps: bump SDK to v0.47-rc3 (#414) Co-authored-by: Carlos Rodriguez <[email protected]> * Consolidate flush command into single command (#417) * Consolidate flush command into single command * Updates for hermes * Bump to rly default with consolidated flush * Flush with channelID since hermes requires it * feat: add ReadFile method for reading files from docker fs (#423) Expose a ReadFile method on ChainNode so that you can read files from the docker filesystem within test cases that exist in downstream repos. --------- Co-authored-by: Justin Tieri <[email protected]> * Add support for genesis sub commands (#384) * Add support for genesis sub commands * Remove logging and add some basic documentation for UsingNewGenesisCommand * Remove UsingNewGenesisCommand from chainspec + add test --------- Co-authored-by: Julien Robert <[email protected]> Co-authored-by: Cian Hatton <[email protected]> Co-authored-by: Damian Nolan <[email protected]> Co-authored-by: Gjermund Garaba <[email protected]> Co-authored-by: bigs <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Justin Tieri <[email protected]> * Add image repo to upgradeVersion (#445) * add image reop to upgradeVersion * container repo * juno container repo * fix order * capitalization * generate explicit port bindings (#444) * generate explicit port bindings * Prevent race to open ports between closing tmp listeners and starting containers * Bump github.com/cosmos/ibc-go/v7 from 7.0.0-rc1 to 7.0.0 (#451) Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.0.0-rc1 to 7.0.0. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/main/CHANGELOG.md) - [Commits](cosmos/ibc-go@v7.0.0-rc1...v7.0.0) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v7 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * Bump golang.org/x/tools from 0.6.0 to 0.7.0 (#449) Bumps [golang.org/x/tools](https://github.com/golang/tools) from 0.6.0 to 0.7.0. - [Release notes](https://github.com/golang/tools/releases) - [Commits](golang/tools@v0.6.0...v0.7.0) --- updated-dependencies: - dependency-name: golang.org/x/tools dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * fix: remove race condition on start container retries (#415) * fix: increase retry to 15 seconds to avoid retrying a command exec * chore: increase log tail, simplify start container logic * chore: cleanup, revert change, add in-code comment * chore: remove reference to retry * Bump github.com/libp2p/go-libp2p-core from 0.15.1 to 0.20.1 (#440) Bumps [github.com/libp2p/go-libp2p-core](https://github.com/libp2p/go-libp2p-core) from 0.15.1 to 0.20.1. - [Release notes](https://github.com/libp2p/go-libp2p-core/releases) - [Commits](libp2p/go-libp2p-core@v0.15.1...v0.20.1) --- updated-dependencies: - dependency-name: github.com/libp2p/go-libp2p-core dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * consolidate create node container (#447) * generate explicit port bindings * Prevent race to open ports between closing tmp listeners and starting containers * Consolidate create node container * Use RWMutex * initialize containerLifecycle for penumbra node * Bump rly to v2.3.0-rc2 * Temporarily modify version * Bump github.com/centrifuge/go-substrate-rpc-client/v4 (#438) Bumps [github.com/centrifuge/go-substrate-rpc-client/v4](https://github.com/centrifuge/go-substrate-rpc-client) from 4.0.10 to 4.0.12. - [Release notes](https://github.com/centrifuge/go-substrate-rpc-client/releases) - [Commits](centrifuge/go-substrate-rpc-client@v4.0.10...v4.0.12) --- updated-dependencies: - dependency-name: github.com/centrifuge/go-substrate-rpc-client/v4 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * Bump actions/setup-go from 3 to 4 (#472) Bumps [actions/setup-go](https://github.com/actions/setup-go) from 3 to 4. - [Release notes](https://github.com/actions/setup-go/releases) - [Commits](actions/setup-go@v3...v4) --- updated-dependencies: - dependency-name: actions/setup-go dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Migrate docker relayer to ContainerLifecycle (#466) * Bump default rly version to v2.3.0-rc4 * update docker relayer to use ContainerLifecycle * Add nil check * Make ICA waits more explicit (#471) * Make ICA waits more explicit * Poll for channel close * Bump modernc.org/sqlite from 1.21.0 to 1.21.1 (#473) Bumps [modernc.org/sqlite](https://gitlab.com/cznic/sqlite) from 1.21.0 to 1.21.1. - [Release notes](https://gitlab.com/cznic/sqlite/tags) - [Commits](https://gitlab.com/cznic/sqlite/compare/v1.21.0...v1.21.1) --- updated-dependencies: - dependency-name: modernc.org/sqlite dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * Bump google.golang.org/grpc from 1.53.0 to 1.54.0 (#463) Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.53.0 to 1.54.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](grpc/grpc-go@v1.53.0...v1.54.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * Bump github.com/cosmos/cosmos-sdk from 0.47.0 to 0.47.1 (#462) Bumps [github.com/cosmos/cosmos-sdk](https://github.com/cosmos/cosmos-sdk) from 0.47.0 to 0.47.1. - [Release notes](https://github.com/cosmos/cosmos-sdk/releases) - [Changelog](https://github.com/cosmos/cosmos-sdk/blob/v0.47.1/CHANGELOG.md) - [Commits](cosmos/cosmos-sdk@v0.47.0...v0.47.1) --- updated-dependencies: - dependency-name: github.com/cosmos/cosmos-sdk dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * Bump github.com/99designs/keyring from 1.2.1 to 1.2.2 (#450) Bumps [github.com/99designs/keyring](https://github.com/99designs/keyring) from 1.2.1 to 1.2.2. - [Release notes](https://github.com/99designs/keyring/releases) - [Commits](99designs/keyring@v1.2.1...v1.2.2) --- updated-dependencies: - dependency-name: github.com/99designs/keyring dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> Co-authored-by: Dan Kanefsky <[email protected]> * Update `ExportState` (#475) * update exportstate * add space * concatenate stdout and stderr * use ChainConfig in RestoreKey relayer * fix mini bug * fix lint * use cmd create node same main --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: bigs <[email protected]> Co-authored-by: Cian Hatton <[email protected]> Co-authored-by: Dan Kanefsky <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> Co-authored-by: Julien Robert <[email protected]> Co-authored-by: Damian Nolan <[email protected]> Co-authored-by: Gjermund Garaba <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Justin Tieri <[email protected]> Co-authored-by: colin axnér <[email protected]> Co-authored-by: Nguyen Thanh Nhan <[email protected]>
* chore: update to 0.47 * bump go * updates * updates * feedback * Fix BroadcastSync for cosmos broadcaster (#360) * Fix BroadcastSync for cosmos broadcaster * Fixing docstring * public grpc, register tmlightclient codecs as no longer by default, attempt base64 decode on key if string value doesn't match for backwards compatibility Add test for IBC with chains pre/post SDK 47 upgrade * Bump ibc-go to latest commit * deps: upgrade to ibc-go/v7 and bump ibctest go mod (#372) * bumping ibc-go to v7 branch * bumping ibctest go mod to v7 * Initial commit for hyperspace relayer * Add support for creating the core config and modifying chain configs. * Move hyperspace to its own package * Populate hyperspace's cosmos config with key information, i.e. KeyEntry * Add wasm code id and wasm client type to cosmos chain config * chore: align User interface with CosmosWallet and fix issue with broadcast mode sync (#388) * Hyperspace create-client is populating client-ids in chain configs. Reverted to using preset keys for both parachain and cosmos. Added 08-wasm types for codec registry. * Add CreateConnections, GetClients, and ParseGetClientsOuput. CreateConnection only commits MsgConnectionOpenInit. Removes wasm_client_type and counterparty_wasm_code_id from chain configs. Increases max rpc body size for a larger contract. Subs out FindTxs for polkadot chains to not impact cosmos chains block db functions. * Temporarily force simd for chain A and rococo-local for chain B * Create connection working in hyperspace/interchaintest * Add create channel and get connections. Create channels works. * Reduce session length to 20 blocks again. Debugging starting the relayer. And using ibc-go-simd from heighliner. * Start hyperspace relayer * IBC transfers working in both directions * Poll for cosmos chain balance to reduce test time * Use interchaintest generated relayer wallet in hyperspace's cosmos chain config. * add query channels (#397) * add query channels * code cleanup from PR commments * fix channel and portID fork polkadot mint * Add test for requiring governance on PushNewWasmCode msg * Fix denom trace param * Add submitting a proposal for a new wasm contract. Clean up the test case. * Bump to ibc-go v7, cosmos-sdk v0.47, polkadot v0.9.36, update parachains mint funds & send ibc tx msgs * Refactor hyperspace into more manageable files * Add GeneratePath() to hyperspace relayer and move hyperspace-specific relayer functions out of DockerRelayer * Add LinkPath to hyperspace relayer interface * Override cosmos chain block time in test * Scale polkadot/parachain additional wallets by a factor of 1mil relative to default * Update test framework with latest changes from ibc-go, hyperspace, grandpa contract, and parachain. * Update to gzip the wasm contract and reduce gas wanted now that gas consumption is better. * Update codecs, update hyperspace/parachain commits to use, and remove enabling ib send/receive on parachain. The parachain now defaults to being enabled for send/receives. * Merge branch 'main' into feat/hyperspace (#492) * Param proposals, balance inquiries, and IBC transfer error handling (#393) * Extract file writing/copying logic to helpers * Add ParamChangeProposal * Update file naming for param change proposal * Use Code to detect errors in IBC transfers * Add AllBalances and ParamChangeProposal * Add support for the Hermes relayer (#396) * adding some scaffolding for hermes relayer * chore: updating interface to accept create connection options type * chore: wip * wip: adding path map to hermes relayer type * writing mnemonic file * correctly reading toml config * ibc test passing with hermes relayer * learn ibc test passing with hermes relayer * adding parse client and connection output * adding hermes test cases * remove unused types * undid import change * reverted import change * reverted some unintentional changes * fix linting error * adding hermes to conformance matrix * adding default value for matrix file for CI * pass absolute value for matrix file * removed extra part of path * adding hermes to Labels function * add capabilities for hermes * temporarily strip down number of tests to verify hermes relayer * fixing conformance tests * fixing channel tests in TestRelayerSetup * revert to go rly to test * bump hermes version * extract json response correctly * extract json result from stdout * correct channel parsing json stdout * set field ClearOnStart to true * switch back to go relayer as default * add hermes to the default relayers list * update penumbra chain spin-up (#409) * Create .github/dependabot.yml Start dependabot. * Bump github.com/BurntSushi/toml from 1.2.0 to 1.2.1 (#432) Bumps [github.com/BurntSushi/toml](https://github.com/BurntSushi/toml) from 1.2.0 to 1.2.1. - [Release notes](https://github.com/BurntSushi/toml/releases) - [Commits](BurntSushi/toml@v1.2.0...v1.2.1) --- updated-dependencies: - dependency-name: github.com/BurntSushi/toml dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump modernc.org/sqlite from 1.17.3 to 1.21.0 (#433) --- updated-dependencies: - dependency-name: modernc.org/sqlite dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump leonsteinhaeuser/project-beta-automations from 2.0.1 to 2.1.0 (#428) Bumps [leonsteinhaeuser/project-beta-automations](https://github.com/leonsteinhaeuser/project-beta-automations) from 2.0.1 to 2.1.0. - [Release notes](https://github.com/leonsteinhaeuser/project-beta-automations/releases) - [Commits](leonsteinhaeuser/project-beta-automations@v2.0.1...v2.1.0) --- updated-dependencies: - dependency-name: leonsteinhaeuser/project-beta-automations dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dan Kanefsky <[email protected]> * Bump golangci/golangci-lint-action from 2 to 3 (#429) Bumps [golangci/golangci-lint-action](https://github.com/golangci/golangci-lint-action) from 2 to 3. - [Release notes](https://github.com/golangci/golangci-lint-action/releases) - [Commits](golangci/golangci-lint-action@v2...v3) --- updated-dependencies: - dependency-name: golangci/golangci-lint-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dan Kanefsky <[email protected]> * Bump actions/checkout from 1 to 3 (#430) Bumps [actions/checkout](https://github.com/actions/checkout) from 1 to 3. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v1...v3) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dan Kanefsky <[email protected]> * Bump github.com/gdamore/tcell/v2 (#434) Bumps [github.com/gdamore/tcell/v2](https://github.com/gdamore/tcell) from 2.4.1-0.20210905002822-f057f0a857a1 to 2.6.0. - [Release notes](https://github.com/gdamore/tcell/releases) - [Commits](https://github.com/gdamore/tcell/commits/v2.6.0) --- updated-dependencies: - dependency-name: github.com/gdamore/tcell/v2 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dan Kanefsky <[email protected]> * Bump github.com/avast/retry-go/v4 from 4.0.4 to 4.3.3 (#437) Bumps [github.com/avast/retry-go/v4](https://github.com/avast/retry-go) from 4.0.4 to 4.3.3. - [Release notes](https://github.com/avast/retry-go/releases) - [Commits](avast/retry-go@4.0.4...4.3.3) --- updated-dependencies: - dependency-name: github.com/avast/retry-go/v4 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump github.com/stretchr/testify from 1.8.1 to 1.8.2 (#436) Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.8.1 to 1.8.2. - [Release notes](https://github.com/stretchr/testify/releases) - [Commits](stretchr/testify@v1.8.1...v1.8.2) --- updated-dependencies: - dependency-name: github.com/stretchr/testify dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * Bump go.uber.org/zap from 1.21.0 to 1.24.0 (#439) Bumps [go.uber.org/zap](https://github.com/uber-go/zap) from 1.21.0 to 1.24.0. - [Release notes](https://github.com/uber-go/zap/releases) - [Changelog](https://github.com/uber-go/zap/blob/master/CHANGELOG.md) - [Commits](uber-go/zap@v1.21.0...v1.24.0) --- updated-dependencies: - dependency-name: go.uber.org/zap dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dan Kanefsky <[email protected]> * Cosmos SDK v0.47 (#359) * chore: update to 0.47 * bump go * updates * updates * feedback * Fix BroadcastSync for cosmos broadcaster (#360) * Fix BroadcastSync for cosmos broadcaster * Fixing docstring * public grpc, register tmlightclient codecs as no longer by default, attempt base64 decode on key if string value doesn't match for backwards compatibility Add test for IBC with chains pre/post SDK 47 upgrade * Bump ibc-go to latest commit * deps: upgrade to ibc-go/v7 and bump ibctest go mod (#372) * bumping ibc-go to v7 branch * bumping ibctest go mod to v7 * chore: align User interface with CosmosWallet and fix issue with broadcast mode sync (#388) * chore: v0.47 branch merge and bumps (#406) * Param proposals, balance inquiries, and IBC transfer error handling (#393) * Extract file writing/copying logic to helpers * Add ParamChangeProposal * Update file naming for param change proposal * Use Code to detect errors in IBC transfers * Add AllBalances and ParamChangeProposal * Add support for the Hermes relayer (#396) * adding some scaffolding for hermes relayer * chore: updating interface to accept create connection options type * chore: wip * wip: adding path map to hermes relayer type * writing mnemonic file * correctly reading toml config * ibc test passing with hermes relayer * learn ibc test passing with hermes relayer * adding parse client and connection output * adding hermes test cases * remove unused types * undid import change * reverted import change * reverted some unintentional changes * fix linting error * adding hermes to conformance matrix * adding default value for matrix file for CI * pass absolute value for matrix file * removed extra part of path * adding hermes to Labels function * add capabilities for hermes * temporarily strip down number of tests to verify hermes relayer * fixing conformance tests * fixing channel tests in TestRelayerSetup * revert to go rly to test * bump hermes version * extract json response correctly * extract json result from stdout * correct channel parsing json stdout * set field ClearOnStart to true * switch back to go relayer as default * add hermes to the default relayers list * Update version of ibc-go from v6 to v7 from merge * Bump sdk and ibc-go --------- Co-authored-by: bigs <[email protected]> Co-authored-by: Cian Hatton <[email protected]> * deps: bump SDK to v0.47-rc3 (#414) Co-authored-by: Carlos Rodriguez <[email protected]> * Consolidate flush command into single command (#417) * Consolidate flush command into single command * Updates for hermes * Bump to rly default with consolidated flush * Flush with channelID since hermes requires it * feat: add ReadFile method for reading files from docker fs (#423) Expose a ReadFile method on ChainNode so that you can read files from the docker filesystem within test cases that exist in downstream repos. --------- Co-authored-by: Justin Tieri <[email protected]> * Add support for genesis sub commands (#384) * Add support for genesis sub commands * Remove logging and add some basic documentation for UsingNewGenesisCommand * Remove UsingNewGenesisCommand from chainspec + add test --------- Co-authored-by: Julien Robert <[email protected]> Co-authored-by: Cian Hatton <[email protected]> Co-authored-by: Damian Nolan <[email protected]> Co-authored-by: Gjermund Garaba <[email protected]> Co-authored-by: bigs <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Justin Tieri <[email protected]> * Add image repo to upgradeVersion (#445) * add image reop to upgradeVersion * container repo * juno container repo * fix order * capitalization * generate explicit port bindings (#444) * generate explicit port bindings * Prevent race to open ports between closing tmp listeners and starting containers * Bump github.com/cosmos/ibc-go/v7 from 7.0.0-rc1 to 7.0.0 (#451) Bumps [github.com/cosmos/ibc-go/v7](https://github.com/cosmos/ibc-go) from 7.0.0-rc1 to 7.0.0. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/main/CHANGELOG.md) - [Commits](cosmos/ibc-go@v7.0.0-rc1...v7.0.0) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v7 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * Bump golang.org/x/tools from 0.6.0 to 0.7.0 (#449) Bumps [golang.org/x/tools](https://github.com/golang/tools) from 0.6.0 to 0.7.0. - [Release notes](https://github.com/golang/tools/releases) - [Commits](golang/tools@v0.6.0...v0.7.0) --- updated-dependencies: - dependency-name: golang.org/x/tools dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * fix: remove race condition on start container retries (#415) * fix: increase retry to 15 seconds to avoid retrying a command exec * chore: increase log tail, simplify start container logic * chore: cleanup, revert change, add in-code comment * chore: remove reference to retry * Bump github.com/libp2p/go-libp2p-core from 0.15.1 to 0.20.1 (#440) Bumps [github.com/libp2p/go-libp2p-core](https://github.com/libp2p/go-libp2p-core) from 0.15.1 to 0.20.1. - [Release notes](https://github.com/libp2p/go-libp2p-core/releases) - [Commits](libp2p/go-libp2p-core@v0.15.1...v0.20.1) --- updated-dependencies: - dependency-name: github.com/libp2p/go-libp2p-core dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * consolidate create node container (#447) * generate explicit port bindings * Prevent race to open ports between closing tmp listeners and starting containers * Consolidate create node container * Use RWMutex * initialize containerLifecycle for penumbra node * Bump rly to v2.3.0-rc2 * Temporarily modify version * Bump github.com/centrifuge/go-substrate-rpc-client/v4 (#438) Bumps [github.com/centrifuge/go-substrate-rpc-client/v4](https://github.com/centrifuge/go-substrate-rpc-client) from 4.0.10 to 4.0.12. - [Release notes](https://github.com/centrifuge/go-substrate-rpc-client/releases) - [Commits](centrifuge/go-substrate-rpc-client@v4.0.10...v4.0.12) --- updated-dependencies: - dependency-name: github.com/centrifuge/go-substrate-rpc-client/v4 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * Bump actions/setup-go from 3 to 4 (#472) Bumps [actions/setup-go](https://github.com/actions/setup-go) from 3 to 4. - [Release notes](https://github.com/actions/setup-go/releases) - [Commits](actions/setup-go@v3...v4) --- updated-dependencies: - dependency-name: actions/setup-go dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Migrate docker relayer to ContainerLifecycle (#466) * Bump default rly version to v2.3.0-rc4 * update docker relayer to use ContainerLifecycle * Add nil check * Make ICA waits more explicit (#471) * Make ICA waits more explicit * Poll for channel close * Bump modernc.org/sqlite from 1.21.0 to 1.21.1 (#473) Bumps [modernc.org/sqlite](https://gitlab.com/cznic/sqlite) from 1.21.0 to 1.21.1. - [Release notes](https://gitlab.com/cznic/sqlite/tags) - [Commits](https://gitlab.com/cznic/sqlite/compare/v1.21.0...v1.21.1) --- updated-dependencies: - dependency-name: modernc.org/sqlite dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * Bump google.golang.org/grpc from 1.53.0 to 1.54.0 (#463) Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.53.0 to 1.54.0. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](grpc/grpc-go@v1.53.0...v1.54.0) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * Bump github.com/cosmos/cosmos-sdk from 0.47.0 to 0.47.1 (#462) Bumps [github.com/cosmos/cosmos-sdk](https://github.com/cosmos/cosmos-sdk) from 0.47.0 to 0.47.1. - [Release notes](https://github.com/cosmos/cosmos-sdk/releases) - [Changelog](https://github.com/cosmos/cosmos-sdk/blob/v0.47.1/CHANGELOG.md) - [Commits](cosmos/cosmos-sdk@v0.47.0...v0.47.1) --- updated-dependencies: - dependency-name: github.com/cosmos/cosmos-sdk dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> * Bump github.com/99designs/keyring from 1.2.1 to 1.2.2 (#450) Bumps [github.com/99designs/keyring](https://github.com/99designs/keyring) from 1.2.1 to 1.2.2. - [Release notes](https://github.com/99designs/keyring/releases) - [Commits](99designs/keyring@v1.2.1...v1.2.2) --- updated-dependencies: - dependency-name: github.com/99designs/keyring dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> Co-authored-by: Dan Kanefsky <[email protected]> * Update `ExportState` (#475) * update exportstate * add space * concatenate stdout and stderr * use ChainConfig in RestoreKey relayer * fix mini bug * fix lint * use cmd create node same main --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: bigs <[email protected]> Co-authored-by: Cian Hatton <[email protected]> Co-authored-by: Dan Kanefsky <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Gouin <[email protected]> Co-authored-by: Julien Robert <[email protected]> Co-authored-by: Damian Nolan <[email protected]> Co-authored-by: Gjermund Garaba <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Justin Tieri <[email protected]> Co-authored-by: colin axnér <[email protected]> Co-authored-by: Nguyen Thanh Nhan <[email protected]> * Change the UidGid of polkadot node to match what is built into the container. * Fix polkadot keys unit tests * Fix go.sum after merge from main * style: go fmt * Update parachain's query ibc balance to match it new api, re-enable verifications of denom balances of our user accounts. * Update cli for wasm light client and update the grandpa contract to support the latest wasm light client interface to contracts. * refactor how hyperspace commander parses get client and get connection output. Fix get channels after the hyperspace cli was removed. * Remove some prints and fix typo * style: go fmt * fix lint errors * update go.mod --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Julien Robert <[email protected]> Co-authored-by: Cian Hatton <[email protected]> Co-authored-by: Andrew Gouin <[email protected]> Co-authored-by: Damian Nolan <[email protected]> Co-authored-by: Eduardo Díaz <[email protected]> Co-authored-by: ducnt131 <[email protected]> Co-authored-by: bigs <[email protected]> Co-authored-by: Dan Kanefsky <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Gjermund Garaba <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Justin Tieri <[email protected]> Co-authored-by: colin axnér <[email protected]> Co-authored-by: Nguyen Thanh Nhan <[email protected]>
This PR adds an
ibc.Relayer
implementation for the Hermes relayer.closes #3
I ran into a few different issues with the Relayer/RelayerCommander interfaces when working on this implementation, but was largely able to work around them, I'll give a little bit of context about what these issues were to clarify the reasoning behind some of the implementation decisions made here.
rly tx link <path>
, hermes does not have a direct equivalent. In this case the "path" concept was implemented at the interchain test layer.DockerRelayer
andRelayerCommander
interfaces assume it is possible to run a single cli command and directly parse the output of that command to extract the desired type. For example the UpdateClients. The golang relayer can do a lot of these things in a single command, however I found that in a lot of cases there was additional logic that needed to happen. For this reason a lot of the commander methods I added are implemented just to satisfy the interface but will panic if called, and the main logic is happening in the new hermes.Relayer type.AddChainConfiguration
assumes a config file per chain config, however in hermes it is a single config that contains an array of chain configs within the single file. This method was re-implemented on the hermes relayer to account for this.I think the
RelayerCommander
interface works perfectly for the go relayer, but I suspect there will be similar issues to the ones I encountered when adding new relayers.