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 sig gallery-application: Migrate to aaz #30678

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

cxznmhdcxz
Copy link
Member

@cxznmhdcxz cxznmhdcxz commented Jan 20, 2025

Related command

az sig gallery-application create/delete/list/show/update/wait

Description
aaz pr Azure/aaz#676

Testing Guide

History Notes

[Compute] BREAKING CHANGE: az sig gallery-application create/update: Output field supportedOsType changed to supportedOSType


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 Jan 20, 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 Jan 20, 2025

⚠️AzureCLI-BreakingChangeTest
⚠️vm
rule cmd_name rule_message suggest_message
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter description: added property aaz_type=string
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter description: added property type=string
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter gallery_application_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter gallery_application_name: added property type=string
⚠️ 1010 - ParaPropUpdate sig gallery-application create cmd sig gallery-application create update parameter gallery_application_name: updated property options from ['--application-name', '--name', '-n'] to ['--application-name', '--gallery-application-name', '--name', '-n']
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter gallery_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter gallery_name: added property type=string
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter location: added property aaz_type=string
⚠️ 1010 - ParaPropUpdate sig gallery-application create cmd sig gallery-application create update parameter location: updated property type from custom_type to string
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter no_wait: added property aaz_type=bool
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter no_wait: added property choices=['0', '1', 'f', 'false', 'n', 'no', 't', 'true', 'y', 'yes']
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter no_wait: added property nargs=?
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter no_wait: added property type=bool
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter os_type: added property aaz_type=string
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter os_type: added property type=string
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter resource_group_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter resource_group_name: added property type=string
⚠️ 1010 - ParaPropUpdate sig gallery-application create cmd sig gallery-application create update parameter resource_group_name: updated property name from resource_group_name to resource_group
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter tags: added property aaz_type=AAZDictArg
⚠️ 1008 - ParaPropAdd sig gallery-application create cmd sig gallery-application create update parameter tags: added property type=Dict<String,String>
⚠️ 1010 - ParaPropUpdate sig gallery-application create cmd sig gallery-application create update parameter tags: updated property nargs from * to +
⚠️ 1006 - ParaAdd sig gallery-application list cmd sig gallery-application list added parameter pagination_limit
⚠️ 1006 - ParaAdd sig gallery-application list cmd sig gallery-application list added parameter pagination_token
⚠️ 1006 - ParaAdd sig gallery-application update cmd sig gallery-application update added parameter generic_update_add
⚠️ 1006 - ParaAdd sig gallery-application update cmd sig gallery-application update added parameter generic_update_force_string
⚠️ 1006 - ParaAdd sig gallery-application update cmd sig gallery-application update added parameter generic_update_remove
⚠️ 1006 - ParaAdd sig gallery-application update cmd sig gallery-application update added parameter generic_update_set
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter description: added property aaz_type=string
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter description: added property type=string
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter gallery_application_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter gallery_application_name: added property id_part=child_name_1
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter gallery_application_name: added property type=string
⚠️ 1010 - ParaPropUpdate sig gallery-application update cmd sig gallery-application update update parameter gallery_application_name: updated property options from ['--application-name', '--name', '-n'] to ['--application-name', '--gallery-application-name', '--name', '-n']
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter gallery_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter gallery_name: added property id_part=name
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter gallery_name: added property type=string
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter location: added property aaz_type=string
⚠️ 1010 - ParaPropUpdate sig gallery-application update cmd sig gallery-application update update parameter location: updated property type from custom_type to string
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter no_wait: added property aaz_type=bool
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter no_wait: added property choices=['0', '1', 'f', 'false', 'n', 'no', 't', 'true', 'y', 'yes']
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter no_wait: added property nargs=?
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter no_wait: added property type=bool
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter resource_group_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter resource_group_name: added property type=string
⚠️ 1010 - ParaPropUpdate sig gallery-application update cmd sig gallery-application update update parameter resource_group_name: updated property name from resource_group_name to resource_group
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter tags: added property aaz_type=AAZDictArg
⚠️ 1008 - ParaPropAdd sig gallery-application update cmd sig gallery-application update update parameter tags: added property type=Dict<String,String>
⚠️ 1010 - ParaPropUpdate sig gallery-application update cmd sig gallery-application update update parameter tags: updated property nargs from * to +
⚠️ 1008 - ParaPropAdd sig gallery-application wait cmd sig gallery-application wait update parameter gallery_application_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd sig gallery-application wait cmd sig gallery-application wait update parameter gallery_application_name: added property id_part=child_name_1
⚠️ 1008 - ParaPropAdd sig gallery-application wait cmd sig gallery-application wait update parameter gallery_application_name: added property type=string
⚠️ 1010 - ParaPropUpdate sig gallery-application wait cmd sig gallery-application wait update parameter gallery_application_name: updated property options from ['--application-name', '--name', '-n'] to ['--application-name', '--gallery-application-name', '--name', '-n']
⚠️ 1008 - ParaPropAdd sig gallery-application wait cmd sig gallery-application wait update parameter gallery_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd sig gallery-application wait cmd sig gallery-application wait update parameter gallery_name: added property id_part=name
⚠️ 1008 - ParaPropAdd sig gallery-application wait cmd sig gallery-application wait update parameter gallery_name: added property type=string
⚠️ 1008 - ParaPropAdd sig gallery-application wait cmd sig gallery-application wait update parameter resource_group_name: added property aaz_type=string
⚠️ 1008 - ParaPropAdd sig gallery-application wait cmd sig gallery-application wait update parameter resource_group_name: added property type=string
⚠️ 1010 - ParaPropUpdate sig gallery-application wait cmd sig gallery-application wait update parameter resource_group_name: updated property name from resource_group_name to resource_group

@yonzhan
Copy link
Collaborator

yonzhan commented Jan 20, 2025

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

self.cmd('sig create -r {gallery} -g {rg}')
self.cmd('sig gallery-application update -n {app_name} -r {gallery} -g {rg} --description test --tags tag=test', checks=[
self.check('name', '{app_name}'),
self.check('supportedOsType', 'Windows'),
self.check('supportedOSType', 'Windows'),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since the output field has been changed and it is actually a breaking change, it is better to add a changelog to the History Notes section like this:
image

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated comment

@yanzhudd
Copy link
Contributor

please remember to raise an AAZ PR as well

@yanzhudd
Copy link
Contributor

It is better to list all the migrated commands of this PR in the Description section

@yanzhudd
Copy link
Contributor

Are the commands only existed in the latest profile?

@cxznmhdcxz
Copy link
Member Author

Are the commands only existed in the latest profile?

Yes, only in latest profile.

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