Skip to content
Merged
Show file tree
Hide file tree
Changes from 12 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
8 changes: 8 additions & 0 deletions docs/changelog/129295.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
pr: 129295
summary: Increase the number of retries for apt-get
area: Security
type: enhancement
issues:
- 124490
- 127738
- 126223
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,7 +133,8 @@ 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(() -> {
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
Loading