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

translator/conventions: add deployment.environment #1691

Closed

Conversation

axw
Copy link
Contributor

@axw axw commented Aug 31, 2020

Add a constant for the recently introduced deployment environment resource attribute: open-telemetry/opentelemetry-specification#606

Add a constant for the recently introduced deployment
environment resource attribute:

open-telemetry/opentelemetry-specification#606
@codecov
Copy link

codecov bot commented Aug 31, 2020

Codecov Report

Merging #1691 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1691      +/-   ##
==========================================
- Coverage   92.33%   92.32%   -0.01%     
==========================================
  Files         265      265              
  Lines       19811    19812       +1     
==========================================
- Hits        18292    18291       -1     
- Misses       1090     1091       +1     
- Partials      429      430       +1     
Impacted Files Coverage Δ
translator/conventions/opentelemetry.go 100.00% <100.00%> (ø)
translator/internaldata/resource_to_oc.go 87.03% <0.00%> (-1.86%) ⬇️

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 c3f7dd5...8421902. Read the comment docs.

@bogdandrutu
Copy link
Member

Please wait for open-telemetry/opentelemetry-specification#851 to finish

@tigrannajaryan
Copy link
Member

open-telemetry/opentelemetry-specification#851 is now merged.

@tigrannajaryan tigrannajaryan self-assigned this Sep 9, 2020
@jrcamp
Copy link
Contributor

jrcamp commented Sep 9, 2020

Didn't see this, I added it in a later PR #1722

@jrcamp jrcamp closed this Sep 9, 2020
@axw axw deleted the conventions-deployment-environment branch September 10, 2020 01:11
@axw
Copy link
Contributor Author

axw commented Sep 10, 2020

No worries, thanks!

hughesjj pushed a commit to hughesjj/opentelemetry-collector that referenced this pull request Apr 27, 2023
…metry#1691)

Bumps [github.com/jaegertracing/jaeger](https://github.com/jaegertracing/jaeger) from 1.35.1 to 1.35.2.
- [Release notes](https://github.com/jaegertracing/jaeger/releases)
- [Changelog](https://github.com/jaegertracing/jaeger/blob/main/CHANGELOG.md)
- [Commits](jaegertracing/jaeger@v1.35.1...v1.35.2)

---
updated-dependencies:
- dependency-name: github.com/jaegertracing/jaeger
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.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.

4 participants