Fix UX issue when exec command fails to run as part of a deploy comma…

…nd (#6673)

For the context, before the changes, the "Execution output" message
was displayed on the same line as the spinner, e.g.:

```
↪ Executing command:
◓  Executing command in container (command: exec-deploy)Execution output:
/bin/sh: line 1: helm: command not found
 ✗  Executing command in container (command: exec-deploy) [8s]
 ✗  failed to execute (command: exec-deploy)
```

This is now fixed by ending the spinner as soon as possible,
and then displaying the command output:

```
↪ Executing command:
 ✗  Executing command in container (command: exec-deploy) [8s]
Execution output:
/bin/sh: line 1: helm: command not found
 ✗  failed to execute (command: exec-deploy)

```

pkg/deploy/deploy.go

@@ -192,6 +192,9 @@ func (o *deployHandler) Execute(command v1alpha2.Command) error {
 	// Wait for the command to complete execution
 	_, err = o.kubeClient.WaitForJobToComplete(createdJob)
 	done <- struct{}{}
+
+	spinner.End(err == nil)
+
 	if err != nil {
 		err = fmt.Errorf("failed to execute (command: %s)", command.Id)
 		// Print the job logs if the job failed
@@ -203,8 +206,6 @@ func (o *deployHandler) Execute(command v1alpha2.Command) error {
 		_ = util.DisplayLog(false, jobLogs, log.GetStderr(), o.componentName, 100)
 	}
 
-	spinner.End(err == nil)
-
 	return err
 }