zipkin tracing: support to make the envoy as the independent hop for tracing (#22987)

* zipkin tracing: support to make the envoy as the independent hop for tracing

Signed-off-by: wbpcode <wangbaiping@corp.netease.com>

* fix test

Signed-off-by: wbpcode <wangbaiping@corp.netease.com>

* simply impl

Signed-off-by: wbpcode <wangbaiping@corp.netease.com>

* Update source/extensions/tracers/zipkin/tracer.cc

Co-authored-by: Adi (Suissa) Peleg <adip@google.com>

* Update source/extensions/tracers/zipkin/tracer.cc

Co-authored-by: Adi (Suissa) Peleg <adip@google.com>

* minor update of comments

Signed-off-by: wbpcode <wangbaiping@corp.netease.com>

* Update api/envoy/config/trace/v3/zipkin.proto

Co-authored-by: Adi (Suissa) Peleg <adip@google.com>
Signed-off-by: code <wangbaiping@corp.netease.com>

* Update api/envoy/config/trace/v3/zipkin.proto

Co-authored-by: Adi (Suissa) Peleg <adip@google.com>
Signed-off-by: code <wangbaiping@corp.netease.com>

* Update changelogs/current.yaml

Co-authored-by: Adi (Suissa) Peleg <adip@google.com>
Signed-off-by: code <wangbaiping@corp.netease.com>

* add attention

Signed-off-by: wbpcode <wangbaiping@corp.netease.com>

* rename the api and add more comment

Signed-off-by: wbpcode <wangbaiping@corp.netease.com>

* fix format

Signed-off-by: wbpcode <wangbaiping@corp.netease.com>

* fix format

Signed-off-by: wbpcode <wangbaiping@corp.netease.com>

* Update api/envoy/config/trace/v3/zipkin.proto

Co-authored-by: Adi (Suissa) Peleg <adip@google.com>
Signed-off-by: code <wangbaiping@corp.netease.com>

* address comment

Signed-off-by: wbpcode <wangbaiping@corp.netease.com>

* Update source/extensions/tracers/zipkin/tracer.cc

Co-authored-by: Adi (Suissa) Peleg <adip@google.com>
Signed-off-by: code <wangbaiping@corp.netease.com>

* fix format

Signed-off-by: wbpcode <wangbaiping@corp.netease.com>

Signed-off-by: wbpcode <wangbaiping@corp.netease.com>
Signed-off-by: code <wangbaiping@corp.netease.com>
Co-authored-by: Adi (Suissa) Peleg <adip@google.com>

Mirrored from https://github.com/envoyproxy/envoy @ ad800292a630c273fb548ac6d320522ffdbd2a8a
pull/626/head
data-plane-api(Azure Pipelines) 2 years ago
parent 79d17fab98
commit a8a27127d5
  1. 17
      envoy/config/trace/v3/zipkin.proto

@ -21,7 +21,7 @@ option (udpa.annotations.file_status).package_version_status = ACTIVE;
// Configuration for the Zipkin tracer.
// [#extension: envoy.tracers.zipkin]
// [#next-free-field: 7]
// [#next-free-field: 8]
message ZipkinConfig {
option (udpa.annotations.versioning).previous_message_type = "envoy.config.trace.v2.ZipkinConfig";
@ -68,4 +68,19 @@ message ZipkinConfig {
// Optional hostname to use when sending spans to the collector_cluster. Useful for collectors
// that require a specific hostname. Defaults to :ref:`collector_cluster <envoy_v3_api_field_config.trace.v3.ZipkinConfig.collector_cluster>` above.
string collector_hostname = 6;
// If this is set to true, then Envoy will be treated as an independent hop in trace chain. A complete span pair will be created for a single
// request. Server span will be created for the downstream request and client span will be created for the related upstream request.
// This should be set to true in the following cases:
//
// * The Envoy Proxy is used as gateway or ingress.
// * The Envoy Proxy is used as sidecar but inbound traffic capturing or outbound traffic capturing is disabled.
// * Any case that the `start_child_span of router <envoy_v3_api_field_extensions.filters.http.router.v3.Router.start_child_span>` is set to true.
//
// .. attention::
//
// If this is set to true, then the
// :ref:`start_child_span of router <envoy_v3_api_field_extensions.filters.http.router.v3.Router.start_child_span>`
// SHOULD be set to true also to ensure the correctness of trace chain.
bool split_spans_for_request = 7;
}

Loading…
Cancel
Save