Always log APF InitialSeats and FinalSeats values
Add apf_additionalLatency field, to have all WorkEstimate data Kubernetes-commit: ed780fb06ebb401f599db1dc9295c73bbc1f565f
This commit is contained in:
parent
8ec6040424
commit
71b30218e5
|
@ -125,14 +125,10 @@ func WithPriorityAndFairness(
|
|||
workEstimate := workEstimator(r, classification.FlowSchemaName, classification.PriorityLevelName)
|
||||
|
||||
fcmetrics.ObserveWorkEstimatedSeats(classification.PriorityLevelName, classification.FlowSchemaName, workEstimate.MaxSeats())
|
||||
// nolint:logcheck // Not using the result of klog.V
|
||||
// inside the if branch is okay, we just use it to
|
||||
// determine whether the additional information should
|
||||
// be added.
|
||||
if klog.V(4).Enabled() {
|
||||
httplog.AddKeyValue(ctx, "apf_iseats", workEstimate.InitialSeats)
|
||||
httplog.AddKeyValue(ctx, "apf_fseats", workEstimate.FinalSeats)
|
||||
}
|
||||
httplog.AddKeyValue(ctx, "apf_iseats", workEstimate.InitialSeats)
|
||||
httplog.AddKeyValue(ctx, "apf_fseats", workEstimate.FinalSeats)
|
||||
httplog.AddKeyValue(ctx, "apf_additionalLatency", workEstimate.AdditionalLatency)
|
||||
|
||||
return workEstimate
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue