Skip to content

Conversation

@davidkyle
Copy link
Member

@davidkyle davidkyle commented May 23, 2025

Adds a test for the fix in #128327

Run the test with this command

./gradlew :x-pack:plugin:inference:qa:inference-with-security:yamlRestTest 

Before #128327 the test would fail because the non admin users cannot call cluster:internal/xpack/inference:

Failed to execute phase [rank-feature], Computing updated ranks for results failed\n\t\t... 11 more\n\tCaused by: 

org.elasticsearch.ElasticsearchSecurityException: action [cluster:internal/xpack/inference] is unauthorized for user [inference_user] with effective roles [monitor_only_user], this action is granted by the cluster privileges [manage,all]

@davidkyle davidkyle added >test Issues or PRs that are addressing/adding tests :ml Machine learning auto-backport Automatically create backport pull requests when merged v8.19.0 labels May 23, 2025
@davidkyle davidkyle marked this pull request as ready for review May 23, 2025 10:48
@elasticsearchmachine elasticsearchmachine added the Team:ML Meta label for the ML team label May 23, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

@davidkyle davidkyle merged commit 5b3a4f3 into elastic:main May 23, 2025
18 checks passed
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.19

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 :ml Machine learning Team:ML Meta label for the ML team >test Issues or PRs that are addressing/adding tests v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants