Removes default 30-minute timeout from gh-trigger-workflow (gravita…

…tional#24178)

The `gh-trigger-workflow` will now default to waiting forever for a
GitHub Workflow to complete. A timeout can be optionally specified
if desired.

build.assets/tooling/cmd/gh-trigger-workflow/args.go

@@ -93,7 +93,7 @@ func parseCommandLine() (args, error) {
 	flag.StringVar(&cliArgs.workflow, "workflow", "", "Path to workflow")
 	flag.StringVar(&cliArgs.workflowRef, "workflow-ref", cliArgs.workflowRef, "Revision reference")
 	flag.BoolVar(&cliArgs.useWorkflowTag, "tag-workflow", false, "Use a workflow input to tag and ID workflows spawned by the event")
-	flag.DurationVar(&cliArgs.timeout, "timeout", 30*time.Minute, "Timeout")
+	flag.DurationVar(&cliArgs.timeout, "timeout", time.Duration(0), "Timeout. If not specified, waits forever.")
 	flag.Var(cliArgs.inputs, "input", "Input to target workflow")
 
 	flag.Parse()

build.assets/tooling/cmd/gh-trigger-workflow/main.go

@@ -69,8 +69,14 @@ func main() {
 		log.Fatal(err.Error())
 	}
 
-	ctx, cancel := context.WithTimeout(context.Background(), args.timeout)
-	defer cancel()
+	ctx := context.Background()
+	if args.timeout != 0 {
+		log.Printf("Setting %v timeout", args.timeout)
+
+		var cancel context.CancelFunc
+		ctx, cancel = context.WithTimeout(ctx, args.timeout)
+		defer cancel()
+	}
 
 	installationID, err := lookupInstallationID(ctx, args)
 	if err != nil {