Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Spec json is invalid: missing field gasLimitBoundDivisor at line 23 column 4 #6939

Closed
stone212 opened this issue Oct 31, 2017 · 4 comments
Closed
Labels
F5-documentation 📑 Documentation needs fixing, improving or augmenting. M3-docs 📑 Documentation. P5-sometimesoon 🌲 Issue is worth doing soon.
Milestone

Comments

@stone212
Copy link

Before filing a new issue, please provide the following information.

I'm running:

  • Parity version: 1.7.8
  • Operating system: Linux
  • And installed: via binaries

I am creating a private chain spec file for a private network. I see that the chain spec file for mainnet has "gasLimitBoundDivisor" in the "params" section:
https://github.com/paritytech/parity/blob/master/ethcore/res/ethereum/foundation.json

So I attempt to duplicate that, and I also place mine in the same section. But when I do, I get this error:

Spec json is invalid: missing field gasLimitBoundDivisor at line 23 column 4

If I add the gasLimitBoundDivisor to the "engine" section (under its params) the error goes away. That matches the example on this page:
https://github.com/paritytech/parity/wiki/Pluggable-Consensus

But it does not match the official mainnet chain spec file linked to above.

@sparshprogrammer
Copy link

    "gasLimitBoundDivisor":"0x0400"

add this on that line

@stone212
Copy link
Author

@sparshprogrammer

Thank you but I already did that, as I said in my initial comment:

If I add the gasLimitBoundDivisor to the "engine" section (under its params) the error goes away.

But the issue that I opened regards the conflicting documentation.

@Office-Julia Office-Julia added F5-documentation 📑 Documentation needs fixing, improving or augmenting. Z0-unconfirmed 🤔 Issue might be valid, but it’s not yet known. labels Nov 1, 2017
@5chdn
Copy link
Contributor

5chdn commented Nov 1, 2017

Duplicate of #6916

https://github.com/paritytech/parity/blob/master/CHANGELOG.md#parity-v180-2017-10-15

Further, users upgrading from 1.7 should acknowledge the following changes:

-The chain-engine was further abstracted and chain-specs need to be upgraded. #6134 #6591

Move registrar, gasLimitBoundDivisor, and eip155Transition from engine (Ethash/AuthorityRound/etc.) to general params section.

@5chdn 5chdn added M2-config 📂 Chain specifications and node configurations. Z7-duplicate 🖨 Issue is a duplicate. Closer should comment with a link to the duplicate. Z1-question 🙋‍♀️ Issue is a question. Closer should answer. and removed F5-documentation 📑 Documentation needs fixing, improving or augmenting. Z0-unconfirmed 🤔 Issue might be valid, but it’s not yet known. labels Nov 1, 2017
@5chdn 5chdn added this to the 1.9 milestone Nov 1, 2017
@5chdn 5chdn closed this as completed Nov 1, 2017
@5chdn
Copy link
Contributor

5chdn commented Nov 1, 2017

Sorry, fat-fingered this issue too quickly, thanks for reporting, I updated the docs.

@5chdn 5chdn added F5-documentation 📑 Documentation needs fixing, improving or augmenting. M3-docs 📑 Documentation. P5-sometimesoon 🌲 Issue is worth doing soon. and removed M2-config 📂 Chain specifications and node configurations. Z7-duplicate 🖨 Issue is a duplicate. Closer should comment with a link to the duplicate. Z1-question 🙋‍♀️ Issue is a question. Closer should answer. labels Nov 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
F5-documentation 📑 Documentation needs fixing, improving or augmenting. M3-docs 📑 Documentation. P5-sometimesoon 🌲 Issue is worth doing soon.
Projects
None yet
Development

No branches or pull requests

4 participants