-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[APM][Otel]Fix get_error_group_main_statistics making trace.id optional field #196822
[APM][Otel]Fix get_error_group_main_statistics making trace.id optional field #196822
Conversation
Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services) |
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 💯 Thank you for the fix 🚀
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]
History
|
Starting backport for target branches: 8.15, 8.16, 8.x https://github.com/elastic/kibana/actions/runs/11412316379 |
…al field (elastic#196822) fixes [196821](elastic#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)
…al field (elastic#196822) fixes [196821](elastic#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)
💔 Some backports could not be created
Note: Successful backport PRs will be merged automatically after passing CI. Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
… optional field (#196822) (#196966) # Backport This will backport the following commits from `main` to `8.16`: - [[APM][Otel]Fix get_error_group_main_statistics making trace.id optional field (#196822)](#196822) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Carlos Crespo","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-18T23:16:36Z","message":"[APM][Otel]Fix get_error_group_main_statistics making trace.id optional field (#196822)\n\nfixes [196821](https://github.com/elastic/kibana/issues/196821)\r\n\r\n## Summary\r\n\r\n`trace.id` should've been sent as an optional field to the\r\n`get_error_group_main_statistics` query. This PR fixes that\r\n\r\n<img width=\"700\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/20b63361-9645-418f-a611-2c0e29b385c7\">\r\n\r\n\r\n## How to test \r\n\r\n- Run the opentelemetry demo:\r\nhttps://github.com/elastic/opentelemetry-demo/tree/rca_ingress_obs\r\n- Navigate to Application > Services\r\n- Open the service details for loadgenerator","sha":"a36fa1151b4af7ecd1565bf2b67fad20835b8209","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-major","ci:project-deploy-observability","Team:obs-ux-infra_services"],"title":"[APM][Otel]Fix get_error_group_main_statistics making trace.id optional field","number":196822,"url":"https://github.com/elastic/kibana/pull/196822","mergeCommit":{"message":"[APM][Otel]Fix get_error_group_main_statistics making trace.id optional field (#196822)\n\nfixes [196821](https://github.com/elastic/kibana/issues/196821)\r\n\r\n## Summary\r\n\r\n`trace.id` should've been sent as an optional field to the\r\n`get_error_group_main_statistics` query. This PR fixes that\r\n\r\n<img width=\"700\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/20b63361-9645-418f-a611-2c0e29b385c7\">\r\n\r\n\r\n## How to test \r\n\r\n- Run the opentelemetry demo:\r\nhttps://github.com/elastic/opentelemetry-demo/tree/rca_ingress_obs\r\n- Navigate to Application > Services\r\n- Open the service details for loadgenerator","sha":"a36fa1151b4af7ecd1565bf2b67fad20835b8209"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196822","number":196822,"mergeCommit":{"message":"[APM][Otel]Fix get_error_group_main_statistics making trace.id optional field (#196822)\n\nfixes [196821](https://github.com/elastic/kibana/issues/196821)\r\n\r\n## Summary\r\n\r\n`trace.id` should've been sent as an optional field to the\r\n`get_error_group_main_statistics` query. This PR fixes that\r\n\r\n<img width=\"700\" alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/20b63361-9645-418f-a611-2c0e29b385c7\">\r\n\r\n\r\n## How to test \r\n\r\n- Run the opentelemetry demo:\r\nhttps://github.com/elastic/opentelemetry-demo/tree/rca_ingress_obs\r\n- Navigate to Application > Services\r\n- Open the service details for loadgenerator","sha":"a36fa1151b4af7ecd1565bf2b67fad20835b8209"}}]}] BACKPORT--> Co-authored-by: Carlos Crespo <[email protected]>
fixes 196821
Summary
trace.id
should've been sent as an optional field to theget_error_group_main_statistics
query. This PR fixes thatHow to test