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

Opentelemetry driver missing spans #5573

Open
himanshu2998 opened this issue Feb 15, 2024 · 4 comments
Open

Opentelemetry driver missing spans #5573

himanshu2998 opened this issue Feb 15, 2024 · 4 comments

Comments

@himanshu2998
Copy link

Hi, I am using tracing service in emissary ingress
https://www.getambassador.io/docs/emissary/latest/topics/running/services/tracing-service
below is the config

{{- if .Values.tracing.enabled }}
apiVersion: getambassador.io/v3alpha1
kind:  TracingService
metadata:
  name:  tracing
  namespace: {{ include "ambassador.namespace" . }}
spec:
  service: {{ .Values.tracing.endpoint }}
  {{- if .Values.env }}
  ambassador_id: [{{ .Values.env.AMBASSADOR_ID | quote }}]
  {{- end }}
  driver: opentelemetry
  sampling:
    overall: {{ .Values.tracing.overall_sampling }}
{{- end }}

Although I am getting traces but for some traces some spans are missing due to which I am getting missing root spans.
This can be due to high load, is there a way to configure timeouts and batching config for tracing data in the service for opentelemetry driver?
Screenshot 2024-02-15 at 4 39 54 PM

@cindymullins-dw
Copy link
Contributor

Hi @himanshu2998 , which version are you on? In 3.4 and 3.5 we had a bug fix and a Community Advocate contribution which both impacted telemetry spans so that'd be one thing to check.

@himanshu2998
Copy link
Author

Can we also set propagation mode for opentelemetry driver?

@himanshu2998
Copy link
Author

Hi @himanshu2998 , which version are you on? In 3.4 and 3.5 we had a bug fix and a Community Advocate contribution which both impacted telemetry spans so that'd be one thing to check.

Hi @cindymullins-dw we are using 3.9 version for ingress

@himanshu2998
Copy link
Author

any updates here?

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

No branches or pull requests

2 participants