chore(reports): Add logging behind feature flag (#29725)

* chore(reports): Add logging behind feature flag

Author: ceorourke

src/sentry/conf/server.py

@@ -1044,6 +1044,8 @@ def create_partitioned_queues(name):
     # Enable usage of external relays, for use with Relay. See
     # https://github.com/getsentry/relay.
     "organizations:relay": True,
+    # Enable logging for weekly reports
+    "organizations:weekly-report-debugging": False,
     # Enable Session Stats down to a minute resolution
     "organizations:minute-resolution-sessions": True,
     # Automatically opt IN users to receiving Slack notifications.

src/sentry/features/__init__.py

@@ -127,6 +127,7 @@
 default_manager.add("organizations:prompt-dashboards", OrganizationFeature)
 default_manager.add("organizations:related-events", OrganizationFeature)
 default_manager.add("organizations:relay", OrganizationFeature)
+default_manager.add("organizations:weekly-report-debugging", OrganizationFeature, True)
 default_manager.add("organizations:release-adoption-chart", OrganizationFeature, True)
 default_manager.add("organizations:release-adoption-stage", OrganizationFeature, True)
 default_manager.add("organizations:release-archives", OrganizationFeature)

src/sentry/tasks/reports.py

@@ -705,8 +705,16 @@ def deliver_organization_user_report(timestamp, duration, organization_id, user_
     user = User.objects.get(id=user_id)
 
     if not user_subscribed_to_organization_reports(user, organization):
+        if features.has("organizations:weekly-report-debugging", organization):
+            logger.info(
+                "reports.user.unsubscribed",
+                extra={
+                    "user": user.id,
+                    "organization_id": organization.id,
+                },
+            )
         logger.debug(
-            "Skipping report for %r to %r, user is not subscribed to reports.", organization, user
+            f"Skipping report for {organization} to {user}, user is not subscribed to reports."
         )
         return Skipped.NotSubscribed
 
@@ -715,10 +723,16 @@ def deliver_organization_user_report(timestamp, duration, organization_id, user_
         projects.update(Project.objects.get_for_user(team, user, _skip_team_check=True))
 
     if not projects:
+        if features.has("organizations:weekly-report-debugging", organization):
+            logger.info(
+                "reports.user.no_projects",
+                extra={
+                    "user": user.id,
+                    "organization_id": organization.id,
+                },
+            )
         logger.debug(
-            "Skipping report for %r to %r, user is not associated with any projects.",
-            organization,
-            user,
+            f"Skipping report for {organization} to {user}, user is not associated with any projects."
         )
         return Skipped.NoProjects
 
@@ -738,8 +752,16 @@ def deliver_organization_user_report(timestamp, duration, organization_id, user_
     )
 
     if not reports:
+        if features.has("organizations:weekly-report-debugging", organization):
+            logger.info(
+                "reports.user.no_reports",
+                extra={
+                    "user": user.id,
+                    "organization_id": organization.id,
+                },
+            )
         logger.debug(
-            "Skipping report for %r to %r, no qualifying reports to deliver.", organization, user
+            f"Skipping report for {organization} to {user}, no qualifying reports to deliver."
         )
         return Skipped.NoReports