Skip to content

Commit

Permalink
Revert "ci: minors"
Browse files Browse the repository at this point in the history
This reverts commit 41ef454.

(cherry picked from commit 52d0ebb)

# Conflicts:
#	.github/workflows/ci.yml
#	.github/workflows/scripts/hermes.txt
#	CHANGELOG.md
#	Cargo.lock
#	Cargo.toml
#	README.md
#	crates/apps/Cargo.toml
#	crates/apps_lib/src/cli.rs
#	crates/apps_lib/src/cli/client.rs
#	crates/apps_lib/src/cli/context.rs
#	crates/apps_lib/src/cli/wallet.rs
#	crates/apps_lib/src/client/tx.rs
#	crates/apps_lib/src/config/genesis/transactions.rs
#	crates/apps_lib/src/config/genesis/utils.rs
#	crates/apps_lib/src/wallet/defaults.rs
#	crates/benches/native_vps.rs
#	crates/core/src/masp.rs
#	crates/ibc/src/lib.rs
#	crates/node/src/bench_utils.rs
#	crates/node/src/shell/testing/client.rs
#	crates/sdk/src/args.rs
#	crates/sdk/src/lib.rs
#	crates/sdk/src/signing.rs
#	crates/sdk/src/tx.rs
#	crates/shielded_token/src/masp.rs
#	crates/shielded_token/src/masp/shielded_wallet.rs
#	crates/shielded_token/src/validation.rs
#	crates/tests/src/e2e/helpers.rs
#	crates/tests/src/e2e/ibc_tests.rs
#	crates/tests/src/e2e/ledger_tests.rs
#	crates/tests/src/e2e/setup.rs
#	crates/tests/src/integration/helpers.rs
#	crates/tests/src/integration/ledger_tests.rs
#	crates/tests/src/integration/masp.rs
#	crates/token/src/lib.rs
#	crates/tx/src/types.rs
#	crates/wallet/src/keys.rs
#	crates/wallet/src/lib.rs
#	crates/wallet/src/store.rs
#	genesis/hardware/src/pre-genesis/wallet.toml
#	genesis/localnet/balances.toml
#	genesis/localnet/src/pre-genesis/signed-transactions.toml
#	genesis/localnet/src/pre-genesis/unsigned-transactions.toml
#	genesis/localnet/src/pre-genesis/wallet.toml
#	genesis/localnet/transactions.toml
#	wasm/Cargo.lock
#	wasm_for_tests/Cargo.lock
  • Loading branch information
