Skip to content

tracing: Update OpenCensus.#7797

Merged
htuch merged 3 commits intoenvoyproxy:masterfrom
g-easy:sdaddr
Aug 5, 2019
Merged

tracing: Update OpenCensus.#7797
htuch merged 3 commits intoenvoyproxy:masterfrom
g-easy:sdaddr

Conversation

@g-easy
Copy link
Contributor

@g-easy g-easy commented Aug 1, 2019

Let the config override the Stackdriver address. This can be used for
proxying and testing.

Signed-off-by: Emil Mikulic g-easy@users.noreply.github.com

Let the config override the Stackdriver address. This can be used for
proxying and testing.

Signed-off-by: Emil Mikulic <g-easy@users.noreply.github.com>
@kyessenov
Copy link
Contributor

kyessenov commented Aug 1, 2019

cc @objectiser @moderation this moves opencensus to master tip (which was requested in a previous PR).

These come from tracing.grpc.pb.h which is autogenerated by the protobuf
compiler.

Signed-off-by: Emil Mikulic <g-easy@users.noreply.github.com>
@repokitteh-read-only
Copy link

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to api/.

🐱

Caused by: #7797 was synchronize by g-easy.

see: more, trace.

Copy link
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

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

Thanks, one major comment on Stackdriver integration...
/wait

@htuch htuch self-assigned this Aug 1, 2019
@htuch
Copy link
Member

htuch commented Aug 2, 2019

@g-easy can you merge master, LGTM.

Signed-off-by: Emil Mikulic <g-easy@users.noreply.github.com>
@g-easy
Copy link
Contributor Author

g-easy commented Aug 5, 2019

Thanks!

@htuch htuch merged commit 3e63182 into envoyproxy:master Aug 5, 2019
@g-easy g-easy deleted the sdaddr branch August 6, 2019 01:35
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