fix(application): syncAppFromRelease panic when helm list failed (#1702)

Co-authored-by: xdonggao <[email protected]>
tkestack · Dec 10, 2021 · b9e45bd · b9e45bd
1 parent 4647d5d
commit b9e45bd
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/pkg/application/controller/app/app_controller.go b/pkg/application/controller/app/app_controller.go
@@ -357,7 +357,11 @@ func (c *Controller) syncAppFromRelease(ctx context.Context, cachedApp *cachedAp
  if app.Status.Phase == applicationv1.AppPhaseSucceeded && hasSynced(app) {
  return app, nil
  }
-
+ defer func() {
+ if r := recover(); r != nil {
+ log.Error("syncAppFromRelease panic")
+ }
+ }()
  newStatus := app.Status.DeepCopy()
  rels, err := action.List(ctx, c.client.ApplicationV1(), c.platformClient, app)
  if err != nil {