Skip to content

[Batch] upgrade azure batch API version to 2024-02-01#28784

Merged
zhoxing-ms merged 10 commits intoAzure:devfrom
wanghoppe:hoppe/0201
Apr 23, 2024
Merged

[Batch] upgrade azure batch API version to 2024-02-01#28784
zhoxing-ms merged 10 commits intoAzure:devfrom
wanghoppe:hoppe/0201

Conversation

@wanghoppe
Copy link
Copy Markdown
Member

@wanghoppe wanghoppe commented Apr 19, 2024

Related command

Description

Upgrade azure batch API version to 2024-02-01
Upgrade azure-batch to 14.2.0
Upgrade azure-mgmt-batch to 17.3.0

Testing Guide

History Notes

[Batch] az batch pool create Add --upgrade-policy-mode argument to support automatic OS upgrade
[Batch] az batch pool create Add --enable-auto-os-upgrade argument to enable automatic OS upgrade
[Batch] az batch pool create Add --disable-auto-rollback argument to disable OS image rollback feature
[Batch] az batch pool create Add --defer-os-rolling-upgrade argument to defer OS upgrades on the TVMs if they are running tasks
[Batch] az batch pool create Add --use-rolling-upgrade-policy argument to support OS rolling upgrade policy
[Batch] az batch pool create Add --enable-cross-zone-upgrade argument to support cross zone OS upgrade
[Batch] az batch pool create Add --max-batch-instance-percent argument to set the maximum percent of total VMs that will be upgraded in one batch
[Batch] az batch pool create Add --max-unhealthy-instance-percent argument to set the maximum percentage of the total VMs can be simultaneously unhealthy
[Batch] az batch pool create Add --max-unhealthy-upgraded-instance-percent argument to set the maximum percentage of upgraded VMs that can be found to be in an unhealthy state
[Batch] az batch pool create Add --pause-time-between-batches argument to set the wait time between batches in rolling OS upgrade
[Batch] az batch pool create Add --prioritize-unhealthy-instances argument to support upgrade all unhealthy VMs first
[Batch] az batch pool create Add --rollback-failed-instances-on-policy-breach argument to enable rollback failed instances to previous model if the Rolling Upgrade policy is violated


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

@azure-client-tools-bot-prd
Copy link
Copy Markdown

azure-client-tools-bot-prd bot commented Apr 19, 2024

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

@azure-client-tools-bot-prd
Copy link
Copy Markdown

Hi @wanghoppe,
Since the current milestone time is less than 7 days, this pr will be reviewed in the next milestone.

@azure-client-tools-bot-prd
Copy link
Copy Markdown

azure-client-tools-bot-prd bot commented Apr 19, 2024

⚠️AzureCLI-BreakingChangeTest
⚠️batch
rule cmd_name rule_message suggest_message
⚠️ 1006 - ParaAdd batch pool create cmd batch pool create added parameter disable_automatic_rollback
⚠️ 1006 - ParaAdd batch pool create cmd batch pool create added parameter enable_automatic_os_upgrade
⚠️ 1006 - ParaAdd batch pool create cmd batch pool create added parameter enable_cross_zone_upgrade
⚠️ 1006 - ParaAdd batch pool create cmd batch pool create added parameter max_batch_instance_percent
⚠️ 1006 - ParaAdd batch pool create cmd batch pool create added parameter max_unhealthy_instance_percent
⚠️ 1006 - ParaAdd batch pool create cmd batch pool create added parameter max_unhealthy_upgraded_instance_percent
⚠️ 1006 - ParaAdd batch pool create cmd batch pool create added parameter mode
⚠️ 1006 - ParaAdd batch pool create cmd batch pool create added parameter os_rolling_upgrade_deferral
⚠️ 1006 - ParaAdd batch pool create cmd batch pool create added parameter pause_time_between_batches
⚠️ 1006 - ParaAdd batch pool create cmd batch pool create added parameter prioritize_unhealthy_instances
⚠️ 1006 - ParaAdd batch pool create cmd batch pool create added parameter rollback_failed_instances_on_policy_breach
⚠️ 1006 - ParaAdd batch pool create cmd batch pool create added parameter use_rolling_upgrade_policy

@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Apr 19, 2024

Batch

@wanghoppe
Copy link
Copy Markdown
Member Author

wanghoppe commented Apr 19, 2024

Hi @zhoxing-ms, I saw you are the first assigned reviewers, and we'd like to include this change in the current milestone, could you review it as soon as possible? Thanks!

@azure-pipelines
Copy link
Copy Markdown

Commenter does not have sufficient privileges for PR 28784 in repo Azure/azure-cli

@zhoxing-ms
Copy link
Copy Markdown
Contributor

[Batch] az batch pool create add new parameters --upgrade-policy-mode, --disable-auto-rollback, --enable-auto-os-upgrade, --defer-os-rolling-upgrade, --use-rolling-upgrade-policy, --enable-cross-zone-upgrade, --max-batch-instance-percent, -max-unhealthy-instance-percent, --max-unhealthy-upgraded-instance-percent, --pause-time-between-batches, --prioritize-unhealthy-instances, and --rollback-failed-instances-on-policy-breach to support automatic OS upgrade

Please refer to this guideline guideline_link to improve the History Notes in your PR description

Comment on lines +285 to +291
c.argument('disable_automatic_rollback', options_list=['--disable-auto-rollback'], arg_type=get_three_state_flag())
c.argument('enable_automatic_os_upgrade', options_list=['--enable-auto-os-upgrade'], arg_type=get_three_state_flag())
c.argument('os_rolling_upgrade_deferral', options_list=['--defer-os-rolling-upgrade'], arg_type=get_three_state_flag())
c.argument('use_rolling_upgrade_policy', arg_type=get_three_state_flag())
c.argument('enable_cross_zone_upgrade', arg_type=get_three_state_flag())
c.argument('prioritize_unhealthy_instances', arg_type=get_three_state_flag())
c.argument('rollback_failed_instances_on_policy_breach', arg_type=get_three_state_flag())
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why don't you add help messages for these new parameters?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I find those arguments have default help message from our python SDK which are already good enough so omitting here.

@wanghoppe
Copy link
Copy Markdown
Member Author

wanghoppe commented Apr 23, 2024

[Batch] az batch pool create add new parameters --upgrade-policy-mode, --disable-auto-rollback, --enable-auto-os-upgrade, --defer-os-rolling-upgrade, --use-rolling-upgrade-policy, --enable-cross-zone-upgrade, --max-batch-instance-percent, -max-unhealthy-instance-percent, --max-unhealthy-upgraded-instance-percent, --pause-time-between-batches, --prioritize-unhealthy-instances, and --rollback-failed-instances-on-policy-breach to support automatic OS upgrade

Please refer to this guideline guideline_link to improve the History Notes in your PR description

I capitalized the "Add" keywords according to the guide, not sure is there anything else needs to be changed?

@zhoxing-ms
Copy link
Copy Markdown
Contributor

zhoxing-ms commented Apr 23, 2024

@wanghoppe No, what I mean is that you need to clearly describe each change in the PR description one by one, so that customers can understand the specific functions of these changes. For example:
image

@zhoxing-ms zhoxing-ms merged commit b0d36d2 into Azure:dev Apr 23, 2024
@wanghoppe wanghoppe deleted the hoppe/0201 branch June 25, 2024 17:46
@wanghoppe wanghoppe restored the hoppe/0201 branch June 25, 2024 17:46
@wanghoppe wanghoppe deleted the hoppe/0201 branch June 25, 2024 17:46
@evelyn-ys
Copy link
Copy Markdown
Member

@wanghoppe Currently we are working on dropping all dependencies on msrestazure and we detected that the latest azure-batch still relies on msrestazure which is already deprecated.

Can you help remove msrestazure dependency from azure-batch and bump it in Azure CLI? Really appreciated.

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 Batch az batch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants