fix(application): rm app phase chart fetched (#1763)

Co-authored-by: xdonggao <[email protected]>
tkestack · Feb 10, 2022 · 97c8d80 · 97c8d80
1 parent 2419eb3
commit 97c8d80
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 16 deletions.
diff --git a/api/application/types.go b/api/application/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/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: