Fixed bundle run for apps when compute is already starting (#3478)

## Changes
It can happen that after creating app the compute state returned from
API is Starting already. Previously we incorrectly handled this and did
not wait for the app to start first

## Why
Fixes nightly tests

## Tests
<!-- How have you tested the changes? -->

<!-- If your PR needs to be included in the release notes for next
release,
add a separate entry in NEXT_CHANGELOG.md as part of your PR. -->

Author: andrewnester

bundle/run/app.go

@@ -37,7 +37,7 @@ func (a *appRunner) Name() string {
 	return a.app.Name
 }
 
-func isAppStopped(app *apps.App) bool {
+func isAppComputeStopped(app *apps.App) bool {
 	return app.ComputeStatus == nil ||
 		(app.ComputeStatus.State == apps.ComputeStateStopped || app.ComputeStatus.State == apps.ComputeStateError)
 }
@@ -70,13 +70,21 @@ func (a *appRunner) Run(ctx context.Context, opts *Options) (output.RunOutput, e
 	// 1. The app is new and was never deployed yet.
 	// 2. The app was stopped (compute not running).
 	// We need to start the app only if the compute is not running.
-	if isAppStopped(createdApp) {
+	if isAppComputeStopped(createdApp) {
 		err := a.start(ctx)
 		if err != nil {
 			return nil, err
 		}
 	}
 
+	// If the app is starting, we need to wait for it to be active before we can deploy it.
+	if isAppComputeStarting(createdApp) {
+		_, err := w.Apps.WaitGetAppActive(ctx, app.Name, 20*time.Minute, nil)
+		if err != nil {
+			return nil, err
+		}
+	}
+
 	// Deploy the app.
 	err = a.deploy(ctx)
 	if err != nil {
@@ -87,6 +95,10 @@ func (a *appRunner) Run(ctx context.Context, opts *Options) (output.RunOutput, e
 	return nil, nil
 }
 
+func isAppComputeStarting(app *apps.App) bool {
+	return app.ComputeStatus != nil && app.ComputeStatus.State == apps.ComputeStateStarting
+}
+
 func (a *appRunner) start(ctx context.Context) error {
 	app := a.app
 	b := a.bundle