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
Make stake pools engine rollback faster. #1285
Conversation
When rolling back, we currently only log monitoring restarts, which can be somewhat confusing when done several times in a row: [pools-engine] Monitoring stake pools. Currently at f4260343-[1158372.5#220] [pools-engine] Monitoring stake pools. Currently at 20bd31a4-[1158372.1#216] [pools-engine] Monitoring stake pools. Currently at 437a7400-[1158371.6#212] [pools-engine] Monitoring stake pools. Currently at 5c874264-[1158371.2#208] Although this is typical of rollbacks, it's better to mention it explicitely to better track what the system is doing.
bors try |
tryBuild succeeded |
@@ -175,7 +177,7 @@ monitorStakePools tr nl db@DBLayer{..} = do | |||
readPoolRegistration poolId >>= \case | |||
Nothing -> putPoolRegistration sl0 r | |||
Just{} -> pure () | |||
readPoolProductionCursor k | |||
readPoolProductionCursor (max 100 k) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why 100 here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, ok. explained below
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Issue Number
#1281
Overview
fe138cc
Log roll back attempts in pools monitoring
When rolling back, we currently only log monitoring restarts, which
can be somewhat confusing when done several times in a row:
[pools-engine] Monitoring stake pools. Currently at f4260343-[1158372.5#220]
[pools-engine] Monitoring stake pools. Currently at 20bd31a4-[1158372.1#216]
[pools-engine] Monitoring stake pools. Currently at 437a7400-[1158371.6#212]
[pools-engine] Monitoring stake pools. Currently at 5c874264-[1158371.2#208]
Although this is typical of rollbacks, it's better to mention it
explicitely to better track what the system is doing.
76bf85a
increase network layer observation window's length
Comments
See details in #1281