Skip to content

Commit 01afe52

Browse files
authored
Merge pull request docker#10527 from ndeloof/build_secret_id
let user declare build secret target
2 parents e88836f + b45ca82 commit 01afe52

File tree

3 files changed

+9
-4
lines changed

3 files changed

+9
-4
lines changed

pkg/compose/build.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -373,15 +373,19 @@ func addSecretsConfig(project *types.Project, service types.ServiceConfig) (sess
373373
var sources []secretsprovider.Source
374374
for _, secret := range service.Build.Secrets {
375375
config := project.Secrets[secret.Source]
376+
id := secret.Source
377+
if secret.Target != "" {
378+
id = secret.Target
379+
}
376380
switch {
377381
case config.File != "":
378382
sources = append(sources, secretsprovider.Source{
379-
ID: secret.Source,
383+
ID: id,
380384
FilePath: config.File,
381385
})
382386
case config.Environment != "":
383387
sources = append(sources, secretsprovider.Source{
384-
ID: secret.Source,
388+
ID: id,
385389
Env: config.Environment,
386390
})
387391
default:

pkg/e2e/fixtures/build-test/secrets/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,5 +22,5 @@ RUN --mount=type=secret,id=mysecret cat /run/secrets/mysecret > /tmp/actual
2222
RUN diff /tmp/expected /tmp/actual
2323

2424
RUN echo "bar" > /tmp/expected
25-
RUN --mount=type=secret,id=envsecret cat /run/secrets/envsecret > tmp/actual
25+
RUN --mount=type=secret,id=build_secret cat /run/secrets/build_secret > tmp/actual
2626
RUN diff --ignore-all-space /tmp/expected /tmp/actual

pkg/e2e/fixtures/build-test/secrets/compose.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,8 @@ services:
55
context: .
66
secrets:
77
- mysecret
8-
- envsecret
8+
- source: envsecret
9+
target: build_secret
910

1011
secrets:
1112
mysecret:

0 commit comments

Comments
 (0)