Skip to content

Commit e57efa7

Browse files
committed
Fix #34605: add docker-group-add
1 parent 5310631 commit e57efa7

2 files changed

Lines changed: 5 additions & 0 deletions

File tree

src/main/java/eu/openanalytics/containerproxy/backend/docker/DockerEngineBackend.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -186,6 +186,8 @@ public Proxy startContainer(Authentication user, Container initialContainer, Con
186186
.build());
187187
}
188188

189+
hostConfigBuilder.groupAdd(spec.getDockerGroupAdd().getValueOrNull());
190+
189191
ContainerConfig containerConfig = ContainerConfig.builder()
190192
.hostConfig(hostConfigBuilder.build())
191193
.image(spec.getImage().getValue())

src/main/java/eu/openanalytics/containerproxy/model/spec/ContainerSpec.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,8 @@ public class ContainerSpec {
8888
@Builder.Default
8989
private SpelField.String dockerIpc = new SpelField.String();
9090
@Builder.Default
91+
private SpelField.StringList dockerGroupAdd = new SpelField.StringList();
92+
@Builder.Default
9193
private List<DockerDeviceRequest> dockerDeviceRequests = new ArrayList<>();
9294

9395
@Builder.Default
@@ -134,6 +136,7 @@ public ContainerSpec firstResolve(SpecExpressionResolver resolver, SpecExpressio
134136
.dockerRuntime(dockerRuntime.resolve(resolver, context))
135137
.dockerUser(dockerUser.resolve(resolver, context))
136138
.dockerIpc(dockerIpc.resolve(resolver, context))
139+
.dockerGroupAdd(dockerGroupAdd.resolve(resolver, context))
137140
.build();
138141
}
139142

0 commit comments

Comments
 (0)