Skip to content
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 Namada #3705

Open
wants to merge 48 commits into
base: master
Choose a base branch
from

Conversation

yito88
Copy link
Contributor

@yito88 yito88 commented Nov 21, 2023

Closes: #XXX

Description

To transfer tokens between Namada and Cosmos-base chains.

  • Add Namada to ChainConfig
  • Reuse CosmosSdkConfig for generating Tendermint light clients
  • Add NamadaChain as ChainEndpoint implementation
    • See crates/relayer/src/chain/namada.rs and files under crates/relayer/src/chain/namada
    • To submit Namada transactions and query
  • Store Namada key to KeyRing

PR author checklist:

  • Added changelog entry, using unclog.
  • Added tests: integration (for Hermes) or unit/mock tests (for modules).
  • Linked to GitHub issue.
  • Updated code comments and documentation (e.g., docs/).
  • Tagged one reviewer who will be the one responsible for shepherding this PR.

Reviewer checklist:

  • Reviewed Files changed in the GitHub PR explorer.
  • Manually tested (in case integration/unit/mock tests are absent).

@yito88 yito88 marked this pull request as ready for review November 21, 2023 16:05
@romac romac changed the title Add Namada ChainEndpoint Add support for Namada Jan 18, 2024
yito88 and others added 20 commits April 12, 2024 23:39
* add a test script

* add shielded transfer tests

* fix comment
* gen-ibc-shielded

* MsgTimeout and MsgAcknowledgement

* add refund flag

* add shielded_sync

* scripts for IBC rate limit

* not to use memo for shielded transfers

* add namada-stargaze-transfer test script

* remove duplicated masp tx

* for Namada query change

* update Namada rev

* Namada 0.33.0
* Initial work to run tests with Namada chain

* Add wait before spawning chain handles

* Removed fixed chain name for Gaia

* Handle namada denom derivation from ibc transfers

* Remove 'get_denom_for_derive' calls

* Add CI job for tests with Namada

* Remove Namada to Gaia job due to yaml syntax error

* Fix bugs when creating Denom in tests and Gaia to Namada CI job

* Update nix flake and add NAMADA_REPO_PATH env variable to CI job

* Fix NAMDA_REPO_PATH setup in CI job

* Switch Nix install step to use DeterminateSystems

* Moved DeterminateSystems nix installer to Namada CI job only

* Add namada-src package to flake.nix

* Add step to generate Namada files

* Update Namada bootstrap in CI

* Update Namada bootstrap for Namada v0.33.0

* Update nix flake and improve Namada CI

* Update nix flake and replace Gaia v13 and v14 with Gaia v15

* Remove unwraps and update comment

* Use rsync instead of cp in CI job

* Fix rsync call to correctly copy in the namada-repo directory

* Fix copying and changing Namada repo permissions in CI

* Free disk space and correctly fix file permissions in Namada test CI

* Run Namada tests only when all other tests are finished

* Add job to run Namada test scripts

* Fix Namada script CI job and skip memo filter with Namada

* Separate CI job namada and namada-scripts to avoid error 137

* Use Gaia v14 for Namada test scripts

* Split Namada scripts CI job

* Remove Namada scripts job

* Fix clippy error from v1.78.0

* Use Namada branch which exposes wasm files in CI

* Use correct Namada repository path in CI

* Fix permission on exposed Namada repository

* Separate Namada to Namada and Gaia to Namada CI jobs from other jobs, and run them sequentially

* Reduce RocksDB cache when running tests with Namada

* Revert "Separate Namada to Namada and Gaia to Namada CI jobs from other jobs, and run them sequentially"

This reverts commit 1d3f7eb.

* Update Namada in Nix flake and fix error from merge

* Update nix flake

* Split Namada integration test in separate workflow

* Fix namada action

* Use custom runner for Namada workflow

* Remove free disk space CI step from Namada workflow

* Fix and improve CometBFT installation for Namada workflow

* Fix download MASP step from Namada workflow

* Retrieve Namada without Nix for workflow

* Fix step fetching Namada binaries

* Only copy Namada binaries

* Add debug output

* Fix retrieving Namada repository

* Add debug output for Namada repo

* Add tree  to Gaia to Namada CI job

* Improve Namada Github workflow

* Add debug ls and pwd in Namada workflow

* Reorder checkout calls in Namada workflow

* Add missing indentation which broke Namada workflow

* Clean-up integration tests and Namada bootstrap

* Use default Github action runner

* Install libssl in Namada workflow

* Update Ubuntu used for Namada workflow to include libssl3

* Clean-up Namada workflow

* Fix Namada to Namada CI job
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants