chore(tracing): fix some service naming tests that were failing silently

[wconti27]

Motivation

See silent failures here.

Checklist

• [x] 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

• [x] 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[bot]]

CODEOWNERS have been resolved as:

ddtrace/contrib/internal/bottle/trace.py                                @DataDog/apm-core-python @DataDog/apm-idm-python
tests/contrib/botocore/test.py                                          @DataDog/apm-core-python @DataDog/apm-idm-python

[datadog-dd-trace-py-rkomorn[bot]]

Datadog Report

Branch report: conti/update-service-naming-algo Commit report: 1844a4e Test service: dd-trace-py

:white_check_mark: 0 Failed, 10 Passed, 1368 Skipped, 55.92s Total duration (35m 13.04s time saved)

[pr-commenter[bot]]

Benchmarks

Benchmark execution time: 2024-11-25 19:40:27

Comparing candidate commit addb591b171db2b15e20624be0d4739351b4e4af in PR branch conti/update-service-naming-algo with baseline commit b5b6dd0634d7027d7056239978de2de2abfdb930 in branch main.

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

[wconti27]

/merge

[dd-devflow[bot]]

Devflow running: /merge

View all feedbacks in Devflow UI.

---

2024-11-21 16:09:29 UTC :information_source: MergeQueue: pull request added to the queue

The median merge time in main is 35m.

---

2024-11-21 17:27:29 UTC :x: MergeQueue: The checks failed on this merge request

Tests failed on this commit 3f108e5:

• test

What to do next?

• Investigate the failures and when ready, re-add your pull request to the queue!
• Any question, go check the FAQ.

[erikayasuda]

/merge

[dd-devflow[bot]]

Devflow running: /merge

View all feedbacks in Devflow UI.

---

2024-11-21 22:28:46 UTC :information_source: MergeQueue: pull request added to the queue

The median merge time in main is 34m.

---

2024-11-21 22:55:29 UTC :x: MergeQueue: The checks failed on this merge request

Tests failed on this commit 25ca482:

• test

What to do next?

• Investigate the failures and when ready, re-add your pull request to the queue!
• Any question, go check the FAQ.

[wconti27]

/merge

[dd-devflow[bot]]

Devflow running: /merge

View all feedbacks in Devflow UI.

---

2024-11-25 18:16:45 UTC :information_source: MergeQueue: pull request added to the queue

The median merge time in main is 35m.

---

2024-11-25 18:19:10 UTC :x: MergeQueue: This merge request was updated

This PR is rejected because it was updated

[wconti27]

/merge

[dd-devflow[bot]]

Devflow running: /merge

View all feedbacks in Devflow UI.

---

2024-11-25 19:32:25 UTC :information_source: 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 19:42:40 UTC :information_source: MergeQueue: merge request added to the queue

The median merge time in main is 35m.