Skip to content

[v2.2-dd] Cherry picks#20

Merged
jaredledvina merged 2 commits into
v2.2-ddfrom
jared.ledvina/v2.2-patches
Apr 3, 2026
Merged

[v2.2-dd] Cherry picks#20
jaredledvina merged 2 commits into
v2.2-ddfrom
jared.ledvina/v2.2-patches

Conversation

@jaredledvina
Copy link
Copy Markdown
Member

Similar to #17

This PR cherry-picks to the 2.2 release the following PR's that are not found upstream:

Looking at release/2.1...v2.1-dd the only other change we had was #18 but that's been merged upstream and is included in 2.2 via containerd#12304

Fricounet and others added 2 commits April 3, 2026 11:36
* [tracing] Create NameSampler

Simple sampler that samples every span having a certain name.
It's best used in conjunction with the ParentBased sampler so that the child spans are also sampled.

* [tracing] Configure otlp plugin to use NameSampler

Allow the plugin to use the custom NameSampler if specified in the usual env vars used to specify the sampler `OTEL_TRACES_SAMPLER` and `OTEL_TRACES_SAMPLER_ARG`. This sampler will allow to avoid emitting a lot of the noisy traces to only keep a subset of meaningful traces.
Co-authored-by: Justin Bera <justin.bera@datadoghq.com>
@jaredledvina jaredledvina requested a review from a team April 3, 2026 15:54
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 13c29fb7f8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread pkg/tracing/plugin/otlp.go
@jaredledvina jaredledvina merged commit b900fad into v2.2-dd Apr 3, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants