Skip to content
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

New VMSS Instance Mix Flex Create and Update Parameters for Powershell #25980

Merged
merged 9 commits into from
Sep 6, 2024

Conversation

msdorahu
Copy link
Contributor

Description

Design spec: https://github.com/Azure/azure-powershell-cmdlet-review-pr/issues/1456

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • For SDK-based development mode, update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • For autorest-based development mode, include the changelog in the PR description.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

Copy link

azure-client-tools-bot-prd bot commented Aug 30, 2024

️✔️Az.Accounts
️✔️Build
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
⚠️Az.Compute
️✔️Build
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️Breaking Change Check
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️Signature Check
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
⚠️Help Example Check
⚠️PowerShell Core - Windows
Type Cmdlet Example Line RuleName Description Extent Remediation
⚠️ New-AzVmssConfig 1 1 Unassigned_Variable New-AzVmssConfig -Location $Loc is a null-valued parameter value. -Location Assign value for $Loc.
⚠️ New-AzVmssConfig 1 1 Unassigned_Variable New-AzVmssConfig -NetworkInterfaceConfiguration $NetCfg is a null-valued parameter value. -NetworkInterfaceConfiguration Assign value for $NetCfg.
⚠️ New-AzVmssConfig 1 2 Unassigned_Variable Add-AzVmssNetworkInterfaceConfiguration -IPConfiguration $IPCfg is a null-valued parameter value. -IPConfiguration Assign value for $IPCfg.
⚠️ New-AzVmssConfig 1 3 Unassigned_Variable Set-AzVmssOsProfile -AdminUsername $adminUsername is a null-valued parameter value. -AdminUsername Assign value for $adminUsername.
⚠️ New-AzVmssConfig 1 3 Unassigned_Variable Set-AzVmssOsProfile -AdminPassword $AdminPassword is a null-valued parameter value. -AdminPassword Assign value for $AdminPassword.
⚠️ New-AzVmssConfig 1 5 Unassigned_Variable Set-AzVmssStorageProfile -ImageReferenceOffer $ImgRef.Offer is a null-valued parameter value. -ImageReferenceOffer Assign value for $ImgRef.Offer.
⚠️ New-AzVmssConfig 1 5 Unassigned_Variable Set-AzVmssStorageProfile -ImageReferenceSku $ImgRef.Skus is a null-valued parameter value. -ImageReferenceSku Assign value for $ImgRef.Skus.
⚠️ New-AzVmssConfig 1 5 Unassigned_Variable Set-AzVmssStorageProfile -ImageReferenceVersion $ImgRef.Version is a null-valued parameter value. -ImageReferenceVersion Assign value for $ImgRef.Version.
⚠️ New-AzVmssConfig 1 6 Unassigned_Variable Set-AzVmssStorageProfile -ImageReferencePublisher $ImgRef.PublisherName is a null-valued parameter value. -ImageReferencePublisher Assign value for $ImgRef.PublisherName.
⚠️ New-AzVmssConfig 1 6 Unassigned_Variable Set-AzVmssStorageProfile -VhdContainer $VHDContainer is a null-valued parameter value. -VhdContainer Assign value for $VHDContainer.
⚠️ New-AzVmssConfig 1 7 Unassigned_Variable Add-AzVmssAdditionalUnattendContent -ComponentName $AUCComponentName is a null-valued parameter value. -ComponentName Assign value for $AUCComponentName.
⚠️ New-AzVmssConfig 1 7 Unassigned_Variable Add-AzVmssAdditionalUnattendContent -Content $AUCContent is a null-valued parameter value. -Content Assign value for $AUCContent.
⚠️ New-AzVmssConfig 1 7 Unassigned_Variable Add-AzVmssAdditionalUnattendContent -PassName $AUCPassName is a null-valued parameter value. -PassName Assign value for $AUCPassName.
⚠️ New-AzVmssConfig 1 7 Unassigned_Variable Add-AzVmssAdditionalUnattendContent -SettingName $AUCSetting is a null-valued parameter value. -SettingName Assign value for $AUCSetting.
⚠️ New-AzVmssConfig 1 9 Unassigned_Variable New-AzVmss -ResourceGroupName $RGName is a null-valued parameter value. -ResourceGroupName Assign value for $RGName.
⚠️ New-AzVmssConfig 1 9 Unassigned_Variable New-AzVmss -Name $VMSSName is a null-valued parameter value. -Name Assign value for $VMSSName.
⚠️ New-AzVmssConfig 4 18 Unassigned_Variable New-AzStorageAccount -ResourceGroupName $rgname is a null-valued parameter value. -ResourceGroupName Assign value for $rgname.
⚠️ New-AzVmssConfig 4 18 Unassigned_Variable New-AzStorageAccount -Location $loc is a null-valued parameter value. -Location Assign value for $loc.
⚠️ New-AzVmssConfig 4 19 Unassigned_Variable Get-AzStorageAccount -ResourceGroupName $rgname is a null-valued parameter value. -ResourceGroupName Assign value for $rgname.
⚠️ New-AzVmssConfig 4 23 Unassigned_Variable New-AzVirtualNetwork -ResourceGroupName $rgname is a null-valued parameter value. -ResourceGroupName Assign value for $rgname.
⚠️ New-AzVmssConfig 4 23 Unassigned_Variable New-AzVirtualNetwork -Location $loc is a null-valued parameter value. -Location Assign value for $loc.
⚠️ New-AzVmssConfig 4 24 Unassigned_Variable Get-AzVirtualNetwork -ResourceGroupName $rgname is a null-valued parameter value. -ResourceGroupName Assign value for $rgname.
⚠️ New-AzVmssConfig 4 30 Unassigned_Variable New-AzVmssConfig -Location $loc is a null-valued parameter value. -Location Assign value for $loc.
⚠️ New-AzVmssConfig 4 32 Unassigned_Variable Set-AzVmssOsProfile -AdminPassword $password is a null-valued parameter value. -AdminPassword Assign value for $password.
⚠️ New-AzVmssConfig 4 37 Unassigned_Variable New-AzVmss -ResourceGroupName $rgname is a null-valued parameter value. -ResourceGroupName Assign value for $rgname.
⚠️ New-AzVmssConfig 4 39 Unassigned_Variable Get-AzVmss -ResourceGroupName $rgname is a null-valued parameter value. -ResourceGroupName Assign value for $rgname.
⚠️ New-AzVmssConfig 5 36 Mismatched_Parameter_Value_Type Set-AzVmssOsProfile -AdminPassword $adminPassword is not an expected parameter value type. -AdminPassword Use correct parameter value type. Expected Type is string. Now the type is securestring.(Command).
⚠️ Update-AzVmss 1 1 Unassigned_Variable Update-AzVmss -VirtualMachineScaleSet $LocalVMSS is a null-valued parameter value. -VirtualMachineScaleSet Assign value for $LocalVMSS.
⚠️Windows PowerShell - Windows
Type Cmdlet Example Line RuleName Description Extent Remediation
⚠️ New-AzVmssConfig 1 1 Unassigned_Variable New-AzVmssConfig -Location $Loc is a null-valued parameter value. -Location Assign value for $Loc.
⚠️ New-AzVmssConfig 1 1 Unassigned_Variable New-AzVmssConfig -NetworkInterfaceConfiguration $NetCfg is a null-valued parameter value. -NetworkInterfaceConfiguration Assign value for $NetCfg.
⚠️ New-AzVmssConfig 1 2 Unassigned_Variable Add-AzVmssNetworkInterfaceConfiguration -IPConfiguration $IPCfg is a null-valued parameter value. -IPConfiguration Assign value for $IPCfg.
⚠️ New-AzVmssConfig 1 3 Unassigned_Variable Set-AzVmssOsProfile -AdminUsername $adminUsername is a null-valued parameter value. -AdminUsername Assign value for $adminUsername.
⚠️ New-AzVmssConfig 1 3 Unassigned_Variable Set-AzVmssOsProfile -AdminPassword $AdminPassword is a null-valued parameter value. -AdminPassword Assign value for $AdminPassword.
⚠️ New-AzVmssConfig 1 5 Unassigned_Variable Set-AzVmssStorageProfile -ImageReferenceOffer $ImgRef.Offer is a null-valued parameter value. -ImageReferenceOffer Assign value for $ImgRef.Offer.
⚠️ New-AzVmssConfig 1 5 Unassigned_Variable Set-AzVmssStorageProfile -ImageReferenceSku $ImgRef.Skus is a null-valued parameter value. -ImageReferenceSku Assign value for $ImgRef.Skus.
⚠️ New-AzVmssConfig 1 5 Unassigned_Variable Set-AzVmssStorageProfile -ImageReferenceVersion $ImgRef.Version is a null-valued parameter value. -ImageReferenceVersion Assign value for $ImgRef.Version.
⚠️ New-AzVmssConfig 1 6 Unassigned_Variable Set-AzVmssStorageProfile -ImageReferencePublisher $ImgRef.PublisherName is a null-valued parameter value. -ImageReferencePublisher Assign value for $ImgRef.PublisherName.
⚠️ New-AzVmssConfig 1 6 Unassigned_Variable Set-AzVmssStorageProfile -VhdContainer $VHDContainer is a null-valued parameter value. -VhdContainer Assign value for $VHDContainer.
⚠️ New-AzVmssConfig 1 7 Unassigned_Variable Add-AzVmssAdditionalUnattendContent -ComponentName $AUCComponentName is a null-valued parameter value. -ComponentName Assign value for $AUCComponentName.
⚠️ New-AzVmssConfig 1 7 Unassigned_Variable Add-AzVmssAdditionalUnattendContent -Content $AUCContent is a null-valued parameter value. -Content Assign value for $AUCContent.
⚠️ New-AzVmssConfig 1 7 Unassigned_Variable Add-AzVmssAdditionalUnattendContent -PassName $AUCPassName is a null-valued parameter value. -PassName Assign value for $AUCPassName.
⚠️ New-AzVmssConfig 1 7 Unassigned_Variable Add-AzVmssAdditionalUnattendContent -SettingName $AUCSetting is a null-valued parameter value. -SettingName Assign value for $AUCSetting.
⚠️ New-AzVmssConfig 1 9 Unassigned_Variable New-AzVmss -ResourceGroupName $RGName is a null-valued parameter value. -ResourceGroupName Assign value for $RGName.
⚠️ New-AzVmssConfig 1 9 Unassigned_Variable New-AzVmss -Name $VMSSName is a null-valued parameter value. -Name Assign value for $VMSSName.
⚠️ New-AzVmssConfig 4 18 Unassigned_Variable New-AzStorageAccount -ResourceGroupName $rgname is a null-valued parameter value. -ResourceGroupName Assign value for $rgname.
⚠️ New-AzVmssConfig 4 18 Unassigned_Variable New-AzStorageAccount -Location $loc is a null-valued parameter value. -Location Assign value for $loc.
⚠️ New-AzVmssConfig 4 19 Unassigned_Variable Get-AzStorageAccount -ResourceGroupName $rgname is a null-valued parameter value. -ResourceGroupName Assign value for $rgname.
⚠️ New-AzVmssConfig 4 23 Unassigned_Variable New-AzVirtualNetwork -ResourceGroupName $rgname is a null-valued parameter value. -ResourceGroupName Assign value for $rgname.
⚠️ New-AzVmssConfig 4 23 Unassigned_Variable New-AzVirtualNetwork -Location $loc is a null-valued parameter value. -Location Assign value for $loc.
⚠️ New-AzVmssConfig 4 24 Unassigned_Variable Get-AzVirtualNetwork -ResourceGroupName $rgname is a null-valued parameter value. -ResourceGroupName Assign value for $rgname.
⚠️ New-AzVmssConfig 4 30 Unassigned_Variable New-AzVmssConfig -Location $loc is a null-valued parameter value. -Location Assign value for $loc.
⚠️ New-AzVmssConfig 4 32 Unassigned_Variable Set-AzVmssOsProfile -AdminPassword $password is a null-valued parameter value. -AdminPassword Assign value for $password.
⚠️ New-AzVmssConfig 4 37 Unassigned_Variable New-AzVmss -ResourceGroupName $rgname is a null-valued parameter value. -ResourceGroupName Assign value for $rgname.
⚠️ New-AzVmssConfig 4 39 Unassigned_Variable Get-AzVmss -ResourceGroupName $rgname is a null-valued parameter value. -ResourceGroupName Assign value for $rgname.
⚠️ New-AzVmssConfig 5 36 Mismatched_Parameter_Value_Type Set-AzVmssOsProfile -AdminPassword $adminPassword is not an expected parameter value type. -AdminPassword Use correct parameter value type. Expected Type is string. Now the type is securestring.(Command).
⚠️ Update-AzVmss 1 1 Unassigned_Variable Update-AzVmss -VirtualMachineScaleSet $LocalVMSS is a null-valued parameter value. -VirtualMachineScaleSet Assign value for $LocalVMSS.
️✔️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
️✔️Windows PowerShell - Windows
️✔️Test
️✔️PowerShell Core - Linux
️✔️PowerShell Core - MacOS
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️Az.KeyVault
️✔️Build
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️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
️✔️Test
️✔️PowerShell Core - Linux
️✔️PowerShell Core - MacOS
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️Az.OperationalInsights
️✔️Build
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️Az.PrivateDns
️✔️Build
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️Az.RecoveryServices
️✔️Build
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️Test
️✔️PowerShell Core - Linux
️✔️PowerShell Core - MacOS
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️Az.Security
️✔️Build
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️Test
️✔️PowerShell Core - Linux
️✔️PowerShell Core - MacOS
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️Az.Sql
️✔️Build
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️Az.Ssh
️✔️Build
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️Test
️✔️PowerShell Core - Linux
️✔️PowerShell Core - MacOS
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows
️✔️Az.Storage
️✔️Build
️✔️PowerShell Core - Windows
️✔️Windows PowerShell - Windows

@haagha haagha added the Compute label Aug 30, 2024
@msdorahu
Copy link
Contributor Author

msdorahu commented Aug 30, 2024 via email

@msdorahu msdorahu force-pushed the cmdassistant-1456-10480367774-1 branch from 194a8ad to 95fe433 Compare August 30, 2024 21:07
@msJinLei
Copy link
Contributor

msJinLei commented Sep 3, 2024

Please rename VmSizes to VmSize
image

Copy link

github-actions bot commented Sep 3, 2024

This PR was labeled "needs-revision" because it has unresolved review comments or CI failures.
Please resolve all open review comments and make sure all CI checks are green. Refer to our guide to troubleshoot common CI failures.

Copy link
Member

@haagha haagha left a comment

Choose a reason for hiding this comment

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

I think SkuProfileVMSize[] VmSizes should be of type string[] so User's can initialize the parameter easily, the conversion from string[] to SkuProfileVmSize[] should take place in the logic.

@haagha haagha requested a review from msJinLei September 4, 2024 17:27
haagha
haagha previously approved these changes Sep 4, 2024
haagha
haagha previously approved these changes Sep 5, 2024
@msJinLei
Copy link
Contributor

msJinLei commented Sep 5, 2024

/azp run azure-powershell - security-tools

Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@msJinLei msJinLei merged commit e5aa20f into Azure:main Sep 6, 2024
13 checks passed
github-actions bot pushed a commit that referenced this pull request Sep 6, 2024
#25980)

* Autogen code

* Autogen code by build fixer

* first pass

* add tests

* fix merge conflict

* change to string

* nit

* Adding help

---------

Co-authored-by: PSCmdAssistant <[email protected]>
Co-authored-by: haagha <[email protected]>
Co-authored-by: Jin Lei <[email protected]>
VeryEarly added a commit that referenced this pull request Sep 9, 2024
#25980) (#26052)

* Autogen code

* Autogen code by build fixer

* first pass

* add tests

* fix merge conflict

* change to string

* nit

* Adding help

---------

Co-authored-by: msdorahu <[email protected]>
Co-authored-by: PSCmdAssistant <[email protected]>
Co-authored-by: haagha <[email protected]>
Co-authored-by: Jin Lei <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants