Skip to content

Conversation

@slobodanadamovic
Copy link
Contributor

Backports the following commits to 9.0:

…les (elastic#122803)

Adds missing `assertBusy` in `testQueryDeleteOrUpdateReservedRoles`. 
We wait for security migration to finish before test execution, but this is not 
enough to guarantee that built-in roles are synchronized into the `.security` 
index. In many cases, the roles have already been synchronized before the 
query gets executed, but in rare cases the cluster is a bit slower and the 
assertion fails because the built-in roles have not been indexed yet.

Resolves elastic#120401
@slobodanadamovic slobodanadamovic added :Security/Security Security issues without another label >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Security Meta label for security team labels Feb 17, 2025
@elasticsearchmachine elasticsearchmachine merged commit 44f99f9 into elastic:9.0 Feb 19, 2025
21 checks passed
@slobodanadamovic slobodanadamovic deleted the backport/9.0/pr-122803 branch February 19, 2025 16:14
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!) backport :Security/Security Security issues without another label Team:Security Meta label for security team >test Issues or PRs that are addressing/adding tests v9.0.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants