You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem?
GCP as platform use Cloud Trace. Currently most of service are able to consume correctly WC3 propagation headers. Notable exception of this is Load Balancer that supports only old header X-Cloud-Trace-Context and looks like it would not change in very fast according to reported issue
This comes out when we enable opentelemetry support for ingress-nginx and integration with LB request logs breaks.
Ingress open-telemetry support only WC3 and B3 headers as they are implemented here and because of it breaks propagation of trace over this request. We would like to add support of Cloud Trace legacy headers to ingress nginx specially for request that are made via GCLB and it would be nice that Importer would comes from same way as B3 and WC3 headers.
Describe the solution you'd like
Implementation of Importer/Exporter in API of legacy Cloud Trace propagation headers. Of course named as deprecated but have global ability to "consume" those traces in uniform way.
Describe alternatives you've considered
If not in main API to not propagate legacy solution in new SDK put that implementation to 3rd party repository.
Is your feature request related to a problem?
GCP as platform use Cloud Trace. Currently most of service are able to consume correctly
WC3
propagation headers. Notable exception of this is Load Balancer that supports only old headerX-Cloud-Trace-Context
and looks like it would not change in very fast according to reported issueThis comes out when we enable opentelemetry support for ingress-nginx and integration with LB request logs breaks.
Ingress open-telemetry support only
WC3
andB3
headers as they are implemented here and because of it breaks propagation of trace over this request. We would like to add support of Cloud Trace legacy headers to ingress nginx specially for request that are made via GCLB and it would be nice thatImporter
would comes from same way asB3
andWC3
headers.Describe the solution you'd like
Implementation of Importer/Exporter in API of legacy Cloud Trace propagation headers. Of course named as deprecated but have global ability to "consume" those traces in uniform way.
Describe alternatives you've considered
If not in main API to not propagate legacy solution in new SDK put that implementation to 3rd party repository.
Additional context
Specification of
X-Cloud-Trace-Context
header: https://cloud.google.com/trace/docs/trace-context#legacy-http-headerI can try prepare pull request for those changes but asking first is usual preferred
The text was updated successfully, but these errors were encountered: