Skip to content

Commit

Permalink
[APM]Latency threshold rule's threshold context variable should use m…
Browse files Browse the repository at this point in the history
…illiseconds instead of microseconds
  • Loading branch information
cauemarcondes committed Feb 2, 2023
1 parent 2fb8620 commit 4d2cc8e
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,7 @@ export function registerTransactionDurationRuleType({
[TRANSACTION_TYPE]: transactionType,
[PROCESSOR_EVENT]: ProcessorEvent.transaction,
[ALERT_EVALUATION_VALUE]: transactionDuration,
[ALERT_EVALUATION_THRESHOLD]: thresholdMicroseconds,
[ALERT_EVALUATION_THRESHOLD]: ruleParams.threshold,
[ALERT_REASON]: reason,
...sourceFields,
},
Expand All @@ -292,7 +292,7 @@ export function registerTransactionDurationRuleType({
interval: `${ruleParams.windowSize}${ruleParams.windowUnit}`,
reason,
serviceName,
threshold: thresholdMicroseconds,
threshold: ruleParams.threshold,
transactionType,
triggerValue: transactionDurationFormatted,
viewInAppUrl,
Expand Down

0 comments on commit 4d2cc8e

Please sign in to comment.