Skip to content

Commit da2a93a

Browse files
mifu67andrewshie-sentry
authored andcommitted
fix(aci milestone 3): make dual processing logs more informative (#89343)
Allow filtering for non-empty results + add aggregation value to logs
1 parent eaf33a7 commit da2a93a

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

src/sentry/incidents/subscription_processor.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -436,7 +436,11 @@ def process_update(self, subscription_update: QuerySubscriptionUpdate) -> None:
436436
logger.info(
437437
"dual processing results for alert rule %s",
438438
self.alert_rule.id,
439-
extra={"results": results},
439+
extra={
440+
"results": results,
441+
"num_results": len(results),
442+
"value": aggregation_value,
443+
},
440444
)
441445

442446
has_anomaly_detection = features.has(

0 commit comments

Comments
 (0)