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

chore: consolidate consensus nodes into one page #1101

Closed
wants to merge 24 commits into from

Conversation

jcstein
Copy link
Member

@jcstein jcstein commented Sep 30, 2023

Overview

PREVIEW

Resolves #556, based on #1093 - and to be merged after #1093
Resolves #505

  1. Tested with full node
  2. tested with self-delegation to valoper address
  3. tested registering validator
  4. tested registering evm address for qgb

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@jcstein jcstein self-assigned this Sep 30, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Sep 30, 2023

PR Preview Action v1.4.4
🚀 Deployed preview to https://celestiaorg.github.io/docs/pr-preview/pr-1101/
on branch gh-pages at 2023-10-03 04:24 UTC

@jcstein
Copy link
Member Author

jcstein commented Oct 3, 2023

if i delegate to my celestiavaloper address before initializing the validator, the TX itself on mintscan shows an error, because the validator does not yet exist

celestia-appd tx staking delegate celestiavaloper1ezv03qp8vvphgkzvmak0efslddwxz9dnusgnc3 1000000utia --from=validator --chain-id=mocha-4 --fees=21000utia
auth_info:
  fee:
    amount:
    - amount: "21000"
      denom: utia
    gas_limit: "210000"
    granter: ""
    payer: ""
  signer_infos: []
  tip: null
body:
  extension_options: []
  memo: ""
  messages:
  - '@type': /cosmos.staking.v1beta1.MsgDelegate
    amount:
      amount: "1000000"
      denom: utia
    delegator_address: celestia1ezv03qp8vvphgkzvmak0efslddwxz9dne022wh
    validator_address: celestiavaloper1ezv03qp8vvphgkzvmak0efslddwxz9dnusgnc3
  non_critical_extension_options: []
  timeout_height: "0"
signatures: []
confirm transaction before signing and broadcasting [y/N]: y
code: 0
codespace: ""
data: ""
events: []
gas_used: "0"
gas_wanted: "0"
height: "0"
info: ""
logs: []
raw_log: '[]'
timestamp: ""
tx: null
txhash: 21167BCFF863B748410EB622C4D0E68F0FA607A363B66C989554AE5B501797EE

@jcstein
Copy link
Member Author

jcstein commented Oct 3, 2023

successfully registered validator by increasing the gas and fees:

celestia-appd tx staking create-validator \
>   --amount=1000000utia \
>   --pubkey=$(celestia-appd tendermint show-validator) \
>   --moniker=$MONIKER \
>   --chain-id=mocha-4 \
>   --commission-rate=0.1 \
>   --commission-max-rate=0.2 \
>   --commission-max-change-rate=0.01 \
>   --min-self-delegation=1000000 \
>   --from=$VALIDATOR_WALLET \
>   --keyring-backend=test \
>   --fees=210000utia \
>   --gas=220000
auth_info:
  fee:
    amount:
    - amount: "210000"
      denom: utia
    gas_limit: "220000"
    granter: ""
    payer: ""
  signer_infos: []
  tip: null
body:
  extension_options: []
  memo: ""
  messages:
  - '@type': /cosmos.staking.v1beta1.MsgCreateValidator
    commission:
      max_change_rate: "0.010000000000000000"
      max_rate: "0.200000000000000000"
      rate: "0.100000000000000000"
    delegator_address: celestia1ezv03qp8vvphgkzvmak0efslddwxz9dne022wh
    description:
      details: ""
      identity: ""
      moniker: jcs
      security_contact: ""
      website: ""
    min_self_delegation: "1000000"
    pubkey:
      '@type': /cosmos.crypto.ed25519.PubKey
      key: BmbFFyPLoe/s8rhLEU8fx0oixCzuv3TDwwN990aV6wU=
    validator_address: celestiavaloper1ezv03qp8vvphgkzvmak0efslddwxz9dnusgnc3
    value:
      amount: "1000000"
      denom: utia
  non_critical_extension_options: []
  timeout_height: "0"
signatures: []
confirm transaction before signing and broadcasting [y/N]: y
code: 0
codespace: ""
data: ""
events: []
gas_used: "0"
gas_wanted: "0"
height: "0"
info: ""
logs: []
raw_log: '[]'
timestamp: ""
tx: null
txhash: A155FA1544B9E984F622C660720D414E15543CC5DCA78FD274C0B86CA3FBA1D5

first time doing this successfully :D
https://testnet.mintscan.io/celestia-testnet/txs/A155FA1544B9E984F622C660720D414E15543CC5DCA78FD274C0B86CA3FBA1D5

@jcstein
Copy link
Member Author

jcstein commented Oct 3, 2023

test register evm address command:

celestia-appd tx qgb register \
>     $VALIDATOR_ADDRESS \
>     $EVM_ADDRESS \
>     --from $VALIDATOR_WALLET \
>     --fees 30000utia \
>     -b block \
>     -y &

# output 
[1] 52647
code: 0
codespace: ""
data: 12300A2E2F63656C65737469612E7167622E76312E4D7367526567697374657245564D41646472657373526573706F6E7365
events:
- attributes:
  - index: true
    key: c3BlbmRlcg==
    value: Y2VsZXN0aWExZXp2MDNxcDh2dnBoZ2t6dm1hazBlZnNsZGR3eHo5ZG5lMDIyd2g=
  - index: true
    key: YW1vdW50
    value: MzAwMDB1dGlh
  type: coin_spent
- attributes:
  - index: true
    key: cmVjZWl2ZXI=
    value: Y2VsZXN0aWExN3hwZnZha20yYW1nOTYyeWxzNmY4NHoza2VsbDhjNWxwbmpzM3M=
  - index: true
    key: YW1vdW50
    value: MzAwMDB1dGlh
  type: coin_received
- attributes:
  - index: true
    key: cmVjaXBpZW50
    value: Y2VsZXN0aWExN3hwZnZha20yYW1nOTYyeWxzNmY4NHoza2VsbDhjNWxwbmpzM3M=
  - index: true
    key: c2VuZGVy
    value: Y2VsZXN0aWExZXp2MDNxcDh2dnBoZ2t6dm1hazBlZnNsZGR3eHo5ZG5lMDIyd2g=
  - index: true
    key: YW1vdW50
    value: MzAwMDB1dGlh
  type: transfer
- attributes:
  - index: true
    key: c2VuZGVy
    value: Y2VsZXN0aWExZXp2MDNxcDh2dnBoZ2t6dm1hazBlZnNsZGR3eHo5ZG5lMDIyd2g=
  type: message
- attributes:
  - index: true
    key: ZmVl
    value: MzAwMDB1dGlh
  - index: true
    key: ZmVlX3BheWVy
    value: Y2VsZXN0aWExZXp2MDNxcDh2dnBoZ2t6dm1hazBlZnNsZGR3eHo5ZG5lMDIyd2g=
  type: tx
- attributes:
  - index: true
    key: YWNjX3NlcQ==
    value: Y2VsZXN0aWExZXp2MDNxcDh2dnBoZ2t6dm1hazBlZnNsZGR3eHo5ZG5lMDIyd2gvNw==
  type: tx
- attributes:
  - index: true
    key: c2lnbmF0dXJl
    value: MXlCQUs4dG5idDBFcTh6RU1yQjBFekZ4TlRXdmFiZVFEa25VaDg2dmVjVUx3WXBnYzJHeTB2ZDFSWUllOWFuVDNLK3VsQ3Y5RVZCa05TNmx1QlRFbGc9PQ==
  type: tx
- attributes:
  - index: true
    key: YWN0aW9u
    value: L2NlbGVzdGlhLnFnYi52MS5Nc2dSZWdpc3RlckVWTUFkZHJlc3M=
  type: message
gas_used: "99278"
gas_wanted: "210000"
height: "196077"
info: ""
logs:
- events:
  - attributes:
    - key: action
      value: /celestia.qgb.v1.MsgRegisterEVMAddress
    type: message
  log: ""
  msg_index: 0
raw_log: '[{"msg_index":0,"events":[{"type":"message","attributes":[{"key":"action","value":"/celestia.qgb.v1.MsgRegisterEVMAddress"}]}]}]'
timestamp: ""
tx: null
txhash: CD5A7058530960BC9BD892C07AA2022715AE3BB3FEBFD94648F8C84214B542F4

@jcstein jcstein added documentation Improvements or additions to documentation enhancement New feature or request labels Oct 3, 2023
@jcstein jcstein marked this pull request as ready for review October 3, 2023 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider consolidating consensus and validator pages on docs Add CLI start to consensus validating node
1 participant