LogLine: Add Explain in Assistant option to explain single log lines
#108387
Conversation
svennergr
added
no-backport
svennergr
requested review from
gabor
and removed request for
a team,
grafanabot and
gabor
|
What do you think if we do the same thing in the panel and explose this everywhere? I can give it a try later today. |
Do you mean in the panel menu? |
|
Ah you already did it! 😆 . I saw this while having lunch and assumed Logs Drilldown. Amazing! |
| @@ -81,6 +84,43 @@ export const LogLineMenu = ({ log, styles }: Props) => { | |||
| } | |||
| }, [log, onPinLine, onUnpinLine, pinned]); | |||
|
|
|||
| const openAssistant = useCallback(async () => { | |||
There was a problem hiding this comment.
This could be moved to https://github.com/grafana/grafana/blob/main/public/app/features/logs/components/panel/LogListContext.tsx, exposing both isAssistantAvailable and openMainAssistant through useLogListContext(), allowing this to be used here and in the new Log Details: https://github.com/grafana/grafana/blob/main/public/app/features/logs/components/panel/LogLineDetailsHeader.tsx
svennergr
requested review from
konsalex,
ivanortegaalba and
bfmatei
and removed request for
a team
svennergr
requested review from
matyax
and removed request for
a team,
konsalex,
ivanortegaalba and
bfmatei
|
Sorry for the spam here. @matyax I moved to the context - that's cool. Would appreciate another look! |
*What is this feature?
Adds an
Explain in Assistantoption to explain single log lines in each logs panel.Screen.Recording.2025-07-22.at.13.43.05.mov