Skip to content

Conversation

@slobodanadamovic
Copy link
Contributor

Adjusting the test assertion to account for the potential 3rd extra group (#125900).

Resolves #126061

@slobodanadamovic slobodanadamovic added >test Issues or PRs that are addressing/adding tests :Security/Security Security issues without another label Team:Security Meta label for security team auto-backport Automatically create backport pull requests when merged v8.19.0 v9.1.0 labels Apr 2, 2025
@slobodanadamovic slobodanadamovic requested a review from n1v0lg April 2, 2025 09:04
@slobodanadamovic slobodanadamovic self-assigned this Apr 2, 2025
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Contributor

@n1v0lg n1v0lg left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this!

@slobodanadamovic slobodanadamovic added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Apr 2, 2025
@elasticsearchmachine elasticsearchmachine merged commit 7b753d0 into elastic:main Apr 2, 2025
22 checks passed
@slobodanadamovic slobodanadamovic deleted the sa-fix-rbac-test branch April 2, 2025 12:57
@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 126106

slobodanadamovic added a commit to slobodanadamovic/elasticsearch that referenced this pull request Apr 3, 2025
Adjusting the test assertion to account for the potential 3rd extra
group (elastic#125900).

Resolves elastic#126061

(cherry picked from commit 7b753d0)

# Conflicts:
#	muted-tests.yml
@slobodanadamovic
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.x

Questions ?

Please refer to the Backport tool documentation

elasticsearchmachine pushed a commit that referenced this pull request Apr 3, 2025
Adjusting the test assertion to account for the potential 3rd extra
group (#125900).

Resolves #126061

(cherry picked from commit 7b753d0)

# Conflicts:
#	muted-tests.yml
andreidan pushed a commit to andreidan/elasticsearch that referenced this pull request Apr 9, 2025
Adjusting the test assertion to account for the potential 3rd extra
group (elastic#125900).

Resolves elastic#126061
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 auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Security/Security Security issues without another label Team:Security Meta label for security 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.

[CI] RBACEngineTests testGetRoleDescriptorsIntersectionForRemoteClusterHasDeterministicOrderForIndicesPrivileges failing

3 participants