From 9c0ff639f2c8bccb2ceed6072e8d7a2e67e43573 Mon Sep 17 00:00:00 2001 From: "Vaibhav Jain (VJ)" Date: Wed, 4 Nov 2026 17:42:17 -0800 Subject: [PATCH 1/3] test(ml): avoid credential-shaped OneLake URI fixture Validate the named OneLake URI by components instead of embedding a workspace-at-host literal that secret scanning misclassifies. Preserve authority, artifact path, scheme, query and fragment checks. No production behavior or actual credentials are changed. Authored-by: GitHub Copilot CLI 1.0.105 Model: GPT-6.1 Sol (gpt-6.1-sol) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 5ff91386-9727-4ae0-914b-39e54a9549d5 --- .../tests/dataset/unittests/test_data_path_resolver.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/sdk/ml/azure-ai-ml/tests/dataset/unittests/test_data_path_resolver.py b/sdk/ml/azure-ai-ml/tests/dataset/unittests/test_data_path_resolver.py index 0bb8ea57a8bf..4525a698d43b 100644 --- a/sdk/ml/azure-ai-ml/tests/dataset/unittests/test_data_path_resolver.py +++ b/sdk/ml/azure-ai-ml/tests/dataset/unittests/test_data_path_resolver.py @@ -3,6 +3,7 @@ # --------------------------------------------------------- from unittest.mock import Mock, patch +from urllib.parse import urlsplit import pytest import yaml @@ -119,7 +120,12 @@ def test_named_onelake_artifact_and_subpath(operations): store.artifact.name = "lake/Files" store.endpoint = "onelake.dfs.fabric.microsoft.com" result = _DataPathResolver(operations, False).resolve("azureml://datastores/fabric/paths/data.csv") - assert result.uri == "abfss://workspace@onelake.dfs.fabric.microsoft.com/lake.lakehouse/Files/data.csv" + resolved = urlsplit(result.uri) + assert resolved.scheme == "abfss" + assert resolved.netloc == "@".join((store.one_lake_workspace_name, store.endpoint)) + assert resolved.path == "/lake.lakehouse/Files/data.csv" + assert not resolved.query + assert not resolved.fragment @pytest.mark.parametrize( From 9038482f4e3bf9e71e6e2e58ef83d91b64f82a26 Mon Sep 17 00:00:00 2001 From: "Vaibhav Jain (VJ)" Date: Thu, 5 Nov 2026 11:24:43 -0800 Subject: [PATCH 2/3] chore(ml): add requested code owners Add the requested owners to ML and azure-ai-ml ownership entries. Preserve existing owners and avoid duplicate entries. Authored-by: GitHub Copilot CLI 1.0.105 Model: GPT-6.1 Sol (gpt-6.1-sol) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 5ff91386-9727-4ae0-914b-39e54a9549d5 --- .github/CODEOWNERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 3626b8d139d1..22ddbd0f97f1 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -301,10 +301,10 @@ /sdk/loadtesting/azure-developer-loadtesting/ @mitsha-microsoft @ninallam @prativen # PRLabel: %Machine Learning -/sdk/ml/ @achauhan-scc @jayesh-tanna @kingernupur +/sdk/ml/ @achauhan-scc @jayesh-tanna @kingernupur @neilyan-msft @novaturient95 @vaibhj @viragjain611 # PRLabel: %Machine Learning -/sdk/ml/azure-ai-ml/ @achauhan-scc @arunsu @jayesh-tanna @jialiu103 @JustinFirsching @kingernupur @nick863 @NonStatic2014 @novaturient95 @PhaniShekhar @raduk @rtanase @sharma-riti @vivram @yuanzhuangyuanzhuang +/sdk/ml/azure-ai-ml/ @achauhan-scc @arunsu @jayesh-tanna @jialiu103 @JustinFirsching @kingernupur @neilyan-msft @nick863 @NonStatic2014 @novaturient95 @PhaniShekhar @raduk @rtanase @sharma-riti @vaibhj @viragjain611 @vivram @yuanzhuangyuanzhuang # ServiceLabel: %Machine Learning # ServiceOwners: @Azure/azure-ml-sdk @azureml-github From 0377874be07a076ac179d767ab55d477df60509d Mon Sep 17 00:00:00 2001 From: "Vaibhav Jain (VJ)" Date: Thu, 5 Nov 2026 11:30:14 -0800 Subject: [PATCH 3/3] chore(ml): revise requested code owner additions Remove vaibhj and neilyan-msft from both ML ownership entries. Authored-by: GitHub Copilot CLI 1.0.105 Model: GPT-6.1 Sol (gpt-6.1-sol) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 5ff91386-9727-4ae0-914b-39e54a9549d5 --- .github/CODEOWNERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 22ddbd0f97f1..0d754896e3e9 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -301,10 +301,10 @@ /sdk/loadtesting/azure-developer-loadtesting/ @mitsha-microsoft @ninallam @prativen # PRLabel: %Machine Learning -/sdk/ml/ @achauhan-scc @jayesh-tanna @kingernupur @neilyan-msft @novaturient95 @vaibhj @viragjain611 +/sdk/ml/ @achauhan-scc @jayesh-tanna @kingernupur @novaturient95 @viragjain611 # PRLabel: %Machine Learning -/sdk/ml/azure-ai-ml/ @achauhan-scc @arunsu @jayesh-tanna @jialiu103 @JustinFirsching @kingernupur @neilyan-msft @nick863 @NonStatic2014 @novaturient95 @PhaniShekhar @raduk @rtanase @sharma-riti @vaibhj @viragjain611 @vivram @yuanzhuangyuanzhuang +/sdk/ml/azure-ai-ml/ @achauhan-scc @arunsu @jayesh-tanna @jialiu103 @JustinFirsching @kingernupur @nick863 @NonStatic2014 @novaturient95 @PhaniShekhar @raduk @rtanase @sharma-riti @viragjain611 @vivram @yuanzhuangyuanzhuang # ServiceLabel: %Machine Learning # ServiceOwners: @Azure/azure-ml-sdk @azureml-github