-
Notifications
You must be signed in to change notification settings - Fork 4
fix(rc): accept RC list-shape tags and honor tracing_sampling_rate #227
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
gh-worker-dd-mergequeue-cf854d
merged 28 commits into
main
from
worktree-brian.marks+adaptive-sampling-fixes
Jun 1, 2026
Merged
Changes from 25 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
32dc0e8
feat(rc): parse tracing_sampling_rate field on LibConfig
bm1549 a3ccbd2
fix(rc): honor tracing_sampling_rate from APM_TRACING payload
bm1549 d58f4ca
test(rc): guard wildcard rate rule appended after rules
bm1549 a8487c2
test(rc): guard explicit-null on either field clears prior RC override
bm1549 d1c21fe
fix(rc): normalize RC list-shape tags before libdd-sampling parse
bm1549 8713abb
refactor(rc): take normalize_rc_tags by slice, drop unreachable!()
bm1549 e159a46
fix(rc): fail closed on malformed adaptive-sampling RC payloads
bm1549 8464dd3
feat(config): expose env-side trace_sampling_rules accessor
bm1549 3dce356
feat(config): add DD_TRACE_SAMPLE_RATE to supported-configurations
bm1549 d658e2b
feat(config): parse DD_TRACE_SAMPLE_RATE and expose accessor
bm1549 cff0275
feat(sampling): apply DD_TRACE_SAMPLE_RATE as implicit catch-all
bm1549 9ba849b
fix(rc): synthetic catch-all uses default provenance, not dynamic
bm1549 e3debe0
fix(rc): respect env rules and env rate in adaptive-sampling composition
bm1549 16f1fab
fix(rc): address pre-push review findings
bm1549 17140e1
fix(rc): propagate RC apply errors, validate rate range, distinguish …
bm1549 c5442b0
fix(config): align DD_TRACE_SAMPLE_RATE entry with Feature Parity reg…
bm1549 cd34441
fix(rc): advertise APM_TRACING_SAMPLE_RATE capability bit
bm1549 12c38b5
Merge branch 'main' into worktree-brian.marks+adaptive-sampling-fixes
bm1549 d379c5b
chore: bump libdd-sampling 1.0.0 -> 2.1.0, drop RC tags shim
bm1549 f448c3b
fix(config): reject out-of-range DD_TRACE_SAMPLE_RATE
bm1549 47fba73
fix(rc): ignore APM_TRACING configs whose service_target mismatches
bm1549 bd077a5
style: drop needless borrow in baggage test
bm1549 7b9919f
docs(changelog): note service_target + DD_TRACE_SAMPLE_RATE fixes
bm1549 0992d20
fix(rc): match service_target against extra services, case-insensitively
bm1549 35bba4b
chore: regenerate LICENSE-3rdparty.csv after libdd-sampling bump
bm1549 dbd0c7a
chore: drop premature CHANGELOG entries
bm1549 9fce918
Update Cargo.toml
bm1549 699192f
docs: fix stale effective_initial_rules catch-all comment
bm1549 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.