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

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

Merged
songy23 merged 1 commit intocensus-instrumentation:masterfrom
songy23:roll-forward-proto
Mar 28, 2019
Merged

Revert "Revert "Upgrade opencensus-proto version to v0.2.0. (#490)" (…#506
songy23 merged 1 commit intocensus-instrumentation:masterfrom
songy23:roll-forward-proto

Conversation

@songy23
Copy link
Contributor

@songy23 songy23 commented Mar 28, 2019

#503)"

This reverts commit b9f473c.

@songy23 songy23 requested review from bogdandrutu and pjanotti March 28, 2019 00:27
@songy23 songy23 requested a review from a team as a code owner March 28, 2019 00:27
@songy23
Copy link
Contributor Author

songy23 commented Mar 28, 2019

Roll forward #490.

@bogdandrutu helped debug the previous issue. The bug that ocagent exporter cannot talk to agent/collector is not because of proto changes. It's because of a gRPC change, which is fixed by #501.

@codecov
Copy link

codecov bot commented Mar 28, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #506      +/-   ##
==========================================
- Coverage   58.81%   58.75%   -0.07%     
==========================================
  Files          69       69              
  Lines        4507     4507              
==========================================
- Hits         2651     2648       -3     
- Misses       1692     1695       +3     
  Partials      164      164
Impacted Files Coverage Δ
receiver/opencensusreceiver/opencensus.go 62.5% <0%> (-2.68%) ⬇️

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 f855869...c32aa0c. Read the comment docs.

@songy23 songy23 merged commit 4036100 into census-instrumentation:master Mar 28, 2019
@songy23 songy23 deleted the roll-forward-proto branch March 28, 2019 00:53
@flands flands added this to the 0.2.0 milestone Mar 28, 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.

5 participants