Skip to content

Commit d6ffeea

Browse files
heschigopherbot
authored andcommitted
internal/relui: write to "ready" file, fix signing arg
Simply opening and closing a file isn't enough to create it. Also, don't use path.Join on URLs: it converts gs://foo to gs:/foo which is a very different URL. For golang/go#51797. Change-Id: Ic681e151f2a2258f93e055564589d4440358e954 Reviewed-on: https://go-review.googlesource.com/c/build/+/412390 Auto-Submit: Heschi Kreinick <[email protected]> Run-TryBot: Heschi Kreinick <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]> TryBot-Result: Gopher Robot <[email protected]>
1 parent 3062ce6 commit d6ffeea

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

internal/relui/workflows.go

+4-1
Original file line numberDiff line numberDiff line change
@@ -571,7 +571,7 @@ func (w *sizeWriter) Write(p []byte) (n int, err error) {
571571
}
572572

573573
func (tasks *BuildReleaseTasks) startSigningCommand(ctx *workflow.TaskContext, version string) (string, error) {
574-
args := fmt.Sprintf("--relui_staging=%q", path.Join(tasks.ScratchURL, signingStagingDir(ctx, version)))
574+
args := fmt.Sprintf("--relui_staging=%q", tasks.ScratchURL+"/"+signingStagingDir(ctx, version))
575575
ctx.Printf("run signer with " + args)
576576
return args, nil
577577
}
@@ -614,6 +614,9 @@ func (tasks *BuildReleaseTasks) copyToStaging(ctx *workflow.TaskContext, version
614614
if err != nil {
615615
return nil, err
616616
}
617+
if _, err := out.Write([]byte("ready")); err != nil {
618+
return nil, err
619+
}
617620
if err := out.Close(); err != nil {
618621
return nil, err
619622
}

0 commit comments

Comments
 (0)