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

checkout opentelemetry-proto v0.11.0 #1055

Merged
merged 5 commits into from
Nov 10, 2021

Conversation

esigo
Copy link
Member

@esigo esigo commented Nov 6, 2021

Related to #1051 (issue)

Changes

checkout opentelemetry-proto v0.11.0 and store the third_party release tags in a file
For significant contributions please make sure you have completed the following items:

  • CHANGELOG.md updated for non-trivial changes
  • Unit tests have been added
  • Changes in public API reviewed

@esigo esigo requested a review from a team November 6, 2021 09:05
@codecov
Copy link

codecov bot commented Nov 6, 2021

Codecov Report

Merging #1055 (4ab2b13) into main (994c08a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1055   +/-   ##
=======================================
  Coverage   94.85%   94.85%           
=======================================
  Files         151      151           
  Lines        5971     5971           
=======================================
  Hits         5663     5663           
  Misses        308      308           

nlohmann-json=v3.9.1
opentelemetry-proto=v0.11.0
prometheus-cpp=v0.11.0
vcpkg=2020.04-2702-g5568f110b
Copy link
Member

Choose a reason for hiding this comment

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

Sorry if I am missing something. What is the purpose of adding this file, and also adding an option in do_ci.sh to create this file? Is it meant to be a central place to list versions of all the otel-cpp dependencies?

Copy link
Member Author

@esigo esigo Nov 8, 2021

Choose a reason for hiding this comment

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

I believe it's a good idea to have the versions documented somewhere so the users can download the versions of the dependencies that a release has been published with. This would be obsolete if the plan is to not support older opentelemetry versions though.

Copy link
Member

@lalitb lalitb Nov 8, 2021

Choose a reason for hiding this comment

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

Ok, that makes sense. Just wondering if this should also contain the dependencies not coming as submodules, eg gRPC(v1.39.1), thrift (0.14.1), abseil(20210324.0)?

Copy link
Member Author

Choose a reason for hiding this comment

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

yes absolutely. Just added them to the script.

@ThomsonTan ThomsonTan merged commit 55aa7ea into open-telemetry:main Nov 10, 2021
@esigo esigo deleted the update-opentelemetry-proto branch November 10, 2021 06:28
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.

3 participants