-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
{Batch} Upgrade api-version to v2023-11-01 #28315
Conversation
️✔️AzureCLI-FullTest
|
|
rule | cmd_name | rule_message | suggest_message |
---|---|---|---|
batch pool create | cmd batch pool create added parameter caching |
||
batch pool create | cmd batch pool create added parameter disk_size_gb |
||
batch pool create | cmd batch pool create added parameter encryption_at_host |
||
batch pool create | cmd batch pool create added parameter placement |
||
batch pool create | cmd batch pool create added parameter resource_tags |
||
batch pool create | cmd batch pool create added parameter secure_boot_enabled |
||
batch pool create | cmd batch pool create added parameter security_type |
||
batch pool create | cmd batch pool create added parameter storage_account_type |
||
batch pool create | cmd batch pool create added parameter v_tpm_enabled |
||
batch pool create | cmd batch pool create added parameter write_accelerator_enabled |
Batch |
a80c653
to
c9cf612
Compare
c9cf612
to
c168049
Compare
Co-authored-by: Xing Zhou <Zhou.Xing@microsoft.com>
Co-authored-by: Xing Zhou <Zhou.Xing@microsoft.com>
Co-authored-by: Xing Zhou <Zhou.Xing@microsoft.com>
Could you please resolve these CI issues? #28315 (comment) |
Please note that we are launching the release for this sprint this week. Please resolve all comments by tomorrow, otherwise the release of this PR will have to be postponed to the next sprint (on 04-02) |
self.cmd( | ||
'batch account login -g {rg} -n {byos_n}' | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This command interferes with batch account login
from test_batch_general_arm_cmd
and causes failure when test_batch_byos_account_cmd
and test_batch_general_arm_cmd
are executed in parallel: #28500
Related command
az batch
Description
--resource-tags
support to pool addSecurityProfile
support toVirtualMachineConfiguration
on pool add--security-type
,--encryption-at-host
,--secure-boot-enabled
, and--v-tpm-enabled
OSDisk
support toVirtualMachineConfiguration
on pool add--caching
,--disk-size-gb
,--write-accelerator-enabled
, and--storage-account-type
Testing Guide
History Notes
[Batch]
az batch pool create
: Add new parameter--resource-tags
to support specifying resource tags for the pool. Any resource created for the pool will then also be tagged by the same resource tags[Batch]
az batch pool create
: Add new parameters--security-type
,--encryption-at-host
,--secure-boot-enabled
, and--v-tpm-enabled
to support Trusted Launch Security Type for VMs/VMSS deployments[Batch]
az batch pool create
: Add new parameters--caching
,--disk-size-gb
,--write-accelerator-enabled
, and--storage-account-type
to support Batch Node Agent temp disk-less SKUsThis checklist is used to make sure that common guidelines for a pull request are followed.
The PR title and description has followed the guideline in Submitting Pull Requests.
I adhere to the Command Guidelines.
I adhere to the Error Handling Guidelines.