-
Notifications
You must be signed in to change notification settings - Fork 10
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
resources are only dd.internal.resource tags #829
base: main
Are you sure you want to change the base?
Conversation
resources.push(resource); | ||
for (key, val) in &self.values { | ||
if key == "dd.internal.resource" { | ||
//anything coming in via dd.internal.resource:<value> has to be a key/value pair |
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.
how was this not caught in testing? is there only unit testing and not integration testing?
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.
The only way to get this in testing would be with a proper json/binary payload to compare, which is not done afaik
BenchmarksComparisonBenchmark execution time: 2025-01-16 20:23:30 Comparing candidate commit a9b35eb in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 52 metrics, 2 unstable metrics. CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
Group 13
BaselineOmitted due to size. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #829 +/- ##
==========================================
- Coverage 71.01% 70.97% -0.04%
==========================================
Files 319 319
Lines 46768 46780 +12
==========================================
- Hits 33210 33201 -9
- Misses 13558 13579 +21
|
avoid sending duplicates tags as resources when they are not prefixed with
dd.internal.resource