Skip to content

[APM] Remove error.id in getErrorGroupMainStatistics query as it's not used#210613

Merged
rmyz merged 1 commit intoelastic:mainfrom
rmyz:210610-apm-fix-error_id-required
Feb 11, 2025
Merged

[APM] Remove error.id in getErrorGroupMainStatistics query as it's not used#210613
rmyz merged 1 commit intoelastic:mainfrom
rmyz:210610-apm-fix-error_id-required

Conversation

@rmyz
Copy link
Copy Markdown
Contributor

@rmyz rmyz commented Feb 11, 2025

Summary

Closes #210610

This PR removes error.id field from being queried at getErrorGroupMainStatistics, as it was not being used, as it was required. If we didn't have this field, the endpoint call would crash.

@rmyz rmyz added release_note:fix v9.0.0 apm Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. backport:version Backport to applied version labels backport:8.17 labels Feb 11, 2025
@rmyz rmyz self-assigned this Feb 11, 2025
@rmyz rmyz marked this pull request as ready for review February 11, 2025 15:43
@rmyz rmyz requested a review from a team February 11, 2025 15:43
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@jennypavlova jennypavlova self-requested a review February 11, 2025 16:29
Copy link
Copy Markdown
Member

@jennypavlova jennypavlova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for fixing that!

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @rmyz

@rmyz rmyz merged commit 46cd29e into elastic:main Feb 11, 2025
16 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 9.0

https://github.com/elastic/kibana/actions/runs/13269645999

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Feb 11, 2025
…s not used (elastic#210613)

## Summary

Closes elastic#210610

This PR removes `error.id` field from being queried at
`getErrorGroupMainStatistics`, as it was not being used, as it was
required. If we didn't have this field, the endpoint call would crash.

(cherry picked from commit 46cd29e)
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
9.0

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Feb 11, 2025
…tistics` query as it's not used (#210613) (#210677)

# Backport

This will backport the following commits from `main` to `9.0`:
- [[APM] Remove `error.id` in
`getErrorGroupMainStatistics` query as it's not used
(#210613)](#210613)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Sergi
Romeu","email":"sergi.romeu@elastic.co"},"sourceCommit":{"committedDate":"2025-02-11T18:13:51Z","message":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used (#210613)\n\n## Summary\n\nCloses #210610\n\nThis PR removes
`error.id` field from being queried at\n`getErrorGroupMainStatistics`,
as it was not being used, as it was\nrequired. If we didn't have this
field, the endpoint call would
crash.","sha":"46cd29e60be24fa3d39b2b364973e0e9b10659b7","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","v9.0.0","apm","Team:obs-ux-infra_services","backport:version","backport:8.17","v9.1.0","backport:8.18"],"title":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used","number":210613,"url":"https://github.com/elastic/kibana/pull/210613","mergeCommit":{"message":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used (#210613)\n\n## Summary\n\nCloses #210610\n\nThis PR removes
`error.id` field from being queried at\n`getErrorGroupMainStatistics`,
as it was not being used, as it was\nrequired. If we didn't have this
field, the endpoint call would
crash.","sha":"46cd29e60be24fa3d39b2b364973e0e9b10659b7"}},"sourceBranch":"main","suggestedTargetBranches":["9.0"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210613","number":210613,"mergeCommit":{"message":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used (#210613)\n\n## Summary\n\nCloses #210610\n\nThis PR removes
`error.id` field from being queried at\n`getErrorGroupMainStatistics`,
as it was not being used, as it was\nrequired. If we didn't have this
field, the endpoint call would
crash.","sha":"46cd29e60be24fa3d39b2b364973e0e9b10659b7"}}]}]
BACKPORT-->

Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
kapral18 added a commit to agusruidiazgd/kibana that referenced this pull request Feb 11, 2025
…on-206439

* main: (402 commits)
  [Search]: Fix Number type field to have correct property (elastic#210462)
  Change filter for rule monitoring gaps (elastic#209983)
  Update Logs Explorer deprecation messages (elastic#201307)
  [APM] Remove `error.id` in `getErrorGroupMainStatistics` query as it's not used (elastic#210613)
  [Embeddable] Fix presentation panel styles (elastic#210113)
  [ci] enable Scout reporter for on-merge-unsupported-ftrs (elastic#210627)
  [Fix][Synonyms UI]Add navigation link to the Detail breadcrumb. (elastic#209574)
  chore(dep): bump `store2` from `2.12.0` to `2.14.4` (elastic#210530)
  [scout] adding test helper `@kbn/scout-oblt` package and uptate onboarding tests (elastic#209761)
  [Cloud Security] Asset Inventory table flyout controls  (elastic#208452)
  [ML] Fix model deployment check in file uploader (elastic#209585)
  Updates archive again (elastic#209828)
  [Security Solution] Added concurrency limits and request throttling to prebuilt rule routes (elastic#209551)
  [Search] [Onboarding] Update search api to use EventEmitter instead of Provider (elastic#209784)
  [maps] lazy load map actions (elastic#210252)
  [Cloud Security] Adding telemetry collection condition based on render condition (elastic#208758)
  [Solution nav] Use flyout for Stack Management in Search and Observability solutions (elastic#208632)
  [Search] Fix Add Inference Endpoint API call (elastic#210243)
  [Agentless Connectors] Integration overview panel (elastic#210222)
  [Lens] Restore dynamic colouring by value for Last value agg (elastic#209110)
  ...
@rmyz
Copy link
Copy Markdown
Contributor Author

rmyz commented Feb 12, 2025

💚 All backports created successfully

Status Branch Result
8.x
8.18

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

rmyz added a commit to rmyz/kibana that referenced this pull request Feb 12, 2025
…s not used (elastic#210613)

## Summary

Closes elastic#210610

This PR removes `error.id` field from being queried at
`getErrorGroupMainStatistics`, as it was not being used, as it was
required. If we didn't have this field, the endpoint call would crash.

(cherry picked from commit 46cd29e)
rmyz added a commit to rmyz/kibana that referenced this pull request Feb 12, 2025
…s not used (elastic#210613)

## Summary

Closes elastic#210610

This PR removes `error.id` field from being queried at
`getErrorGroupMainStatistics`, as it was not being used, as it was
required. If we didn't have this field, the endpoint call would crash.

(cherry picked from commit 46cd29e)
@rmyz
Copy link
Copy Markdown
Contributor Author

rmyz commented Feb 12, 2025

💚 All backports created successfully

Status Branch Result
8.17

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

rmyz added a commit to rmyz/kibana that referenced this pull request Feb 12, 2025
…s not used (elastic#210613)

## Summary

Closes elastic#210610

This PR removes `error.id` field from being queried at
`getErrorGroupMainStatistics`, as it was not being used, as it was
required. If we didn't have this field, the endpoint call would crash.

(cherry picked from commit 46cd29e)
@rmyz rmyz deleted the 210610-apm-fix-error_id-required branch February 12, 2025 09:45
rmyz added a commit that referenced this pull request Feb 12, 2025
…as it's not used (#210613) (#210761)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[APM] Remove &#x60;error.id&#x60; in
&#x60;getErrorGroupMainStatistics&#x60; query as it&#x27;s not used
(#210613)](#210613)

<!--- Backport version: 9.6.4 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Sergi
Romeu","email":"sergi.romeu@elastic.co"},"sourceCommit":{"committedDate":"2025-02-11T18:13:51Z","message":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used (#210613)\n\n## Summary\n\nCloses #210610\n\nThis PR removes
`error.id` field from being queried at\n`getErrorGroupMainStatistics`,
as it was not being used, as it was\nrequired. If we didn't have this
field, the endpoint call would
crash.","sha":"46cd29e60be24fa3d39b2b364973e0e9b10659b7","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","v9.0.0","apm","Team:obs-ux-infra_services","backport:version","backport:8.17","v9.1.0","backport:8.18"],"title":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used","number":210613,"url":"https://github.com/elastic/kibana/pull/210613","mergeCommit":{"message":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used (#210613)\n\n## Summary\n\nCloses #210610\n\nThis PR removes
`error.id` field from being queried at\n`getErrorGroupMainStatistics`,
as it was not being used, as it was\nrequired. If we didn't have this
field, the endpoint call would
crash.","sha":"46cd29e60be24fa3d39b2b364973e0e9b10659b7"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/210677","number":210677,"state":"MERGED","mergeCommit":{"sha":"dbe7cc3b4f4b21484f5bb4b26e47fbe541b83fab","message":"[9.0]
[APM] Remove &#x60;error.id&#x60; in
&#x60;getErrorGroupMainStatistics&#x60; query as it&#x27;s not used
(#210613) (#210677)\n\n# Backport\n\nThis will backport the following
commits from `main` to `9.0`:\n- [[APM] Remove &#x60;error.id&#x60;
in\n&#x60;getErrorGroupMainStatistics&#x60; query as it&#x27;s not
used\n(#210613)](https://github.com/elastic/kibana/pull/210613)\n\n<!---
Backport version: 9.4.3 -->\n\n### Questions ?\nPlease refer to the
[Backport
tool\ndocumentation](https://github.com/sqren/backport)\n\n<!--BACKPORT
[{\"author\":{\"name\":\"Sergi\nRomeu\",\"email\":\"sergi.romeu@elastic.co\"},\"sourceCommit\":{\"committedDate\":\"2025-02-11T18:13:51Z\",\"message\":\"[APM]\nRemove
`error.id` in `getErrorGroupMainStatistics` query as it's not\nused
(#210613)\\n\\n## Summary\\n\\nCloses #210610\\n\\nThis PR
removes\n`error.id` field from being queried
at\\n`getErrorGroupMainStatistics`,\nas it was not being used, as it
was\\nrequired. If we didn't have this\nfield, the endpoint call
would\ncrash.\",\"sha\":\"46cd29e60be24fa3d39b2b364973e0e9b10659b7\",\"branchLabelMapping\":{\"^v9.1.0$\":\"main\",\"^v8.19.0$\":\"8.x\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"release_note:fix\",\"v9.0.0\",\"apm\",\"Team:obs-ux-infra_services\",\"backport:version\",\"backport:8.17\",\"v9.1.0\",\"backport:8.18\"],\"title\":\"[APM]\nRemove
`error.id` in `getErrorGroupMainStatistics` query as it's
not\nused\",\"number\":210613,\"url\":\"https://github.com/elastic/kibana/pull/210613\",\"mergeCommit\":{\"message\":\"[APM]\nRemove
`error.id` in `getErrorGroupMainStatistics` query as it's not\nused
(#210613)\\n\\n## Summary\\n\\nCloses #210610\\n\\nThis PR
removes\n`error.id` field from being queried
at\\n`getErrorGroupMainStatistics`,\nas it was not being used, as it
was\\nrequired. If we didn't have this\nfield, the endpoint call
would\ncrash.\",\"sha\":\"46cd29e60be24fa3d39b2b364973e0e9b10659b7\"}},\"sourceBranch\":\"main\",\"suggestedTargetBranches\":[\"9.0\"],\"targetPullRequestStates\":[{\"branch\":\"9.0\",\"label\":\"v9.0.0\",\"branchLabelMappingKey\":\"^v(\\\\d+).(\\\\d+).\\\\d+$\",\"isSourceBranch\":false,\"state\":\"NOT_CREATED\"},{\"branch\":\"main\",\"label\":\"v9.1.0\",\"branchLabelMappingKey\":\"^v9.1.0$\",\"isSourceBranch\":true,\"state\":\"MERGED\",\"url\":\"https://github.com/elastic/kibana/pull/210613\",\"number\":210613,\"mergeCommit\":{\"message\":\"[APM]\nRemove
`error.id` in `getErrorGroupMainStatistics` query as it's not\nused
(#210613)\\n\\n## Summary\\n\\nCloses #210610\\n\\nThis PR
removes\n`error.id` field from being queried
at\\n`getErrorGroupMainStatistics`,\nas it was not being used, as it
was\\nrequired. If we didn't have this\nfield, the endpoint call
would\ncrash.\",\"sha\":\"46cd29e60be24fa3d39b2b364973e0e9b10659b7\"}}]}]\nBACKPORT-->\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>"}},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210613","number":210613,"mergeCommit":{"message":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used (#210613)\n\n## Summary\n\nCloses #210610\n\nThis PR removes
`error.id` field from being queried at\n`getErrorGroupMainStatistics`,
as it was not being used, as it was\nrequired. If we didn't have this
field, the endpoint call would
crash.","sha":"46cd29e60be24fa3d39b2b364973e0e9b10659b7"}}]}]
BACKPORT-->
rmyz added a commit that referenced this pull request Feb 12, 2025
… as it's not used (#210613) (#210762)

# Backport

This will backport the following commits from `main` to `8.18`:
- [[APM] Remove &#x60;error.id&#x60; in
&#x60;getErrorGroupMainStatistics&#x60; query as it&#x27;s not used
(#210613)](#210613)

<!--- Backport version: 9.6.4 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Sergi
Romeu","email":"sergi.romeu@elastic.co"},"sourceCommit":{"committedDate":"2025-02-11T18:13:51Z","message":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used (#210613)\n\n## Summary\n\nCloses #210610\n\nThis PR removes
`error.id` field from being queried at\n`getErrorGroupMainStatistics`,
as it was not being used, as it was\nrequired. If we didn't have this
field, the endpoint call would
crash.","sha":"46cd29e60be24fa3d39b2b364973e0e9b10659b7","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","v9.0.0","apm","Team:obs-ux-infra_services","backport:version","backport:8.17","v9.1.0","backport:8.18"],"title":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used","number":210613,"url":"https://github.com/elastic/kibana/pull/210613","mergeCommit":{"message":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used (#210613)\n\n## Summary\n\nCloses #210610\n\nThis PR removes
`error.id` field from being queried at\n`getErrorGroupMainStatistics`,
as it was not being used, as it was\nrequired. If we didn't have this
field, the endpoint call would
crash.","sha":"46cd29e60be24fa3d39b2b364973e0e9b10659b7"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/210677","number":210677,"state":"MERGED","mergeCommit":{"sha":"dbe7cc3b4f4b21484f5bb4b26e47fbe541b83fab","message":"[9.0]
[APM] Remove &#x60;error.id&#x60; in
&#x60;getErrorGroupMainStatistics&#x60; query as it&#x27;s not used
(#210613) (#210677)\n\n# Backport\n\nThis will backport the following
commits from `main` to `9.0`:\n- [[APM] Remove &#x60;error.id&#x60;
in\n&#x60;getErrorGroupMainStatistics&#x60; query as it&#x27;s not
used\n(#210613)](https://github.com/elastic/kibana/pull/210613)\n\n<!---
Backport version: 9.4.3 -->\n\n### Questions ?\nPlease refer to the
[Backport
tool\ndocumentation](https://github.com/sqren/backport)\n\n<!--BACKPORT
[{\"author\":{\"name\":\"Sergi\nRomeu\",\"email\":\"sergi.romeu@elastic.co\"},\"sourceCommit\":{\"committedDate\":\"2025-02-11T18:13:51Z\",\"message\":\"[APM]\nRemove
`error.id` in `getErrorGroupMainStatistics` query as it's not\nused
(#210613)\\n\\n## Summary\\n\\nCloses #210610\\n\\nThis PR
removes\n`error.id` field from being queried
at\\n`getErrorGroupMainStatistics`,\nas it was not being used, as it
was\\nrequired. If we didn't have this\nfield, the endpoint call
would\ncrash.\",\"sha\":\"46cd29e60be24fa3d39b2b364973e0e9b10659b7\",\"branchLabelMapping\":{\"^v9.1.0$\":\"main\",\"^v8.19.0$\":\"8.x\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"release_note:fix\",\"v9.0.0\",\"apm\",\"Team:obs-ux-infra_services\",\"backport:version\",\"backport:8.17\",\"v9.1.0\",\"backport:8.18\"],\"title\":\"[APM]\nRemove
`error.id` in `getErrorGroupMainStatistics` query as it's
not\nused\",\"number\":210613,\"url\":\"https://github.com/elastic/kibana/pull/210613\",\"mergeCommit\":{\"message\":\"[APM]\nRemove
`error.id` in `getErrorGroupMainStatistics` query as it's not\nused
(#210613)\\n\\n## Summary\\n\\nCloses #210610\\n\\nThis PR
removes\n`error.id` field from being queried
at\\n`getErrorGroupMainStatistics`,\nas it was not being used, as it
was\\nrequired. If we didn't have this\nfield, the endpoint call
would\ncrash.\",\"sha\":\"46cd29e60be24fa3d39b2b364973e0e9b10659b7\"}},\"sourceBranch\":\"main\",\"suggestedTargetBranches\":[\"9.0\"],\"targetPullRequestStates\":[{\"branch\":\"9.0\",\"label\":\"v9.0.0\",\"branchLabelMappingKey\":\"^v(\\\\d+).(\\\\d+).\\\\d+$\",\"isSourceBranch\":false,\"state\":\"NOT_CREATED\"},{\"branch\":\"main\",\"label\":\"v9.1.0\",\"branchLabelMappingKey\":\"^v9.1.0$\",\"isSourceBranch\":true,\"state\":\"MERGED\",\"url\":\"https://github.com/elastic/kibana/pull/210613\",\"number\":210613,\"mergeCommit\":{\"message\":\"[APM]\nRemove
`error.id` in `getErrorGroupMainStatistics` query as it's not\nused
(#210613)\\n\\n## Summary\\n\\nCloses #210610\\n\\nThis PR
removes\n`error.id` field from being queried
at\\n`getErrorGroupMainStatistics`,\nas it was not being used, as it
was\\nrequired. If we didn't have this\nfield, the endpoint call
would\ncrash.\",\"sha\":\"46cd29e60be24fa3d39b2b364973e0e9b10659b7\"}}]}]\nBACKPORT-->\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>"}},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210613","number":210613,"mergeCommit":{"message":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used (#210613)\n\n## Summary\n\nCloses #210610\n\nThis PR removes
`error.id` field from being queried at\n`getErrorGroupMainStatistics`,
as it was not being used, as it was\nrequired. If we didn't have this
field, the endpoint call would
crash.","sha":"46cd29e60be24fa3d39b2b364973e0e9b10659b7"}}]}]
BACKPORT-->
rmyz added a commit that referenced this pull request Feb 12, 2025
… as it's not used (#210613) (#210763)

# Backport

This will backport the following commits from `main` to `8.17`:
- [[APM] Remove &#x60;error.id&#x60; in
&#x60;getErrorGroupMainStatistics&#x60; query as it&#x27;s not used
(#210613)](#210613)

<!--- Backport version: 9.6.4 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Sergi
Romeu","email":"sergi.romeu@elastic.co"},"sourceCommit":{"committedDate":"2025-02-11T18:13:51Z","message":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used (#210613)\n\n## Summary\n\nCloses #210610\n\nThis PR removes
`error.id` field from being queried at\n`getErrorGroupMainStatistics`,
as it was not being used, as it was\nrequired. If we didn't have this
field, the endpoint call would
crash.","sha":"46cd29e60be24fa3d39b2b364973e0e9b10659b7","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","v9.0.0","apm","Team:obs-ux-infra_services","backport:version","backport:8.17","v9.1.0","backport:8.18"],"title":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used","number":210613,"url":"https://github.com/elastic/kibana/pull/210613","mergeCommit":{"message":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used (#210613)\n\n## Summary\n\nCloses #210610\n\nThis PR removes
`error.id` field from being queried at\n`getErrorGroupMainStatistics`,
as it was not being used, as it was\nrequired. If we didn't have this
field, the endpoint call would
crash.","sha":"46cd29e60be24fa3d39b2b364973e0e9b10659b7"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/210677","number":210677,"state":"MERGED","mergeCommit":{"sha":"dbe7cc3b4f4b21484f5bb4b26e47fbe541b83fab","message":"[9.0]
[APM] Remove &#x60;error.id&#x60; in
&#x60;getErrorGroupMainStatistics&#x60; query as it&#x27;s not used
(#210613) (#210677)\n\n# Backport\n\nThis will backport the following
commits from `main` to `9.0`:\n- [[APM] Remove &#x60;error.id&#x60;
in\n&#x60;getErrorGroupMainStatistics&#x60; query as it&#x27;s not
used\n(#210613)](https://github.com/elastic/kibana/pull/210613)\n\n<!---
Backport version: 9.4.3 -->\n\n### Questions ?\nPlease refer to the
[Backport
tool\ndocumentation](https://github.com/sqren/backport)\n\n<!--BACKPORT
[{\"author\":{\"name\":\"Sergi\nRomeu\",\"email\":\"sergi.romeu@elastic.co\"},\"sourceCommit\":{\"committedDate\":\"2025-02-11T18:13:51Z\",\"message\":\"[APM]\nRemove
`error.id` in `getErrorGroupMainStatistics` query as it's not\nused
(#210613)\\n\\n## Summary\\n\\nCloses #210610\\n\\nThis PR
removes\n`error.id` field from being queried
at\\n`getErrorGroupMainStatistics`,\nas it was not being used, as it
was\\nrequired. If we didn't have this\nfield, the endpoint call
would\ncrash.\",\"sha\":\"46cd29e60be24fa3d39b2b364973e0e9b10659b7\",\"branchLabelMapping\":{\"^v9.1.0$\":\"main\",\"^v8.19.0$\":\"8.x\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"release_note:fix\",\"v9.0.0\",\"apm\",\"Team:obs-ux-infra_services\",\"backport:version\",\"backport:8.17\",\"v9.1.0\",\"backport:8.18\"],\"title\":\"[APM]\nRemove
`error.id` in `getErrorGroupMainStatistics` query as it's
not\nused\",\"number\":210613,\"url\":\"https://github.com/elastic/kibana/pull/210613\",\"mergeCommit\":{\"message\":\"[APM]\nRemove
`error.id` in `getErrorGroupMainStatistics` query as it's not\nused
(#210613)\\n\\n## Summary\\n\\nCloses #210610\\n\\nThis PR
removes\n`error.id` field from being queried
at\\n`getErrorGroupMainStatistics`,\nas it was not being used, as it
was\\nrequired. If we didn't have this\nfield, the endpoint call
would\ncrash.\",\"sha\":\"46cd29e60be24fa3d39b2b364973e0e9b10659b7\"}},\"sourceBranch\":\"main\",\"suggestedTargetBranches\":[\"9.0\"],\"targetPullRequestStates\":[{\"branch\":\"9.0\",\"label\":\"v9.0.0\",\"branchLabelMappingKey\":\"^v(\\\\d+).(\\\\d+).\\\\d+$\",\"isSourceBranch\":false,\"state\":\"NOT_CREATED\"},{\"branch\":\"main\",\"label\":\"v9.1.0\",\"branchLabelMappingKey\":\"^v9.1.0$\",\"isSourceBranch\":true,\"state\":\"MERGED\",\"url\":\"https://github.com/elastic/kibana/pull/210613\",\"number\":210613,\"mergeCommit\":{\"message\":\"[APM]\nRemove
`error.id` in `getErrorGroupMainStatistics` query as it's not\nused
(#210613)\\n\\n## Summary\\n\\nCloses #210610\\n\\nThis PR
removes\n`error.id` field from being queried
at\\n`getErrorGroupMainStatistics`,\nas it was not being used, as it
was\\nrequired. If we didn't have this\nfield, the endpoint call
would\ncrash.\",\"sha\":\"46cd29e60be24fa3d39b2b364973e0e9b10659b7\"}}]}]\nBACKPORT-->\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>"}},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210613","number":210613,"mergeCommit":{"message":"[APM]
Remove `error.id` in `getErrorGroupMainStatistics` query as it's not
used (#210613)\n\n## Summary\n\nCloses #210610\n\nThis PR removes
`error.id` field from being queried at\n`getErrorGroupMainStatistics`,
as it was not being used, as it was\nrequired. If we didn't have this
field, the endpoint call would
crash.","sha":"46cd29e60be24fa3d39b2b364973e0e9b10659b7"}},{"url":"https://github.com/elastic/kibana/pull/210761","number":210761,"branch":"8.x","state":"OPEN"},{"url":"https://github.com/elastic/kibana/pull/210762","number":210762,"branch":"8.18","state":"OPEN"}]}]
BACKPORT-->
@jennypavlova jennypavlova linked an issue Mar 6, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

4 participants