[Az.Databricks] Update generation tool version: autorest.powershell v3->v4 #28554
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,9 +1,9 @@ | ||
| # region Generated | ||
| # Load the private module dll | ||
| $null = Import-Module -PassThru -Name (Join-Path $PSScriptRoot '..\bin\Az.Databricks.private.dll') | ||
|
|
||
| # Load the internal module | ||
| $internalModulePath = Join-Path $PSScriptRoot '..\internal\Az.Databricks.internal.psm1' | ||
|
Comment on lines
+3
to
+6
There was a problem hiding this comment. These Join-Path child paths use Windows-style backslashes (e.g., '..\bin...'). This can break on non-Windows platforms because the backslash may be treated as a literal character. Prefer passing clean relative segments (e.g., '../bin/Az.Databricks.private.dll') or joining multiple segments so path separators are platform-correct. |
||
| if(Test-Path $internalModulePath) { | ||
| $null = Import-Module -Name $internalModulePath | ||
| } | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -24,106 +24,130 @@ Update-AzDatabricksVNetPeering -Name vnet-peering-t1 -WorkspaceName azps-databri | |||||
| Get-AzDatabricksVNetPeering -WorkspaceName azps-databricks-workspace-t1 -ResourceGroupName azps_test_gp_db -Name vnet-peering-t1 | Update-AzDatabricksVNetPeering -AllowGatewayTransit $true | ||||||
|
|
||||||
| .Outputs | ||||||
| Microsoft.Azure.PowerShell.Cmdlets.Databricks.Models.IVirtualNetworkPeering | ||||||
| .Link | ||||||
| https://learn.microsoft.com/powershell/module/az.databricks/update-azdatabricksvnetpeering | ||||||
| #> | ||||||
| function Update-AzDatabricksVNetPeering { | ||||||
| [OutputType([Microsoft.Azure.PowerShell.Cmdlets.Databricks.Models.IVirtualNetworkPeering])] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Runtime.OutputBreakingChangeAttribute("Microsoft.Azure.PowerShell.Cmdlets.Databricks.Models.IVirtualNetworkPeering", "16.0.0", "2.0.0", "May 2026", ReplacementCmdletOutputType = "Microsoft.Azure.PowerShell.Cmdlets.Databricks.Models.IVirtualNetworkPeering", DeprecatedOutputProperties = ("DatabrickAddressSpaceAddressPrefix, RemoteAddressSpaceAddressPrefix 'System.String[]'"), NewOutputProperties = ("DatabrickAddressSpaceAddressPrefix, RemoteAddressSpaceAddressPrefix 'System.Collections.Generic.List`1[System.String]'"))] | ||||||
|
There was a problem hiding this comment. The breaking-change effective date is set to "May 2026". In this repo, breaking change attributes use a strict "YYYY/MM/DD" date format (e.g., "2025/11/03" in src/CloudService/CloudService.Autorest/custom/New-AzCloudService.ps1:25). Please update this value to the required format.
Suggested change
|
||||||
| [CmdletBinding(DefaultParameterSetName='UpdateExpanded', PositionalBinding=$false, SupportsShouldProcess, ConfirmImpact='Medium')] | ||||||
| param( | ||||||
| [Parameter(ParameterSetName='UpdateExpanded', Mandatory)] | ||||||
| [Parameter(ParameterSetName='UpdateViaJsonFilePath', Mandatory)] | ||||||
| [Parameter(ParameterSetName='UpdateViaJsonString', Mandatory)] | ||||||
| [Alias('PeeringName')] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Path')] | ||||||
| [System.String] | ||||||
| # The name of the workspace vNet peering. | ||||||
| ${Name}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateExpanded', Mandatory)] | ||||||
| [Parameter(ParameterSetName='UpdateViaJsonFilePath', Mandatory)] | ||||||
| [Parameter(ParameterSetName='UpdateViaJsonString', Mandatory)] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Path')] | ||||||
| [System.String] | ||||||
| # The name of the resource group. | ||||||
| # The name is case insensitive. | ||||||
| ${ResourceGroupName}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateExpanded', Mandatory)] | ||||||
| [Parameter(ParameterSetName='UpdateViaJsonFilePath', Mandatory)] | ||||||
| [Parameter(ParameterSetName='UpdateViaJsonString', Mandatory)] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Path')] | ||||||
| [System.String] | ||||||
| # The name of the workspace. | ||||||
| ${WorkspaceName}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateExpanded')] | ||||||
| [Parameter(ParameterSetName='UpdateViaJsonFilePath')] | ||||||
| [Parameter(ParameterSetName='UpdateViaJsonString')] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Path')] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Runtime.DefaultInfo(Script='(Get-AzContext).Subscription.Id')] | ||||||
| [System.String] | ||||||
| # The ID of the target subscription. | ||||||
| # The value must be an UUID. | ||||||
| ${SubscriptionId}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateViaIdentityExpanded', Mandatory, ValueFromPipeline)] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Path')] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Models.IDatabricksIdentity] | ||||||
| # Identity Parameter | ||||||
| ${InputObject}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateExpanded')] | ||||||
| [Parameter(ParameterSetName='UpdateViaIdentityExpanded')] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Body')] | ||||||
| [System.Management.Automation.SwitchParameter] | ||||||
| # Whether the forwarded traffic from the VMs in the local virtual network will be allowed/disallowed in remote virtual network. | ||||||
| ${AllowForwardedTraffic}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateExpanded')] | ||||||
| [Parameter(ParameterSetName='UpdateViaIdentityExpanded')] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Body')] | ||||||
| [System.Management.Automation.SwitchParameter] | ||||||
| # If gateway links can be used in remote virtual networking to link to this virtual network. | ||||||
| ${AllowGatewayTransit}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateExpanded')] | ||||||
| [Parameter(ParameterSetName='UpdateViaIdentityExpanded')] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Body')] | ||||||
| [System.Management.Automation.SwitchParameter] | ||||||
|
Collaborator
There was a problem hiding this comment. why change the type from boolean to switch?
Contributor
There was a problem hiding this comment. Hi @VeryEarly, this change in parameter type is due to the internals of Update-AzDatabricksVNetPeering. The AllowVirtualNetworkAccess parameter is defined as a SwitchParameter internally, so we're following the internal parameter type definition. |
||||||
| # Whether the VMs in the local virtual network space would be able to access the VMs in remote virtual network space. | ||||||
| ${AllowVirtualNetworkAccess}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateExpanded')] | ||||||
| [Parameter(ParameterSetName='UpdateViaIdentityExpanded')] | ||||||
| [AllowEmptyCollection()] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Body')] | ||||||
| [System.String[]] | ||||||
| # A list of address blocks reserved for this virtual network in CIDR notation. | ||||||
| ${DatabricksAddressSpacePrefix}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateExpanded')] | ||||||
| [Parameter(ParameterSetName='UpdateViaIdentityExpanded')] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Body')] | ||||||
| [System.String] | ||||||
| # The Id of the databricks virtual network. | ||||||
| ${DatabricksVirtualNetworkId}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateExpanded')] | ||||||
| [Parameter(ParameterSetName='UpdateViaIdentityExpanded')] | ||||||
| [AllowEmptyCollection()] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Body')] | ||||||
| [System.String[]] | ||||||
| # A list of address blocks reserved for this virtual network in CIDR notation. | ||||||
| ${RemoteAddressSpacePrefix}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateExpanded')] | ||||||
| [Parameter(ParameterSetName='UpdateViaIdentityExpanded')] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Body')] | ||||||
| [System.String] | ||||||
| # The Id of the remote virtual network. | ||||||
| ${RemoteVirtualNetworkId}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateExpanded')] | ||||||
| [Parameter(ParameterSetName='UpdateViaIdentityExpanded')] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Body')] | ||||||
| [System.Management.Automation.SwitchParameter] | ||||||
| # If remote gateways can be used on this virtual network. | ||||||
| # If the flag is set to true, and allowGatewayTransit on remote peering is also true, virtual network will use gateways of remote virtual network for transit. | ||||||
| # Only one peering can have this flag set to true. | ||||||
| # This flag cannot be set if virtual network already has a gateway. | ||||||
| ${UseRemoteGateway}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateViaJsonFilePath', Mandatory)] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Body')] | ||||||
| [System.String] | ||||||
| # Path of Json file supplied to the Update operation | ||||||
| ${JsonFilePath}, | ||||||
|
|
||||||
| [Parameter(ParameterSetName='UpdateViaJsonString', Mandatory)] | ||||||
| [Microsoft.Azure.PowerShell.Cmdlets.Databricks.Category('Body')] | ||||||
| [System.String] | ||||||
| # Json string supplied to the Update operation | ||||||
| ${JsonString}, | ||||||
|
|
||||||
| [Parameter()] | ||||||
| [Alias('AzureRMContext', 'AzureCredential')] | ||||||
| [ValidateNotNull()] | ||||||
|
|
||||||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
A new
model-cmdletdirective was added without an explanatory comment. In AutoRest README.md files, directives should be commented to explain the rationale/purpose so future maintainers can safely modify regeneration settings.