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

[rm3l]

What type of PR is this:
/kind bug
/area UX
/area deploy

What does this PR do / why we need it:
I stumbled upon this issue while reviewing #6672.

Before the changes, the Execution output: message was displayed on the same line as the spinner, like so:

↪ 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 the command is done, and then displaying the command output. Example 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)

Which issue(s) this PR fixes:
Related to #5701

PR acceptance criteria:

• Unit test

• Integration test

• Documentation

How to test changes / Special notes to the reviewer:

[netlify]

✅ Deploy Preview for odo-docusaurus-preview canceled.

Name	Link
🔨 Latest commit	aa675ae
🔍 Latest deploy log	https://app.netlify.com/sites/odo-docusaurus-preview/deploys/641b2aaa2a01710008fdc6a6

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[odo-robot]

OpenShift Unauthenticated Tests on commit finished successfully.
View logs: TXT HTML

[odo-robot]

NoCluster Tests on commit finished successfully.
View logs: TXT HTML

[odo-robot]

Unit Tests on commit finished successfully.
View logs: TXT HTML

[odo-robot]

Validate Tests on commit finished successfully.
View logs: TXT HTML

[odo-robot]

Kubernetes Tests on commit finished successfully.
View logs: TXT HTML

[odo-robot]

Windows Tests (OCP) on commit finished with errors.
View logs: TXT HTML

[odo-robot]

OpenShift Tests on commit finished successfully.
View logs: TXT HTML

[odo-robot]

Kubernetes Docs Tests on commit 47234b9 finished successfully.
View logs: TXT HTML

[rm3l]

  [FAILED] Expected
      <*url.Error | 0xc000439260>: {
          Op: "Post",
          URL: "http://127.0.0.1:51302/api/newuser",
          Err: <*errors.errorString | 0xc000088130>{s: "EOF"},
      }
  to be nil
  In [It] at: C:/Users/Administrator.ANSIBLE-TEST-VS/3457/tests/e2escenarios/e2e_test.go:306 @ 03/22/23 11:44:12.853

  There were additional failures detected.  To view them in detail run ginkgo -vv
------------------------------

Summarizing 1 Failure:
  [FAIL] E2E Test starting with non-empty Directory add Binding [It] should verify developer workflow of using binding as env in innerloop
  C:/Users/Administrator.ANSIBLE-TEST-VS/3457/tests/e2escenarios/e2e_test.go:306

Flaky E2E test - reported in #6582

/override windows-integration-test/Windows-test

[openshift-ci]

@rm3l: Overrode contexts on behalf of rm3l: windows-integration-test/Windows-test

In response to this:

 [FAILED] Expected
     <*url.Error | 0xc000439260>: {
         Op: "Post",
         URL: "http://127.0.0.1:51302/api/newuser",
         Err: <*errors.errorString | 0xc000088130>{s: "EOF"},
     }
 to be nil
 In [It] at: C:/Users/Administrator.ANSIBLE-TEST-VS/3457/tests/e2escenarios/e2e_test.go:306 @ 03/22/23 11:44:12.853

 There were additional failures detected.  To view them in detail run ginkgo -vv
------------------------------

Summarizing 1 Failure:
 [FAIL] E2E Test starting with non-empty Directory add Binding [It] should verify developer workflow of using binding as env in innerloop
 C:/Users/Administrator.ANSIBLE-TEST-VS/3457/tests/e2escenarios/e2e_test.go:306

Flaky E2E test - reported in #6582

/override windows-integration-test/Windows-test

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.