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.

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

Invalid tick count causes slot dumping #10082

Open
sakridge opened this issue May 16, 2020 · 4 comments
Open

Invalid tick count causes slot dumping #10082

sakridge opened this issue May 16, 2020 · 4 comments
Assignees
Milestone

Comments

@sakridge
Copy link
Member

Problem

Validators can dump their slots if they encounter an invalid tick count slot, but this error was expected and dumping/redownloading the slot is just wasted effort.

Proposed Solution

Detect this case and don't dump, or at least detect if there is some other different version of the slot available.

@mvines mvines added this to the v1.3.0 milestone May 21, 2020
@mvines mvines modified the milestones: v1.3.0, v1.4.0 Aug 5, 2020
@brianlong
Copy link
Contributor

Whenever I see this error in my logs, the current slot is spoiled along with the three following slots. In this sequence, the leader made the 1st block, messed-up the 2nd block, and didn't submit the 3rd or 4th.

{"leader":"BSF2yD9mqzaixDaLEraF1en82EWaXx7wbaCqSuKppqG5","skipped":false,"slot":30817244},
{"leader":"BSF2yD9mqzaixDaLEraF1en82EWaXx7wbaCqSuKppqG5","skipped":true,"slot":30817245},
{"leader":"BSF2yD9mqzaixDaLEraF1en82EWaXx7wbaCqSuKppqG5","skipped":true,"slot":30817246},
{"leader":"BSF2yD9mqzaixDaLEraF1en82EWaXx7wbaCqSuKppqG5","skipped":true,"slot":30817247}

(30817246 & 30817247 do not appear in my logs)

The next leader in line skipped 30817248 and then submitted blocks in the next 3 slots.

{"leader":"Db2V7nPHc4sPHne87nYXPGn8Kv8rMsiWCAjgAXmpqcpC","skipped":true,"slot":30817248},
{"leader":"Db2V7nPHc4sPHne87nYXPGn8Kv8rMsiWCAjgAXmpqcpC","skipped":false,"slot":30817249},
{"leader":"Db2V7nPHc4sPHne87nYXPGn8Kv8rMsiWCAjgAXmpqcpC","skipped":false,"slot":30817250},
{"leader":"Db2V7nPHc4sPHne87nYXPGn8Kv8rMsiWCAjgAXmpqcpC","skipped":false,"slot":30817251}

@brianlong
Copy link
Contributor

@sakridge Is the problem that I highlighted above directly related to this particular Issue #? If not, I will create a new Issue to track it.

@brianlong
Copy link
Contributor

I think I will create a new Issue # since they seem like different issues.

@sakridge
Copy link
Member Author

@brianlong I think this is a different issue. I think this is not an issue in the current code because the slot dumping is disabled at the moment.

@mvines mvines modified the milestones: v1.4.0, v1.5.0 Oct 8, 2020
@mvines mvines modified the milestones: v1.5.0, v1.6.0 Dec 17, 2020
@mvines mvines modified the milestones: v1.6.0, v1.7.0 Mar 11, 2021
@mvines mvines modified the milestones: v1.7.0, v1.8.0 May 10, 2021
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

No branches or pull requests

4 participants