Skip to content

Commit 8e47e0d

Browse files
authored
Jphenow/different reactions (#4706)
* Only guess that we can resolve prebuilt from deployers * trace the error regardless
1 parent 8702b3b commit 8e47e0d

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

internal/command/deploy/deploy_build.go

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import (
44
"context"
55
"errors"
66
"fmt"
7+
"os"
78
"path/filepath"
89

910
"github.com/dustin/go-humanize"
@@ -146,12 +147,16 @@ func determineImage(ctx context.Context, app *flaps.App, appConfig *appconfig.Co
146147
img, err = resolver.ResolveReference(ctx, io, opts)
147148
if err != nil {
148149
tracing.RecordError(span, err, "failed to resolve reference for prebuilt docker image")
149-
img = &imgsrc.DeploymentImage{
150-
ID: imageRef,
151-
Tag: imageRef,
150+
if trigger := os.Getenv("DEPLOY_TRIGGER"); trigger == "" {
151+
return
152+
} else {
153+
img = &imgsrc.DeploymentImage{
154+
ID: imageRef,
155+
Tag: imageRef,
156+
}
157+
terminal.Debugf("Failed to resolve reference for prebuilt docker image, using imageRef %s: %v\n", img.String(), err)
158+
err = nil
152159
}
153-
terminal.Debugf("Failed to resolve reference for prebuilt docker image, using imageRef %s: %v\n", img.String(), err)
154-
err = nil
155160
}
156161

157162
span.AddEvent("using pre-built docker image")

0 commit comments

Comments
 (0)