Skip to content

Commit

Permalink
get hasSeparateEnginePod from boolean annotations instead of existence
Browse files Browse the repository at this point in the history
  • Loading branch information
chengchengpei committed Jul 1, 2020
1 parent b0ebe9e commit 76cbc8c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion operator/controllers/seldondeployment_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -396,7 +396,7 @@ func (r *SeldonDeploymentReconciler) createComponents(mlDep *machinelearningv1.S
pSvcName := machinelearningv1.GetPredictorKey(mlDep, &p)
log.Info("pSvcName", "val", pSvcName)
// Add engine deployment if separate
_, hasSeparateEnginePod := mlDep.Spec.Annotations[machinelearningv1.ANNOTATION_SEPARATE_ENGINE]
hasSeparateEnginePod := strings.ToLower(mlDep.Spec.Annotations[machinelearningv1.ANNOTATION_SEPARATE_ENGINE]) == "true"
if hasSeparateEnginePod && !noEngine {
deploy, err := createEngineDeployment(mlDep, &p, pSvcName, engine_http_port, engine_grpc_port)
if err != nil {
Expand Down

0 comments on commit 76cbc8c

Please sign in to comment.