Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

{Compute} az disk-encryption-set: Migrate to AAZ #30894

Merged
merged 7 commits into from
Mar 6, 2025

Conversation

cxznmhdcxz
Copy link
Member

@cxznmhdcxz cxznmhdcxz commented Feb 26, 2025

Related command

az disk-encryption-set create/delete/list/show/update
az disk-encryption-set identity assign/remove/show

Description

Migrate to AAZ based
Azure/aaz#701

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature


This checklist is used to make sure that common guidelines for a pull request are followed.

Copy link

azure-client-tools-bot-prd bot commented Feb 26, 2025

️✔️AzureCLI-FullTest
️✔️acr
️✔️2020-09-01-hybrid
️✔️3.12
️✔️3.9
️✔️latest
️✔️3.12
️✔️3.9
️✔️acs
️✔️2020-09-01-hybrid
️✔️3.12
️✔️3.9
️✔️latest
️✔️3.12
️✔️3.9
️✔️advisor
️✔️latest
️✔️3.12
️✔️3.9
️✔️ams
️✔️latest
️✔️3.12
️✔️3.9
️✔️apim
️✔️latest
️✔️3.12
️✔️3.9
️✔️appconfig
️✔️latest
️✔️3.12
️✔️3.9
️✔️appservice
️✔️latest
️✔️3.12
️✔️3.9
️✔️aro
️✔️latest
️✔️3.12
️✔️3.9
️✔️backup
️✔️latest
️✔️3.12
️✔️3.9
️✔️batch
️✔️latest
️✔️3.12
️✔️3.9
️✔️batchai
️✔️latest
️✔️3.12
️✔️3.9
️✔️billing
️✔️latest
️✔️3.12
️✔️3.9
️✔️botservice
️✔️latest
️✔️3.12
️✔️3.9
️✔️cdn
️✔️latest
️✔️3.12
️✔️3.9
️✔️cloud
️✔️latest
️✔️3.12
️✔️3.9
️✔️cognitiveservices
️✔️latest
️✔️3.12
️✔️3.9
️✔️compute_recommender
️✔️latest
️✔️3.12
️✔️3.9
️✔️computefleet
️✔️latest
️✔️3.12
️✔️3.9
️✔️config
️✔️latest
️✔️3.12
️✔️3.9
️✔️configure
️✔️latest
️✔️3.12
️✔️3.9
️✔️consumption
️✔️latest
️✔️3.12
️✔️3.9
️✔️container
️✔️latest
️✔️3.12
️✔️3.9
️✔️containerapp
️✔️latest
️✔️3.12
️✔️3.9
️✔️core
️✔️2018-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️2019-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️2020-09-01-hybrid
️✔️3.12
️✔️3.9
️✔️latest
️✔️3.12
️✔️3.9
️✔️cosmosdb
️✔️latest
️✔️3.12
️✔️3.9
️✔️databoxedge
️✔️2019-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️2020-09-01-hybrid
️✔️3.12
️✔️3.9
️✔️latest
️✔️3.12
️✔️3.9
️✔️dls
️✔️latest
️✔️3.12
️✔️3.9
️✔️dms
️✔️latest
️✔️3.12
️✔️3.9
️✔️eventgrid
️✔️latest
️✔️3.12
️✔️3.9
️✔️eventhubs
️✔️latest
️✔️3.12
️✔️3.9
️✔️feedback
️✔️latest
️✔️3.12
️✔️3.9
️✔️find
️✔️latest
️✔️3.12
️✔️3.9
️✔️hdinsight
️✔️latest
️✔️3.12
️✔️3.9
️✔️identity
️✔️latest
️✔️3.12
️✔️3.9
️✔️iot
️✔️2019-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️2020-09-01-hybrid
️✔️3.12
️✔️3.9
️✔️latest
️✔️3.12
️✔️3.9
️✔️keyvault
️✔️2018-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️2020-09-01-hybrid
️✔️3.12
️✔️3.9
️✔️latest
️✔️3.12
️✔️3.9
️✔️lab
️✔️latest
️✔️3.12
️✔️3.9
️✔️managedservices
️✔️latest
️✔️3.12
️✔️3.9
️✔️maps
️✔️latest
️✔️3.12
️✔️3.9
️✔️marketplaceordering
️✔️latest
️✔️3.12
️✔️3.9
️✔️monitor
️✔️latest
️✔️3.12
️✔️3.9
️✔️mysql
️✔️latest
️✔️3.12
️✔️3.9
️✔️netappfiles
️✔️latest
️✔️3.12
️✔️3.9
️✔️network
️✔️2018-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️latest
️✔️3.12
️✔️3.9
️✔️policyinsights
️✔️latest
️✔️3.12
️✔️3.9
️✔️privatedns
️✔️latest
️✔️3.12
️✔️3.9
️✔️profile
️✔️latest
️✔️3.12
️✔️3.9
️✔️rdbms
️✔️latest
️✔️3.12
️✔️3.9
️✔️redis
️✔️latest
️✔️3.12
️✔️3.9
️✔️relay
️✔️latest
️✔️3.12
️✔️3.9
️✔️resource
️✔️2018-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️2019-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️latest
️✔️3.12
️✔️3.9
️✔️role
️✔️latest
️✔️3.12
️✔️3.9
️✔️search
️✔️latest
️✔️3.12
️✔️3.9
️✔️security
️✔️latest
️✔️3.12
️✔️3.9
️✔️servicebus
️✔️latest
️✔️3.12
️✔️3.9
️✔️serviceconnector
️✔️latest
️✔️3.12
️✔️3.9
️✔️servicefabric
️✔️latest
️✔️3.12
️✔️3.9
️✔️signalr
️✔️latest
️✔️3.12
️✔️3.9
️✔️sql
️✔️latest
️✔️3.12
️✔️3.9
️✔️sqlvm
️✔️latest
️✔️3.12
️✔️3.9
️✔️storage
️✔️2018-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️2019-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️2020-09-01-hybrid
️✔️3.12
️✔️3.9
️✔️latest
️✔️3.12
️✔️3.9
️✔️synapse
️✔️latest
️✔️3.12
️✔️3.9
️✔️telemetry
️✔️2018-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️2019-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️2020-09-01-hybrid
️✔️3.12
️✔️3.9
️✔️latest
️✔️3.12
️✔️3.9
️✔️util
️✔️latest
️✔️3.12
️✔️3.9
️✔️vm
️✔️2018-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️2019-03-01-hybrid
️✔️3.12
️✔️3.9
️✔️2020-09-01-hybrid
️✔️3.12
️✔️3.9
️✔️latest
️✔️3.12
️✔️3.9

