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

Adds support for timeout on the otlp/gRPC exporter #1821

Merged
merged 8 commits into from
Apr 20, 2021
Merged

Adds support for timeout on the otlp/gRPC exporter #1821

merged 8 commits into from
Apr 20, 2021

Conversation

paivagustavo
Copy link
Member

Closes #1085

Adds support for timeout on the otlp/gRPC exporter.

@codecov
Copy link

codecov bot commented Apr 19, 2021

Codecov Report

Merging #1821 (dc4ed38) into main (081cc61) will increase coverage by 0.2%.
The diff coverage is 71.4%.

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #1821     +/-   ##
=======================================
+ Coverage   78.2%   78.4%   +0.2%     
=======================================
  Files        135     135             
  Lines       7245    7252      +7     
=======================================
+ Hits        5667    5688     +21     
+ Misses      1330    1316     -14     
  Partials     248     248             
Impacted Files Coverage Δ
exporters/otlp/otlpgrpc/options.go 43.5% <33.3%> (-0.9%) ⬇️
exporters/otlp/otlpgrpc/driver.go 92.0% <100.0%> (+3.2%) ⬆️
exporters/otlp/otlpgrpc/connection.go 92.5% <0.0%> (+1.8%) ⬆️
exporters/otlp/internal/otlptest/data.go 88.0% <0.0%> (+24.0%) ⬆️

@MrAlias MrAlias added pkg:exporter:otlp Related to the OTLP exporter package release:1.0.0-rc.1 labels Apr 20, 2021
@MrAlias MrAlias added this to the RC1 milestone Apr 20, 2021
@MrAlias MrAlias merged commit 70bc9eb into open-telemetry:main Apr 20, 2021
@paivagustavo paivagustavo deleted the otlp_grpc_timeout branch April 20, 2021 17:53
@MrAlias MrAlias mentioned this pull request Apr 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg:exporter:otlp Related to the OTLP exporter package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update OTLP exporter configuration to conform to the specification
3 participants