fix: restart running app on push

mogul · mogul · commit e2066806ce76 · 2025-11-17T10:48:15.000-08:00
diff --git a/operation/push.go b/operation/push.go
@@ -305,6 +305,7 @@ func (p *AppPushOperation) pushApp(ctx context.Context, space *resource.Space, m
 	if err != nil {
 		return nil, err
 	}
+	appWasStarted := app.State == "STARTED"
 
 	var pkg *resource.Package
 	// Check if lifecycle is explicitly set in manifest
@@ -342,9 +343,11 @@ func (p *AppPushOperation) pushApp(ctx context.Context, space *resource.Space, m
 
 	if p.stopped {
 		return p.client.Applications.Stop(ctx, app.GUID)
-	} else {
-		return p.client.Applications.Start(ctx, app.GUID)
 	}
+	if appWasStarted {
+		return p.client.Applications.Restart(ctx, app.GUID)
+	}
+	return p.client.Applications.Start(ctx, app.GUID)
 }
 
 func (p *AppPushOperation) applySpaceManifest(ctx context.Context, space *resource.Space, manifest *AppManifest) error {

Original file line number	Diff line number	Diff line change
`@@ -305,6 +305,7 @@ func (p AppPushOperation) pushApp(ctx context.Context, space resource.Space, m`
`305`	`305`	`if err != nil {`
`306`	`306`	`return nil, err`
`307`	`307`	`}`
	`308`	`+ appWasStarted := app.State == "STARTED"`
`308`	`309`
`309`	`310`	`var pkg *resource.Package`
`310`	`311`	`// Check if lifecycle is explicitly set in manifest`
`@@ -342,9 +343,11 @@ func (p AppPushOperation) pushApp(ctx context.Context, space resource.Space, m`
`342`	`343`
`343`	`344`	`if p.stopped {`
`344`	`345`	`return p.client.Applications.Stop(ctx, app.GUID)`
`345`		`- } else {`
`346`		`- return p.client.Applications.Start(ctx, app.GUID)`
`347`	`346`	`}`
	`347`	`+ if appWasStarted {`
	`348`	`+ return p.client.Applications.Restart(ctx, app.GUID)`
	`349`	`+ }`
	`350`	`+ return p.client.Applications.Start(ctx, app.GUID)`
`348`	`351`	`}`
`349`	`352`
`350`	`353`	`func (p AppPushOperation) applySpaceManifest(ctx context.Context, space resource.Space, manifest *AppManifest) error {`