Deprecated ModelDeploymentProperties, ModelDeployer and updated ModelDeployment class.

Author: lu-ohai

ads/model/deployment/model_deployer.py

@@ -51,6 +51,16 @@
 from .model_deployment import DEFAULT_POLL_INTERVAL, DEFAULT_WAIT_TIME, ModelDeployment
 from .model_deployment_properties import ModelDeploymentProperties
 
+warnings.warn(
+    (
+        "The `ads.model.deployment.model_deployer` is deprecated in `oracle-ads 2.8.7` and will be removed in `oracle-ads 3.0`."
+        "Use `ModelDeployment` class in `ads.model.deployment` module for initializing and deploy model deployment. "
+        "Check https://accelerated-data-science.readthedocs.io/en/latest/user_guide/model_registration/introduction.html"
+    ),
+    DeprecationWarning,
+    stacklevel=2,
+)
+
 
 class ModelDeployer:
     """ModelDeployer is the class responsible for deploying the ModelDeployment
@@ -96,7 +106,7 @@ def __init__(self, config: dict = None, ds_client: DataScienceClient = None):
         """
         if config:
             warnings.warn(
-                "`config` will be deprecated in 3.0.0 and will be ignored now. Please use `ads.set_auth()` to config the auth information."
+                "`config` will be removed in 3.0.0 and will be ignored now. Please use `ads.set_auth()` to config the auth information."
             )
 
         self.ds_client = None

ads/model/deployment/model_deployment.py

@@ -70,6 +70,11 @@
 MODEL_DEPLOYMENT_INSTANCE_COUNT = 1
 MODEL_DEPLOYMENT_BANDWIDTH_MBPS = 10
 
+MODEL_DEPLOYMENT_RUNTIMES = {
+    ModelDeploymentRuntimeType.CONDA: ModelDeploymentCondaRuntime,
+    ModelDeploymentRuntimeType.CONTAINER: ModelDeploymentContainerRuntime,
+}
+
 
 class ModelDeploymentLogType:
     PREDICT = "predict"
@@ -292,17 +297,17 @@ def __init__(
 
         if config:
             warnings.warn(
-                "`config` will be deprecated in 3.0.0 and will be ignored now. Please use `ads.set_auth()` to config the auth information."
+                "Parameter `config` will be removed from ModelDeployment constructor in 3.0.0 and will be ignored now. Please use `ads.set_auth()` to config the auth information."
             )
 
         if properties:
             warnings.warn(
-                "`properties` will be deprecated in 3.0.0. Please use `spec` or the builder pattern to initialize model deployment instance."
+                "Parameter `properties` will be removed from ModelDeployment constructor in 3.0.0. Please use `spec` or the builder pattern to initialize model deployment instance."
             )
 
         if model_deployment_url or model_deployment_id:
             warnings.warn(
-                "`model_deployment_url` and `model_deployment_id` will be deprecated in 3.0.0 and will be ignored now. These two fields will be auto-populated from the service side."
+                "Parameter `model_deployment_url` and `model_deployment_id` will be removed from ModelDeployment constructor in 3.0.0 and will be ignored now. These two fields will be auto-populated from the service side."
             )
 
         initialize_spec = {}
@@ -694,6 +699,11 @@ def update(
         ModelDeployment
             The instance of ModelDeployment.
         """
+        if properties:
+            warnings.warn(
+                "Parameter `properties` will be removed from ModelDeployment update() in 3.0.0. Please use the builder pattern to update model deployment instance."
+            )
+
         updated_properties = properties
         if not isinstance(properties, ModelDeploymentProperties):
             updated_properties = ModelDeploymentProperties(
@@ -703,7 +713,7 @@ def update(
         update_model_deployment_details = (
             updated_properties.to_update_deployment()
             if properties or updated_properties.oci_model_deployment or kwargs
-            else self._update_model_deployment_details()
+            else self._update_model_deployment_details(**kwargs)
         )
 
         response = self.dsc_model_deployment.update(
@@ -1487,7 +1497,7 @@ def _build_model_deployment_details(self) -> CreateModelDeploymentDetails:
             **create_model_deployment_details
         ).to_oci_model(CreateModelDeploymentDetails)
 
-    def _update_model_deployment_details(self) -> UpdateModelDeploymentDetails:
+    def _update_model_deployment_details(self, **kwargs) -> UpdateModelDeploymentDetails:
         """Builds UpdateModelDeploymentDetails from model deployment instance.
 
         Returns
@@ -1499,7 +1509,7 @@ def _update_model_deployment_details(self) -> UpdateModelDeploymentDetails:
             raise ValueError(
                 "Missing parameter runtime or infrastructure. Try reruning it after parameters are fully configured."
             )
-
+        self._update_spec(**kwargs)
         update_model_deployment_details = {
             self.CONST_DISPLAY_NAME: self.display_name,
             self.CONST_DESCRIPTION: self.description,
@@ -1511,6 +1521,65 @@ def _update_model_deployment_details(self) -> UpdateModelDeploymentDetails:
         return OCIDataScienceModelDeployment(
             **update_model_deployment_details
         ).to_oci_model(UpdateModelDeploymentDetails)
+    
+    def _update_spec(self, **kwargs) -> "ModelDeployment":
+        """Updates model deployment specs from kwargs.
+
+        Parameters
+        ----------
+        kwargs:
+            display_name: (str)
+                Model deployment display name
+            description: (str)
+                Model deployment description
+            freeform_tags: (dict)
+                Model deployment freeform tags
+            defined_tags: (dict)
+                Model deployment defined tags
+            
+            Additional kwargs arguments.
+            Can be any attribute that `ads.model.deployment.ModelDeploymentCondaRuntime`, `ads.model.deployment.ModelDeploymentContainerRuntime`
+            and `ads.model.deployment.ModelDeploymentInfrastructure` accepts.
+
+        Returns
+        -------
+        ModelDeployment
+            The instance of ModelDeployment.
+        """
+        if not kwargs:
+            return self
+
+        converted_specs = ads_utils.batch_convert_case(kwargs, "camel")
+        specs = {
+            "self": self._spec,
+            "runtime": self.runtime._spec,
+            "infrastructure": self.infrastructure._spec
+        }
+        sub_set = {
+            self.infrastructure.CONST_ACCESS_LOG,
+            self.infrastructure.CONST_PREDICT_LOG,
+            self.infrastructure.CONST_SHAPE_CONFIG_DETAILS
+        }
+        for spec_value in specs.values():
+            for key in spec_value:
+                if key in converted_specs:
+                    if key in sub_set:
+                        for sub_key in converted_specs[key]:
+                            converted_sub_key = ads_utils.snake_to_camel(sub_key)
+                            spec_value[key][converted_sub_key] = converted_specs[key][sub_key]
+                    else:
+                        spec_value[key] = copy.deepcopy(converted_specs[key])
+        self = (
+            ModelDeployment(spec=specs["self"])
+            .with_runtime(
+                MODEL_DEPLOYMENT_RUNTIMES[self.runtime.type](spec=specs["runtime"])
+            )
+            .with_infrastructure(
+                ModelDeploymentInfrastructure(spec=specs["infrastructure"])
+            )
+        )
+
+        return self
 
     def _build_model_deployment_configuration_details(self) -> Dict:
         """Builds model deployment configuration details from model deployment instance.

ads/model/deployment/model_deployment_properties.py

@@ -12,6 +12,18 @@
 
 from .common.utils import OCIClientManager
 
+import warnings
+
+warnings.warn(
+    (
+        "The `ads.model.deployment.model_deployment_properties` is deprecated in `oracle-ads 2.8.7` and will be removed in `oracle-ads 3.0`."
+        "Use `ModelDeploymentInfrastructure` and `ModelDeploymentRuntime` classes in `ads.model.deployment` module for configuring model deployment. "
+        "Check https://accelerated-data-science.readthedocs.io/en/latest/user_guide/model_registration/introduction.html"
+    ),
+    DeprecationWarning,
+    stacklevel=2,
+)
+
 
 class ModelDeploymentProperties(
     OCIDataScienceMixin, data_science_models.ModelDeployment

ads/model/generic_model.py

@@ -10,6 +10,7 @@
 import tempfile
 from enum import Enum
 from typing import Any, Callable, Dict, List, Optional, Tuple, Type, TypeVar, Union
+import warnings
 
 import numpy as np
 import pandas as pd
@@ -44,7 +45,6 @@
 from ads.model.deployment import (
     DEFAULT_POLL_INTERVAL,
     DEFAULT_WAIT_TIME,
-    ModelDeployer,
     ModelDeployment,
     ModelDeploymentMode,
     ModelDeploymentProperties,
@@ -1565,9 +1565,7 @@ def from_model_deployment(
                 "Only SparkPipelineModel framework supports object storage path as `artifact_dir`."
             )
 
-        model_deployment = ModelDeployer(config=auth).get_model_deployment(
-            model_deployment_id=model_deployment_id
-        )
+        model_deployment = ModelDeployment.from_id(model_deployment_id)
 
         current_state = model_deployment.state.name.upper()
         if current_state != ModelDeploymentState.ACTIVE.name:
@@ -1618,13 +1616,22 @@ def update_deployment(
         Examples
         --------
         >>> # Update access log id, freeform tags and description for the model deployment
+        >>> # Deprecated
         >>> model.update_deployment(
         >>>     properties=ModelDeploymentProperties(
         >>>         access_log_id=<log_ocid>,
         >>>         description="Description for Custom Model",
         >>>         freeform_tags={"key": "value"},
         >>>     )
         >>> )
+        >>> # New way to update access log id, freeform tags and description for the model deployment
+        >>> model.update_deployment(
+        ...     access_log={
+        ...         log_id=<log_ocid>
+        ...     },
+        ...     description="Description for Custom Model",
+        ...     freeform_tags={"key": "value"},
+        ... )
 
         Parameters
         ----------
@@ -1647,12 +1654,28 @@ def update_deployment(
                 If you need to override the default, use the `ads.common.auth.api_keys` or
                 `ads.common.auth.resource_principal` to create appropriate authentication signer
                 and kwargs required to instantiate IdentityClient object.
+            display_name: (str)
+                Model deployment display name
+            description: (str)
+                Model deployment description
+            freeform_tags: (dict)
+                Model deployment freeform tags
+            defined_tags: (dict)
+                Model deployment defined tags
+            
+            Additional kwargs arguments.
+            Can be any attribute that `ads.model.deployment.ModelDeploymentCondaRuntime`, `ads.model.deployment.ModelDeploymentContainerRuntime`
+            and `ads.model.deployment.ModelDeploymentInfrastructure` accepts.
 
         Returns
         -------
         ModelDeployment
             An instance of ModelDeployment class.
         """
+        if properties:
+            warnings.warn(
+                "Parameter `properties` will be removed from GenericModel `update_deployment()` in 3.0.0. Please use kwargs to update model deployment."
+            )
         if not inspect.isclass(cls):
             if cls.model_deployment:
                 return cls.model_deployment.update(
@@ -1666,10 +1689,7 @@ def update_deployment(
         if not model_deployment_id:
             raise ValueError("Parameter `model_deployment_id` must be provided.")
 
-        auth = kwargs.pop("auth", authutil.default_signer())
-        model_deployment = ModelDeployer(config=auth).get_model_deployment(
-            model_deployment_id=model_deployment_id
-        )
+        model_deployment = ModelDeployment.from_id(model_deployment_id)
         return model_deployment.update(
             properties=properties,
             wait_for_completion=wait_for_completion,

tests/unitary/default_setup/model_deployment/test_model_deployment_v2.py

@@ -1389,3 +1389,47 @@ def test_model_deployment_with_subnet_id(self):
 
         model_deployment.infrastructure.with_subnet_id("test_id")
         assert model_deployment.infrastructure.subnet_id == "test_id"
+
+    def test_update_spec(self):
+        model_deployment = self.initialize_model_deployment()
+        model_deployment._update_spec(
+            display_name="test_updated_name",
+            freeform_tags={"test_updated_key":"test_updated_value"},
+            access_log={
+                "log_id": "test_updated_access_log_id"
+            },
+            predict_log={
+                "log_group_id": "test_updated_predict_log_group_id"
+            },
+            shape_config_details={
+                "ocpus": 100,
+                "memoryInGBs": 200
+            },
+            replica=20,
+            image="test_updated_image",
+            env={
+                "test_updated_env_key":"test_updated_env_value"
+            }
+        )
+
+        assert model_deployment.display_name == "test_updated_name"
+        assert model_deployment.freeform_tags == {
+            "test_updated_key":"test_updated_value"
+        }
+        assert model_deployment.infrastructure.access_log == {
+            "logId": "test_updated_access_log_id",
+            "logGroupId": "fakeid.loggroup.oc1.iad.xxx"
+        }
+        assert model_deployment.infrastructure.predict_log == {
+            "logId": "fakeid.log.oc1.iad.xxx",
+            "logGroupId": "test_updated_predict_log_group_id"
+        }
+        assert model_deployment.infrastructure.shape_config_details == {
+            "ocpus": 100,
+            "memoryInGBs": 200
+        }
+        assert model_deployment.infrastructure.replica == 20
+        assert model_deployment.runtime.image == "test_updated_image"
+        assert model_deployment.runtime.env == {
+            "test_updated_env_key":"test_updated_env_value"
+        }