Skip to content

Commit 20a9c7d

Browse files
committed
Surface embedded cluster upgrade failures on panics
1 parent b51214b commit 20a9c7d

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

pkg/upgradeservice/deploy/deploy.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -119,6 +119,9 @@ func Deploy(opts DeployOptions) error {
119119

120120
go func() (finalError error) {
121121
defer func() {
122+
if r := recover(); r != nil {
123+
finalError = fmt.Errorf("recovered from panic during cluster upgrade: %v", r)
124+
}
122125
if finalError != nil {
123126
logger.Error(errors.Wrap(finalError, "failed to deploy"))
124127
if err := task.SetStatusUpgradeFailed(opts.Params.AppSlug, finalError.Error()); err != nil {

0 commit comments

Comments
 (0)