feat(taskworker) Make status codes easier to understand (#92224)

Metric tags with named labels are much easier to understand than integer
values.

Author: markstory

src/sentry/taskworker/workerchild.py

@@ -83,6 +83,17 @@ def get_at_most_once_key(namespace: str, taskname: str, task_id: str) -> str:
     return f"tw:amo:{namespace}:{taskname}:{task_id}"
 
 
+def status_name(status: TaskActivationStatus.ValueType) -> str:
+    """Convert a TaskActivationStatus to a human readable name"""
+    if status == TASK_ACTIVATION_STATUS_COMPLETE:
+        return "complete"
+    if status == TASK_ACTIVATION_STATUS_FAILURE:
+        return "failure"
+    if status == TASK_ACTIVATION_STATUS_RETRY:
+        return "retry"
+    return f"unknown-{status}"
+
+
 def child_process(
     child_tasks: queue.Queue[TaskActivation],
     processed_tasks: queue.Queue[ProcessingResult],
@@ -334,15 +345,15 @@ def record_task_execution(
                 "taskname": activation.taskname,
                 "execution_duration": execution_duration,
                 "execution_latency": execution_latency,
-                "status": status,
+                "status": status_name(status),
             },
         )
         metrics.incr(
             "taskworker.worker.execute_task",
             tags={
                 "namespace": activation.namespace,
                 "taskname": activation.taskname,
-                "status": status,
+                "status": status_name(status),
                 "processing_pool": processing_pool_name,
             },
         )