-
Notifications
You must be signed in to change notification settings - Fork 687
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
40x & 50x Emissary Traces not categorised as ERRORS in Datadog #5803
Comments
emissary-ingress 3.9.1 is on envoy v1.27.2 which have this issue |
I see the last General Release has been Nov 20th 2023. |
There is indeed a plan to make a proper 3.10.0 GA, with a goal of that happening this month. |
Just wanted to check again if there has been any update on the GA release for v3.10.0? Thanks! |
+1 |
I see a new version on Artifact Hub v8.12.2, but I do not see the release anywhere on this github repo. The emissary image is coming from dockerHub. Is this the right version? Why is the code not published here on github? Are we safe to upgrade our Emissary to v8.12.2? |
Describe the bug
We have 2 emissary ingresses running in our EKS cluster.
They are successfully integrated with a Datadog Agent pushing traces to our DataDog Backend.
We are now receiving the traces/spans in DataDog but their statuses are not being classified correctly - everything is classified as "OK".
According to Datadog support, this is an issue with how the Ambassador team sends the attributes with their traces.
This is the response we have gotten from DataDog Support :
Links sent by DataDog:
https://docs.datadoghq.com/tracing/trace_collection/tracing_naming_convention/#span-tags
https://docs.datadoghq.com/tracing/trace_collection/custom_instrumentation/ruby/dd-api/#setting-errors-on-a-span
To Reproduce
Steps to reproduce the behavior:
Expected behavior
We are supposed to see traces with HTTP error code 50x & 40x as errors. Instead they are being ingested as "OK" similar to 200 status codes.
Versions (please complete the following information):
Please help. Thanks!
The text was updated successfully, but these errors were encountered: