Skip to content

[9.0] Fix unsupported privileges error message during role and API key creation (#128858)#129158

Merged
gmjehovich merged 6 commits intoelastic:9.0from
gmjehovich:fix/unsupported-privilege-error-msg-9.0
Jun 11, 2025
Merged

[9.0] Fix unsupported privileges error message during role and API key creation (#128858)#129158
gmjehovich merged 6 commits intoelastic:9.0from
gmjehovich:fix/unsupported-privilege-error-msg-9.0

Conversation

@gmjehovich
Copy link
Contributor

@gmjehovich gmjehovich commented Jun 9, 2025

Backport of #128858
@gmjehovich gmjehovich requested a review from jfreden June 9, 2025 18:04
@gmjehovich gmjehovich added >enhancement :Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC Team:Security Meta label for security team auto-backport Automatically create backport pull requests when merged v8.19.0 v8.17.8 v8.18.3 labels Jun 9, 2025
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Jun 9, 2025

Pinging @elastic/es-security (Team:Security)
@gmjehovich gmjehovich self-assigned this Jun 9, 2025
jfreden
jfreden reviewed Jun 10, 2025
View reviewed changes
Copy link
Contributor

@jfreden jfreden left a comment

Choose a reason for hiding this comment

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

Code looks good but you're missing the changelog and the the original PR should be in the title, so:
[9.0] Fix unsupported privileges error message during role and API key creation (#128858)

Since this is the backport PR, you also don't need the labels with all the other versions this will be backported to, that's only needed in the main PR.
@gmjehovich gmjehovich changed the title [9.0] Fix unsupported privileges error message during role and API key creation Jun 10, 2025
jfreden
jfreden approved these changes Jun 11, 2025
View reviewed changes
Copy link
Contributor

@jfreden jfreden left a comment

Choose a reason for hiding this comment

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

LGTM!
@gmjehovich gmjehovich merged commit e649019 into elastic:9.0 Jun 11, 2025
21 checks passed
This was referenced Jun 11, 2025
Merged
Merged
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Jun 11, 2025

💔 Backport failed

Status Branch Result
❌ 8.19 Commit could not be cherrypicked due to conflicts
✅ 8.17
✅ 8.18

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 129158
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Jun 11, 2025

@gmjehovich according to this PR's labels, it should not have a changelog YAML, but I can't delete it because it is closed. Please either delete the changelog from the appropriate branch yourself, or adjust the labels.
gmjehovich added a commit to gmjehovich/elasticsearch that referenced this pull request Jun 11, 2025
@gmjehovich
[9.0] Fix unsupported privileges error message during role and API ke…
491bb74 
…y creation (elastic#128858) (elastic#129158)

* Fix unsupported privileges error message during role and API key creation

* [CI] Auto commit changes from spotless

* Add changelog file

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
elasticsearchmachine pushed a commit that referenced this pull request Jun 11, 2025
@gmjehovich
[9.0] Fix unsupported privileges error message during role and API ke…
5d7bdfd 
…y creation (#128858) (#129158) (#129276)

* Fix unsupported privileges error message during role and API key creation

* [CI] Auto commit changes from spotless

* Add changelog file

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
elasticsearchmachine pushed a commit that referenced this pull request Jun 11, 2025
@gmjehovich
[9.0] Fix unsupported privileges error message during role and API ke…
3509cbb 
…y creation (#128858) (#129158) (#129274)

* Fix unsupported privileges error message during role and API key creation

* [CI] Auto commit changes from spotless

* Add changelog file

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged backport pending backport >enhancement :Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC Team:Security Meta label for security team v8.17.8 v8.18.3 v8.19.0 v9.0.3

3 participants

@gmjehovich @elasticsearchmachine @jfreden