fix(celery): change out.host tags to point to celery broker

[ZStriker19]

Motivation

Change out.host tags to point towards the celery broker, instead of the local celery hostname. Fixes service-representation issues.

Fixes 11491

Checklist

• PR author has checked that all the criteria below are met
• The PR description includes an overview of the change
• The PR description articulates the motivation for the change
• The change includes tests OR the PR description describes a testing strategy
• The PR description notes risks associated with the change, if any
• Newly-added code is easy to change
• The change follows the library release note guidelines
• The change includes or references documentation updates if necessary
• Backport labels are set (if applicable)

Reviewer Checklist

• Reviewer has checked that all the criteria below are met
• Title is accurate
• All changes are related to the pull request's stated goal
• Avoids breaking API changes
• Testing strategy adequately addresses listed risks
• Newly-added code is easy to change
• Release note makes sense to a user of the library
• If necessary, author has acknowledged and discussed the performance implications of this PR as reported in the benchmarks PR comment
• Backport labels are set in a manner that is consistent with the release branch maintenance policy

[github-actions]

CODEOWNERS have been resolved as:

.riot/requirements/1509aa1.txt                                          @DataDog/apm-python
.riot/requirements/1df4aa0.txt                                          @DataDog/apm-python
.riot/requirements/654f8c0.txt                                          @DataDog/apm-python
releasenotes/notes/update-celery-out-host-tag-be8da4f2ab88b4cf.yaml     @DataDog/apm-python
tests/contrib/celery/test_tagging.py                                    @DataDog/apm-core-python @DataDog/apm-idm-python
ddtrace/contrib/internal/celery/signals.py                              @DataDog/apm-core-python @DataDog/apm-idm-python
riotfile.py                                                             @DataDog/apm-python
tests/contrib/celery/base.py                                            @DataDog/apm-core-python @DataDog/apm-idm-python
tests/contrib/celery/test_integration.py                                @DataDog/apm-core-python @DataDog/apm-idm-python
tests/contrib/jobspec.yml                                               @DataDog/apm-core-python @DataDog/apm-idm-python

[datadog-dd-trace-py-rkomorn]

Datadog Report

Branch report: zachg/fix_celery_outgoing_tags
Commit report: d644a36
Test service: dd-trace-py

✅ 0 Failed, 1097 Passed, 281 Skipped, 8m 50.59s Total duration (17m 43.95s time saved)

[pr-commenter]

Benchmarks

Benchmark execution time: 2024-11-25 20:34:07

Comparing candidate commit 96695f1 in PR branch zachg/fix_celery_outgoing_tags with baseline commit b5b6dd0 in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 388 metrics, 2 unstable metrics.

[djova]

👍 with a suggestion

[wconti27]

/merge

[dd-devflow]

Devflow running: /merge

View all feedbacks in Devflow UI.

---

2024-11-25 19:54:57 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.

---

2024-11-25 20:36:50 UTC ℹ️ MergeQueue: merge request added to the queue

The median merge time in main is 34m.

[wantsui]

I am also approving, but left a note about a possible edge case.