Skip to content

[EDR Workflows] Deprecate public endpoint/suggestions api endpoint in favour of an internal one #194832

Merged
szwarckonrad merged 16 commits intoelastic:mainfrom
szwarckonrad:deprecate-public-suggestions-api-endpoint
Oct 8, 2024
Merged

[EDR Workflows] Deprecate public endpoint/suggestions api endpoint in favour of an internal one #194832
szwarckonrad merged 16 commits intoelastic:mainfrom
szwarckonrad:deprecate-public-suggestions-api-endpoint

Conversation

@szwarckonrad
Copy link
Copy Markdown
Contributor

@szwarckonrad szwarckonrad commented Oct 3, 2024

New internal GET /internal/api/endpoint/suggestions/{suggestion_type} route.

Current public GET /api/endpoint/suggestions/{suggestion_type} route is set to deprecated.

UI uses now the internal GET /internal/api/endpoint/suggestions/{suggestion_type} api route

@szwarckonrad szwarckonrad self-assigned this Oct 3, 2024
@szwarckonrad szwarckonrad added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:Defend Workflows “EDR Workflows” sub-team of Security Solution v8.16.0 backport:version Backport to applied version labels labels Oct 3, 2024
@szwarckonrad szwarckonrad marked this pull request as ready for review October 7, 2024 08:48
@szwarckonrad szwarckonrad requested a review from a team as a code owner October 7, 2024 08:48
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/security-defend-workflows (Team:Defend Workflows)

Comment thread oas_docs/output/kibana.serverless.staging.yaml
Copy link
Copy Markdown
Contributor

@paul-tavares paul-tavares left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you

@kibana-ci
Copy link
Copy Markdown

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] x-pack/test_serverless/functional/test_suites/search/common_configs/config.group6.ts / discover/esql discover esql view switch modal should not show switch modal when switching to a data view while a saved search is open
  • [job] [logs] Jest Tests #9 / query tab with unified timeline Leading actions - notes securitySolutionNotesEnabled = true should have the notification dot & correct tooltip

Metrics [docs]

Async chunks

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

id before after diff
securitySolution 20.6MB 20.6MB +9.0B

History

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

cc @szwarckonrad

Copy link
Copy Markdown
Contributor

@gergoabraham gergoabraham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@szwarckonrad szwarckonrad merged commit ef4755a into elastic:main Oct 8, 2024
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11236616061

kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Oct 8, 2024
… favour of an internal one (elastic#194832)

New internal GET `/internal/api/endpoint/suggestions/{suggestion_type}`
route.

Current public GET `/api/endpoint/suggestions/{suggestion_type}` route
is set to deprecated.

UI uses now the internal GET
`/internal/api/endpoint/suggestions/{suggestion_type}` api route

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit ef4755a)
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Oct 8, 2024
…int in favour of an internal one (#194832) (#195416)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[EDR Workflows] Deprecate public endpoint/suggestions api endpoint in
favour of an internal one
(#194832)](#194832)

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

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

<!--BACKPORT [{"author":{"name":"Konrad
Szwarc","email":"konrad.szwarc@elastic.co"},"sourceCommit":{"committedDate":"2024-10-08T13:34:27Z","message":"[EDR
Workflows] Deprecate public endpoint/suggestions api endpoint in favour
of an internal one (#194832)\n\nNew internal GET
`/internal/api/endpoint/suggestions/{suggestion_type}`\r\nroute.\r\n\r\nCurrent
public GET `/api/endpoint/suggestions/{suggestion_type}` route\r\nis set
to deprecated.\r\n\r\n\r\nUI uses now the internal
GET\r\n`/internal/api/endpoint/suggestions/{suggestion_type}` api
route\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"ef4755a063e5e08af3ebd44c2a866ac804eabcd4","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Defend
Workflows","v8.16.0","backport:version"],"title":"[EDR Workflows]
Deprecate public endpoint/suggestions api endpoint in favour of an
internal one
","number":194832,"url":"https://github.com/elastic/kibana/pull/194832","mergeCommit":{"message":"[EDR
Workflows] Deprecate public endpoint/suggestions api endpoint in favour
of an internal one (#194832)\n\nNew internal GET
`/internal/api/endpoint/suggestions/{suggestion_type}`\r\nroute.\r\n\r\nCurrent
public GET `/api/endpoint/suggestions/{suggestion_type}` route\r\nis set
to deprecated.\r\n\r\n\r\nUI uses now the internal
GET\r\n`/internal/api/endpoint/suggestions/{suggestion_type}` api
route\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"ef4755a063e5e08af3ebd44c2a866ac804eabcd4"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/194832","number":194832,"mergeCommit":{"message":"[EDR
Workflows] Deprecate public endpoint/suggestions api endpoint in favour
of an internal one (#194832)\n\nNew internal GET
`/internal/api/endpoint/suggestions/{suggestion_type}`\r\nroute.\r\n\r\nCurrent
public GET `/api/endpoint/suggestions/{suggestion_type}` route\r\nis set
to deprecated.\r\n\r\n\r\nUI uses now the internal
GET\r\n`/internal/api/endpoint/suggestions/{suggestion_type}` api
route\r\n\r\n---------\r\n\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"ef4755a063e5e08af3ebd44c2a866ac804eabcd4"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Konrad Szwarc <konrad.szwarc@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:Defend Workflows “EDR Workflows” sub-team of Security Solution v8.16.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants