Azure / azure-powershell

Microsoft Azure PowerShell
Other
4.27k stars 3.87k forks source link

Add Diagnostics and Warnings to Deployments WhatIf/Validate cmdlets #26732

Closed tsmallig33 closed 4 days ago

tsmallig33 commented 6 days ago

Description

Add diagnostic messages and warnings to Resource Deployments WhatIf and Test cmdlets (e.g. TestDeploymentCmdletBaseand DeploymentWhatIfCmdlet)

Mandatory Checklist

azure-client-tools-bot-prd[bot] commented 6 days ago
️✔️Az.Accounts >
> ️✔️Build > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>>
>> ️✔️Windows PowerShell - Windows >> >> >>
>
⚠️Az.Blueprint >
> ️✔️Build > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>>
>> ️✔️Windows PowerShell - Windows >> >> >>
>
>
> ⚠️Test > >>
>> ⚠️ - Linux >> >>|Type|Title|Current Coverage|Description| >>|---|---|---|---| >>|⚠️|Test Coverage Less Than 50%|23.08 %|Test coverage for the module cannot be lower than 50%.| >> >>
>>
>> ⚠️ - MacOS >> >>|Type|Title|Current Coverage|Description| >>|---|---|---|---| >>|⚠️|Test Coverage Less Than 50%|23.08%|Test coverage for the module cannot be lower than 50%.| >> >>
>>
>> ⚠️PowerShell Core - Windows >> >>|Type|Title|Current Coverage|Description| >>|---|---|---|---| >>|⚠️|Test Coverage Less Than 50%|23.08%|Test coverage for the module cannot be lower than 50%.| >> >>
>>
>> ⚠️Windows PowerShell - Windows >> >>|Type|Title|Current Coverage|Description| >>|---|---|---|---| >>|⚠️|Test Coverage Less Than 50%|23.08%|Test coverage for the module cannot be lower than 50%.| >> >>
>
️✔️Az.CosmosDB >
> ️✔️Build > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>>
>> ️✔️Windows PowerShell - Windows >> >> >>
>
>
> ️✔️Test > >>
>> ️✔️ - Linux >> >> >>
>>
>> ️✔️ - MacOS >> >> >>
>>
>> ️✔️PowerShell Core - Windows >> >> >>
>>
>> ️✔️Windows PowerShell - Windows >> >> >>
>
️✔️Az.KeyVault >
> ️✔️Build > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>>
>> ️✔️Windows PowerShell - Windows >> >> >>
>
⚠️Az.MachineLearning >
> ️✔️Build > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>>
>> ️✔️Windows PowerShell - Windows >> >> >>
>
>
> ⚠️Test > >>
>> ⚠️ - Linux >> >>|Type|Title|Current Coverage|Description| >>|---|---|---|---| >>|⚠️|Test Coverage Less Than 50%|0.00 %|Test coverage for the module cannot be lower than 50%.| >> >>
>>
>> ⚠️ - MacOS >> >>|Type|Title|Current Coverage|Description| >>|---|---|---|---| >>|⚠️|Test Coverage Less Than 50%|0.00%|Test coverage for the module cannot be lower than 50%.| >> >>
>>
>> ⚠️PowerShell Core - Windows >> >>|Type|Title|Current Coverage|Description| >>|---|---|---|---| >>|⚠️|Test Coverage Less Than 50%|0.00%|Test coverage for the module cannot be lower than 50%.| >> >>
>>
>> ⚠️Windows PowerShell - Windows >> >>|Type|Title|Current Coverage|Description| >>|---|---|---|---| >>|⚠️|Test Coverage Less Than 50%|0.00%|Test coverage for the module cannot be lower than 50%.| >> >>
>
️✔️Az.ManagedServiceIdentity >
> ️✔️Build > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>>
>> ️✔️Windows PowerShell - Windows >> >> >>
>
️✔️Az.Monitor >
> ️✔️Build > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>>
>> ️✔️Windows PowerShell - Windows >> >> >>
>
️✔️Az.Network >
> ️✔️Build > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>>
>> ️✔️Windows PowerShell - Windows >> >> >>
>
⚠️Az.Resources >
> ️✔️Build > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>>
>> ️✔️Windows PowerShell - Windows >> >> >>
>
>
> ️✔️Breaking Change Check > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>>
>> ️✔️Windows PowerShell - Windows >> >> >>
>
>
> ⚠️Signature Check > >>
>> ⚠️PowerShell Core - Windows >> >>|Type|Cmdlet|Description|Remediation| >>|---|---|---|---| >>|⚠️|Get-AzADGroupOwner|Get-AzADGroupOwner Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzADGroupOwner|Get-AzADGroupOwner changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzADServicePrincipalAppRoleAssignment|Get-AzADServicePrincipalAppRoleAssignment Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzADServicePrincipalAppRoleAssignment|Get-AzADServicePrincipalAppRoleAssignment changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzDataBoundaryScope|Get-AzDataBoundaryScope Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzDataBoundaryScope|Get-AzDataBoundaryScope changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzDataBoundaryTenant|Get-AzDataBoundaryTenant Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzDataBoundaryTenant|Get-AzDataBoundaryTenant changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzPolicyAssignment|Get-AzPolicyAssignment Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzPolicyAssignment|Get-AzPolicyAssignment changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzPolicyDefinition|Get-AzPolicyDefinition Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzPolicyDefinition|Get-AzPolicyDefinition changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzPolicyExemption|Get-AzPolicyExemption Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzPolicyExemption|Get-AzPolicyExemption changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzPolicySetDefinition|Get-AzPolicySetDefinition Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzPolicySetDefinition|Get-AzPolicySetDefinition changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >> >>
>>
>> ⚠️Windows PowerShell - Windows >> >>|Type|Cmdlet|Description|Remediation| >>|---|---|---|---| >>|⚠️|Get-AzADGroupOwner|Get-AzADGroupOwner Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzADGroupOwner|Get-AzADGroupOwner changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzADServicePrincipalAppRoleAssignment|Get-AzADServicePrincipalAppRoleAssignment Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzADServicePrincipalAppRoleAssignment|Get-AzADServicePrincipalAppRoleAssignment changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzDataBoundaryScope|Get-AzDataBoundaryScope Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzDataBoundaryScope|Get-AzDataBoundaryScope changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzDataBoundaryTenant|Get-AzDataBoundaryTenant Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzDataBoundaryTenant|Get-AzDataBoundaryTenant changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzPolicyAssignment|Get-AzPolicyAssignment Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzPolicyAssignment|Get-AzPolicyAssignment changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzPolicyDefinition|Get-AzPolicyDefinition Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzPolicyDefinition|Get-AzPolicyDefinition changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzPolicyExemption|Get-AzPolicyExemption Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzPolicyExemption|Get-AzPolicyExemption changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >>|⚠️|Get-AzPolicySetDefinition|Get-AzPolicySetDefinition Changes the ConfirmImpact but does not set the SupportsShouldProcess property to true in the cmdlet attribute.|Determine if the cmdlet should implement ShouldProcess and if so determine if it should implement Force / ShouldContinue| >>|⚠️|Get-AzPolicySetDefinition|Get-AzPolicySetDefinition changes the confirm impact. Please ensure that the change in ConfirmImpact is justified|Verify that ConfirmImpact is changed appropriately by the cmdlet. It is very rare for a cmdlet to change the ConfirmImpact.| >> >>
>
>
> ️✔️Help File Existence Check > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>>
>> ️✔️Windows PowerShell - Windows >> >> >>
>
>
> ️✔️File Change Check > >>
>> ️✔️PowerShell Core - Windows >> >> >>
>>
>> ️✔️Windows PowerShell - Windows >> >> >>
>
>
> ⚠️UX Metadata Check > >>
>> ⚠️PowerShell Core - Windows >> >>|Type|Module|ResourceType|SubResourceType|Command|Description| >>|---|---|---|---|---|---| >>|⚠️|Az.Resources|Microsoft.Resources|subscriptionsResourcegroups|Get-AzResourceGroup|The path /subscriptions/{subscriptionId}/resourceGroups/{name} doesn't contains the right resource tpye: Microsoft.Resources| >>|⚠️|Az.Resources|Microsoft.Resources|subscriptionsResourcegroups|Remove-AzResourceGroup|The path /subscriptions/{subscriptionId}/resourceGroups/{name} doesn't contains the right resource tpye: Microsoft.Resources| >> >>
>>
>> ⚠️Windows PowerShell - Windows >> >>|Type|Module|ResourceType|SubResourceType|Command|Description| >>|---|---|---|---|---|---| >>|⚠️|Az.Resources|Microsoft.Resources|subscriptionsResourcegroups|Get-AzResourceGroup|The path /subscriptions/{subscriptionId}/resourceGroups/{name} doesn't contains the right resource tpye: Microsoft.Resources| >>|⚠️|Az.Resources|Microsoft.Resources|subscriptionsResourcegroups|Remove-AzResourceGroup|The path /subscriptions/{subscriptionId}/resourceGroups/{name} doesn't contains the right resource tpye: Microsoft.Resources| >> >>
>
>
> ⚠️Test > >>
>> ⚠️ - Linux >> >>|Type|Title|Current Coverage|Description| >>|---|---|---|---| >>|⚠️|Test Coverage Less Than 50%|49.76 %|Test coverage for the module cannot be lower than 50%.| >> >>
>>
>> ⚠️ - MacOS >> >>|Type|Title|Current Coverage|Description| >>|---|---|---|---| >>|⚠️|Test Coverage Less Than 50%|49.76%|Test coverage for the module cannot be lower than 50%.| >> >>
>>
>> ⚠️PowerShell Core - Windows >> >>|Type|Title|Current Coverage|Description| >>|---|---|---|---| >>|⚠️|Test Coverage Less Than 50%|49.76%|Test coverage for the module cannot be lower than 50%.| >> >>
>>
>> ⚠️Windows PowerShell - Windows >> >>|Type|Title|Current Coverage|Description| >>|---|---|---|---| >>|⚠️|Test Coverage Less Than 50%|49.76%|Test coverage for the module cannot be lower than 50%.| >> >>
>
VeryEarly commented 4 days ago

/azp run azure-powershell - security-tools

azure-pipelines[bot] commented 4 days ago
Azure Pipelines successfully started running 1 pipeline(s).
VeryEarly commented 4 days ago

/azp run azure-powershell - security-tools

azure-pipelines[bot] commented 4 days ago
Azure Pipelines successfully started running 1 pipeline(s).