Skip to content

add legacy-consensus simulators#1122

Closed
winsvega wants to merge 1 commit intoethereum:masterfrom
winsvega:master
Closed

add legacy-consensus simulators#1122
winsvega wants to merge 1 commit intoethereum:masterfrom
winsvega:master

Conversation

@winsvega
Copy link
Contributor

A bit of copy paste, but its ok because legacy tests won't be changing.
So this adds simulators

legacy-consensus - to run all the tests we had at the times generated from Frontier to Constantinople revisions
legacy-consensus-cancun - to run all the tests from Constantinople up to Cancun

and from >=Cancun and beyond the tests might change and will be migrated to Pyspecs. so the format might change.
those will be fun by consensus simulator regulary. those tests are all vectors we have in Cancun only revision. so each run will save time.

then legacy- ones could be run once a week or so. this will save hive stats the running time.

@fjl
Copy link
Collaborator

fjl commented Jun 20, 2024

It's not great to simply copy the consensus simulator including its implementation. We could just integrate it into the existing consensus simulator as a separate test suite.

@winsvega
Copy link
Contributor Author

The legacy simulators should run not on a daily basis.

Ideally, the logic should be abstracted out.
Because only the folder change

@winsvega
Copy link
Contributor Author

@fjl ok, how about this approach?
#1130

@fjl fjl closed this Jul 15, 2024
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

Successfully merging this pull request may close these issues.

2 participants