[8.x] [Obs AI Assistant] Include an AdHoc instruction about the slack connector to avoid executing a loop (#199531) #199860
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ctor to avoid executing a loop (elastic#199531) Closes elastic#185028 ## Summary ### Problem The ticket mentions that there was an issue with displaying results. However both `display results` and `visualize query` are working as expected based on my investigation. More details including a video are attached [here](elastic#185028 (comment)). The function calling loop seems to occur when the AI Assistant is trying to send the output to Slack via the Kibana Slack connector. In order to do this, the LLM invokes the function `execute_connector`. For the Slack connector, `id` and `params` properties are required. However, the LLM only populated `id` and not `params` which causes an error when validated against the schema for the Slack connector. - Sometimes, it's able to retry a few times and successfully send the output to Slack. - Sometimes, it goes into a loop trying to find `params` and failing repeatedly. Attaching another similar issue for more context - elastic#195564 _With the solution below, my intention is to send the output to Slack in one go, without retrying the function `execute_connector`._ ### Solution Based on the solutions I experimented with, seems like we need to force the LLM to understand what's needed for the Slack connector. I tried a few options here and the combination of updates that worked are as follows: - Appending an AdHoc instruction about the Slack connector properties. (This gives the LLM some additional information about the required properties) - Updating the `properties` attached to the connector (when passing the connector list to the LLM), to reflect both `id` and `params` with `message`. With the above change, the AI Assistant has managed to consistently send the output to Slack _**without any retries**_ because of missing `params`. ### Screenshots: (all alert triggers successfully sent the output to Slack without having to retry the function) <img width="1452" alt="success-attempts-to-slack-connector" src="https://github.com/user-attachments/assets/715a5957-2c04-4a55-868f-34abe564f6d4"> (cherry picked from commit ed4c0df)
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
botelastic
bot
added
ci:project-deploy-observability
|
Pinging @elastic/obs-ai-assistant (Team:Obs AI Assistant) |
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport
This will backport the following commits from
mainto8.x:Questions ?
Please refer to the Backport tool documentation