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.

Sign up for GitHub

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

Jump to bottom

Trigger fixes #6792

Closed
wants to merge 947 commits into from
Closed

Trigger fixes #6792

wants to merge 947 commits into from

Conversation

AdoAdoAdo
Copy link
Contributor

Reasoning behind the pull request

Proposed changes

Testing procedure

Pre-requisites

Based on the Contributing Guidelines the PR author and the reviewers must check the following requirements are met:

  • was the PR targeted to the correct branch?
  • if this is a larger feature that probably needs more than one PR, is there a feat branch created?
  • if this is a feat branch merging, do all satellite projects have a proper tag inside go.mod?

ssd04 and others added 30 commits December 17, 2024 11:09
@ssd04
Merge branch 'feat/equivalent-messages' into integrate-proof-check-on…
1715ed0 
…-sync
@ssd04
fix conflicts
8a95e05
@ssd04
added flag for proofs check in header intercetor
89a246e
@ssd04
remove todo in sync process for proofs
cb82037
@ssd04
remove accumulation maps for epoch start confirmation block
87a635e
@ssd04
rename chan
44ab7fa
@ssd04
more fixes after review
555ea3d
@ssd04
added more unit tests on handling equivalent proofs on sync
ebb3fc3
@ssd04
check for already existing equivalent proof
39af8f1
@sstanculeanu
Merge pull request #6501 from multiversx/integrate-proof-check-on-sync
7b02128 
Integrate proof check on sync
@ssd04
subround endround fixes
5b4a098
@ssd04
nil proof check in worker
537eb34
@ssd04
remove debug logs
2f56ebd
@ssd04
return true if subround already finished
f9527f6
@ssd04
remove extra empty line
5d2773f
@ssd04
adapt integration tests for consensus v2
6ecd517
@AdoAdoAdo
Merge pull request #6683 from multiversx/subround-endround-v2-fixes
7e35a87 
Subround endround v2 fixes
@AdoAdoAdo
Merge branch 'feat/equivalent-messages' into consensus-v2-integration…
5c42932 
…-tests
@AdoAdoAdo
use EpochNotifier instead of EpochStartNotifier to transition to the …
be02d30 
…new consensus version
@AdoAdoAdo
trigger consensus version change as validator
97b9e06
@AdoAdoAdo
temp change of equivalent messages enable epoch to shorten transition…
ccdd6cb 
… tests
@AdoAdoAdo
fixes integration tests
75e042a
@AdoAdoAdo
Merge branch 'feat/equivalent-messages' into equivalent-proofs-feat-s…
6092d21 
…tabilization
@AdoAdoAdo
fixes integration tests
5cd3d46
@AdoAdoAdo
fixes check proof on first block after genesis
330eb4a
@AdoAdoAdo
fix header finality check for equivalent proofs
7832305
@ssd04
added separate single keys test with equivalent proofs
30f6c12
@AdoAdoAdo
adapt integration tests
60584a0
@AdoAdoAdo
adapt integration tests
a8fb6f1
@AdoAdoAdo
fixes & debug logs
df47f28
AdoAdoAdo and others added 29 commits February 6, 2025 10:47
@AdoAdoAdo
Merge pull request #6769 from multiversx/consensus-verifier-fix
1694574 
add header verification unit tests
@AdoAdoAdo
set fork detector checkpoints on bootstrapper applyBlock
5453171
@AdoAdoAdo
update condition to detect correctly the final blocks
068e81d
@AdoAdoAdo
update checks
181c610
@andreibancioiu
Fix some long tests.
74a2e3e
@ssd04
remove duplicated code
8a3ca7a
@andreibancioiu
Merge pull request #6785 from multiversx/fix-long-tests
308bb27 
Fix some long tests
@ssd04
update check + additional check for genesis block
e5eb36a
@sstanculeanu
Merge branch 'feat/equivalent-messages' into consensus-v2-integration…
14e6645 
…-tests
@ssd04
fix genesis time
ecbb55f
@AdoAdoAdo
update trigger epoch start prepare
a715be2
@AdoAdoAdo
use mutex for trigger
0c31a4b
@AdoAdoAdo
Merge pull request #6685 from multiversx/consensus-v2-integration-tests
4a354c1 
adapt integration tests for consensus v2
@AdoAdoAdo
Merge branch 'feat/equivalent-messages' into bootstrap-fixes
b6715a9
@sstanculeanu
better handling of missing attesting headers
ce10695
@sstanculeanu
proper fix for attesting headers
1ecff9a
@sstanculeanu
Merge pull request #6783 from multiversx/bootstrap-fixes
3909d04 
set fork detector checkpoints on bootstrapper applyBlock
@sstanculeanu
Merge branch 'feat/equivalent-messages' into handle_missing_attenstin…
788d72b 
…g_better
@sstanculeanu
fixes after review
113c0ad
@sstanculeanu
fixes after discussion
30a98a8
@AdoAdoAdo
Merge pull request #6786 from multiversx/handle_missing_attensting_be…
ad3f872 
…tter

better handling of missing attesting headers
@AdoAdoAdo
recover round at transition
cb4b243
@AdoAdoAdo
Merge branch 'feat/equivalent-messages' into consensus-transition-spe…
3eeecf7 
…edup
@AdoAdoAdo
fix unit tests + fixes after review
c2c3315
@AdoAdoAdo
add logging for trigger activation
aa29fdd
@AdoAdoAdo
Merge pull request #6787 from multiversx/consensus-transition-speedup
8d0c6fc 
Consensus transition speedup
@AdoAdoAdo
Merge branch 'feat/equivalent-messages' into trigger-fixes
b526ad9
@AdoAdoAdo
fix check finality attesting headers with proofs
e0cdb85
@AdoAdoAdo
fix cyclic import
dcac9c1
@AdoAdoAdo AdoAdoAdo closed this Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AdoAdoAdo @ssd04 @sstanculeanu @andreibancioiu