[ML]: fix: Index Operation Fixes (#35434)

* fix: Re-add missing @properties decorator

* style: Run isort on ml_client.py

* feat: Set default Index.stage to Development

* fix: Format Index.path as a datastore uri instead of a storage uri

    This in support of the work done for build_index

Author: kdestin

sdk/ml/azure-ai-ml/azure/ai/ml/_ml_client.py

@@ -44,11 +44,11 @@
 from azure.ai.ml._scope_dependent_operations import OperationConfig, OperationsContainer, OperationScope
 from azure.ai.ml._telemetry.logging_handler import get_appinsights_log_handler
 from azure.ai.ml._user_agent import USER_AGENT
+from azure.ai.ml._utils._experimental import experimental
 from azure.ai.ml._utils._http_utils import HttpPipeline
 from azure.ai.ml._utils._preflight_utils import get_deployments_operation
 from azure.ai.ml._utils._registry_utils import get_registry_client
 from azure.ai.ml._utils.utils import _is_https_url
-from azure.ai.ml._utils._experimental import experimental
 from azure.ai.ml.constants._common import AzureMLResourceType, DefaultOpenEncoding
 from azure.ai.ml.entities import (
     BatchDeployment,
@@ -59,16 +59,16 @@
     Environment,
     Index,
     Job,
+    MarketplaceSubscription,
     Model,
     ModelBatchDeployment,
     OnlineDeployment,
     OnlineEndpoint,
     PipelineComponentBatchDeployment,
     Registry,
     Schedule,
-    Workspace,
     ServerlessEndpoint,
-    MarketplaceSubscription,
+    Workspace,
 )
 from azure.ai.ml.entities._assets import WorkspaceAssetReference
 from azure.ai.ml.exceptions import ErrorCategory, ErrorTarget, ValidationException
@@ -77,19 +77,19 @@
     BatchEndpointOperations,
     ComponentOperations,
     ComputeOperations,
+    ConnectionsOperations,
     DataOperations,
     DatastoreOperations,
     EnvironmentOperations,
     IndexOperations,
     JobOperations,
+    MarketplaceSubscriptionOperations,
     ModelOperations,
     OnlineDeploymentOperations,
     OnlineEndpointOperations,
     RegistryOperations,
-    ConnectionsOperations,
-    WorkspaceOperations,
     ServerlessEndpointOperations,
-    MarketplaceSubscriptionOperations,
+    WorkspaceOperations,
 )
 from azure.ai.ml.operations._code_operations import CodeOperations
 from azure.ai.ml.operations._feature_set_operations import FeatureSetOperations
@@ -1033,6 +1033,7 @@ def marketplace_subscriptions(self) -> MarketplaceSubscriptionOperations:
         """
         return self._marketplace_subscriptions
 
+    @property
     def indexes(self) -> IndexOperations:
         """A collection of index related operations.
 

sdk/ml/azure-ai-ml/azure/ai/ml/entities/_assets/_artifacts/index.py

@@ -6,8 +6,9 @@
 
 # cspell:disable-next-line
 from azure.ai.ml._restclient.azure_ai_assets_v2024_04_01.azureaiassetsv20240401.models import Index as RestIndex
-from azure.ai.ml._utils._arm_id_utils import AMLAssetId
+from azure.ai.ml._utils._arm_id_utils import AMLAssetId, AMLNamedArmId
 from azure.ai.ml._utils._experimental import experimental
+from azure.ai.ml.constants._common import LONG_URI_FORMAT
 from azure.ai.ml.entities._assets import Artifact
 from azure.ai.ml.entities._assets._artifacts.artifact import ArtifactStorageInfo
 from azure.ai.ml.entities._system_data import RestSystemData, SystemData
@@ -42,7 +43,7 @@ def __init__(
         *,
         name: str,
         version: str,
-        stage: str,
+        stage: str = "Development",
         description: Optional[str] = None,
         tags: Optional[Dict[str, str]] = None,
         properties: Optional[Dict[str, str]] = None,
@@ -117,4 +118,11 @@ def _update_path(self, asset_artifact: ArtifactStorageInfo) -> None:
 
         :param ArtifactStorageInfo asset_artifact: The asset storage info of the artifact
         """
-        self.path = asset_artifact.full_storage_path
+        aml_datastore_id = AMLNamedArmId(asset_artifact.datastore_arm_id)
+        self.path = LONG_URI_FORMAT.format(
+            aml_datastore_id.subscription_id,
+            aml_datastore_id.resource_group_name,
+            aml_datastore_id.workspace_name,
+            aml_datastore_id.asset_name,
+            asset_artifact.relative_path,
+        )