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] CompositeRolesStoreTests stalls indefinitely #37657

Closed
markharwood opened this issue Jan 21, 2019 · 4 comments
Closed

[CI] CompositeRolesStoreTests stalls indefinitely #37657

markharwood opened this issue Jan 21, 2019 · 4 comments
Assignees
@bizybot
Labels
:Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC >test-failure Triaged test failures from CI

Comments

@markharwood
Copy link
Contributor

Stalls in the testNegativeLookupsAreCached test.

Build: https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+intake/1392/console

Reproduced with

./gradlew :x-pack:plugin:security:unitTest -Dtests.seed=C0AE2ED0439D26F6 -Dtests.class=org.elasticsearch.xpack.security.authz.store.CompositeRolesStoreTests -Dtests.method="testNegativeLookupsAreCached" -Dtests.security.manager=true -Dtests.locale=ja-JP -Dtests.timezone=America/Recife -Dcompiler.java=11 -Druntime.java=8

Maybe related to recent change to test subject class ff0f540
@markharwood markharwood added >test-failure Triaged test failures from CI :Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC labels Jan 21, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security

@bizybot bizybot self-assigned this Jan 21, 2019
bizybot pushed a commit to bizybot/elasticsearch that referenced this issue Jan 21, 2019
Fix a test failure in CompositeRolesStoreTests
ba84dd2 
Due to missing stubbing for `NativePrivilegeStore#getPrivileges`
the test `testNegativeLookupsAreCached` failed
when the superuser role name was present in the role names.
This commit adds missing stubbing.

Closes: elastic#37657
@bizybot bizybot mentioned this issue Jan 21, 2019
Merged
@markharwood
Copy link
Contributor Author

Note happening on 6.x too: https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+6.x+intake/1089/console

markharwood added a commit that referenced this issue Jan 21, 2019
@markharwood
Mute test.
08341c7 
Relates #37657
@markharwood
Copy link
Contributor Author

Muted on 6.x in 08341c7

albertzaharovits added a commit that referenced this issue Jan 21, 2019
@albertzaharovits
Mute test.
f349372 
 Relates #37657
@albertzaharovits
Copy link
Contributor

Muted on master as well f349372

bizybot added a commit to bizybot/elasticsearch that referenced this issue Jan 21, 2019
@bizybot
Fix a test failure in CompositeRolesStoreTests (elastic#37661)
ca4b586 
Due to missing stubbing for `NativePrivilegeStore#getPrivileges`
the test `testNegativeLookupsAreCached` failed
when the superuser role name was present in the role names.
This commit adds missing stubbing.

Closes: elastic#37657
bizybot added a commit to bizybot/elasticsearch that referenced this issue Jan 21, 2019
@bizybot
Fix a test failure in CompositeRolesStoreTests (elastic#37661)
3d8c4a3 
Due to missing stubbing for `NativePrivilegeStore#getPrivileges`
the test `testNegativeLookupsAreCached` failed
when the superuser role name was present in the role names.
This commit adds missing stubbing.

Closes: elastic#37657
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bizybot bizybot

Labels
:Security/Authorization Roles, Privileges, DLS/FLS, RBAC/ABAC >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@markharwood @bizybot @albertzaharovits @elasticmachine