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

Always add tags for observability, other metrics may use them #1312

Merged
merged 1 commit into from
Jul 10, 2020

Conversation

bogdandrutu
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Jul 9, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1312      +/-   ##
==========================================
- Coverage   89.34%   89.33%   -0.01%     
==========================================
  Files         212      212              
  Lines       15144    15138       -6     
==========================================
- Hits        13530    13524       -6     
  Misses       1178     1178              
  Partials      436      436              
Impacted Files Coverage Δ
obsreport/obsreport_exporter.go 97.91% <100.00%> (-0.03%) ⬇️
obsreport/obsreport_processor.go 97.82% <100.00%> (-0.05%) ⬇️
obsreport/obsreport_receiver.go 100.00% <100.00%> (ø)
translator/internaldata/resource_to_oc.go 86.04% <0.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 4272781...54a3444. Read the comment docs.

Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
Copy link
Contributor

@pjanotti pjanotti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pjanotti pjanotti merged commit 474407c into open-telemetry:master Jul 10, 2020
@bogdandrutu bogdandrutu deleted the newtagsalways branch July 10, 2020 00:24
wyTrivail pushed a commit to mxiamxia/opentelemetry-collector that referenced this pull request Jul 13, 2020
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this pull request Nov 11, 2021
Remove cruft about spanStore.

Update documentation about how span.mu is used (related to open-telemetry#1311).

Reformat to conform with idiomatic field documentation.
Troels51 pushed a commit to Troels51/opentelemetry-collector that referenced this pull request Jul 5, 2024
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.

2 participants