-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Batch breaking changes #28767
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 breaking changes #28767
Conversation
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
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.
Pull Request Overview
This PR removes deprecated certificate-related cmdlets and deprecated pool configuration properties from the Az.Batch module as part of version 4.0.0. The changes align with Azure Batch service deprecations for certificate management and cloud service configuration features.
Key changes:
- Removes 5 certificate-related cmdlets and their associated code
- Removes deprecated pool configuration parameters including CloudServiceConfiguration, CertificateReferences, ApplicationLicenses, ResourceTags, and node communication mode properties
- Adds new properties to support updated Azure Batch API features (deallocated node states, gallery images, VNet management, disk security, container bind mounts)
- Updates SDK dependency from Microsoft.Azure.Batch 16.2.0 to 16.3.1
- Updates test recordings to use new API version (2024-07-01.20.0)
Reviewed Changes
Copilot reviewed 33 out of 202 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| ChangeLog.md | Documents all breaking changes and new features for version 4.0.0 |
| Help documentation files | Removes documentation for deprecated certificate commands and pool parameters |
| Command files | Removes certificate and RDP file command implementations |
| Model files | Removes deprecated model classes and properties from pool-related types |
| Utils.cs | Removes certificate reference conversion logic |
| BatchClient files | Removes certificate operations and deprecated pool property handling |
| Batch.csproj | Updates SDK version and package references |
| Az.Batch.psd1 | Updates module manifest to remove deprecated cmdlets |
| Test files | Updates test recordings and removes deprecated test scenarios |
|
To the author of the pull request, |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
c3c2145 to
9ccd9ff
Compare
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command. |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command. |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
@wiboris, Could you help authorize a markdown like this for each of the breaking change? We'll need it for publishing migration guides. Just leave a comment in the issue would be fine. Thank you! ## Az.Accounts
### `Get-AzAccessToken`
The default output type is changed from `PSAccessToken` to `PSSecureAccessToken`.That is to change plaintext `PSAccessToken.Token` to `SecureString PSSecureAccessToken.Token`
#### Before
```powershell
$authHeader = @{
'Content-Type' = 'application/json'
'Authorization' = 'Bearer ' + (Get-AccessToken).Token
}
$response = Invoke-RestMethod -Method Get -Headers $authHeader -Uri $uri
```
#### After
```powershell
$secureToken = (Get-AzAccessToken).Token
$ssPtr = [System.Runtime.InteropServices.Marshal]::SecureStringToBSTR($secureToken)
try {
$plaintextToken = [System.Runtime.InteropServices.Marshal]::PtrToStringBSTR($ssPtr)
}
finally {
[System.Runtime.InteropServices.Marshal]::ZeroFreeBSTR($ssPtr)
}
$authHeader = @{
'Content-Type' = 'application/json'
'Authorization' = 'Bearer ' + $plaintextToken
}
$response = Invoke-RestMethod -Method Get -Headers $authHeader -Uri $uri
``` |
Az.Batch
|
Description
Updating the batch api's to 2024-07-01 and removing deprecated commands and parameters that were documented in the last release.
Removed command
Get-AzBatchRemoteDesktopProtocolFile.Removed command
Get-AzBatchCertificate.Removed command
New-AzBatchCertificate.Removed command
Remove-AzBatchCertificate.Removed command
Stop-AzBatchCertificateDeletion.Removed parameter
CloudServiceConfigurationfromNew-AzBatchPoolcommand.Removed parameter
CertificateReferencesfromNew-AzBatchPoolcommand.Removed parameter
TargetNodeCommunicationfromNew-AzBatchPoolcommand.Removed parameter
CurrentNodeCommunicationfromNew-AzBatchPoolcommand.Removed parameter
ResourceTagfromNew-AzBatchPoolcommand.Removed parameter
ApplicationLicensesfromNew-AzBatchPoolcommand.Removed properties
CloudServiceConfiguration,CertificateReferences,TargetNodeCommunication,CurrentNodeCommunication,ResourceTag, andApplicationLicensesfromPsCloudPoolmodel.Removed property
CertificateReferencesfromPsComputeNodemodel.Removed properties
CloudServiceConfiguration,CertificateReferences,TargetNodeCommunication,CurrentNodeCommunication,ResourceTag, andApplicationLicensesfromPsPoolSpecificationmodel.Added new enum values
DeallocatedandDeallocatingtoPSNodeCountsmodel.Added properties
CommunityGalleryImageIdandSharedGalleryImageIdtoPSImageReferencemodel.Added property
SkipWithdrawFromVNettoPSJobNetworkConfigurationmodel.Added property
SecurityProfiletoPSManagedDiskmodel.Added property
ContainerHostBatchBindMountstoPSTaskContainerSettingsmodel.Mandatory Checklist
Please choose the target release of Azure PowerShell. (⚠️ Target release is a different concept from API readiness. Please click below links for details.)
Check this box to confirm: I have read the Submitting Changes section of
CONTRIBUTING.mdand reviewed the following information:ChangeLog.mdfile(s) appropriatelysrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.## Upcoming Releaseheader in the past tense.ChangeLog.mdif no new release is required, such as fixing test case only.