Skip to content

Commit

Permalink
Correctness sev
Browse files Browse the repository at this point in the history
  • Loading branch information
zktaiga committed Sep 2, 2024
1 parent be4e486 commit 3f470cd
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
5 changes: 4 additions & 1 deletion internal/benchmark/metric.go
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,10 @@ func LoadEnabledMetrics(config configs.Config) map[metric.Group][]metricService
configs.Values.Benchmark.Consensus.Address,
"Attestation",
network.GenesisTime[network.Name(config.Benchmark.Network)],
[]metric.HealthCondition[float64]{},
[]metric.HealthCondition[float64]{
{Name: consensus.CorrectnessMeasurement, Threshold: 97, Operator: metric.OperatorLessThanOrEqual, Severity: metric.SeverityHigh},
{Name: consensus.CorrectnessMeasurement, Threshold: 98.5, Operator: metric.OperatorLessThanOrEqual, Severity: metric.SeverityMedium},
},
))
}

Expand Down
6 changes: 4 additions & 2 deletions internal/benchmark/metrics/consensus/attestation.go
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ const (
ReceivedBlockMeasurement = "ReceivedBlock"
MissedAttestationMeasurement = "MissedAttestation"
FreshAttestationMeasurement = "FreshAttestation"
CorrectnessMeasurement = "Correctness"
)

var (
Expand Down Expand Up @@ -158,14 +159,15 @@ func (a *AttestationMetric) fetchAttestationBlockRoot(ctx context.Context, slot
}

func (a *AttestationMetric) AggregateResults() string {
var missedAttestations, freshAttestations, missedBlocks, receivedBlocks, unreadyBlocks float64
var missedAttestations, freshAttestations, missedBlocks, receivedBlocks, unreadyBlocks, correctness float64

for _, point := range a.DataPoints {
missedAttestations += point.Values[MissedAttestationMeasurement]
missedBlocks += point.Values[MissedBlockMeasurement]
freshAttestations += point.Values[FreshAttestationMeasurement]
receivedBlocks += point.Values[ReceivedBlockMeasurement]
unreadyBlocks += point.Values[UnreadyBlockMeasurement]
correctness += freshAttestations / receivedBlocks * 100
}

return fmt.Sprintf(
Expand All @@ -175,7 +177,7 @@ func (a *AttestationMetric) AggregateResults() string {
missedBlocks,
freshAttestations,
receivedBlocks,
freshAttestations/receivedBlocks*100)
correctness)
}

func (a *AttestationMetric) calculateMeasurements(slot phase0.Slot) {
Expand Down

0 comments on commit 3f470cd

Please sign in to comment.