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

pull constantinople on ethereum network #10189

Merged
merged 5 commits into from
Jan 15, 2019
Merged

pull constantinople on ethereum network #10189

merged 5 commits into from
Jan 15, 2019

Conversation

5chdn
Copy link
Contributor

@5chdn 5chdn commented Jan 15, 2019

@5chdn 5chdn added A0-pleasereview 🤓 Pull request needs code review. P0-dropeverything 🌋 Everyone should address the issue now. B1-patch-beta 🕷🕷 M2-config 📂 Chain specifications and node configurations. B9-blocker 🚧 This pull request blocks the next release from happening. Use only in extreme cases. B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. labels Jan 15, 2019
@5chdn 5chdn added this to the 2.4 milestone Jan 15, 2019
@5chdn 5chdn changed the title A5 config const pull constantinople on ethereum network Jan 15, 2019
Copy link
Contributor

@andresilva andresilva left a comment

Choose a reason for hiding this comment

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

I know 0xffffffffffffdead is cooler, but we could remove the parameters instead.

@@ -16,9 +16,9 @@ track = "nightly"
# Latest supported fork blocks.
# Indicates a critical release in this track (i.e. consensus issue).
[package.metadata.networks]
foundation = { forkBlock = 7080000, critical = false }
foundation = { forkBlock = 7080000, critical = true }
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't we revert the forkBlock as well to 4370000?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch!

@5chdn 5chdn merged commit a6c6c7c into master Jan 15, 2019
@5chdn 5chdn deleted the a5-config-const branch January 15, 2019 20:51
5chdn added a commit that referenced this pull request Jan 15, 2019
* ethcore: pull constantinople on ethereum network

* version: mark update as critical

* ethcore: remove constantinople alltogether from chain spec

* version: revert fork block for ethereum
5chdn added a commit that referenced this pull request Jan 15, 2019
* ethcore: pull constantinople on ethereum network

* version: mark update as critical

* ethcore: remove constantinople alltogether from chain spec

* version: revert fork block for ethereum
This was referenced Jan 15, 2019
5chdn added a commit that referenced this pull request Jan 15, 2019
* version: bump stable to 2.2.7

* version: mark 2.2 track stable

* version: mark update critical on all networks

* version: commit cargo lock

* Ping nodes from discovery (#10167)

* snap: fix path in script (#10157)

* snap: fix path in script

* debug, revert me

* fix

* necromancer awk magic

* awk necromancy and path fixing

* working track selection

* Fix _cannot recursively call into `Core`_ issue (#10144)

* Change igd to github:maufl/rust-igd

* Run `igd::search_gateway_from_timeout` from own thread

* Handle the case for contract creation on an empty but exist account with storage items (#10065)

* Add is_base_storage_root_unchanged

* Fix compile, use a shortcut for check, and remove ignored tests

* Add a warn!

* Update ethereum/tests to v6.0.0-beta.2

* grumble: use {:#x} instead of 0x{:x}

Co-Authored-By: sorpaas <accounts@that.world>

* version: bump fork blocks for kovan and foundation (#10186)

* pull constantinople on ethereum network (#10189)

* ethcore: pull constantinople on ethereum network

* version: mark update as critical

* ethcore: remove constantinople alltogether from chain spec

* version: revert fork block for ethereum
5chdn added a commit that referenced this pull request Jan 15, 2019
* version: mark 2.3 track beta

* version: mark update critical on all networks

* Ping nodes from discovery (#10167)

* Fix _cannot recursively call into `Core`_ issue (#10144)

* Change igd to github:maufl/rust-igd

* Run `igd::search_gateway_from_timeout` from own thread

* Handle the case for contract creation on an empty but exist account with storage items (#10065)

* Add is_base_storage_root_unchanged

* Fix compile, use a shortcut for check, and remove ignored tests

* Add a warn!

* Update ethereum/tests to v6.0.0-beta.2

* grumble: use {:#x} instead of 0x{:x}

Co-Authored-By: sorpaas <accounts@that.world>

* version: bump fork blocks for kovan and foundation (#10186)

* pull constantinople on ethereum network (#10189)

* ethcore: pull constantinople on ethereum network

* version: mark update as critical

* ethcore: remove constantinople alltogether from chain spec

* version: revert fork block for ethereum
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-pleasereview 🤓 Pull request needs code review. B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. B9-blocker 🚧 This pull request blocks the next release from happening. Use only in extreme cases. M2-config 📂 Chain specifications and node configurations. P0-dropeverything 🌋 Everyone should address the issue now.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants