[chore] consolidate scope/resource creation in transformer #4429
Conversation
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #4429 +/- ##
==========================================
- Coverage 92.19% 90.76% -1.44%
==========================================
Files 336 87 -249
Lines 9511 1906 -7605
Branches 2016 405 -1611
==========================================
- Hits 8769 1730 -7039
+ Misses 742 176 -566 |
| return { | ||
| name: scope.name, | ||
| version: scope.version, | ||
| droppedAttributesCount: 0, |
There was a problem hiding this comment.
I think this can be removed.
| droppedAttributesCount: 0, |
There was a problem hiding this comment.
It's actually part of the proto (see https://github.com/open-telemetry/opentelemetry-proto/blob/b5c1a7882180a26bb7794594e8546798ecb68103/opentelemetry/proto/common/v1/common.proto#L80)
I think the tests will need to be adapted to expect this new value.
There was a problem hiding this comment.
Already looks very good, thanks for taking care of this. 🙂
A few comments:
- I think the tests need to be adapted to include the
droppedAttributeCountin the resource assertions. npm run lint:fixwill address any lint issues that are currently causing the lint workflow fail.- please add an entry in
experimental/CHANGELOG.mdfor this PR (this will make the changelog workflow pass).
| return { | ||
| name: scope.name, | ||
| version: scope.version, | ||
| droppedAttributesCount: 0, |
There was a problem hiding this comment.
It's actually part of the proto (see https://github.com/open-telemetry/opentelemetry-proto/blob/b5c1a7882180a26bb7794594e8546798ecb68103/opentelemetry/proto/common/v1/common.proto#L80)
I think the tests will need to be adapted to expect this new value.
|
This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
|
Merged with the proposed changes and lint fixes in #4600 |
No description provided.