Skip to content

chore: Add display name to Span Pruning Processor#47572

Merged
mx-psi merged 3 commits into
open-telemetry:mainfrom
jaydeluca:span-processor-name
Apr 13, 2026
Merged

chore: Add display name to Span Pruning Processor#47572
mx-psi merged 3 commits into
open-telemetry:mainfrom
jaydeluca:span-processor-name

Conversation

@jaydeluca
Copy link
Copy Markdown
Member

Related to #47277 (comment)

Added display_name to the metadata file and ran go generate from the module to update the generated readme.

The changes in the other files (import ordering) are also from running that command.

cc @portertech

@jaydeluca jaydeluca requested a review from a team as a code owner April 12, 2026 15:37
@jaydeluca jaydeluca requested a review from bogdandrutu April 12, 2026 15:37
@github-actions github-actions Bot requested a review from portertech April 12, 2026 15:37
@jaydeluca jaydeluca changed the title Add display name to Span Pruning Processor chore: Add display name to Span Pruning Processor Apr 12, 2026
@paulojmdias paulojmdias added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Apr 12, 2026
Copy link
Copy Markdown
Member

@paulojmdias paulojmdias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

go package imports not formatted, please run "make gogci" and commit the changes in this PR.

Copy link
Copy Markdown
Member

@mx-psi mx-psi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. @jaydeluca if you want to we could make this field required on mdatagen to prevent this kind of thing in the future

@mx-psi
Copy link
Copy Markdown
Member

mx-psi commented Apr 13, 2026

needs make gogci

@mx-psi mx-psi merged commit 20d37ed into open-telemetry:main Apr 13, 2026
189 checks passed
AndrewCharlesHay pushed a commit to AndrewCharlesHay/opentelemetry-collector-contrib that referenced this pull request Apr 23, 2026
Related to
open-telemetry#47277 (comment)

Added `display_name` to the metadata file and ran `go generate` from the
module to update the generated readme.

The changes in the other files (import ordering) are also from running
that command.

cc @portertech
gracewehner pushed a commit to gracewehner/opentelemetry-collector-contrib that referenced this pull request Apr 29, 2026
Related to
open-telemetry#47277 (comment)

Added `display_name` to the metadata file and ran `go generate` from the
module to update the generated readme.

The changes in the other files (import ordering) are also from running
that command.

cc @portertech
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

processor/spanpruning Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants