Add celery task_id into scope tags (getsentry#596)

lemonlatte · web-flow · commit ca3c6b5e9656 · 2020-02-10T09:33:59.000+01:00
diff --git a/sentry_sdk/integrations/celery.py b/sentry_sdk/integrations/celery.py
@@ -169,6 +169,8 @@ def event_processor(event, hint):
         # type: (Event, Hint) -> Optional[Event]
 
         with capture_internal_exceptions():
+            tags = event.setdefault("tags", {})
+            tags["celery_task_id"] = uuid
             extra = event.setdefault("extra", {})
             extra["celery-job"] = {
                 "task_name": task.name,
diff --git a/tests/integrations/celery/test_celery.py b/tests/integrations/celery/test_celery.py
@@ -83,6 +83,7 @@ def dummy_task(x, y):
     assert event["contexts"]["trace"]["trace_id"] == span.trace_id
     assert event["contexts"]["trace"]["span_id"] != span.span_id
     assert event["transaction"] == "dummy_task"
+    assert "celery_task_id" in event["tags"]
     assert event["extra"]["celery-job"] == dict(
         task_name="dummy_task", **expected_context
     )

Original file line number	Diff line number	Diff line change
`@@ -83,6 +83,7 @@ def dummy_task(x, y):`
`83`	`83`	`assert event["contexts"]["trace"]["trace_id"] == span.trace_id`
`84`	`84`	`assert event["contexts"]["trace"]["span_id"] != span.span_id`
`85`	`85`	`assert event["transaction"] == "dummy_task"`
	`86`	`+ assert "celery_task_id" in event["tags"]`
`86`	`87`	`assert event["extra"]["celery-job"] == dict(`
`87`	`88`	`task_name="dummy_task", **expected_context`
`88`	`89`	`)`