tzemanovic authored and mergify[bot] committed Dec 11, 2024
1 parent 127dd4b commit 834329c
Show file tree
Hide file tree
Showing 52 changed files with 3,366 additions and 0 deletions.
2 changes: 2 additions & 0 deletions .changelog/unreleased/CI/4140-ledger-namada-unit-tests-ci.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
- Add Ledger app unit tests to the Namada CI.
([\#4140](https://github.com/anoma/namada/pull/4140))
2 changes: 2 additions & 0 deletions .changelog/unreleased/SDK/4127-masp-for-namadillo.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
- Allow MASP transaction building without knowledge of the spend authorization
key. ([\#4127](https://github.com/anoma/namada/pull/4127))
2 changes: 2 additions & 0 deletions .changelog/v1.0.0/miscellaneous/4123-apps-pre-release.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
- updated namada libraries to libs-v0.46.1
([\#4123](https://github.com/anoma/namada/pull/4123))
2 changes: 2 additions & 0 deletions .changelog/v1.0.0/summary.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Namada v1.0.0 is the first stable release of the Namada apps.

26 changes: 26 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,10 @@ env:
AWS_REGION: us-west-2
NIGHTLY: nightly-2024-09-08
NAMADA_MASP_PARAMS_DIR: /masp/.masp-params
<<<<<<< HEAD
=======
LEDGER_APP_VERSION: "1.0.6-ci-patch"
>>>>>>> 52d0ebbd7c (Revert "ci: minors")

jobs:
changelog:
Expand Down Expand Up @@ -651,11 +655,20 @@ jobs:
fail_ci_if_error: true
token: ${{ secrets.CODECOV_TOKEN }}

<<<<<<< HEAD
<<<<<<< HEAD
=======
test-ledger-app:
timeout-minutes: 30
runs-on: [self-hosted, 4vcpu-8ram-ubuntu22-namada-x86]
=======
test-ledger-app:
timeout-minutes: 30
runs-on: [ubuntu-latest]

env:
RUSTC_WRAPPER: ""
>>>>>>> 52d0ebbd7c (Revert "ci: minors")

steps:
- name: Checkout repo
Expand All @@ -666,11 +679,14 @@ jobs:
if: ${{ github.event_name == 'pull_request_target' }}
with:
ref: ${{ github.event.pull_request.head.sha }}
<<<<<<< HEAD
- name: Configure AWS
uses: aws-actions/configure-aws-credentials@v4
with:
aws-region: ${{ env.AWS_REGION }}
role-to-assume: ${{ secrets.AWS_ROLE }}
=======
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
- name: Checkout ledger-namada
run: |
echo "Using Namada Ledger App version: v${LEDGER_APP_VERSION}"
Expand All @@ -680,6 +696,13 @@ jobs:
git submodule update --init --recursive
sudo update-alternatives --install /usr/bin/python python /usr/bin/python3 10
make deps
<<<<<<< HEAD
=======
- name: Install rust
uses: actions-rs/toolchain@v1
with:
toolchain: stable
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
- name: Generate test vectors
run: |
# The path where the Ledger app test suite will locate test vectors
Expand All @@ -692,7 +715,10 @@ jobs:
cd ../ledger-namada
make cpp_test
<<<<<<< HEAD
>>>>>>> b2e33f943 (Revert "ci/test-ledger-app: skip unnecessary steps")
=======
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
test-e2e-with-device-automation:
runs-on: [self-hosted, 4vcpu-8ram-ubuntu22-namada-x86]
container:
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/scripts/hermes.txt
Original file line number Diff line number Diff line change
@@ -1 +1,5 @@
<<<<<<< HEAD
1.10.4-namada-beta17-rc2
=======
1.10.4-namada-beta18-test
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
12 changes: 12 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,17 @@
# CHANGELOG

<<<<<<< HEAD
=======
## v1.0.0

Namada v1.0.0 is the first stable release of the Namada apps.

### MISCELLANEOUS

- updated namada libraries to libs-v0.46.1
([\#4123](https://github.com/anoma/namada/pull/4123))

>>>>>>> 52d0ebbd7c (Revert "ci: minors")
## libs-v0.46.1

Namada 0.46.1 is a libaries patch release containing couple improvements and a bug fix in preparetion for a stable apps release.
Expand Down
20 changes: 20 additions & 0 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 9 additions & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -134,15 +134,24 @@ konst = { version = "0.3.8", default-features = false }
lazy_static = "1.4.0"
# TODO: upstreamed in https://github.com/ledger-community/rust-ledger/pull/9
ledger-lib = { git = "https://github.com/heliaxdev/rust-ledger", rev = "f96f4559b3237d09218f7583df01acf36034ea79", default-features = false, features = ["transport_tcp"] }
<<<<<<< HEAD
ledger-namada-rs = { git = "https://github.com/Zondax/ledger-namada", tag = "v0.0.24" }
=======
ledger-namada-rs = { git = "https://github.com/Zondax/ledger-namada", tag = "v1.0.5" }
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
ledger-transport = "0.10.0"
ledger-transport-hid = "0.10.0"
libc = "0.2.97"
libloading = "0.7.2"
linkme = "0.3.24"
# branch = "tomas/arbitrary"
<<<<<<< HEAD
masp_primitives = { git = "https://github.com/anoma/masp", rev = "12ed8b060b295c06502a2ff8468e4a941cb7cca4" }
masp_proofs = { git = "https://github.com/anoma/masp", rev = "12ed8b060b295c06502a2ff8468e4a941cb7cca4", default-features = false, features = ["local-prover"] }
=======
masp_primitives = { git = "https://github.com/anoma/masp", rev = "0d0da3507a6f9ad135f00fd8201dc54c2f1d9efe" }
masp_proofs = { git = "https://github.com/anoma/masp", rev = "0d0da3507a6f9ad135f00fd8201dc54c2f1d9efe", default-features = false, features = ["local-prover"] }
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
num256 = "0.3.5"
num_cpus = "1.13.0"
num-derive = "0.4"
Expand Down
29 changes: 29 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,11 @@ the form of native protocol tokens. A multi-asset shielded transfer
wallet is provided in order to facilitate safe and private user
interaction with the protocol.

<<<<<<< HEAD
* Blogpost: [Introducing Namada: Interchain Asset-agnostic Privacy](https://blog.namada.net/introducing-namada-interchain-asset-agnostic-privacy/)
=======
* Blogpost: [Introducing Namada: Interchain Asset-agnostic Privacy](https://namada.net/blog/introducing-namada-multichain-asset-agnostic-data-protection)
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
## 📓 Docs

Expand All @@ -38,10 +42,17 @@ make install

After installation, the main `namada` executable will be available on path.

<<<<<<< HEAD
To find how to use it, check out the [User Guide section of the docs](https://docs.namada.net/user-guide/index.html).

For more detailed instructions and more install options, see the [Install
section](https://docs.namada.net/user-guide/install/index.html) of the User
=======
To find how to use it, check out the [User Guide section of the docs](https://docs.namada.net/users).

For more detailed instructions and more install options, see the [Install
section](https://docs.namada.net/introduction/install) of the User
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
Guide.

## ⚙️ Development
Expand All @@ -51,7 +62,11 @@ Guide.
make build-wasm-scripts-docker
```

<<<<<<< HEAD
### Before submitting a PR, pls make sure to run the following
=======
### Before submitting a PR, please make sure to run the following
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
```shell
# Format the code
Expand All @@ -71,7 +86,11 @@ To change the log level, set `NAMADA_LOG` environment variable to one of:
* `debug`
* `trace`

<<<<<<< HEAD
The default is set to `info` for all the modules, expect for CometBFT ABCI, which has a lot of `debug` logging.
=======
The default is set to `info` for all the modules, except for CometBFT ABCI, which has a lot of `debug` logging.
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
For more fine-grained logging levels settings, please refer to the [tracing subscriber docs](https://docs.rs/tracing-subscriber/0.2.18/tracing_subscriber/struct.EnvFilter.html#directives) for more information.

Expand All @@ -83,14 +102,24 @@ Please see the [contributing page](./CONTRIBUTING.md).

### Dependencies

<<<<<<< HEAD
The ledger currently requires [CometBFT v0.37.11](https://github.com/cometbft/cometbft/releases/tag/v0.37.11) is installed and available on path. This can be achieved through following [these instructions](https://github.com/cometbft/cometbft/blob/main/docs/tutorials/install.md)
=======
The ledger currently requires [CometBFT v0.37.11](https://github.com/cometbft/cometbft/releases/tag/v0.37.11) is installed and available on path. This can be achieved through following [these instructions](https://github.com/cometbft/cometbft/blob/main/docs/tutorials/install.md).
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
#### Hermes

We maintain a fork of [hermes](https://github.com/heliaxdev/hermes) that adds support for Namada.

Compatibility table with Namada:

<<<<<<< HEAD
| Namada | Hermes |
| ----------- | ----------- |
| v0.44.0 | 1.10.4-namada-beta17-rc2 |
=======
| Namada binaries | Hermes |
| ----------- | ----------- |
| v1.0.0 | [1.10.4-namada-beta17-rc2](https://github.com/heliaxdev/hermes/releases/tag/v1.10.4-namada-beta17-rc2) |
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
4 changes: 4 additions & 0 deletions crates/apps/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,11 @@ keywords.workspace = true
license.workspace = true
readme.workspace = true
repository.workspace = true
<<<<<<< HEAD
version = "0.46.0"
=======
version = "1.0.0"
>>>>>>> 52d0ebbd7c (Revert "ci: minors")

# Namada CLI that aggregates some common commands for the node and the client
[[bin]]
Expand Down
27 changes: 27 additions & 0 deletions crates/apps_lib/src/cli.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3547,6 +3547,10 @@ pub mod args {
let raw = "http://127.0.0.1:26657";
Url::from_str(raw).unwrap()
}));
<<<<<<< HEAD
=======
pub const LEDGER_ZIP32: ArgFlag = flag("ledger-zip32");
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
pub const LIST_FIND_ADDRESSES_ONLY: ArgFlag = flag("addr");
pub const LIST_FIND_KEYS_ONLY: ArgFlag = flag("keys");
pub const LOCALHOST: ArgFlag = flag("localhost");
Expand Down Expand Up @@ -7762,8 +7766,12 @@ pub mod args {
find_viewing_key(&mut wallet)
} else {
find_viewing_key(&mut ctx.borrow_mut_chain_or_exit().wallet)
<<<<<<< HEAD
}
.key;
=======
};
>>>>>>> 52d0ebbd7c (Revert "ci: minors")

Ok(PayAddressGen::<SdkTypes> {
alias: self.alias,
Expand Down Expand Up @@ -7811,12 +7819,20 @@ pub mod args {
HD_PROMPT_BIP39_PASSPHRASE.parse(matches);
let use_device = USE_DEVICE.parse(matches);
let device_transport = DEVICE_TRANSPORT.parse(matches);
<<<<<<< HEAD
=======
let ledger_zip32 = LEDGER_ZIP32.parse(matches);
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
Self {
scheme,
shielded,
alias,
alias_force,
unsafe_dont_encrypt,
<<<<<<< HEAD
=======
ledger_zip32,
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
derivation_path,
allow_non_compliant,
prompt_bip39_passphrase,
Expand Down Expand Up @@ -7887,6 +7903,17 @@ pub mod args {
.arg(HD_PROMPT_BIP39_PASSPHRASE.def().help(wrap!(
"Use an additional passphrase for HD-key generation."
)))
<<<<<<< HEAD
=======
.arg(
LEDGER_ZIP32.def().requires(SHIELDED.name).help(wrap!(
"Use the modified ZIP 32 algorithm supported by Ledger \
devices. This flag is necessary if importing the \
generated mnemonic code onto the Ledger device at some \
future time is a requirement."
)),
)
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
}
}

Expand Down
17 changes: 17 additions & 0 deletions crates/apps_lib/src/cli/client.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,10 @@ use std::io::Read;
use color_eyre::eyre::Result;
use namada_sdk::io::{display_line, Io, NamadaIo};
use namada_sdk::masp::ShieldedContext;
<<<<<<< HEAD
=======
use namada_sdk::wallet::DatedViewingKey;
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
use namada_sdk::{Namada, NamadaImpl};

use crate::cli;
Expand Down Expand Up @@ -349,8 +353,21 @@ impl CliApi {
chain_ctx
.wallet
.get_viewing_keys()
<<<<<<< HEAD
.values()
.copied(),
=======
.into_iter()
.map(|(k, v)| {
DatedViewingKey::new(
v,
chain_ctx
.wallet
.find_birthday(k)
.copied(),
)
}),
>>>>>>> 52d0ebbd7c (Revert "ci: minors")
);

crate::client::masp::syncing(
Expand Down
Loading

0 comments on commit 834329c

Please sign in to comment.