diff --git a/python/az/aro/azext_aro/_validators.py b/python/az/aro/azext_aro/_validators.py index 96b8941323a..34b1435cc75 100644 --- a/python/az/aro/azext_aro/_validators.py +++ b/python/az/aro/azext_aro/_validators.py @@ -284,6 +284,8 @@ def validate_refresh_cluster_credentials(namespace): return if namespace.client_secret is not None or namespace.client_id is not None: raise RequiredArgumentMissingError('--client-id and --client-secret must be not set with --refresh-credentials.') # pylint: disable=line-too-long + if namespace.platform_workload_identities is not None: + raise MutuallyExclusiveArgumentError('--platform-workload-identities must be not set with --refresh-credentials.') # pylint: disable=line-too-long if namespace.upgradeable_to is not None: raise MutuallyExclusiveArgumentError('Must not specify --refresh-credentials when --upgradeable-to is used.') # pylint: disable=line-too-long diff --git a/python/az/aro/azext_aro/tests/latest/unit/test_validators.py b/python/az/aro/azext_aro/tests/latest/unit/test_validators.py index 0df05cf9c94..3fce1c5a546 100644 --- a/python/az/aro/azext_aro/tests/latest/unit/test_validators.py +++ b/python/az/aro/azext_aro/tests/latest/unit/test_validators.py @@ -815,9 +815,14 @@ def test_validate_worker_vm_disk_size_gb(test_description, namespace, expected_e Mock(client_id="client_id_456"), RequiredArgumentMissingError ), + ( + "should raise MutuallyExclusiveArgumentError Exception because namespace.platform_workload_identities is present", + Mock(platform_workload_identities=[Mock(resource_id='Foo')], client_id=None, client_secret=None), + MutuallyExclusiveArgumentError + ), ( "should not raise any Exception because namespace.client_secret is None and namespace.client_id is None", - Mock(upgradeable_to=None, client_secret=None, client_id=None), + Mock(upgradeable_to=None, client_secret=None, client_id=None, platform_workload_identities=None), None ), (