Merge from relayer #169
Draft
Merge from relayer #169
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Include begin and end block events * disable flushing when termination condition is set * Still flush for FlushLifecycle * Add sort for flush logging to avoid confusion
* Wire initial messages into path processor caches so that retry logic will occur * Fix counterparty keys * Remove debug log
* fix slip44 default * Add test case
* build: bump to Go 1.20 + bump deps This bumps the Go version to 1.20 and also bumps the SDK version to 0.47.0 and ibc-go to v7.0.0 * chore: update GH workflows to use Go 1.20 + update interchaintest deps * chore: update missing deps in go.sum + use 1.20 in dockerfiles * chore: bump to `setup-go/v4` and remove caching step * chore: bump to `checkout/v3` and remove caching step * chore: bump 1.20.2
* Fix ordered channel closure * Increase timeout for scenarios test * Fix tracking of init messages when IDs aren't the same * bump interchaintest to include explicit port bindings
* Add channel close correlation * Switch to pre-close key * make tx channel-close cli command work, add test coverage * more sweet code removals * update comment
* Fix flush on ordered channels * Queue all packets at nextseqrecv or above
* Lock config file for all write operations * Fix linter errs * tidy * more tidy
* Penumbra buf go gen * Use go prefix override * wip: penumbra relayer provider remove copied-over cosmos provider tests, rename processor cosmos -> penumbra rename ccp -> pcp reformat into new relayer dir structure update penumbra types to point to buf.build building again * fix penumbra * fix: implement MsgSubmitMisbehaviour * fix: remove unnecessary proto file + fix msg type cast * chore: add removal of penumbra client protos in protocgen * working penumbra relayer functionality wip: unbase64? wip: multiple messages per penumbra tx wip: stub SendMessages impl wip: attempt to split out common method wip: use random anchor and work around path renaming wip: improve logging wip: changes during pairing https://www.youtube.com/watch?v=RYonSOkZ5ZE clean up logs skip height bug workaround drop debug panic * update penumbra chain processor connection and channel message processing * cleanup logging statements for review Responding to review comments, honoring the style guide for logging, and removing some unused reference code that was commented out while debugging. --------- Co-authored-by: Andrew Gouin <[email protected]> Co-authored-by: Ava Howell <[email protected]> Co-authored-by: jtieri <[email protected]> Co-authored-by: Ava Howell <[email protected]> Co-authored-by: Conor Schaefer <[email protected]> Co-authored-by: Justin Tieri <[email protected]>
* Make ICA waits more explicit * poll for timeout * poll for channel close confirm * Comment out sqlite db file for scenarios tests
Bumps [github.com/docker/docker](https://github.com/docker/docker) from 20.10.19+incompatible to 20.10.24+incompatible. - [Release notes](https://github.com/docker/docker/releases) - [Commits](moby/moby@v20.10.19...v20.10.24) --- updated-dependencies: - dependency-name: github.com/docker/docker dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [github.com/opencontainers/runc](https://github.com/opencontainers/runc) from 1.1.3 to 1.1.5. - [Release notes](https://github.com/opencontainers/runc/releases) - [Changelog](https://github.com/opencontainers/runc/blob/v1.1.5/CHANGELOG.md) - [Commits](opencontainers/runc@v1.1.3...v1.1.5) --- updated-dependencies: - dependency-name: github.com/opencontainers/runc dependency-type: indirect ... 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]>
* Migrate to cometbft * Additional replaces * Register comet proto types * Add sr25519 support * bump ictest * Add keys test * Update supported algos comment
Updating the generated protos for Penumbra support. After lightly editing the `scripts/protocgen.sh` script, I ran `make proto-gen` and then committed the changes `relayer/chains/penumbra/`. Other automatically updated protos I intentionally excluded from this PR, to avoid side-effects. Co-authored-by: Conor Schaefer <[email protected]>
* Register Counterparty relayer cmd and fee middleware test * debugging the command * debugging and finalizing the fee_middleware_test. * debugging and finalizing the fee_middleware_test. * merged with updated repo * clear out some commanded code * nits and suggestions post review * more nits * added one val no fullnode as chainspec --------- Co-authored-by: Harry <[email protected]>
* fix nil receiver initiate for path ensure path get written to config * add change doc
* add max fee * add test * add change doc * Update cregistry/chain_info.go
* feat: save snapshot of serviced height in relay * fix: use milliseconds * chore: refactor get retry block after to provider config interface * chore: rename variable * fix: headercache copy by cloning --------- Co-authored-by: izyak <[email protected]> Co-authored-by: viveksharmapoudel <[email protected]>
* fix: generalize archway module as wasm module * chore: gitignore rename * chore: add block interval to provider cfg * chore: add block interval * chore: block interval --------- Co-authored-by: izyak <[email protected]>
* fix: filter wasm contract address * fix: show balance of wallet address wasm * fix: json file name * chore: rename file name * fix: allow query balance in icon --------- Co-authored-by: izyak <[email protected]>
* fix: packet retention not clear * fix: set key to counterpart * fix: handle tx archway if updateClient gives error * fix: refractor * fix: for write ack * chore: remove unnecessary block of code * fix: handle fixes for packet timeout --------- Co-authored-by: izyak <[email protected]>
* fix: params for packet receipt and acknowledgement * fix: params for packet receipt and acknowledgement --------- Co-authored-by: izyak <[email protected]>
Co-authored-by: izyak <[email protected]>
Co-authored-by: izyak <[email protected]> Co-authored-by: viveksharmapoudel <[email protected]>
* fix: send packet on chain with icon client only after client is updated * fix: update client only retry * fix: check from clientState instead of querying consensus State * fix: tx wasm multiple retry * fix: build and next sequence not fetched issue * fix: test message save position during debug position change
* feat: log height every hour simulateneously * fix: add missing method * fix: remove should snapshot --------- Co-authored-by: izyak <[email protected]> Co-authored-by: izyak <[email protected]> Co-authored-by: viveksharmapoudel <[email protected]>
…icon (#134) Co-authored-by: izyak <[email protected]>
* fix: change way of handling keystore on icon * feat: wallet and deletion * fix: handle keys delete on icon * chore: config sample files updated --------- Co-authored-by: izyak <[email protected]>
* docs: update readme for ibc-icon-integration * chore: cleanup * docs: add example for icon to edit keys on config * docs: add command for client creating and handshaking * chore: change addresses for security notice * chore: social site --------- Co-authored-by: izyak <[email protected]>
* fix: sdk prefix change handle for neutron and archway * fix: wasm prefix change optimize * fix: add config during successlogtx as well due to feepayer address
* docs: add README for relay setup on localnet and testnet * docs: running relayer locally * docs: update reference * chore: remove demo/dev environmtn --------- Co-authored-by: izyak <[email protected]>
* test: fix test for icon keys * test: fix tests * chore: change addr for cosmos * chore: update keystore path for relayer docker image --------- Co-authored-by: izyak <[email protected]>
Co-authored-by: viveksharmapoudel <[email protected]>
* fix: add missing mutex in query proof * chore: add retry logic in wasm query method * fix: add delay for pathprocessor sync * fix: remove tp when getting client state * fix: adding defer in queryWasmProof * fix: change to processheight * fix: limit max block fetch at a time in wasm processor * chore: add height to event --------- Co-authored-by: izyak <[email protected]>
* ci: change relay deployment script repo * ci: download contracts from latest artifacts * ci: adjust changes in icon-ibc deployment script" Removed submodule fix: typo fix: node deploy debug * ci: update build event trigger
* fix: add missing mutex in query proof * chore: add retry logic in wasm query method * fix: add delay for pathprocessor sync * fix: remove tp when getting client state * fix: adding defer in queryWasmProof * fix: change to processheight * fix: limit max block fetch at a time in wasm processor * fix: btp block update miss * fix:add values * fix: the logic * fix: queue logic * fix:refractor * fix: implement mutex in queue * fix: add prevconsensusstateheight method and use it to fetch trusted height --------- Co-authored-by: izyak <[email protected]>
…158) * fix: icon module validate packet revision_number from chainId revert * fix: revision number
* feat: add step estimation for icon * chore: remove icon bridge dependencies from ibc-relay * chore: return if step estimation fails * chore: cleanup --------- Co-authored-by: izyak <[email protected]> Co-authored-by: viveksharmapoudel <[email protected]>
Co-authored-by: izyak <[email protected]>
Co-authored-by: izyak <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.