Fix unspecified model_observability parameter on sdk

caraml-dev · Dec 2, 2024 · af2e8b6 · af2e8b6
1 parent c2bcce2
commit af2e8b6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/api/models/service.go b/api/models/service.go
@@ -84,7 +84,7 @@ func NewService(model *Model, version *Version, modelOpt *ModelOption, endpoint
 		AutoscalingPolicy:           endpoint.AutoscalingPolicy,
 		Protocol:                    endpoint.Protocol,
 		CurrentIsvcName:             endpoint.InferenceServiceName,
-		EnabledModelObservability:   endpoint.EnableModelObservability,
+		EnabledModelObservability:   endpoint.IsModelMonitoringEnabled(),
 		ModelSchema:                 version.ModelSchema,
 		PredictorUPIOverHTTPEnabled: predictorUPIOverHTTPEnabled(endpoint.Transformer, endpoint.Protocol),
 	}

diff --git a/python/sdk/merlin/fluent.py b/python/sdk/merlin/fluent.py
@@ -418,6 +418,7 @@ def deploy(
             autoscaling_policy,
             protocol,
             enable_model_observability,
+            model_observability
         )
 
     return _merlin_client.deploy(  # type: ignore
@@ -432,6 +433,7 @@ def deploy(
         autoscaling_policy,
         protocol,
         enable_model_observability,
+        model_observability
     )