fix(application): rm app phase chart fetched

tkestack · Feb 9, 2022 · 7e8a866 · 7e8a866
1 parent be8d413
commit 7e8a866
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 13 deletions.
diff --git a/api/application/v1/types.go b/api/application/v1/types.go
@@ -226,9 +226,6 @@ const (
 type AppPhase string
 
 const (
-	// ChartFetched means the chart to which the App refers
-	// has been fetched successfully
-	AppPhaseChartFetched AppPhase = "ChartFetched"
 	// ChartFetchedFailed means the chart to which the App
 	// refers could not be fetched.
 	AppPhaseChartFetchFailed AppPhase = "ChartFetchFailed"

diff --git a/pkg/application/controller/app/action/pull.go b/pkg/application/controller/app/action/pull.go
@@ -61,14 +61,6 @@ func Pull(ctx context.Context,
 			updateStatusFunc(ctx, app, &app.Status, newStatus)
 			return destfile, err
 		}
-		newStatus.Phase = applicationv1.AppPhaseChartFetched
-		newStatus.Message = ""
-		newStatus.Reason = ""
-		newStatus.LastTransitionTime = metav1.Now()
-		_, err := updateStatusFunc(ctx, app, &app.Status, newStatus)
-		if err != nil {
-			return destfile, err
-		}
 	}
 	return destfile, err
 }
diff --git a/pkg/application/controller/app/app_controller.go b/pkg/application/controller/app/app_controller.go
@@ -341,8 +341,6 @@ func (c *Controller) handlePhase(ctx context.Context, key string, cachedApp *cac
 		return c.syncAppFromRelease(ctx, cachedApp, app)
 	case applicationv1.AppPhaseRollbackFailed:
 		break
-	case applicationv1.AppPhaseChartFetched:
-		break
 	case applicationv1.AppPhaseChartFetchFailed:
 		break
 	case applicationv1.AppPhaseSyncFailed: