log `delay` relative to deadline (#5769)

To better analyze logs across different `SECONDS_PER_SLOT`, log delays
for duties relative to their deadline instead of relative to slot start.
This commit is contained in:
Etan Kissling 2024-01-17 15:52:19 +01:00 committed by GitHub
parent f409ed97ae
commit e7794b6aec
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 8 additions and 5 deletions

View File

@ -381,7 +381,7 @@ proc processAttestation*(
return errIgnore("Attestation before genesis") return errIgnore("Attestation before genesis")
# Potential under/overflows are fine; would just create odd metrics and logs # Potential under/overflows are fine; would just create odd metrics and logs
let delay = wallTime - attestation.data.slot.start_beacon_time let delay = wallTime - attestation.data.slot.attestation_deadline
debug "Attestation received", delay debug "Attestation received", delay
# Now proceed to validation # Now proceed to validation
@ -431,8 +431,9 @@ proc processSignedAggregateAndProof*(
return errIgnore("Aggregate before genesis") return errIgnore("Aggregate before genesis")
# Potential under/overflows are fine; would just create odd logs # Potential under/overflows are fine; would just create odd logs
let delay = let
wallTime - signedAggregateAndProof.message.aggregate.data.slot.start_beacon_time slot = signedAggregateAndProof.message.aggregate.data.slot
delay = wallTime - slot.aggregate_deadline
debug "Aggregate received", delay debug "Aggregate received", delay
let v = let v =
@ -578,7 +579,7 @@ proc processSyncCommitteeMessage*(
wallSlot wallSlot
# Potential under/overflows are fine; would just create odd metrics and logs # Potential under/overflows are fine; would just create odd metrics and logs
let delay = wallTime - syncCommitteeMsg.slot.start_beacon_time let delay = wallTime - syncCommitteeMsg.slot.sync_committee_message_deadline
debug "Sync committee message received", delay debug "Sync committee message received", delay
# Now proceed to validation # Now proceed to validation
@ -624,7 +625,9 @@ proc processSignedContributionAndProof*(
wallSlot wallSlot
# Potential under/overflows are fine; would just create odd metrics and logs # Potential under/overflows are fine; would just create odd metrics and logs
let delay = wallTime - contributionAndProof.message.contribution.slot.start_beacon_time let
slot = contributionAndProof.message.contribution.slot
delay = wallTime - slot.sync_contribution_deadline
debug "Contribution received", delay debug "Contribution received", delay
# Now proceed to validation # Now proceed to validation