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

Eager dlinear reporting consensus duration greater than legacy inc timer #2337

Closed
3 tasks
dB2510 opened this issue Jun 20, 2023 · 1 comment
Closed
3 tasks
Assignees
Labels
bug Something isn't working protocol Protocol Team tickets

Comments

@dB2510
Copy link
Contributor

dB2510 commented Jun 20, 2023

🎯 Problem to be solved

eager_dlinear timer is reporting consensus duration metric greater than legacy increasing time. The eager_dlinear starts measuring duration BEFORE it can propose so the duration of eager timers start before normal times. How do we compare the duration of eager timers vs normal timers, since they do not measure the same thing?

🛠️ Proposed solution

There are some ways to approach this:

  • Consensus delay from start of slot (or start of duty offset) is probably best. But that includes fetcher delay, so the data wouldn't be focussed on consensus only, so biased by beacon node score. At least the bias introduced should be uncorrelated to timer type.
  • Other option:
    • Duration from consensus propose (post fetcher) to consensus decide
    • This could however result in negative values for eager timers, but that is probably fine.
    • This would at least focus on consensus only.
    • duration=consensus_decided_at-consensus_proposed_at

🧪 Tests

  • Tested by new automated unit/integration/smoke tests
  • Manually tested on core team/canary/test clusters
  • Manually tested on local compose simnet
@github-actions github-actions bot added the protocol Protocol Team tickets label Jun 20, 2023
@OisinKyne
Copy link
Contributor

What is the root problem here? Is our QBFT experiment borked because of this bug?

@boulder225 boulder225 added the bug Something isn't working label Jul 5, 2023
@corverroos corverroos added question Further information is requested bug Something isn't working and removed bug Something isn't working question Further information is requested labels Jul 12, 2023
obol-bulldozer bot pushed a commit that referenced this issue Jul 13, 2023
Refactors the consensus duration metric to `duration=consensus_decided_at-consensus_proposed_at` so that eager timer durations are reflected correctly since they start much earlier than normal timing strategies. 

category: refactor
ticket: #2337
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working protocol Protocol Team tickets
Projects
None yet
Development

No branches or pull requests

4 participants