Copy link

azure-client-tools-bot-prd bot commented Feb 26, 2025

⚠️AzureCLI-BreakingChangeTest
⚠️vm
rule cmd_name rule_message suggest_message
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter disk_encryption_set_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter disk_encryption_set_name: added property type=string
⚠️ 1009 - ParaPropRemove disk-encryption-set create cmd disk-encryption-set create update parameter disk_encryption_set_name: removed property id_part=name
⚠️ 1010 - ParaPropUpdate disk-encryption-set create cmd disk-encryption-set create update parameter disk_encryption_set_name: updated property options from ['--name', '-n'] to ['--disk-encryption-set-name', '--name', '-n']
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter enable_auto_key_rotation: added property aaz_type=bool
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter enable_auto_key_rotation: added property type=bool
⚠️ 1010 - ParaPropUpdate disk-encryption-set create cmd disk-encryption-set create update parameter enable_auto_key_rotation: updated property choices from ['false', 'true'] to ['0', '1', 'f', 'false', 'n', 'no', 't', 'true', 'y', 'yes']
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter encryption_type: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter encryption_type: added property type=string
⚠️ 1010 - ParaPropUpdate disk-encryption-set create cmd disk-encryption-set create update parameter encryption_type: updated property choices from ['ConfidentialVmEncryptedWithCustomerKey', 'EncryptionAtRestWithCustomerKey', 'EncryptionAtRestWithPlatformAndCustomerKeys', 'EncryptionAtRestWithPlatformKey'] to ['ConfidentialVmEncryptedWithCustomerKey', 'EncryptionAtRestWithCustomerKey', 'EncryptionAtRestWithPlatformAndCustomerKeys']
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter federated_client_id: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter federated_client_id: added property type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter key_url: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter key_url: added property type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter location: added property aaz_type=string
⚠️ 1010 - ParaPropUpdate disk-encryption-set create cmd disk-encryption-set create update parameter location: updated property type from custom_type to string
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter mi_system_assigned: added property aaz_type=bool
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter mi_system_assigned: added property type=bool
⚠️ 1010 - ParaPropUpdate disk-encryption-set create cmd disk-encryption-set create update parameter mi_system_assigned: updated property choices from ['false', 'true'] to ['0', '1', 'f', 'false', 'n', 'no', 't', 'true', 'y', 'yes']
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter mi_user_assigned: added property aaz_type=AAZListArg
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter mi_user_assigned: added property type=List<String>
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter no_wait: added property aaz_type=bool
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter no_wait: added property choices=['0', '1', 'f', 'false', 'n', 'no', 't', 'true', 'y', 'yes']
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter no_wait: added property nargs=?
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter no_wait: added property type=bool
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter resource_group_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter resource_group_name: added property type=string
⚠️ 1010 - ParaPropUpdate disk-encryption-set create cmd disk-encryption-set create update parameter resource_group_name: updated property name from resource_group_name to resource_group
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter source_vault: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter source_vault: added property type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter tags: added property aaz_type=AAZDictArg
⚠️ 1008 - ParaPropAdd disk-encryption-set create cmd disk-encryption-set create update parameter tags: added property type=Dict<String,String>
⚠️ 1010 - ParaPropUpdate disk-encryption-set create cmd disk-encryption-set create update parameter tags: updated property nargs from * to +
⚠️ 1008 - ParaPropAdd disk-encryption-set identity assign cmd disk-encryption-set identity assign update parameter disk_encryption_set_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set identity assign cmd disk-encryption-set identity assign update parameter disk_encryption_set_name: added property type=string
⚠️ 1010 - ParaPropUpdate disk-encryption-set identity assign cmd disk-encryption-set identity assign update parameter disk_encryption_set_name: updated property options from ['--name', '-n'] to ['--disk-encryption-set-name', '--name', '-n']
⚠️ 1008 - ParaPropAdd disk-encryption-set identity assign cmd disk-encryption-set identity assign update parameter mi_system_assigned: added property aaz_type=bool
⚠️ 1008 - ParaPropAdd disk-encryption-set identity assign cmd disk-encryption-set identity assign update parameter mi_system_assigned: added property type=bool
⚠️ 1010 - ParaPropUpdate disk-encryption-set identity assign cmd disk-encryption-set identity assign update parameter mi_system_assigned: updated property choices from ['false', 'true'] to ['0', '1', 'f', 'false', 'n', 'no', 't', 'true', 'y', 'yes']
⚠️ 1010 - ParaPropUpdate disk-encryption-set identity assign cmd disk-encryption-set identity assign update parameter mi_system_assigned: updated property name from mi_system_assigned to system_assigned
⚠️ 1008 - ParaPropAdd disk-encryption-set identity assign cmd disk-encryption-set identity assign update parameter mi_user_assigned: added property aaz_type=AAZListArg
⚠️ 1008 - ParaPropAdd disk-encryption-set identity assign cmd disk-encryption-set identity assign update parameter mi_user_assigned: added property type=List<String>
⚠️ 1010 - ParaPropUpdate disk-encryption-set identity assign cmd disk-encryption-set identity assign update parameter mi_user_assigned: updated property name from mi_user_assigned to user_assigned
⚠️ 1008 - ParaPropAdd disk-encryption-set identity assign cmd disk-encryption-set identity assign update parameter resource_group_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set identity assign cmd disk-encryption-set identity assign update parameter resource_group_name: added property type=string
⚠️ 1010 - ParaPropUpdate disk-encryption-set identity assign cmd disk-encryption-set identity assign update parameter resource_group_name: updated property name from resource_group_name to resource_group
⚠️ 1004 - CmdPropRemove disk-encryption-set identity remove cmd disk-encryption-set identity remove removed property confirmation
⚠️ 1008 - ParaPropAdd disk-encryption-set identity remove cmd disk-encryption-set identity remove update parameter disk_encryption_set_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set identity remove cmd disk-encryption-set identity remove update parameter disk_encryption_set_name: added property type=string
⚠️ 1010 - ParaPropUpdate disk-encryption-set identity remove cmd disk-encryption-set identity remove update parameter disk_encryption_set_name: updated property options from ['--name', '-n'] to ['--disk-encryption-set-name', '--name', '-n']
⚠️ 1008 - ParaPropAdd disk-encryption-set identity remove cmd disk-encryption-set identity remove update parameter mi_system_assigned: added property aaz_type=bool
⚠️ 1008 - ParaPropAdd disk-encryption-set identity remove cmd disk-encryption-set identity remove update parameter mi_system_assigned: added property type=bool
⚠️ 1010 - ParaPropUpdate disk-encryption-set identity remove cmd disk-encryption-set identity remove update parameter mi_system_assigned: updated property choices from ['false', 'true'] to ['0', '1', 'f', 'false', 'n', 'no', 't', 'true', 'y', 'yes']
⚠️ 1010 - ParaPropUpdate disk-encryption-set identity remove cmd disk-encryption-set identity remove update parameter mi_system_assigned: updated property name from mi_system_assigned to system_assigned
⚠️ 1008 - ParaPropAdd disk-encryption-set identity remove cmd disk-encryption-set identity remove update parameter mi_user_assigned: added property aaz_type=AAZListArg
⚠️ 1008 - ParaPropAdd disk-encryption-set identity remove cmd disk-encryption-set identity remove update parameter mi_user_assigned: added property type=List<String>
⚠️ 1010 - ParaPropUpdate disk-encryption-set identity remove cmd disk-encryption-set identity remove update parameter mi_user_assigned: updated property name from mi_user_assigned to user_assigned
⚠️ 1008 - ParaPropAdd disk-encryption-set identity remove cmd disk-encryption-set identity remove update parameter resource_group_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set identity remove cmd disk-encryption-set identity remove update parameter resource_group_name: added property type=string
⚠️ 1010 - ParaPropUpdate disk-encryption-set identity remove cmd disk-encryption-set identity remove update parameter resource_group_name: updated property name from resource_group_name to resource_group
⚠️ 1006 - ParaAdd disk-encryption-set update cmd disk-encryption-set update added parameter no_wait
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter disk_encryption_set_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter disk_encryption_set_name: added property type=string
⚠️ 1010 - ParaPropUpdate disk-encryption-set update cmd disk-encryption-set update update parameter disk_encryption_set_name: updated property options from ['--name', '-n'] to ['--disk-encryption-set-name', '--name', '-n']
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter enable_auto_key_rotation: added property aaz_type=bool
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter enable_auto_key_rotation: added property type=bool
⚠️ 1010 - ParaPropUpdate disk-encryption-set update cmd disk-encryption-set update update parameter enable_auto_key_rotation: updated property choices from ['false', 'true'] to ['0', '1', 'f', 'false', 'n', 'no', 't', 'true', 'y', 'yes']
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter federated_client_id: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter federated_client_id: added property type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter key_url: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter key_url: added property type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter properties_to_add: added property aaz_type=AAZGenericUpdateAddArg
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter properties_to_remove: added property aaz_type=AAZGenericUpdateRemoveArg
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter properties_to_set: added property aaz_type=AAZGenericUpdateSetArg
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter resource_group_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter resource_group_name: added property type=string
⚠️ 1010 - ParaPropUpdate disk-encryption-set update cmd disk-encryption-set update update parameter resource_group_name: updated property name from resource_group_name to resource_group
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter source_vault: added property aaz_type=string
⚠️ 1008 - ParaPropAdd disk-encryption-set update cmd disk-encryption-set update update parameter source_vault: added property type=string

@yonzhan
Copy link
Collaborator

yonzhan commented Feb 26, 2025

Thank you for your contribution! We will review the pull request and get back to you soon.

Copy link

The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR.

Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions).
After that please run the following commands to enable git hooks:

pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>

@cxznmhdcxz cxznmhdcxz marked this pull request as ready for review February 27, 2025 07:57
@yanzhudd yanzhudd merged commit 26ee3d1 into Azure:dev Mar 6, 2025
53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Auto-Assign Auto assign by bot Compute az vm/vmss/image/disk/snapshot
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants