-
Notifications
You must be signed in to change notification settings - Fork 293
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
add remote config support for sampling rules #4242
Conversation
Overall package sizeSelf size: 6.59 MB Dependency sizes
🤖 This report was automatically generated by heaviest-objects-in-the-universe |
BenchmarksBenchmark execution time: 2024-05-30 22:58:57 Comparing candidate commit 83a26ee in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 264 metrics, 2 unstable metrics. |
…e-js into ida613/rc-sampling-rules
Co-authored-by: simon-id <simon.id@datadoghq.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but wait for APM review too
…e-js into ida613/rc-sampling-rules
* added rc support for sampling rules * run system test * Update config.spec.js * add rc capability and update code styling * removed unnessary check * add unit test * unit test fix * fix lint errors * fix lint error * Update packages/dd-trace/test/config.spec.js Co-authored-by: simon-id <simon.id@datadoghq.com> * simplified code * ignore sampling rules tags for now * fix lint errors * remove unnecessary comments and ci command --------- Co-authored-by: simon-id <simon.id@datadoghq.com>
* added rc support for sampling rules * run system test * Update config.spec.js * add rc capability and update code styling * removed unnessary check * add unit test * unit test fix * fix lint errors * fix lint error * Update packages/dd-trace/test/config.spec.js Co-authored-by: simon-id <simon.id@datadoghq.com> * simplified code * ignore sampling rules tags for now * fix lint errors * remove unnecessary comments and ci command --------- Co-authored-by: simon-id <simon.id@datadoghq.com>
What does this PR do?
https://docs.google.com/document/d/12lXVniuaYenanxnpt5gJJ76a1X7gFOfmCYVQVXlHNco/edit?usp=sharing
forcing system test to run with changes in .github/workflows/system-tests.yml, will delete the change after PR passes review
Motivation
Plugin Checklist
Additional Notes