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

Test failure: GetUserPrivilegesResponseTests.testSerializationForCurrentVersion #108285

Closed
edsavage opened this issue May 5, 2024 · 3 comments
Closed

Test failure: GetUserPrivilegesResponseTests.testSerializationForCurrentVersion #108285

edsavage opened this issue May 5, 2024 · 3 comments
Assignees
@slobodanadamovic
Labels
needs:risk Requires assignment of a risk label (low, medium, blocker) :Security/Security Security issues without another label Team:Security Meta label for security team >test-failure Triaged test failures from CI

Comments

@edsavage
Copy link
Contributor

edsavage commented May 5, 2024

CI Link

https://buildkite.com/elastic/elasticsearch-periodic-platform-support/builds/2633#018f4a59-f51e-476a-a6ac-a1aa31ac2646

Repro line

REPRODUCE WITH: ./gradlew ':x-pack:plugin:core:test' --tests "org.elasticsearch.xpack.core.security.action.user.GetUserPrivilegesResponseTests.testSerializationForCurrentVersion" -Dtests.seed=BA6487ACE905F3F5 -Dtests.locale=zh-Hant-HK -Dtests.timezone=Asia/Aqtau -Druntime.java=21

Does it reproduce?

Yes

Applicable branches

main

Failure history

No response

Failure excerpt

org.elasticsearch.xpack.core.security.action.user.GetUserPrivilegesResponseTests > testSerializationForCurrentVersion FAILED
java.lang.AssertionError:
Expected: org.elasticsearch.xpack.core.security.action.user.GetUserPrivilegesResponse@8f05791f
but: was org.elasticsearch.xpack.core.security.action.user.GetUserPrivilegesResponse@6db43f9
at __randomizedtesting.SeedInfo.seed([BA6487ACE905F3F5:7D3BD42FD3942A82]:0)
at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:6)
at org.elasticsearch.test.ESTestCase.assertThat(ESTestCase.java:2160)
at org.elasticsearch.xpack.core.security.action.user.GetUserPrivilegesResponseTests.testSerializationForCurrentVersion(GetUserPrivilegesResponseTests.java:84)
@edsavage edsavage added >test-failure Triaged test failures from CI :Security/Security Security issues without another label needs:triage Requires assignment of a team area label labels May 5, 2024
@elasticsearchmachine elasticsearchmachine added Team:Security Meta label for security team needs:risk Requires assignment of a risk label (low, medium, blocker) and removed needs:triage Requires assignment of a team area label labels May 5, 2024
@elasticsearchmachine
Copy link
Collaborator

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

@edsavage edsavage mentioned this issue May 5, 2024
Merged
@slobodanadamovic slobodanadamovic mentioned this issue May 6, 2024
Merged
@slobodanadamovic slobodanadamovic self-assigned this May 6, 2024
@slobodanadamovic
Copy link
Contributor

Will be fixed by: #108255

elasticsearchmachine pushed a commit that referenced this issue May 6, 2024
@slobodanadamovic
Fix role descriptor's hashCode, equals and isEmpty implementations (#…
0378a77 
…108255)

Added missing `remoteClusterPermissions` to `hashCode`, `equals` and
`isEmpty` implementations.

Resolves #108253,
#108285

Note: Marking as `>non-issue` since the PR that introduced
`remoteClusterPermissions` is not released yet.
@slobodanadamovic
Copy link
Contributor

Fixed by #108255

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@slobodanadamovic slobodanadamovic

Labels
needs:risk Requires assignment of a risk label (low, medium, blocker) :Security/Security Security issues without another label Team:Security Meta label for security team >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@slobodanadamovic @edsavage @elasticsearchmachine