Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(sync): mark blocks which diverge from checkpoint as bad #12763

Merged
merged 2 commits into from
Dec 9, 2024

Conversation

Kubuxu
Copy link
Contributor

@Kubuxu Kubuxu commented Dec 5, 2024

Resolves flake in TestSyncCheckpointHead

Not certain why this was flaky, it would mean that sometimes the forking
tipset would not hit this case.

Resolves: #12758

Resolves flake in `TestSyncCheckpointHead`

Not certain why this was flaky, it would mean that sometimes the forking
tipset would not hit this case.

Signed-off-by: Jakub Sztandera <oss@kubuxu.com>
@Kubuxu Kubuxu added the skip/changelog This change does not require CHANGELOG.md update label Dec 5, 2024
Signed-off-by: Jakub Sztandera <oss@kubuxu.com>
@BigLep BigLep requested a review from rvagg December 5, 2024 17:33
@Kubuxu Kubuxu requested a review from Stebalien December 6, 2024 14:14
Copy link
Member

@rvagg rvagg left a comment

Choose a reason for hiding this comment

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

nice

@rvagg rvagg merged commit e145ef5 into master Dec 9, 2024
83 checks passed
@rvagg rvagg deleted the kubuxu/fix/sync-mark-bad branch December 9, 2024 04:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
skip/changelog This change does not require CHANGELOG.md update
Projects
Status: ☑️ Done (Archive)
Development

Successfully merging this pull request may close these issues.

New flaky: TestSyncCheckpointHead
2 participants