Implement EIP-1559 feature #1120
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Move PolyBFT to edge Co-authored-by: Igor Crevar <[email protected]> Co-authored-by: Sasa Prsic <[email protected]> Co-authored-by: Stefan Negovanović <[email protected]> Co-authored-by: Victor Castell <[email protected]>
Co-authored-by: Stefan Negovanović <[email protected]> Co-authored-by: Sasa Prsic <[email protected]>
* EVM-57 PolyBFT bridge enabled state txs Co-authored-by: Sasa Prsic <[email protected]> * EVM-57 PolyBFT bridge enabled state txs Co-authored-by: Sasa Prsic <[email protected]> * tidy. * pr fixes * pr fix 2 * small fix * TransactionType should be populated in receipt * nonce fix * write change original tx fix * createStateTransactionWithData change * pr fix more * revert ComputeHash for tx because id does not really matter Co-authored-by: Sasa Prsic <[email protected]>
Back-port the e2e framework + tests from v3 to edge as a separate set of tests.
* Initial changes * Init v3-contract submodule * Uptime changes and linter fixes * Use local build and test workflows (temporary) * Test workflow expand with init git submodules * Native transfer precompile * CommitTransaction in blockBuilder * adding workflow call secrets input (#799) * Add BLS aggregated signatures verification precompile * Change required gas for introduced precompiles * Add console precompile * Additional functions in bls package * Register validator command * Linter fixes, register subcommand for register validator command * Adapt genesis command * Cleanup * Change addresses of sidechain contracts so that they don't clash with precompile addresses * Marshal from address in transaction * Remove CommitTransaction and use WriteTx instead * Fix panic in register validator command * Get rid of panics in register validator * Increase gas limit for ChildValidatorSet initialization * Cleanup GenesisPostHookFactory * Fix register validator command * Change gas limit to 100M for initialize child validator set * Change make compile v3-contracts * Recover CI (1st attempt) * Recover CI (2nd attempt) * Recover CI (3rd attempt) * Cleanup Makefile (remove compilation of SCs when unneeded). Remove cloning submodules in lint workflow * Recover lint * Remove init submodules from test workflow * Address some go security issues * small make fix * init root contracts command * Remove rootchain contracts deployment from rootchain server command * Provide new url for smart contracts * Introduce constants for register validator and contracts root folder * Comments * Comments (part 2) * Reference dev-ethernal commit from core-contracts * Fix polybft e2e workflow * Fix genesis cmd invocation from e2e framework * contract_initializer fix * Fix polybft e2e tests * Comments (part 3) * Populate validators balance based on provided genesis configuration. Comments (part 4) * Fix balance unmarshal for polybft.Validator * Initialize native transfer contract Co-authored-by: Goran Rojovic <[email protected]> Co-authored-by: Epikichi <[email protected]> Co-authored-by: Igor Crevar <[email protected]>
* Add bridge flag. * Populate bridge config. * Fix bridge address. * rootchain panic fixes * changes. deploying contracts to root still does not work in test * fix deploying contracts and running bridge. Emit command change * Read balance from genesis * Remove Geth reference from Edge module * bridge test fixes * remove test-bootnode. add WithBootnodeCount to test-cluster. Change network_test to be more relaxed Co-authored-by: Sasa Prsic <[email protected]> Co-authored-by: Igor Crevar <[email protected]>
* test-bridge fix + temporary fix * Read balance from genesis * state, state transaction, event tracker * FSM tests (part 1) * ConsensusRuntime tests * FSM tests (part 2) * FSM tests (part 3) * SystemState tests * ValidatorAccount and ValidatorSnapshot tests * buildLogsFromReceipts test * FSM tests (part 4) * FSM fix remaining tests * t.parallel * Introduce parallel execution in tests (part 2) * Lint fixes * Rename test helpers file to exclude it from build * Add test suffix to mocks * Install solidity compiler on test workflow * Assign execute permission to setup-ci.sh script * polybft tests * all tests merged * setting epoch snapshot added * Lint fixes * Use constant for IstanbulExtraVanity * Description of polybft test Co-authored-by: Igor Crevar <[email protected]> Co-authored-by: Nemanja0x <[email protected]> Co-authored-by: Goran Rojovic <[email protected]>
* Exit event storage and query functions * ExitProof generation * Reorganize tests * Optimize key composition * Extract common function for decoding state sync and exit events * Comments fix * Comments fix * [Checkpointing] JSON RPC point for generating exit proofs (#844) * JSON RPC point for generating exit proofs * Comments fix Co-authored-by: Stefan Negovanović <[email protected]>
* Add CheckpointData to Extra * Don't store chainID, blockNumber and blockHash to the blockchain * Set checkpoint data to the Header.Extra * Fix and simplify FSM unit tests * getSignHash static function, fix verify header implementation * Increase block waiting time in bridge test * Remove unnecessary condition * Set checkpoint to each extra * Sign checkpoint hash instead of aggregated hash * Make nextValidatorsHash same as currentValidatorsHash for non-epoch ending blocks * Set event root to the checkpoint object
* Parse receipt logs and insert exit events * Comments fix * Comments fix * Comments fix * Remove TODO * Comments fix * Comments fix * Comments fix
* Change error messages for checkpoint hash calculation * Determine checkpoint blocks * Checkpoint submission (draft) * Introduce checkpoint manager * Lint fixes * Fix tests * Checkpoint manager checkpoint data ABI encode unit tests * Unit test for getCurrentCheckpointID * Unit test for submitCheckpoint * Remove leftover code * Comments * Determine epoch ending blocks by comparing epoch numbers in adjacent headers * Micro optimization * Skip querying first block for end of epoch * Optimization * Simplify checkpoints offset * Adapt comment * Checkpoint block detection optimization (#872) * Simplify checkpoint block detection * Resolve conflicts and pass tests Co-authored-by: Boris <[email protected]> * Simplify * Submit pending checkpoints in unit test * Extract parentHeader retrieval outside of the for * Submit end of epoch blocks for pending checkpoints * Update submitCheckpoint comment Co-authored-by: Victor Castell <[email protected]> * Feedback * Separate import declarations * Create var block for `parentExtra` and `parentHeader` Co-authored-by: Victor Castell <[email protected]> Co-authored-by: Goran Rojovic <[email protected]> Co-authored-by: Boris <[email protected]> Co-authored-by: Victor Castell <[email protected]>
* Mock Host implementation in native transfer test * Lint fix * Fix polybft e2e tests
* Implement the logic for quorum check Depending on the message type it will implement different kinds of quorum checks. * Remove MaximumFaulty
* EVM 149 - syncer go ibft coordination (#843) * consensus runtime backend metohod (not buildable) * transport * polybft - run cycle * transport fix 2 * small changes * less complex ibft consennsus wrapper.Example: https://replit.com/@crewce/Newest#main.go * small changes * runSequence better * remove pbftTransportWrapper * noone likes temp variables * Comment fixed * jm fixes Co-authored-by: Igor Crevar <[email protected]> * EVM-151 Use (or not) edge hooks support (#849) * FSM that supports go-ibft backend interface (#865) * consensus runtime backend metohod (not buildable) * transport * polybft - run cycle * transport fix 2 * less complex ibft consennsus wrapper.Example: https://replit.com/@crewce/Newest#main.go * Implement MessageConstructor interface. * Tidy linter errors. * runSequence better * remove pbftTransportWrapper * Add quorum size implementation. * Remove one part of pbft.NodeID reference. * Add some comments. * IsValid, Build proposal on FSM * Add block validation. * Add stub for IsValidSender method. * Sender validation is always true. * fsm - Insert block * simplify build proposal * Tidy logs. * IsValidSender * pbft.Proposal removed * Tidy errors. * restartEpoch if epoch is nil, remove hook in fsm * initialization fix * Fix compare. * quorum fixes and error propagation * evm-151 changes + additional needed in this task * Increase test timeout. * bring back some part of fsm * just execute syn test 100 times in a row * Tidy part of errors. * Revert e2e test. * Remove unused code and move message creator interface implementation. * minor fix * Add comments. * code organization - valdateSender + recoverAddress * Fix compile after merge. * small change * restart epoch on better place. lastBuiltBlock and epoch should not be nil * small reorg in polybft * pr fix * Update comments * pr fixes no 2 * Add voting power to the validators (#880) * Add voting power to the validator account * Rename ValidatorAccount to ValidatorMetadata * Set voting power to the existing test * Comment fixes * Rename files * Address comments * Remove extra space from the comment * pr fixes * Rename GetValidatorAccount to GetValidatorMetadata Co-authored-by: Nemanja0x <[email protected]> Co-authored-by: Igor Crevar <[email protected]> Co-authored-by: Stefan Negovanović <[email protected]> Co-authored-by: Stefan Negovanović <[email protected]> * Accommodate go-ibft `BuildProposal` updated function (#893) * Populate round info and change proposal hash * Adopt currentRound from BuildProposal * Fix logs * Fix proposal hash calculation inside IsValidProposalHash * Update go-ibft version * Igor's feedback addressed * Merge fix * EVM-187 Handling bundles on both sync and consensus (#882) * EVM-187 Handling bundles on both sync and consensus * consensus_runtime tests * UT fix Co-authored-by: Goran Rojovic <[email protected]> * Implement HasQuorum function in consensusRuntime Co-authored-by: Nemanja Nedic <[email protected]> Co-authored-by: Igor Crevar <[email protected]> Co-authored-by: Saša Pršić <[email protected]> Co-authored-by: Nemanja0x <[email protected]> Co-authored-by: Goran Rojovic <[email protected]>
Co-authored-by: Goran Rojovic <[email protected]>
…or other workflows (#916)
* Introduce local development cluster script
* Fix linters * Fix TestEventTracker_TrackSyncEvents * Comment
begmaroman
removed
the
don't merge
…P-1559 # Conflicts: # chain/params.go # command/genesis/params.go # command/genesis/polybft_params.go # e2e-polybft/e2e/helpers_test.go # e2e-polybft/framework/test-cluster.go
vcastellm
reviewed
Apr 22, 2023
vcastellm
reviewed
Apr 22, 2023
vcastellm
reviewed
Apr 22, 2023
…P-1559 # Conflicts: # consensus/polybft/block_builder.go # consensus/polybft/contractsapi/gen_sc_data.go # state/executor.go
vcastellm
approved these changes
Apr 24, 2023
Stefan-Ethernal
approved these changes
Apr 24, 2023
igorcrevar
approved these changes
Apr 24, 2023
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Implement EIP-1559
Changes include
Checklist
Testing