Skip to content

Commit 2062f8e

Browse files
Merge branch 'dev' into unblock-ha-enablement-on-ssdv2
2 parents e82082b + 4ef9898 commit 2062f8e

File tree

427 files changed

+146020
-86006
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

427 files changed

+146020
-86006
lines changed

.github/workflows/RunIssueSentinel.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,3 +14,4 @@ jobs:
1414
with:
1515
enable-similar-issues-scanning: true
1616
enable-security-issues-scanning: true
17+
enable-ux-tag: true

build_scripts/windows/scripts/build.cmd

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -154,8 +154,6 @@ if %errorlevel% neq 0 goto ERROR
154154
pushd %BUILDING_DIR%
155155
%BUILDING_DIR%\python.exe -I %REPO_ROOT%\scripts\compact_aaz.py
156156
if %errorlevel% neq 0 goto ERROR
157-
%BUILDING_DIR%\python.exe -I %~dp0\patch_models_v2.py
158-
if %errorlevel% neq 0 goto ERROR
159157
%BUILDING_DIR%\python.exe -I %REPO_ROOT%\scripts\trim_sdk.py
160158
if %errorlevel% neq 0 goto ERROR
161159
popd

src/azure-cli-core/azure/cli/core/profiles/_shared.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -212,7 +212,7 @@ def default_api_version(self):
212212
ResourceType.MGMT_SERVICEBUS: None,
213213
ResourceType.MGMT_EVENTHUB: None,
214214
ResourceType.MGMT_MONITOR: None,
215-
ResourceType.MGMT_MSI: '2023-01-31',
215+
ResourceType.MGMT_MSI: '2024-11-30',
216216
ResourceType.MGMT_APPSERVICE: '2024-11-01',
217217
ResourceType.MGMT_IOTHUB: None,
218218
ResourceType.MGMT_IOTDPS: None,

src/azure-cli/azure/cli/command_modules/acr/_docker_utils.py

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@
2828
from ._constants import get_managed_sku
2929
from ._constants import ACR_AUDIENCE_RESOURCE_NAME
3030
from ._utils import get_registry_by_name, ResourceNotFound
31-
from .policy import acr_config_authentication_as_arm_show
3231
from ._format import add_timestamp
3332
from ._errors import CONNECTIVITY_TOOMANYREQUESTS_ERROR
3433

@@ -135,18 +134,14 @@ def _get_aad_token_after_challenge(cli_ctx,
135134
artifact_repository,
136135
permission,
137136
is_diagnostics_context,
138-
use_acr_audience,
139137
verify_user_permissions):
140138
authurl = urlparse(token_params['realm'])
141139
authhost = urlunparse((authurl[0], authurl[1], '/oauth2/exchange', '', '', ''))
142140

143141
from azure.cli.core._profile import Profile
144142
profile = Profile(cli_ctx=cli_ctx)
145143

146-
scope = None
147-
if use_acr_audience:
148-
logger.debug("Using ACR audience token for authentication")
149-
scope = "https://{}.azure.net".format(ACR_AUDIENCE_RESOURCE_NAME)
144+
scope = "https://{}.azure.net".format(ACR_AUDIENCE_RESOURCE_NAME)
150145

151146
# this might be a cross tenant scenario, so pass subscription to get_raw_token
152147
creds, _, tenant = profile.get_raw_token(subscription=get_subscription_id(cli_ctx),
@@ -267,7 +262,6 @@ def _get_aad_token(cli_ctx,
267262
artifact_repository=None,
268263
permission=None,
269264
is_diagnostics_context=False,
270-
use_acr_audience=False,
271265
verify_user_permissions=False):
272266
"""Obtains refresh and access tokens for an AAD-enabled registry. Will return the allowed actions if
273267
verify_user_permissions is set to True.
@@ -296,7 +290,6 @@ def _get_aad_token(cli_ctx,
296290
artifact_repository,
297291
permission,
298292
is_diagnostics_context,
299-
use_acr_audience,
300293
verify_user_permissions)
301294

302295

@@ -453,19 +446,12 @@ def _get_credentials(cmd, # pylint: disable=too-many-statements
453446
if not registry or registry.sku.name in get_managed_sku(cmd):
454447
logger.info("Attempting to retrieve AAD refresh token...")
455448
try:
456-
use_acr_audience = False
457-
458-
if registry:
459-
aad_auth_policy = acr_config_authentication_as_arm_show(cmd, registry_name, resource_group_name)
460-
use_acr_audience = (aad_auth_policy and aad_auth_policy.status == 'disabled')
461-
462449
return login_server, EMPTY_GUID, _get_aad_token(cli_ctx,
463450
login_server,
464451
only_refresh_token,
465452
repository,
466453
artifact_repository,
467-
permission,
468-
use_acr_audience=use_acr_audience)
454+
permission)
469455
except CLIError as e:
470456
raise_toomanyrequests_error(str(e))
471457
logger.warning("%s: %s", AAD_TOKEN_BASE_ERROR_MESSAGE, str(e))

src/azure-cli/azure/cli/command_modules/acr/tests/latest/recordings/test_acr_abac_task.yaml

Lines changed: 736 additions & 585 deletions
Large diffs are not rendered by default.

src/azure-cli/azure/cli/command_modules/acr/tests/latest/recordings/test_acr_identity.yaml

Lines changed: 546 additions & 494 deletions
Large diffs are not rendered by default.

src/azure-cli/azure/cli/command_modules/acr/tests/latest/test_acr_commands_mock.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1206,6 +1206,7 @@ def _core_token_scenarios(self, mock_get_raw_token, mock_requests_get, mock_requ
12061206

12071207
# Test get refresh token
12081208
get_login_credentials(cmd, registry_name, tenant_suffix=tenant_suffix)
1209+
self._validate_raw_token_request(mock_get_raw_token)
12091210
self._validate_refresh_token_request(mock_requests_get, mock_requests_post, login_server)
12101211

12111212
# Test get access token for container image repository
@@ -1237,6 +1238,9 @@ def _setup_mock_token_requests(self, mock_get_aad_token, mock_requests_get, mock
12371238
'access_token': TEST_ACR_ACCESS_TOKEN}).encode()
12381239
mock_requests_post.return_value = token_response
12391240

1241+
def _validate_raw_token_request(self, mock_get_raw_token):
1242+
mock_get_raw_token.assert_called_with(mock.ANY, resource="https://containerregistry.azure.net", subscription=mock.ANY)
1243+
12401244
def _validate_refresh_token_request(self, mock_requests_get, mock_requests_post, login_server):
12411245
mock_requests_get.assert_called_with('https://{}/v2/'.format(login_server), verify=mock.ANY)
12421246
mock_requests_post.assert_called_with(

src/azure-cli/azure/cli/command_modules/acs/tests/latest/recordings/test_aks_abort.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ interactions:
1515
User-Agent:
1616
- AZURECLI/2.72.0 azsdk-python-core/1.31.0 Python/3.10.11 (Windows-10-10.0.26100-SP0)
1717
method: GET
18-
uri: https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest000001/providers/Microsoft.ContainerService/managedClusters/cliakstest000002?api-version=2025-09-01
18+
uri: https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest000001/providers/Microsoft.ContainerService/managedClusters/cliakstest000002?api-version=2025-10-01
1919
response:
2020
body:
2121
string: '{"error":{"code":"ResourceNotFound","message":"The Resource ''Microsoft.ContainerService/managedClusters/cliakstest000002''
@@ -125,7 +125,7 @@ interactions:
125125
User-Agent:
126126
- AZURECLI/2.72.0 azsdk-python-core/1.31.0 Python/3.10.11 (Windows-10-10.0.26100-SP0)
127127
method: PUT
128-
uri: https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest000001/providers/Microsoft.ContainerService/managedClusters/cliakstest000002?api-version=2025-09-01
128+
uri: https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest000001/providers/Microsoft.ContainerService/managedClusters/cliakstest000002?api-version=2025-10-01
129129
response:
130130
body:
131131
string: "{\n \"id\": \"/subscriptions/00000000-0000-0000-0000-000000000000/resourcegroups/clitest000001/providers/Microsoft.ContainerService/managedClusters/cliakstest000002\",\n
@@ -221,7 +221,7 @@ interactions:
221221
User-Agent:
222222
- AZURECLI/2.72.0 azsdk-python-core/1.31.0 Python/3.10.11 (Windows-10-10.0.26100-SP0)
223223
method: GET
224-
uri: https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest000001/providers/Microsoft.ContainerService/managedClusters/cliakstest000002?api-version=2025-09-01
224+
uri: https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest000001/providers/Microsoft.ContainerService/managedClusters/cliakstest000002?api-version=2025-10-01
225225
response:
226226
body:
227227
string: "{\n \"id\": \"/subscriptions/00000000-0000-0000-0000-000000000000/resourcegroups/clitest000001/providers/Microsoft.ContainerService/managedClusters/cliakstest000002\",\n
@@ -313,7 +313,7 @@ interactions:
313313
User-Agent:
314314
- AZURECLI/2.72.0 azsdk-python-core/1.31.0 Python/3.10.11 (Windows-10-10.0.26100-SP0)
315315
method: POST
316-
uri: https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest000001/providers/Microsoft.ContainerService/managedclusters/cliakstest000002/abort?api-version=2025-09-01
316+
uri: https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest000001/providers/Microsoft.ContainerService/managedclusters/cliakstest000002/abort?api-version=2025-10-01
317317
response:
318318
body:
319319
string: "{\n \"id\": \"/subscriptions/00000000-0000-0000-0000-000000000000/resourcegroups/clitest000001/providers/Microsoft.ContainerService/managedClusters/cliakstest000002\",\n
@@ -565,7 +565,7 @@ interactions:
565565
User-Agent:
566566
- AZURECLI/2.72.0 azsdk-python-core/1.31.0 Python/3.10.11 (Windows-10-10.0.26100-SP0)
567567
method: GET
568-
uri: https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest000001/providers/Microsoft.ContainerService/managedClusters/cliakstest000002?api-version=2025-09-01
568+
uri: https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/clitest000001/providers/Microsoft.ContainerService/managedClusters/cliakstest000002?api-version=2025-10-01
569569
response:
570570
body:
571571
string: "{\n \"id\": \"/subscriptions/00000000-0000-0000-0000-000000000000/resourcegroups/clitest000001/providers/Microsoft.ContainerService/managedClusters/cliakstest000002\",\n

0 commit comments

Comments
 (0)