From 190858b9e4639586a08ab65f3cf6972de909cae9 Mon Sep 17 00:00:00 2001 From: mrinalirao Date: Mon, 20 Feb 2023 17:18:13 +1100 Subject: [PATCH] Bug Fix where CLI exits if run is not confirmable. --- internal/cloud/backend_taskStages.go | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/internal/cloud/backend_taskStages.go b/internal/cloud/backend_taskStages.go index 36dab7629a..bf5dc18895 100644 --- a/internal/cloud/backend_taskStages.go +++ b/internal/cloud/backend_taskStages.go @@ -97,8 +97,15 @@ func (b *Cloud) runTaskStage(ctx *IntegrationContext, output IntegrationOutputWr case tfe.TaskStagePending: // Waiting for it to start return true, nil + case tfe.TaskStageRunning: + _, e := processSummarizers(ctx, output, stage, summarizers, errs) + if e != nil { + errs = e + } + // not a terminal status so we continue to poll + return true, nil // Note: Terminal statuses need to print out one last time just in case - case tfe.TaskStageRunning, tfe.TaskStagePassed: + case tfe.TaskStagePassed: ok, e := processSummarizers(ctx, output, stage, summarizers, errs) if e != nil { errs = e @@ -176,6 +183,8 @@ func (b *Cloud) processStageOverrides(context *IntegrationContext, output Integr if err != errRunOverridden { if _, err = b.client.TaskStages.Override(context.StopContext, taskStageID, tfe.TaskStageOverrideOptions{}); err != nil { return false, generalError(fmt.Sprintf("Failed to override policy check.\n%s", runUrl), err) + } else { + return true, nil } } else { output.Output(fmt.Sprintf("The run needs to be manually overridden or discarded.\n%s\n", runUrl))