Skip to content
This repository was archived by the owner on Nov 7, 2022. It is now read-only.

Revert "Upgrade opencensus-proto version to v0.2.0. (#490)"#503

Merged
pjanotti merged 1 commit intocensus-instrumentation:masterfrom
pjanotti:revert-proto-up
Mar 27, 2019
Merged

Revert "Upgrade opencensus-proto version to v0.2.0. (#490)"#503
pjanotti merged 1 commit intocensus-instrumentation:masterfrom
pjanotti:revert-proto-up

Conversation

@pjanotti
Copy link

This reverts commit 392ee4a.

Fixes #502

@pjanotti pjanotti requested a review from a team as a code owner March 26, 2019 22:14
@codecov
Copy link

codecov bot commented Mar 26, 2019

Codecov Report

Merging #503 into master will increase coverage by 0.06%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #503      +/-   ##
=========================================
+ Coverage   58.13%   58.2%   +0.06%     
=========================================
  Files          69      69              
  Lines        4500    4500              
=========================================
+ Hits         2616    2619       +3     
+ Misses       1719    1716       -3     
  Partials      165     165
Impacted Files Coverage Δ
receiver/opencensusreceiver/opencensus.go 64.54% <0%> (+2.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 405c9e3...12568e9. Read the comment docs.

@odeke-em
Copy link
Member

Oh oh, what happened?

@pjanotti
Copy link
Author

@odeke-em I'm not sure but the RPC between agent and collector in the demo/traces is broken with the upgrade, reverting it fixes the issue.

@pjanotti
Copy link
Author

Using https://github.com/grpc/grpc-go/releases/tag/v1.18.1 also fixes demo/trace perhaps will go with that instead of the revert.

@songy23
Copy link
Contributor

songy23 commented Mar 27, 2019

Using https://github.com/grpc/grpc-go/releases/tag/v1.18.1 also fixes demo/trace perhaps will go with that instead of the revert.

Great to know, so the issue may be the RPC client and server generated with protoc v3.6.1 don't work with this gRPC version.

@pjanotti
Copy link
Author

Per conversation on issue #502 going with the revert.

@pjanotti pjanotti merged commit b9f473c into census-instrumentation:master Mar 27, 2019
@pjanotti pjanotti deleted the revert-proto-up branch March 27, 2019 01:18
songy23 added a commit to songy23/opencensus-service that referenced this pull request Mar 28, 2019
songy23 added a commit that referenced this pull request Mar 28, 2019
@flands flands added this to the 0.2.0 milestone Mar 28, 2019
fivesheep pushed a commit to fivesheep/opencensus-service that referenced this pull request Jun 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants