Skip to content

Conversation

@gmarouli
Copy link
Contributor

@gmarouli gmarouli commented Apr 9, 2025

In this PR we extend DataStreamLifecycleServiceRuntimeSecurityIT.java to cover DLM security access towards failure store as well.

The system data stream tests are excluded because failure store is not supported yet.

@gmarouli gmarouli added >test Issues or PRs that are addressing/adding tests :Data Management/Data streams Data streams and their lifecycles v8.19.0 v9.1.0 labels Apr 9, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Data Management Meta label for data/management team label Apr 9, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@gmarouli gmarouli added the auto-backport Automatically create backport pull requests when merged label Apr 9, 2025
@gmarouli gmarouli changed the title [Failure store] Bug fix & extend security test [Failure store] Extend DLM security test to cover failure store Apr 10, 2025
@slobodanadamovic slobodanadamovic self-requested a review April 10, 2025 13:51
Copy link
Contributor

@slobodanadamovic slobodanadamovic left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@gmarouli
Copy link
Contributor Author

@slobodanadamovic thank you!

@gmarouli gmarouli merged commit 424a257 into elastic:main Apr 11, 2025
17 checks passed
@gmarouli gmarouli deleted the failure-store-dlm-security branch April 11, 2025 07:12
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
8.x Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 126543

@gmarouli
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.x

Questions ?

Please refer to the Backport tool documentation

gmarouli added a commit to gmarouli/elasticsearch that referenced this pull request Apr 11, 2025
…tic#126543)

(cherry picked from commit 424a257)

# Conflicts:
#	x-pack/plugin/security/src/internalClusterTest/java/org/elasticsearch/integration/DataStreamLifecycleServiceRuntimeSecurityIT.java
elasticsearchmachine pushed a commit that referenced this pull request Apr 11, 2025
) (#126659)

(cherry picked from commit 424a257)

# Conflicts:
#	x-pack/plugin/security/src/internalClusterTest/java/org/elasticsearch/integration/DataStreamLifecycleServiceRuntimeSecurityIT.java
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 :Data Management/Data streams Data streams and their lifecycles Team:Data Management Meta label for data/management 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