-
Notifications
You must be signed in to change notification settings - Fork 426
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
feat(config): support libdatadog's library_config for config through file #12260
base: main
Are you sure you want to change the base?
Conversation
|
e4869ac
to
56082d1
Compare
Datadog ReportBranch report: ✅ 0 Failed, 130 Passed, 1184 Skipped, 3m 25.74s Total duration (25m 43.72s time saved) |
56082d1
to
ca62de8
Compare
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.
Ship it 🔥 🔥 🔥
BenchmarksBenchmark execution time: 2025-02-11 17:13:36 Comparing candidate commit 14950cf in PR branch Found 0 performance improvements and 1 performance regressions! Performance is the same for 393 metrics, 2 unstable metrics. scenario:iast_aspects-ospathbasename_aspect
|
6f77853
to
a40b960
Compare
471a266
to
1ebacb5
Compare
9fabaf0
to
1ebacb5
Compare
5b4cb54
to
1ebacb5
Compare
1ebacb5
to
14950cf
Compare
Description of changes
Most changes were taken from #11839
libdatadog
to v16.0.2libdatadog
that allows reading configuration from a file with a higher precedence than environment variablesSupported config entries
As of libdatadog v16.0.2:
Testing strategy
Added a unit test.
To test this PR, you can:
/etc/datadog-agent/application_monitoring.yaml
Additional concerns
Relevant system test: DataDog/system-tests#3982
Checklist
Reviewer Checklist