Skip to content

Commit

Permalink
[RayService] Ignore deployments status to decide whether to deploy se…
Browse files Browse the repository at this point in the history
…rve application (ray-project#1014)

Ignore deployments status to decide whether to deploy serve application
  • Loading branch information
sihanwang41 authored Apr 13, 2023
1 parent ff08129 commit 874c45e
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions ray-operator/controllers/ray/rayservice_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -545,9 +545,6 @@ func (r *RayServiceReconciler) checkIfNeedSubmitServeDeployment(rayServiceInstan
} else if !utils.CompareJsonStruct(cachedServeConfig, rayServiceInstance.Spec.ServeDeploymentGraphSpec) {
shouldUpdate = true
reason = fmt.Sprintf("Current Serve config doesn't match cached Serve config for cluster %s with key %s", rayClusterInstance.Name, cacheKey)
} else if len(serveStatus.ServeStatuses) == 0 {
shouldUpdate = true
reason = fmt.Sprintf("No Serve deployments have started deploying for cluster %s with key %s", rayClusterInstance.Name, cacheKey)
}

r.Log.V(1).Info("shouldUpdate", "shouldUpdateServe", shouldUpdate, "reason", reason, "cachedServeConfig", cachedServeConfig, "current Serve config", rayServiceInstance.Spec.ServeDeploymentGraphSpec)
Expand Down

0 comments on commit 874c45e

Please sign in to comment.