Skip to content

Conversation

@gmarouli
Copy link
Contributor

@gmarouli gmarouli commented May 30, 2023

We did not define if cluster:monitor/xpack/usage/data_lifecycle is operator only or non operator action which resulted in the following failure:

java.lang.AssertionError: Actions are neither operator-only nor non-operator: [[cluster:monitor/xpack/usage/data_lifecycle]]. An action should be declared to be either operator-only in [org.elasticsearch.xpack.security.operator.OperatorOnlyRegistry] or non-operator in [org.elasticsearch.xpack.security.operator.Constants] |   

https://gradle-enterprise.elastic.co/s/qs25bf66nam6s/tests/:x-pack:plugin:security:qa:operator-privileges-tests:javaRestTest/org.elasticsearch.xpack.security.operator.OperatorPrivilegesIT/testEveryActionIsEitherOperatorOnlyOrNonOperator?top-execution=1

@gmarouli gmarouli added >test Issues or PRs that are addressing/adding tests :Data Management/DLM v8.9.0 labels May 30, 2023
@elasticsearchmachine elasticsearchmachine added the Team:Data Management Meta label for data/management team label May 30, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

Copy link
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks for fixing it.

@gmarouli gmarouli added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label May 30, 2023
@elasticsearchmachine elasticsearchmachine merged commit b7bbc19 into elastic:main May 30, 2023
@gmarouli gmarouli deleted the make-dlm-usage-non-operator branch May 30, 2023 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) Team:Data Management Meta label for data/management team >test Issues or PRs that are addressing/adding tests v8.9.0

3 participants