Skip to content

Commit 8014bb7

Browse files
authored
chore(uptime): Always log results in uptime result processor (#74673)
We want to be able to inspect results regardless of whether a subscription is present, so just move the logging up <!-- Describe your PR here. -->
1 parent 96acd8d commit 8014bb7

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/sentry/uptime/consumers/results_consumer.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,8 @@ def get_subscription_id(self, result: CheckResult) -> str:
6262
return result["subscription_id"]
6363

6464
def handle_result(self, subscription: UptimeSubscription | None, result: CheckResult):
65+
logger.info("process_result", extra=result)
66+
6567
if subscription is None:
6668
# TODO: We probably want to want to publish a tombstone
6769
# subscription here
@@ -79,8 +81,6 @@ def handle_result(self, subscription: UptimeSubscription | None, result: CheckRe
7981
last_update_ms = 0 if last_update_raw is None else int(last_update_raw)
8082
self.handle_result_for_project(project_subscription, result, last_update_ms)
8183

82-
logger.info("process_result", extra=result)
83-
8484
def handle_result_for_project(
8585
self,
8686
project_subscription: ProjectUptimeSubscription,

0 commit comments

Comments
 (0)