-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Security GenAI] Remove assistantNaturalLanguageESQLTool
feature flag and enable by default
#195480
Conversation
Pinging @elastic/security-solution (Team: SecuritySolution) |
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here. |
⏳ Build in-progress, with failures
Failed CI StepsTest Failures
History
|
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Async chunks
Page load bundle
History
|
Starting backport for target branches: 8.x https://github.com/elastic/kibana/actions/runs/11256813292 |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…ure flag and enable by default (#195480) (#195674) # Backport This will backport the following commits from `main` to `8.x`: - [[Security GenAI] Remove `assistantNaturalLanguageESQLTool` feature flag and enable by default (#195480)](#195480) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Steph Milovic","email":"stephanie.milovic@elastic.co"},"sourceCommit":{"committedDate":"2024-10-09T14:13:05Z","message":"[Security GenAI] Remove `assistantNaturalLanguageESQLTool` feature flag and enable by default (#195480)","sha":"3dd1ee8ae00a40b582f2d1f784c96269a1d2e016","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:enhancement","v9.0.0","Team: SecuritySolution","backport:prev-minor","Feature:Security Assistant","Team:Security Generative AI","v8.16.0"],"number":195480,"url":"https://github.com/elastic/kibana/pull/195480","mergeCommit":{"message":"[Security GenAI] Remove `assistantNaturalLanguageESQLTool` feature flag and enable by default (#195480)","sha":"3dd1ee8ae00a40b582f2d1f784c96269a1d2e016"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/195480","number":195480,"mergeCommit":{"message":"[Security GenAI] Remove `assistantNaturalLanguageESQLTool` feature flag and enable by default (#195480)","sha":"3dd1ee8ae00a40b582f2d1f784c96269a1d2e016"}},{"branch":"8.x","label":"v8.16.0","labelRegex":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT-->
Summary
After several weeks of evaluation testing, we feel confident to switch from
ESQLKnowledgeBaseTool
toNaturalLanguageESQLTool
as our default ESQL tool. This PR takes the following steps to do so:assistantNaturalLanguageESQLTool
feature flag and enables theNaturalLanguageESQLTool
by defaultESQLKnowledgeBaseTool
loadESQL
function as we no longer use them with theNaturalLanguageESQLTool
esql_exists
fromisKnowledgeBaseSetup
checkesql/documentation
andesql/language_definition
asNaturalLanguageESQLTool
already has these referencesesql/example_queries
as I am going to spike on including these in theNaturalLanguageESQLTool
To test
NaturalLanguageESQLTool
has been called.Free feel to run the ESQL Generation Regression Suite but I've already done it for you!