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

Simple instrumented gRPC example #729

Merged
merged 24 commits into from
Jun 3, 2021

Conversation

Hablapatabla
Copy link
Contributor

Changes

Add a simple instrumented client/server example using gRPC. Instructions on running the example are held in the README. The example requires gRPC and Protobuf as dependencies. This should be no problem but in the project-level CMakeLists, they are only loaded when -DWITH_OTLP=ON, which I will look out for.

@Hablapatabla Hablapatabla requested a review from a team May 8, 2021 02:26
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented May 8, 2021

CLA Signed

The committers are authorized under a signed CLA.

@codecov
Copy link

codecov bot commented May 8, 2021

Codecov Report

Merging #729 (9404cf9) into main (1c70f0e) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #729   +/-   ##
=======================================
  Coverage   96.19%   96.19%           
=======================================
  Files         153      153           
  Lines        6444     6444           
=======================================
  Hits         6198     6198           
  Misses        246      246           

Copy link
Member

@lalitb lalitb left a comment

Choose a reason for hiding this comment

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

Thank you for the PR. This is definitely a good start.

examples/grpc/client.cc Outdated Show resolved Hide resolved
examples/grpc/client.cc Outdated Show resolved Hide resolved
examples/grpc/experimental/greeter_interceptor.cc Outdated Show resolved Hide resolved
examples/grpc/grpc_foo_lib/foo_split.cc Outdated Show resolved Hide resolved
Version: 1.5.1
Homepage: https://github.com/google/benchmark
Description: A library to support the benchmarking of functions, similar to unit-tests.
Source: benchmark
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here. No changes, unrelated to your work, revert this one.

@lalitb
Copy link
Member

lalitb commented Jun 2, 2021

@Hablapatabla - Hope you are fine with me taking over these changes, as we need them urgently now. I will try to see if I have permission to commit to your branch, else will move these changes to my fork and update.

@lalitb
Copy link
Member

lalitb commented Jun 2, 2021

@Hablapatabla - Hope you are fine with me taking over these changes, as we need them urgently now. I will try to see if I have permission to commit to your branch, else will move these changes to my fork and update.

Ok, I have fixed the propagation, need to add cmake option to optionally enable building this example ( as this needs grpc ), and also need to fix README. Will do that and remove WIP tag afterwards.

@lalitb lalitb changed the title Simple instrumented gRPC example [WIP] Simple instrumented gRPC example Jun 2, 2021
@lalitb
Copy link
Member

lalitb commented Jun 2, 2021

@open-telemetry/cpp-approvers - Can you please review this PR, have done the needed changes to make it run end to end with context propagation.

Copy link
Contributor

@jsuereth jsuereth left a comment

Choose a reason for hiding this comment

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

Great work!

This looks great, can you open a bug to add a bazel build for this?

@lalitb lalitb merged commit 0f6199f into open-telemetry:main Jun 3, 2021
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.

4 participants