Skip to content
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.

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

Fix FileSettingsRoleMappingUpgradeIT assertions (#115422) #115469

Merged
merged 2 commits into from
Oct 24, 2024

Conversation

n1v0lg
Copy link
Contributor

@n1v0lg n1v0lg commented Oct 23, 2024

Backports 115422 with commit:

Fixes some faulty assertions in an upgrade test. Test failures only
manifest on the 8.16 branch since 9.x does not qualify for these upgrade
tests, and the change is not backported to 8.17 yet (unrelated CI
failures).

I validated this works by running it locally from the 8.16 branch.

Resolves: elastic#115410
Resolves: elastic#115411
@n1v0lg n1v0lg added >test Issues or PRs that are addressing/adding tests :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) backport Team:Security Meta label for security team v8.17.0 labels Oct 23, 2024
@n1v0lg n1v0lg self-assigned this Oct 23, 2024
@n1v0lg n1v0lg added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Oct 23, 2024
@n1v0lg
Copy link
Contributor Author

n1v0lg commented Oct 24, 2024

@elasticmachine update branch

@elasticsearchmachine elasticsearchmachine merged commit 2e58124 into elastic:8.x Oct 24, 2024
20 checks passed
@n1v0lg n1v0lg deleted the 8-x-backport-115422 branch October 24, 2024 08:43
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/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) Team:Security Meta label for security team >test Issues or PRs that are addressing/adding tests v8.17.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants