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

[bridge cli] print data in json and print more data #18742

Merged
merged 2 commits into from
Jul 25, 2024

Conversation

longbowlu
Copy link
Contributor

@longbowlu longbowlu commented Jul 19, 2024

Description

  1. now you can use --network testnet to avoid having to provide proxy address
  2. add more stuff to print functions, including nonces
  3. print everything in json, for easier piped processing

See offline nodes:

sui-bridge-cli view-sui-bridge  --sui-rpc-url https://rpc.testnet.sui.io:443 --ping --hex | jq '.result.committee[] | select(.status == "offline")'

See eth nonce:

sui-bridge-cli view-eth-bridge --network testnet --eth-rpc-url https://ethereum-sepolia-rpc.publicnode.com

See sui nonce:

sui-bridge-cli view-sui-bridge  --sui-rpc-url https://rpc.testnet.sui.io:443 --ping --hex | jq '.result.nonces'

Test plan

How did you test the new or updated feature?


Release notes

Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.

For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

  • Protocol:
  • Nodes (Validators and Full nodes):
  • Indexer:
  • JSON-RPC:
  • GraphQL:
  • CLI:
  • Rust SDK:
  • REST API:

Copy link

vercel bot commented Jul 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sui-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 25, 2024 9:15pm
3 Skipped Deployments
Name Status Preview Comments Updated (UTC)
multisig-toolkit ⬜️ Ignored (Inspect) Visit Preview Jul 25, 2024 9:15pm
sui-kiosk ⬜️ Ignored (Inspect) Visit Preview Jul 25, 2024 9:15pm
sui-typescript-docs ⬜️ Ignored (Inspect) Visit Preview Jul 25, 2024 9:15pm

@longbowlu longbowlu changed the title [bridgeprint evm upgrade nonces for proxys [bridge cli] print evm upgrade nonces for proxys Jul 19, 2024
@longbowlu longbowlu force-pushed the add-upgrade-evm-nonces-in-display-command branch from 6340594 to 7ead3fd Compare July 19, 2024 23:53
@longbowlu longbowlu force-pushed the add-upgrade-evm-nonces-in-display-command branch from 7ead3fd to 87e057b Compare July 19, 2024 23:55
@longbowlu longbowlu changed the title [bridge cli] print evm upgrade nonces for proxys [bridge cli] print data in json and print more data Jul 19, 2024
@longbowlu longbowlu force-pushed the add-upgrade-evm-nonces-in-display-command branch from 87e057b to ef7e8b4 Compare July 20, 2024 00:14
Copy link
Contributor

@dariorussi dariorussi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

crates/sui-bridge-cli/src/lib.rs Show resolved Hide resolved
crates/sui-bridge-cli/src/main.rs Outdated Show resolved Hide resolved
@longbowlu longbowlu force-pushed the add-upgrade-evm-nonces-in-display-command branch from ef7e8b4 to d23d268 Compare July 25, 2024 21:08
@longbowlu longbowlu enabled auto-merge (squash) July 25, 2024 21:09
@longbowlu longbowlu merged commit ec7527c into main Jul 25, 2024
49 checks passed
@longbowlu longbowlu deleted the add-upgrade-evm-nonces-in-display-command branch July 25, 2024 21:34
suiwombat pushed a commit that referenced this pull request Sep 16, 2024
## Description 

1. now you can use `--network testnet` to avoid having to provide proxy
address
2. add more stuff to print functions, including nonces
3. print everything in json, for easier piped processing


See offline nodes:
```
sui-bridge-cli view-sui-bridge  --sui-rpc-url https://rpc.testnet.sui.io:443 --ping --hex | jq '.result.committee[] | select(.status == "offline")'
```

See eth nonce:
```
sui-bridge-cli view-eth-bridge --network testnet --eth-rpc-url https://ethereum-sepolia-rpc.publicnode.com
```

See sui nonce:
```
sui-bridge-cli view-sui-bridge  --sui-rpc-url https://rpc.testnet.sui.io:443 --ping --hex | jq '.result.nonces'
```

## Test plan 

How did you test the new or updated feature?

---

## Release notes

Check each box that your changes affect. If none of the boxes relate to
your changes, release notes aren't required.

For each box you select, include information after the relevant heading
that describes the impact of your changes that a user might notice and
any actions they must take to implement updates.

- [ ] Protocol: 
- [ ] Nodes (Validators and Full nodes): 
- [ ] Indexer: 
- [ ] JSON-RPC: 
- [ ] GraphQL: 
- [ ] CLI: 
- [ ] Rust SDK:
- [ ] REST API:
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.

2 participants