Skip to content

Commit c0c2e4f

Browse files
committed
🐛 fix: subscription can be null
1 parent 211ecc9 commit c0c2e4f

File tree

1 file changed

+3
-10
lines changed

1 file changed

+3
-10
lines changed

src/sentry/incidents/typings/metric_detector.py

+3-10
Original file line numberDiff line numberDiff line change
@@ -111,8 +111,8 @@ class MetricIssueContext:
111111
id: int
112112
open_period_identifier: int # Used for link building
113113
snuba_query: SnubaQuery
114-
subscription: QuerySubscription
115114
new_status: IncidentStatus
115+
subscription: QuerySubscription | None
116116
metric_value: float | None
117117

118118
@classmethod
@@ -136,15 +136,8 @@ def _get_snuba_query(cls, occurrence: IssueOccurrence) -> SnubaQuery:
136136
return query
137137

138138
@classmethod
139-
def _get_subscription(cls, occurrence: IssueOccurrence) -> QuerySubscription:
140-
subscription_id = occurrence.evidence_data.get("subscription_id")
141-
if not subscription_id:
142-
raise ValueError("Subscription ID is required for alert context")
143-
try:
144-
subscription = QuerySubscription.objects.get(id=subscription_id)
145-
except QuerySubscription.DoesNotExist as e:
146-
raise ValueError("Subscription does not exist") from e
147-
return subscription
139+
def _get_subscription(cls, occurrence: IssueOccurrence) -> QuerySubscription | None:
140+
return occurrence.evidence_data.get("subscription_id")
148141

149142
@classmethod
150143
def _get_metric_value(cls, occurrence: IssueOccurrence) -> float:

0 commit comments

Comments
 (0)