Skip to content
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

[next] merge changes from main #4363

Merged
merged 35 commits into from
Dec 13, 2023
Merged

Conversation

dyladan
Copy link
Member

@dyladan dyladan commented Dec 13, 2023

Merge all changes from main into next

dyladan and others added 30 commits November 20, 2023 10:33
… listener

Fixes a memory leak where unhandled response bodies pile up in node 20
…lemetry#4315)

* feat: add script to update changelogs on releases

* fix: address comments

* Apply suggestions from code review

Co-authored-by: Trent Mick <trentm@gmail.com>

* fix: apply suggestions from code review

* fix: use packageJson.version instead of version

---------

Co-authored-by: Trent Mick <trentm@gmail.com>
fix(instrumentation-http): resume responses when there is no response…
…open-telemetry#4289)

* feat(sdk-logs): add droppedAttributesCount field to ReadableLogRecord

* chore: check droppedAttributesCount value in test case

* feat(otlp-transformer): make toLogRecord() use ReadableLogRecord.droppedAttributesCount

---------

Co-authored-by: Marc Pichler <marc.pichler@dynatrace.com>
…ry#4345)

* fix: allow passing in TimeInput for LogRecord

* chore: update changelog
open-telemetry#4334)

* fix: programmatic url and headers take precedence in metric exporters (open-telemetry#2370)

* chore: adjust grpc exporter metrics test

* chore(changelog): update changelog
@dyladan dyladan requested a review from a team December 13, 2023 13:43
@pichlermarc
Copy link
Member

This also includes everything from #4331, right?

Copy link

codecov bot commented Dec 13, 2023

Codecov Report

Merging #4363 (a942cb7) into next (543f0b4) will decrease coverage by 0.02%.
Report is 2 commits behind head on next.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             next    #4363      +/-   ##
==========================================
- Coverage   92.24%   92.23%   -0.02%     
==========================================
  Files         332      332              
  Lines        9437     9447      +10     
  Branches     1999     2003       +4     
==========================================
+ Hits         8705     8713       +8     
- Misses        732      734       +2     
Files Coverage Δ
...erimental/packages/api-logs/src/types/LogRecord.ts 100.00% <ø> (ø)
...porter-metrics-otlp-grpc/src/OTLPMetricExporter.ts 93.75% <100.00%> (+0.20%) ⬆️
...-otlp-http/src/platform/node/OTLPMetricExporter.ts 100.00% <100.00%> (ø)
...orter-metrics-otlp-proto/src/OTLPMetricExporter.ts 100.00% <100.00%> (ø)
...ges/opentelemetry-instrumentation-http/src/http.ts 93.97% <100.00%> (+0.05%) ⬆️
...ackages/otlp-exporter-base/src/OTLPExporterBase.ts 95.23% <100.00%> (ø)
...mental/packages/otlp-transformer/src/logs/index.ts 100.00% <ø> (ø)
experimental/packages/sdk-logs/src/LogRecord.ts 98.05% <100.00%> (+0.07%) ⬆️

... and 2 files with indirect coverage changes

@dyladan
Copy link
Member Author

dyladan commented Dec 13, 2023

This also includes everything from #4331, right?

Yes. The changes were made in #4332

@dyladan dyladan merged commit ea8bfa2 into open-telemetry:next Dec 13, 2023
19 checks passed
@dyladan dyladan deleted the next-main-12-13 branch December 13, 2023 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants