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.

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

[CI] FileSettingsRoleMappingUpgradeIT testRoleMappingsAppliedOnUpgrade {upgradedNodes=3} failing #115410

Closed
elasticsearchmachine opened this issue Oct 23, 2024 · 2 comments · Fixed by #115422
Closed

[CI] FileSettingsRoleMappingUpgradeIT testRoleMappingsAppliedOnUpgrade {upgradedNodes=3} failing #115410

elasticsearchmachine opened this issue Oct 23, 2024 · 2 comments · Fixed by #115422
Assignees
@n1v0lg
Labels
needs:risk Requires assignment of a risk label (low, medium, blocker) :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) Team:Security Meta label for security team >test-failure Triaged test failures from CI

Comments

@elasticsearchmachine
Copy link
Collaborator

Build Scans:

Reproduction Line:

./gradlew ":qa:rolling-upgrade:v8.7.1#bwcTest" -Dtests.class="org.elasticsearch.upgrades.FileSettingsRoleMappingUpgradeIT" -Dtests.method="testRoleMappingsAppliedOnUpgrade {upgradedNodes=3}" -Dtests.seed=F6ADB9CD20C85AAC -Dtests.bwc=true -Dtests.locale=nb -Dtests.timezone=Pacific/Port_Moresby -Druntime.java=22

Applicable branches:
8.16

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

java.lang.AssertionError: 
Expected: is an instance of java.util.Map
     but: <[{metadata={_es_reserved_role_mapping_name=everyone_kibana}, roles=[kibana_user], rules={field={username=*}}, enabled=true}]> is a java.util.ArrayList

Issue Reasons:

  • [8.16] 2 consecutive failures in test testRoleMappingsAppliedOnUpgrade {upgradedNodes=3}
  • [8.16] 2 failures in test testRoleMappingsAppliedOnUpgrade {upgradedNodes=3} (5.7% fail rate in 35 executions)

Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.
@elasticsearchmachine elasticsearchmachine added :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) >test-failure Triaged test failures from CI labels Oct 23, 2024
@elasticsearchmachine
Copy link
Collaborator Author

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

@elasticsearchmachine elasticsearchmachine added Team:Security Meta label for security team needs:risk Requires assignment of a risk label (low, medium, blocker) labels Oct 23, 2024
@n1v0lg n1v0lg self-assigned this Oct 23, 2024
@n1v0lg
Copy link
Contributor

n1v0lg commented Oct 23, 2024

Duplicate of: #115411

@n1v0lg n1v0lg mentioned this issue Oct 23, 2024
Merged
elasticsearchmachine pushed a commit that referenced this issue Oct 23, 2024
@n1v0lg
Fix FileSettingsRoleMappingUpgradeIT assertions (#115422)
c6bd53f 
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: #115410
Resolves: #115411
n1v0lg added a commit to n1v0lg/elasticsearch that referenced this issue Oct 23, 2024
@n1v0lg
Fix FileSettingsRoleMappingUpgradeIT assertions (elastic#115422)
a1d0120 
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 added a commit to n1v0lg/elasticsearch that referenced this issue Oct 23, 2024
@n1v0lg
Fix FileSettingsRoleMappingUpgradeIT assertions (elastic#115422)
905d3b0 
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
elasticsearchmachine pushed a commit that referenced this issue Oct 23, 2024
@n1v0lg
Fix FileSettingsRoleMappingUpgradeIT assertions (#115422) (#115468)
67a7682 
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: #115410
Resolves: #115411
smalyshev pushed a commit to smalyshev/elasticsearch that referenced this issue Oct 23, 2024
@n1v0lg @smalyshev
Fix FileSettingsRoleMappingUpgradeIT assertions (elastic#115422)
cb20409 
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
elasticsearchmachine pushed a commit that referenced this issue Oct 24, 2024
@n1v0lg @elasticmachine
Fix FileSettingsRoleMappingUpgradeIT assertions (#115422) (#115469)
2e58124 
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: #115410
Resolves: #115411

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
georgewallace pushed a commit to georgewallace/elasticsearch that referenced this issue Oct 25, 2024
@n1v0lg @georgewallace
Fix FileSettingsRoleMappingUpgradeIT assertions (elastic#115422)
82e285f 
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
jfreden pushed a commit to jfreden/elasticsearch that referenced this issue Nov 4, 2024
@n1v0lg @jfreden
Fix FileSettingsRoleMappingUpgradeIT assertions (elastic#115422)
9f4f7f9 
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@n1v0lg n1v0lg

Labels
needs:risk Requires assignment of a risk label (low, medium, blocker) :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) Team:Security Meta label for security team >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix FileSettingsRoleMappingUpgradeIT assertions n1v0lg/elasticsearch
2 participants
@n1v0lg @elasticsearchmachine