Skip to content

[8.14] [Obs AI Assistant] Move alert context from apm to obs api (#182243)#182375

Merged
kibanamachine merged 1 commit intoelastic:8.14from
kibanamachine:backport/8.14/pr-182243
May 2, 2024
Merged

[8.14] [Obs AI Assistant] Move alert context from apm to obs api (#182243)#182375
kibanamachine merged 1 commit intoelastic:8.14from
kibanamachine:backport/8.14/pr-182243

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.14:

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine added the backport This PR is a backport of another PR label May 2, 2024
@kibanamachine kibanamachine enabled auto-merge (squash) May 2, 2024 14:08
@botelastic botelastic bot added Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. labels May 2, 2024
@elasticmachine
Copy link
Copy Markdown
Contributor

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

@sorenlouv sorenlouv force-pushed the backport/8.14/pr-182243 branch from 8f81e57 to 7e816c3 Compare May 2, 2024 15:09
…2243)

Follow-up to elastic#181501

This moves the alert details context used by the AI Assistant from the
apm api `GET /internal/apm/assistant/get_obs_alert_details_context` to
the Obs api
`/internal/observability/assistant/alert_details_contextual_insights`.

This also involves adding a type safe API client to the observability
API tests

### Other changes:

- Rename `synthtraceEsClient` to `apmSynthtraceEsClient` to indicate it
is specific to APM

(cherry picked from commit 0c9a323)
@sorenlouv sorenlouv force-pushed the backport/8.14/pr-182243 branch from 7e816c3 to a447638 Compare May 2, 2024 17:59
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
observability 669 670 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
observability 286.3KB 286.5KB +208.0B
Unknown metric groups

API count

id before after diff
observability 678 679 +1

History

  • 💔 Build #207557 failed 7e816c31d8ea2162248682ccc7cf14d2c9f2be4a
  • 💔 Build #207528 failed 8f81e571028f5abebc3ebe6e834332c1c93fe3de

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @sorenlouv

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants