Skip to content

Commit

Permalink
[APM][Otel]Fix get_error_group_main_statistics making trace.id option…
Browse files Browse the repository at this point in the history
…al field (#196822)

fixes [196821](#196821)

## Summary

`trace.id` should've been sent as an optional field to the
`get_error_group_main_statistics` query. This PR fixes that

<img width="700" alt="image"
src="https://github.com/user-attachments/assets/20b63361-9645-418f-a611-2c0e29b385c7">

## How to test

- Run the opentelemetry demo:
https://github.com/elastic/opentelemetry-demo/tree/rca_ingress_obs
- Navigate to Application > Services
- Open the service details for loadgenerator

(cherry picked from commit a36fa11)
  • Loading branch information
crespocarlos committed Oct 18, 2024
1 parent 77aa57d commit 7ecb58a
Showing 1 changed file with 2 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -97,14 +97,10 @@ export async function getErrorGroupMainStatistics({
]
: [];

const requiredFields = asMutableArray([
TRACE_ID,
AT_TIMESTAMP,
ERROR_GROUP_ID,
ERROR_ID,
] as const);
const requiredFields = asMutableArray([AT_TIMESTAMP, ERROR_GROUP_ID, ERROR_ID] as const);

const optionalFields = asMutableArray([
TRACE_ID,
ERROR_CULPRIT,
ERROR_LOG_MESSAGE,
ERROR_EXC_MESSAGE,
Expand Down

0 comments on commit 7ecb58a

Please sign in to comment.