Skip to content

Sample rate setter #507

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Jun 4, 2025
Merged

Sample rate setter #507

merged 8 commits into from
Jun 4, 2025

Conversation

LikeTheSalad
Copy link
Contributor

Addresses #480

Copy link

github-actions bot commented Jun 4, 2025

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@LikeTheSalad LikeTheSalad marked this pull request as ready for review June 4, 2025 08:31
@LikeTheSalad LikeTheSalad requested review from theletterf, hegerchr and a team June 4, 2025 08:31
theletterf
theletterf previously approved these changes Jun 4, 2025
Copy link
Contributor

@theletterf theletterf left a comment

Choose a reason for hiding this comment

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

Approved with a couple of suggestions. I'll make sure to bring this over to elastic/opentelemetry#244

Co-authored-by: Fabrizio Ferri-Benedetti <algernon@fastmail.com>
@LikeTheSalad LikeTheSalad merged commit 479bc59 into main Jun 4, 2025
9 checks passed
@LikeTheSalad LikeTheSalad deleted the sample-rate-setter branch June 16, 2025 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants