Skip to content
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

[One Discover] Enable O11y LogAIAssistant prompts for log detail flyout #202191

Closed
tonyghiani opened this issue Nov 28, 2024 · 1 comment · Fixed by #204339
Closed

[One Discover] Enable O11y LogAIAssistant prompts for log detail flyout #202191

tonyghiani opened this issue Nov 28, 2024 · 1 comment · Fixed by #204339
Assignees
Labels
Project:OneDiscover Enrich Discover with contextual awareness Team:obs-ux-logs Observability Logs User Experience Team

Comments

@tonyghiani
Copy link
Contributor

📓 Summary

As a follow-up work of #181976 and given the contextual awareness architecture is now a solid concept in Discover, we want to bring parity with Logs Explorer and LogsStream into the O11y Discover experience adding the support for the Logs AI Assistant in the log detail flyout.

Image

✔ Acceptance Criteria

  • The registered Logs AI Assistant should be rendered on the Logs Overview DocView when it matches the O11y logs document profile.
  • The AI assistant prompts should be displayed only when the message field is available on the document.

💡 Implementation hints

  • The logs ai assistant is already registered by the discover-shared plugin as a feature to prevent cyclic dependency between plugins. It should be consumed from that registry into the log document profile.
@tonyghiani tonyghiani added Project:OneDiscover Enrich Discover with contextual awareness Team:obs-ux-logs Observability Logs User Experience Team labels Nov 28, 2024
@elasticmachine
Copy link
Contributor

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

@tonyghiani tonyghiani self-assigned this Dec 16, 2024
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Dec 19, 2024
…ic#204339)

## 📓 Summary

Closes elastic#202191

This work brings into the O11y solution contextual experience in
Discover the Logs AI Assistant prompts already adopted in LogStream and
Logs Explorer, which are now set for deprecation in favour of the unique
OneDiscover experience.

The AIAssistant is consumed by the `discover-shared` features registry,
which registers the features from the logs-shared plugin. Once the
codebase for LogStream and Logs Explorer that consume this component is
removed, it might make sense to do some clean-up and move the
LogsAIAssistant component implementation among the other contextual
components (log summary columns, etc...).

https://github.com/user-attachments/assets/f97d5a01-3896-4a86-9a3c-443151a9d04e

Co-authored-by: Marco Antonio Ghiani <[email protected]>
(cherry picked from commit 8b3281d)
kibanamachine added a commit that referenced this issue Dec 19, 2024
…204339) (#204910)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[One Discover] Add AIAssistant prompts to logs overview detail
(#204339)](#204339)

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

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

<!--BACKPORT [{"author":{"name":"Marco Antonio
Ghiani","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-19T10:37:08Z","message":"[One
Discover] Add AIAssistant prompts to logs overview detail
(#204339)\n\n## 📓 Summary\r\n\r\nCloses #202191 \r\n\r\nThis work brings
into the O11y solution contextual experience in\r\nDiscover the Logs AI
Assistant prompts already adopted in LogStream and\r\nLogs Explorer,
which are now set for deprecation in favour of the unique\r\nOneDiscover
experience.\r\n\r\nThe AIAssistant is consumed by the `discover-shared`
features registry,\r\nwhich registers the features from the logs-shared
plugin. Once the\r\ncodebase for LogStream and Logs Explorer that
consume this component is\r\nremoved, it might make sense to do some
clean-up and move the\r\nLogsAIAssistant component implementation among
the other contextual\r\ncomponents (log summary columns,
etc...).\r\n\r\n\r\nhttps://github.com/user-attachments/assets/f97d5a01-3896-4a86-9a3c-443151a9d04e\r\n\r\nCo-authored-by:
Marco Antonio Ghiani
<[email protected]>","sha":"8b3281d4843efff3be4d291453c1aa6a61cd75fc","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:DataDiscovery","backport:prev-minor","ci:project-deploy-observability","Team:obs-ux-logs","Project:OneDiscover"],"title":"[One
Discover] Add AIAssistant prompts to logs overview
detail","number":204339,"url":"https://github.com/elastic/kibana/pull/204339","mergeCommit":{"message":"[One
Discover] Add AIAssistant prompts to logs overview detail
(#204339)\n\n## 📓 Summary\r\n\r\nCloses #202191 \r\n\r\nThis work brings
into the O11y solution contextual experience in\r\nDiscover the Logs AI
Assistant prompts already adopted in LogStream and\r\nLogs Explorer,
which are now set for deprecation in favour of the unique\r\nOneDiscover
experience.\r\n\r\nThe AIAssistant is consumed by the `discover-shared`
features registry,\r\nwhich registers the features from the logs-shared
plugin. Once the\r\ncodebase for LogStream and Logs Explorer that
consume this component is\r\nremoved, it might make sense to do some
clean-up and move the\r\nLogsAIAssistant component implementation among
the other contextual\r\ncomponents (log summary columns,
etc...).\r\n\r\n\r\nhttps://github.com/user-attachments/assets/f97d5a01-3896-4a86-9a3c-443151a9d04e\r\n\r\nCo-authored-by:
Marco Antonio Ghiani
<[email protected]>","sha":"8b3281d4843efff3be4d291453c1aa6a61cd75fc"}},"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/204339","number":204339,"mergeCommit":{"message":"[One
Discover] Add AIAssistant prompts to logs overview detail
(#204339)\n\n## 📓 Summary\r\n\r\nCloses #202191 \r\n\r\nThis work brings
into the O11y solution contextual experience in\r\nDiscover the Logs AI
Assistant prompts already adopted in LogStream and\r\nLogs Explorer,
which are now set for deprecation in favour of the unique\r\nOneDiscover
experience.\r\n\r\nThe AIAssistant is consumed by the `discover-shared`
features registry,\r\nwhich registers the features from the logs-shared
plugin. Once the\r\ncodebase for LogStream and Logs Explorer that
consume this component is\r\nremoved, it might make sense to do some
clean-up and move the\r\nLogsAIAssistant component implementation among
the other contextual\r\ncomponents (log summary columns,
etc...).\r\n\r\n\r\nhttps://github.com/user-attachments/assets/f97d5a01-3896-4a86-9a3c-443151a9d04e\r\n\r\nCo-authored-by:
Marco Antonio Ghiani
<[email protected]>","sha":"8b3281d4843efff3be4d291453c1aa6a61cd75fc"}}]}]
BACKPORT-->

---------

Co-authored-by: Marco Antonio Ghiani <[email protected]>
JoseLuisGJ pushed a commit to JoseLuisGJ/kibana that referenced this issue Dec 19, 2024
…ic#204339)

## 📓 Summary

Closes elastic#202191 

This work brings into the O11y solution contextual experience in
Discover the Logs AI Assistant prompts already adopted in LogStream and
Logs Explorer, which are now set for deprecation in favour of the unique
OneDiscover experience.

The AIAssistant is consumed by the `discover-shared` features registry,
which registers the features from the logs-shared plugin. Once the
codebase for LogStream and Logs Explorer that consume this component is
removed, it might make sense to do some clean-up and move the
LogsAIAssistant component implementation among the other contextual
components (log summary columns, etc...).


https://github.com/user-attachments/assets/f97d5a01-3896-4a86-9a3c-443151a9d04e

Co-authored-by: Marco Antonio Ghiani <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Project:OneDiscover Enrich Discover with contextual awareness Team:obs-ux-logs Observability Logs User Experience Team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants