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

bumping app to sdk 0.45.1: panic message redacted to hide potentially sensitive system info #11140

Closed
4 tasks
yaruwangway opened this issue Feb 8, 2022 · 7 comments

Comments

@yaruwangway
Copy link
Contributor

yaruwangway commented Feb 8, 2022

Summary of Bug

When bumping gaia dependency version to sdk 0.45.1, gaia could not run from height 0,
panic message redacted to hide potentially sensitive system info: panic

But if I run gaiad from existing database(start not from height 0), then it could run. Details please find in Steps to Reproduce.

I see a similar err reported but I have make sure the module init genesis orders is following the comment.

I am not sure if the code change in this pr is wrong so I imported sdk from my local, and try to add more print to debug, however, this will introduce another issue .String() panics with nested Anys

Version

Steps to Reproduce

  1. Build gaiad from https://github.com/cosmos/gaia/pull/1219/files
  2. Init a single node validator network locally and run gaia
    will see the panic:
4:49PM INF asserting crisis invariants inv=9/12 module=x/crisis name=bank/nonnegative-outstanding
4:49PM INF asserting crisis invariants inv=10/12 module=x/crisis name=bank/total-supply
4:49PM INF asserting crisis invariants inv=11/12 module=x/crisis name=gov/module-account
4:49PM INF asserted all invariants duration=0.95175 height=0 module=x/crisis
panic message redacted to hide potentially sensitive system info: panic
  1. run this gaiad version on existing data (use gaiad v6 to init and run to produce a few blocks, stop the chain, change to the gaiad from this pr), no panic/err will occur.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@yaruwangway
Copy link
Contributor Author

It seems that gaia iterates the genesis and cannot get the validator...it gets an empty validator set

@alexanderbez
Copy link
Contributor

It seems that gaia iterates the genesis and cannot get the validator...it gets an empty validator set

Are there gentxs in the genesis state?

@yaruwangway
Copy link
Contributor Author

yaruwangway commented Feb 8, 2022

at .gaia/config/gentx/ there is a json file @alexanderbez

@yaruwangway
Copy link
Contributor Author

with the same genesis, if I use gaiad v6 rather than gaiad from this pr, I can start the chain ....

@yaruwangway
Copy link
Contributor Author

in genesis there is gen_tx @alexanderbez

 "genutil": {
      "gen_txs": [
        {
          "body": {
            "messages": [
              {
                "@type": "/cosmos.staking.v1beta1.MsgCreateValidator",
                "description": {
                  "moniker": "test",
                  "identity": "",
                  "website": "",
                  "security_contact": "",
                  "details": ""
                },
                "commission": {
                  "rate": "0.100000000000000000",
                  "max_rate": "0.200000000000000000",
                  "max_change_rate": "0.010000000000000000"
                },
                "min_self_delegation": "1",
                "delegator_address": "cosmos140r2fz7a4u79al39f2krwxaxsnz4qh3agz8ge4",
                "validator_address": "cosmosvaloper140r2fz7a4u79al39f2krwxaxsnz4qh3adkna4x",
                "pubkey": {
                  "@type": "/cosmos.crypto.ed25519.PubKey",
                  "key": "TElEvoTtpTnqYygjj+8LfxbhpNNwBAfwTwFq0PVf+wg="
                },
                "value": {
                  "denom": "stake",
                  "amount": "1000000"
                }
              }
            ],

@yaruwangway
Copy link
Contributor Author

genutil module should init after staking and auth in SetOrderInitGenesis() because genutil need to access the params from there !

@AmauryM thank you for helping!

@RaulBernal
Copy link

A guide to bump to sdk 0.45.1 from v.0.44.x would be very appreciated

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

No branches or pull requests

3 participants