Skip to content

add support for OTEL_SERVICE_NAME environment variable - #302

Merged
tsloughter merged 3 commits into
open-telemetry:mainfrom
tsloughter:env-service-name
Oct 30, 2021
Merged

add support for OTEL_SERVICE_NAME environment variable #302
tsloughter merged 3 commits into
open-telemetry:mainfrom
tsloughter:env-service-name

Conversation

@tsloughter

Copy link
Copy Markdown
Member

No description provided.

@codecov

codecov Bot commented Oct 29, 2021

Copy link
Copy Markdown

Codecov Report

Merging #302 (97f8106) into main (2b6a188) will increase coverage by 0.07%.
The diff coverage is 92.85%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #302      +/-   ##
==========================================
+ Coverage   37.27%   37.35%   +0.07%     
==========================================
  Files          46       46              
  Lines        3262     3266       +4     
==========================================
+ Hits         1216     1220       +4     
  Misses       2046     2046              
Flag Coverage Δ
api 65.14% <100.00%> (ø)
elixir 14.41% <0.00%> (ø)
erlang 37.35% <92.85%> (+0.07%) ⬆️
exporter 19.75% <ø> (ø)
sdk 76.74% <92.30%> (+0.16%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
apps/opentelemetry/src/otel_resource_detector.erl 91.89% <92.30%> (+0.46%) ⬆️
...pps/opentelemetry_api/src/otel_tracer_provider.erl 81.25% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2b6a188...97f8106. Read the comment docs.

@tsloughter
tsloughter merged commit aa1ed7b into open-telemetry:main Oct 30, 2021
@tsloughter
tsloughter deleted the env-service-name branch October 30, 2021 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants