[Role] az role assignment delete: If --ids is provided, ignore other arguments, instead of raising error

[Jiashuo Li (jiasli)]

Fix IcM 341728422

Related command
az role assignment delete

Description
If defaults.group config option is set via:

az config set defaults.group=testrg

az role assignment delete --ids fails:

> az role assignment delete --ids /subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590/providers/Microsoft.Authorization/roleAssignments/54df05ac-c9d3-4b71-a005-dec43d813f87
When assignment ids are used, other parameter values are not required

Root Cause
This command disallows using --ids with other arguments:

azure-cli/src/azure-cli/azure/cli/command_modules/role/custom.py

Lines 497 to 499 in 98f7865

	if ids:
	if assignee or role or resource_group_name or scope or include_inherited:
	raise CLIError('When assignment ids are used, other parameter values are not required')

Argument resource_group_name is registered at

azure-cli/src/azure-cli-core/azure/cli/core/__init__.py

Line 492 in d5271c3

c.argument('resource_group_name', resource_group_name_type)

resource_group_name_type has a configured_default:

azure-cli/src/azure-cli-core/azure/cli/core/commands/parameters.py

Line 242 in fa2a2d8

configured_default='group',

When defaults.group config option is set, resource_group_name variable will be populated, causing this failure.

Proposed solution
Removing the configured default value for --resource-group in az role assignment delete will be a BREAKING CHANGE and will break customers who rely on --resource-group's configured default value.

Therefore, this PR changes the behavior so that if --ids is provided, it ignores other arguments, instead of raising error.

Testing Guide

az role assignment delete --ids {id} --assignee test --role test --resource-group test --scope test --include-inherit

[Jiashuo Li (jiasli)]

Will refine the test after #24324 is merged.

[Jiashuo Li (jiasli)]

This mimics the behavior of core for --ids handling:

azure-cli/src/azure-cli-core/azure/cli/core/commands/arm.py

Lines 265 to 268 in ce3a1f1

	for _, data in other_values.items():
	if data['value'] and not getattr(data['value'], 'is_default', None):
	logger.warning("option '%s' will be ignored due to use of '--ids'.",
	data['arg'].type.settings['options_list'][0])

A difference is that core doesn't show warning if default value is used, but we do here. I feel being explicit is better than being implicit.

[Yong Zhang (yonzhan)]

Role