Skip to content

[CI] IndicesPermissionTests testAuthorizeDataStreamFailureIndices failing #125555

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
elasticsearchmachine opened this issue Mar 25, 2025 · 5 comments · Fixed by #126112
Closed

[CI] IndicesPermissionTests testAuthorizeDataStreamFailureIndices failing #125555

elasticsearchmachine opened this issue Mar 25, 2025 · 5 comments · Fixed by #126112
Assignees
@n1v0lg
Labels
low-risk An open issue or test failure that is a low risk to future releases :Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC Team:Security Meta label for security team >test-failure Triaged test failures from CI

Comments

@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Mar 25, 2025 •
edited
Loading

Build Scans:

Reproduction Line:

./gradlew ":x-pack:plugin:security:test" --tests "org.elasticsearch.xpack.security.authz.accesscontrol.IndicesPermissionTests.testAuthorizeDataStreamFailureIndices" -Dtests.seed=93669AFB5AAE2CBD -Dbuild.snapshot=false -Dtests.jvm.argline="-Dbuild.snapshot=false" -Dlicense.key=x-pack/license-tools/src/test/resources/public.key -Dtests.locale=syr-SY -Dtests.timezone=SystemV/YST9YDT -Druntime.java=24

Applicable branches:
8.x

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

java.lang.AssertionError: for privilege [read_failure_store]
Expected: is <true>
     but: was <false>

Issue Reasons:

  • [8.x] 10 consecutive failures in step release-tests
  • [8.x] 10 failures in test testAuthorizeDataStreamFailureIndices (2.3% fail rate in 432 executions)
  • [8.x] 10 failures in step release-tests (100.0% fail rate in 10 executions)
  • [8.x] 10 failures in pipeline elasticsearch-periodic (100.0% fail rate in 10 executions)

Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.
@elasticsearchmachine elasticsearchmachine added :Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC >test-failure Triaged test failures from CI Team:Security Meta label for security team needs:risk Requires assignment of a risk label (low, medium, blocker) labels Mar 25, 2025
@elasticsearchmachine
Copy link
Collaborator Author

Pinging @elastic/es-security (Team:Security)

@n1v0lg n1v0lg self-assigned this Mar 25, 2025
@n1v0lg
Copy link
Contributor

n1v0lg commented Mar 25, 2025

Forgot to feature flag the test.

@n1v0lg n1v0lg added low-risk An open issue or test failure that is a low risk to future releases and removed needs:risk Requires assignment of a risk label (low, medium, blocker) labels Mar 25, 2025
@elasticsearchmachine
Copy link
Collaborator Author

This has been muted on branch main

Mute Reasons:

  • [main] 3 consecutive failures in step release-tests
  • [main] 4 failures in test testAuthorizeDataStreamFailureIndices (2.7% fail rate in 147 executions)
  • [main] 3 failures in step release-tests (100.0% fail rate in 3 executions)
  • [main] 3 failures in pipeline elasticsearch-periodic (100.0% fail rate in 3 executions)

Build Scans:

elasticsearchmachine added a commit that referenced this issue Mar 25, 2025
@elasticsearchmachine
Mute org.elasticsearch.xpack.security.authz.accesscontrol.IndicesPerm…
2331051 
…issionTests testAuthorizeDataStreamFailureIndices #125555
@n1v0lg
Copy link
Contributor

n1v0lg commented Mar 25, 2025

Urgh yes fix incoming.

@n1v0lg
Copy link
Contributor

n1v0lg commented Mar 27, 2025

Closed by: #125591

@n1v0lg n1v0lg closed this as completed Mar 27, 2025
omricohenn pushed a commit to omricohenn/elasticsearch that referenced this issue Mar 28, 2025
@elasticsearchmachine @omricohenn
Mute org.elasticsearch.xpack.security.authz.accesscontrol.IndicesPerm…
a9e447d 
…issionTests testAuthorizeDataStreamFailureIndices elastic#125555
slobodanadamovic added a commit to slobodanadamovic/elasticsearch that referenced this issue Apr 2, 2025
@slobodanadamovic
[Test] Skip testAuthorizeDataStreamFailureIndices for release tests
347b662 
Adding a missing feature flag for
`IndicesPermissionTests#testAuthorizeDataStreamFailureIndices` test.

Resolves elastic#125555
@slobodanadamovic slobodanadamovic mentioned this issue Apr 2, 2025
Merged
elasticsearchmachine pushed a commit that referenced this issue Apr 2, 2025
@slobodanadamovic
[Test] Skip testAuthorizeDataStreamFailureIndices for release tests (#…
ea8fd3b 
…126112)

Adding a missing feature flag for
`IndicesPermissionTests#testAuthorizeDataStreamFailureIndices` test.

Resolves #125555 Resolves #126040
slobodanadamovic added a commit to slobodanadamovic/elasticsearch that referenced this issue Apr 3, 2025
@slobodanadamovic
[Test] Skip testAuthorizeDataStreamFailureIndices for release tests (e…
500445b 
…lastic#126112)

Adding a missing feature flag for
`IndicesPermissionTests#testAuthorizeDataStreamFailureIndices` test.

Resolves elastic#125555 Resolves elastic#126040

(cherry picked from commit ea8fd3b)

# Conflicts:
#	muted-tests.yml
elasticsearchmachine pushed a commit that referenced this issue Apr 3, 2025
@slobodanadamovic
[Test] Skip testAuthorizeDataStreamFailureIndices for release tests (#…
684f460 
…126112) (#126184)

Adding a missing feature flag for
`IndicesPermissionTests#testAuthorizeDataStreamFailureIndices` test.

Resolves #125555 Resolves #126040

(cherry picked from commit ea8fd3b)

# Conflicts:
#	muted-tests.yml
andreidan pushed a commit to andreidan/elasticsearch that referenced this issue Apr 9, 2025
@slobodanadamovic @andreidan
[Test] Skip testAuthorizeDataStreamFailureIndices for release tests (e…
232efa7 
…lastic#126112)

Adding a missing feature flag for
`IndicesPermissionTests#testAuthorizeDataStreamFailureIndices` test.

Resolves elastic#125555 Resolves elastic#126040
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@n1v0lg n1v0lg

Labels
low-risk An open issue or test failure that is a low risk to future releases :Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC Team:Security Meta label for security team >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
2 participants
@n1v0lg @elasticsearchmachine