Skip to content

tornado: reduce cardinality with old semconv metrics attributes#4666

Open
xrmx wants to merge 4 commits into
open-telemetry:mainfrom
xrmx:tornado-http-target-template
Open

tornado: reduce cardinality with old semconv metrics attributes#4666
xrmx wants to merge 4 commits into
open-telemetry:mainfrom
xrmx:tornado-http-target-template

Conversation

@xrmx

@xrmx xrmx commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Description

Reduce the http.target cardinality in metrics when the old semantic conventions are configured.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A

Does This PR Require a Core Repo Change?

  • Yes. - Link to PR:
  • No.

Checklist:

See contributing.md for styleguide, changelog guidelines, and more.

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

Reduce the `http.target` cardinality in metrics when the old semantic
conventions are configured.
@xrmx xrmx requested a review from a team as a code owner June 5, 2026 09:43
@xrmx xrmx moved this to Ready for review in Python PR digest Jun 5, 2026
@github-project-automation github-project-automation Bot moved this from Ready for review to Approved PRs in Python PR digest Jun 5, 2026
xrmx and others added 2 commits June 11, 2026 09:09
Co-authored-by: Lukas Hering <40302054+herin049@users.noreply.github.com>
@xrmx xrmx enabled auto-merge June 11, 2026 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Approved PRs

Development

Successfully merging this pull request may close these issues.

3 participants