Log usage of niche features (#893)

codecov · Nov 14, 2024 · 20861f4 · 20861f4
1 parent b3f5e8d
commit 20861f4
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 6 deletions.
diff --git a/services/report/raw_upload_processor.py b/services/report/raw_upload_processor.py
@@ -129,6 +129,10 @@ def clear_carryforward_sessions(
     if to_partially_overwrite_flags:
         # NOTE: this here might be the most accurate counter, as it takes into account the
         # actual `to_merge_flags` that were used for this particular upload.
+        sentry_sdk.capture_message(
+            "Customer is using `carryforward_mode=labels` feature",
+            extras={"flags": to_partially_overwrite_flags},
+        )
         LABELS_USAGE.labels(codepath="carryforward_cleanup").inc()
 
     to_fully_overwrite_flags = flags_under_carryforward_rules.difference(

diff --git a/tasks/upload.py b/tasks/upload.py
@@ -78,8 +78,8 @@ def __init__(
         repoid: int,
         commitid: str,
         report_type: ReportType = ReportType.COVERAGE,
-        report_code: Optional[str] = None,
-        redis_connection: Optional[Redis] = None,
+        report_code: str | None = None,
+        redis_connection: Redis | None = None,
     ):
         self.repoid = repoid
         self.commitid = commitid
@@ -89,8 +89,6 @@ def __init__(
 
     def log_extra(self, **kwargs) -> dict:
         return dict(
-            repoid=self.repoid,
-            commit=self.commitid,
             report_type=self.report_type.value,
             report_code=self.report_code,
             **kwargs,
@@ -286,8 +284,8 @@ def run_impl(
         db_session: Session,
         repoid: int,
         commitid: str,
-        report_type="coverage",
-        report_code=None,
+        report_type: str = "coverage",
+        report_code: str | None = None,
         *args,
         **kwargs,
     ):
@@ -297,6 +295,11 @@ def run_impl(
             report_type=ReportType(report_type),
             report_code=report_code,
         )
+        if report_code:
+            sentry_sdk.capture_message(
+                "Customer is using non-default `report_code`",
+                tags={"report_type": report_type, "report_code": report_code},
+            )
         checkpoints = upload_context.get_checkpoints(kwargs)
         log.info("Received upload task", extra=upload_context.log_extra())