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

config: add meter/tracer provider config #4617

Conversation

codeboten
Copy link
Contributor

@codeboten codeboten commented Nov 24, 2023

This pulls in the code that was in the collector to configure the MeterProvider and TracerProvider.

Fixes #4371

Copy link

codecov bot commented Nov 24, 2023

Codecov Report

Attention: 30 lines in your changes are missing coverage. Please review.

Comparison is base (a001fcc) 81.1% compared to head (977e67f) 81.3%.

❗ Current head 977e67f differs from pull request most recent head 5db40c0. Consider uploading reports for the commit 5db40c0 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #4617     +/-   ##
=======================================
+ Coverage   81.1%   81.3%   +0.1%     
=======================================
  Files        150     150             
  Lines      10759   11034    +275     
=======================================
+ Hits        8731    8975    +244     
- Misses      1872    1895     +23     
- Partials     156     164      +8     
Files Coverage Δ
config/config.go 84.2% <64.0%> (-15.8%) ⬇️
config/trace.go 91.7% <91.6%> (-8.3%) ⬇️
config/metric.go 91.6% <92.2%> (-8.4%) ⬇️

@codeboten codeboten force-pushed the codeboten/implement-tracer-provider-sdk branch 2 times, most recently from 7e22836 to 4327828 Compare December 18, 2023 21:55
@codeboten codeboten marked this pull request as ready for review December 18, 2023 21:55
@codeboten codeboten requested a review from a team as a code owner December 18, 2023 21:55
Alex Boten added 2 commits December 18, 2023 14:04
This update includes:
- schema_url for resource
- without_type_suffix and without_units for prometheus exporter

Signed-off-by: Alex Boten <aboten@lightstep.com>
Signed-off-by: Alex Boten <aboten@lightstep.com>
codeboten pushed a commit to codeboten/opentelemetry-collector that referenced this pull request Dec 18, 2023
This code will live in the otel-go-contrib `config` package instead. This PR depends on open-telemetry/opentelemetry-go-contrib#4617

Signed-off-by: Alex Boten <aboten@lightstep.com>
Alex Boten added 5 commits December 19, 2023 11:42
Signed-off-by: Alex Boten <aboten@lightstep.com>
This pulls in the code that was in the collector to configure the MeterProvider and TracerProvider.

Signed-off-by: Alex Boten <aboten@lightstep.com>
Signed-off-by: Alex Boten <aboten@lightstep.com>
Signed-off-by: Alex Boten <aboten@lightstep.com>
@codeboten codeboten force-pushed the codeboten/implement-tracer-provider-sdk branch from 977e67f to 5db40c0 Compare December 20, 2023 23:01
codeboten pushed a commit to codeboten/opentelemetry-go-contrib that referenced this pull request Dec 21, 2023
This PR is another attempt at filling in the SDK details for NewSDK. It breaks the open-telemetry#4617 per signal to make it more manageable to review.

Part of open-telemetry#4371

Signed-off-by: Alex Boten <aboten@lightstep.com>
@codeboten
Copy link
Contributor Author

Split into two PRs #4741 & #4804

@codeboten codeboten closed this Jan 15, 2024
codeboten pushed a commit to codeboten/opentelemetry-collector that referenced this pull request Jan 16, 2024
This code will live in the otel-go-contrib `config` package instead. This PR depends on open-telemetry/opentelemetry-go-contrib#4617

Signed-off-by: Alex Boten <aboten@lightstep.com>
codeboten pushed a commit to codeboten/opentelemetry-collector that referenced this pull request Jan 17, 2024
This code will live in the otel-go-contrib `config` package instead. This PR depends on open-telemetry/opentelemetry-go-contrib#4617

Signed-off-by: Alex Boten <aboten@lightstep.com>
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.

config: Create SDK from model
1 participant