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

fix aleth genesis file and instructions #143

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

fix aleth genesis file and instructions #143

wants to merge 3 commits into from

Conversation

hugo-dc
Copy link
Member

@hugo-dc hugo-dc commented Dec 16, 2018

I updated the genesis file and instructions to connect an aleth node to the testnet.
Fixes #140
Please let me know if not needed files for dummymetering and dummyevm2wasm has to be removed in this PR as well.

@hugo-dc hugo-dc requested a review from axic December 16, 2018 02:40
@chfast chfast added this to Reviewer approved in Public testnet (2nd milestone) Jan 24, 2019
@hugo-dc
Copy link
Member Author

hugo-dc commented Jan 24, 2019

I removed startingBlock for precompiles as all precompiles should be able to be callled since first block.

@jwasinger
Copy link
Contributor

I'm trying to figure out why I'm getting genesis mismatch errors trying to connect to the testnet using this PR.

@hugo-dc
Copy link
Member Author

hugo-dc commented Jan 25, 2019

@jwasinger this genesis file won't work for the current testnet, the faucet balance was changed 931c304#diff-8ff30e3d87b04b0d6767d45bce187afd
If you want to sync aleth to the testnet you must use the genesis file in the cluster repo (ewasm-spec.json)

@jwasinger
Copy link
Contributor

LGTM. can we merge this?

@axic
Copy link
Member

axic commented Feb 5, 2019

@jwasinger how about @hugo-dc's last comment?

@jwasinger
Copy link
Contributor

Okay, genesis file should work now.

"0000000000000000000000000000000000000007": { "precompiled": { "name": "alt_bn128_G1_mul", "linear": { "base": 40000, "word": 0 } } },
"0000000000000000000000000000000000000008": { "precompiled": { "name": "alt_bn128_pairing_product" } },
"000000000000000000000000000000000000000a": { "codeFromFile": "metering.wasm", "balance": "0x01", "nonce": "0x00", "storage": {} },
"9fa4f23079bae7a7a5c392b67e18093310315bfc": { "balance": "0x200000000000000000000000000000000000000000000000000000000000000", "storage": {}, "nonce": "0x0", "code": "" }
Copy link
Member

Choose a reason for hiding this comment

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

Balance in geth seems to be:

    "9fa4f23079bae7a7a5c392b67e18093310315bfc": {
      "balance": "100000000000000000000000000"
    }

Copy link
Contributor

Choose a reason for hiding this comment

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

Right.. I only tested syncing against the public testnet (milestone-v1 genesis) and not staging

Copy link
Contributor

Choose a reason for hiding this comment

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

Something else must have changed with the genesis we have in staging because I am getting genesis mismatch when trying to connect after making this change.

Copy link
Contributor

Choose a reason for hiding this comment

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

Okay it syncs against staging. We need a tagged release so that #143 can be merged and then, the aleth branches changed here and this PR merged.

Copy link
Member

Choose a reason for hiding this comment

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

What do you mean exactly?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Public testnet (2nd milestone)
  
Reviewer approved
Development

Successfully merging this pull request may close these issues.

None yet

4 participants