Skip to content

[Kibana Management] Add missing API endpoint authz info#209758

Merged
sabarasaba merged 5 commits intoelastic:mainfrom
sabarasaba:add_missing_authz_content
Feb 7, 2025
Merged

[Kibana Management] Add missing API endpoint authz info#209758
sabarasaba merged 5 commits intoelastic:mainfrom
sabarasaba:add_missing_authz_content

Conversation

@sabarasaba
Copy link
Copy Markdown
Member

@sabarasaba sabarasaba commented Feb 5, 2025

Summary

Part of #204681. Adds missing authz info to the API endpoints of some of our apps.

@sabarasaba sabarasaba added Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// release_note:skip Skip the PR/issue when compiling release notes v9.0.0 backport:prev-minor v8.19.0 labels Feb 5, 2025
@sabarasaba sabarasaba self-assigned this Feb 5, 2025
@ersin-erdal ersin-erdal assigned sabarasaba and unassigned sabarasaba Feb 5, 2025
@sabarasaba sabarasaba marked this pull request as ready for review February 6, 2025 08:49
@sabarasaba sabarasaba requested review from a team as code owners February 6, 2025 08:49
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-management (Team:Kibana Management)

@sabarasaba
Copy link
Copy Markdown
Member Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #7 / useGetFeaturesIds shows a toast error when the api return an error

Metrics [docs]

✅ unchanged

History

cc @sabarasaba

Copy link
Copy Markdown
Contributor

@mattkime mattkime left a comment

Choose a reason for hiding this comment

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

changes look good, thanks!

@sabarasaba sabarasaba added backport:version Backport to applied version labels Authz: API migration labels Feb 7, 2025
@sabarasaba sabarasaba merged commit a468965 into elastic:main Feb 7, 2025
5 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.x, 9.0

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

@kibanamachine
Copy link
Copy Markdown
Contributor

💔 Some backports could not be created

Status Branch Result
8.x
9.0 Backport failed because of merge conflicts

You might need to backport the following PRs to 9.0:
- chore(slo): factorize error handler (#209671)
- [Security Solution] SIEM Migrations RBAC (#207087)
- Remove beta badge for manual rule run (#209535)
- [EDR Workflows] Fix agent count on policy deploy modal (#209593)
- [GenAI Connectors] Add support for telemetry metadata (#208180)

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 209758

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Feb 7, 2025
… (#210135)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Kibana Management] Add missing API endpoint authz info
(#209758)](#209758)

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

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

<!--BACKPORT [{"author":{"name":"Ignacio
Rivas","email":"rivasign@gmail.com"},"sourceCommit":{"committedDate":"2025-02-07T07:51:22Z","message":"[Kibana
Management] Add missing API endpoint authz info
(#209758)","sha":"a468965588704f61a493eacd691da98877c177e2","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Kibana
Management","release_note:skip","v9.0.0","backport:prev-minor","backport:version","Authz:
API migration","v9.1.0","v8.19.0"],"title":"[Kibana Management] Add
missing API endpoint authz
info","number":209758,"url":"https://github.com/elastic/kibana/pull/209758","mergeCommit":{"message":"[Kibana
Management] Add missing API endpoint authz info
(#209758)","sha":"a468965588704f61a493eacd691da98877c177e2"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.x"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/209758","number":209758,"mergeCommit":{"message":"[Kibana
Management] Add missing API endpoint authz info
(#209758)","sha":"a468965588704f61a493eacd691da98877c177e2"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Ignacio Rivas <rivasign@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Authz: API migration backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// v8.19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants