Remove logic to copy config files to user bucket

Author: VipulMascarenhas

ads/aqua/model/model.py

@@ -29,7 +29,6 @@
     LifecycleStatus,
     _build_resource_identifier,
     cleanup_local_hf_model_artifact,
-    copy_model_config,
     create_word_icon,
     generate_tei_cmd_var,
     get_artifact_path,
@@ -969,24 +968,6 @@ def _create_model_catalog_entry(
                 )
                 tags[Tags.LICENSE] = validation_result.tags.get(Tags.LICENSE, UNKNOWN)
 
-        try:
-            # If verified model already has a artifact json, use that.
-            artifact_path = metadata.get(MODEL_BY_REFERENCE_OSS_PATH_KEY).value
-            logger.info(
-                f"Found model artifact in the service bucket. "
-                f"Using artifact from service bucket instead of {os_path}."
-            )
-
-            # todo: implement generic copy_folder method
-            # copy model config from artifact path to user bucket
-            copy_model_config(
-                artifact_path=artifact_path, os_path=os_path, auth=default_signer()
-            )
-        except Exception:
-            logger.debug(
-                f"Proceeding with model registration without copying model config files at {os_path}. "
-                f"Default configuration will be used for deployment and fine-tuning."
-            )
         # Set artifact location to user bucket, and replace existing key if present.
         metadata.add(
             key=MODEL_BY_REFERENCE_OSS_PATH_KEY,

tests/unitary/with_extras/aqua/test_model.py

@@ -665,7 +665,6 @@ def test_get_model_fine_tuned(
     @patch("ads.model.datascience_model.DataScienceModel.sync")
     @patch("ads.model.datascience_model.DataScienceModel.upload_artifact")
     @patch.object(AquaModelApp, "_find_matching_aqua_model")
-    @patch("ads.aqua.common.utils.copy_file")
     @patch("ads.common.object_storage_details.ObjectStorageDetails.list_objects")
     @patch("ads.aqua.common.utils.load_config", return_value={})
     @patch("huggingface_hub.snapshot_download")
@@ -676,7 +675,6 @@ def test_import_verified_model(
         mock_snapshot_download,
         mock_load_config,
         mock_list_objects,
-        mock_copy_file,
         mock__find_matching_aqua_model,
         mock_upload_artifact,
         mock_sync,
@@ -788,8 +786,6 @@ def test_import_verified_model(
             mock_subprocess.assert_not_called()
             mock_load_config.assert_called()
 
-        if not artifact_location_set:
-            mock_copy_file.assert_called()
         ds_freeform_tags.pop(
             "ready_to_import"
         )  # The imported model should not have this tag