Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 0 additions & 3 deletions muted-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -301,9 +301,6 @@ tests:
- class: org.elasticsearch.index.engine.ThreadPoolMergeSchedulerTests
method: testSchedulerCloseWaitsForRunningMerge
issue: https://github.com/elastic/elasticsearch/issues/125236
- class: org.elasticsearch.xpack.security.SecurityRolesMultiProjectIT
method: testUpdatingFileBasedRoleAffectsAllProjects
issue: https://github.com/elastic/elasticsearch/issues/126223
- class: org.elasticsearch.packaging.test.DockerTests
method: test020PluginsListWithNoPlugins
issue: https://github.com/elastic/elasticsearch/issues/126232
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@
import java.io.IOException;
import java.util.Collection;
import java.util.Map;
import java.util.concurrent.TimeUnit;

import static org.hamcrest.Matchers.contains;
import static org.hamcrest.Matchers.containsInAnyOrder;
Expand Down Expand Up @@ -132,14 +133,15 @@ public void testUpdatingFileBasedRoleAffectsAllProjects() throws Exception {
assertBusy(() -> {
assertThat(getClusterPrivileges(project1, username1), contains("monitor"));
assertThat(getClusterPrivileges(project2, username2), contains("monitor"));
});
}, 20, TimeUnit.SECONDS); // increasing this to try and solve for a rare failure

rolesFile.update(Resource.fromString(""));

assertBusy(() -> {
// Both projects should automatically reflect that the role has been removed
assertThat(getClusterPrivileges(project1, username1), empty());
assertThat(getClusterPrivileges(project2, username2), empty());
});
}, 20, TimeUnit.SECONDS);
}

private void createUser(ProjectId projectId, String username, String roleName) throws IOException {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,10 @@ public SmbTestContainer() {
new ImageFromDockerfile("es-smb-fixture").withDockerfileFromBuilder(
builder -> builder.from(DOCKER_BASE_IMAGE)
.env("TZ", "Etc/UTC")
.run("DEBIAN_FRONTEND=noninteractive apt-get update -qqy && apt-get install -qqy tzdata winbind samba ldap-utils")
.run(
"DEBIAN_FRONTEND=noninteractive apt-get -o Acquire::Retries=10 update -qqy && apt-get install -qqy tzdata"
+ " winbind samba ldap-utils "
)
.copy("fixture/provision/installsmb.sh", "/fixture/provision/installsmb.sh")
.copy("fixture/certs/ca.key", "/fixture/certs/ca.key")
.copy("fixture/certs/ca.pem", "/fixture/certs/ca.pem")
Expand Down