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

Remove extra send/receive annotations with using zipkin v1 #993

Merged
merged 2 commits into from
Jun 2, 2020

Conversation

chris-smith-zocdoc
Copy link
Contributor

Fixes #960

@codecov-commenter
Copy link

codecov-commenter commented May 20, 2020

Codecov Report

Merging #993 into master will decrease coverage by 0.13%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #993      +/-   ##
==========================================
- Coverage   85.68%   85.54%   -0.14%     
==========================================
  Files         184      184              
  Lines       13061    13063       +2     
==========================================
- Hits        11191    11175      -16     
- Misses       1419     1436      +17     
- Partials      451      452       +1     
Impacted Files Coverage Δ
translator/trace/zipkin/zipkinv1_to_protospan.go 82.95% <100.00%> (-8.90%) ⬇️
service/service.go 49.81% <0.00%> (+0.72%) ⬆️

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 f588c89...c2e9f2d. Read the comment docs.

Copy link
Member

@dmitryax dmitryax left a comment

Choose a reason for hiding this comment

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

Looks good to me

@dmitryax
Copy link
Member

@chris-smith-zocdoc please sign the Contributor License Agreement

@chris-smith-zocdoc
Copy link
Contributor Author

I signed it

@chris-smith-zocdoc
Copy link
Contributor Author

Was there anything else blocking this?

@bogdandrutu bogdandrutu merged commit 665d64e into open-telemetry:master Jun 2, 2020
@flands flands added this to the Beta 0.4 milestone Jun 4, 2020
wyTrivail pushed a commit to mxiamxia/opentelemetry-collector that referenced this pull request Jul 13, 2020
…metry#993)

* Remove extra send/receive annotations with using zipkin v1
Fixes open-telemetry#960

* Update test to use assert.EqualValues
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this pull request Nov 11, 2021
* Update release docs

Clean up language and include directions to handle the Changelog during
a release.

* Fix grammar
hughesjj pushed a commit to hughesjj/opentelemetry-collector that referenced this pull request Apr 27, 2023
* update-deps: support 1.16 go mod tidy

* Update Collector deps to v0.40.0
swiatekm pushed a commit to swiatekm/opentelemetry-collector that referenced this pull request Oct 9, 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.

Zipkin V1 Receiver + V2 Exporter creates empty annotations
5 participants