[BC]StackHCI Module Breaking Changes Announcement Date Update#29134
[BC]StackHCI Module Breaking Changes Announcement Date Update#29134JiaSeng-v wants to merge 1 commit intoAzure:mainfrom
Conversation
JiaSeng-v
added
Breaking Change Preannounce
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
There was a problem hiding this comment.
Pull request overview
This pull request updates the breaking change announcement dates for the Az.StackHCI module and adds several new cmdlets for MSI-based registration and Arc enablement functionality.
Changes:
- Updated breaking change announcement dates from Azure PowerShell version 15.0.0 (November 2025) to version 16.0.0 (May 2026) across all breaking change directives in README.md
- Added seven new cmdlets: Enable-ArcOnNodes, Invoke-MSIFlow, Invoke-MSIUnregistrationFlow, Test-ArcNodeClusterLink, Test-ClusterArcEnabled, Test-ClusterMsiSupport, and Test-ComputerNameHasDnsSuffix
- Added new parameter
-useStableAgentVersionto Register-AzStackHCI cmdlet to enable using the stable HCI Arc agent for server registration
Reviewed changes
Copilot reviewed 37 out of 37 changed files in this pull request and generated 15 comments.
Show a summary per file
| File | Description |
|---|---|
| src/StackHCI/StackHCI.Autorest/README.md | Updated breaking change dates from 2025/11 to 2026/05 and version from 15.0.0 to 16.0.0 across 15 breaking change directives |
| src/StackHCI/StackHCI.Autorest/Properties/AssemblyInfo.cs | Updated assembly version from 2.6.1 to 2.6.6 |
| src/StackHCI/StackHCI/Az.StackHCI.psd1 | Updated module manifest generation date, uncommented ScriptsToProcess and TypesToProcess, added new cmdlets to FunctionsToExport, updated Tags formatting |
| src/StackHCI/StackHCI.sln | Updated project GUID reference for AutoRest project |
| src/StackHCI/StackHCI.Autorest/generate-info.json | Updated generate_Id GUID |
| src/StackHCI/StackHCI/help/*.md | Added help documentation files for 7 new cmdlets with incomplete placeholder content |
| src/StackHCI/StackHCI.Autorest/docs/*.md | Added documentation files for 7 new cmdlets, updated module index, and added new parameter documentation to Register-AzStackHCI |
| src/StackHCI/StackHCI.Autorest/test/*.Tests.ps1 | Added 7 new Pester test files for new cmdlets with placeholder/skipped test implementations |
| src/StackHCI/StackHCI.Autorest/examples/*.md | Added 7 example documentation files with placeholder template content |
| ## SYNOPSIS | ||
|
|
||
| ## SYNTAX | ||
|
|
||
| ``` | ||
| Test-ComputerNameHasDnsSuffix [-ComputerName] <String> | ||
| [<CommonParameters>] | ||
| ``` | ||
|
|
||
| ## DESCRIPTION | ||
|
|
||
| ## EXAMPLES | ||
|
|
||
| ### Example 1: {{ Add title here }} | ||
| ```powershell | ||
| {{ Add code here }} | ||
| ``` | ||
|
|
||
| ```output | ||
| {{ Add output here }} | ||
| ``` | ||
|
|
||
| {{ Add description here }} | ||
|
|
||
| ### Example 2: {{ Add title here }} | ||
| ```powershell | ||
| {{ Add code here }} | ||
| ``` | ||
|
|
||
| ```output | ||
| {{ Add output here }} | ||
| ``` | ||
|
|
||
| {{ Add description here }} | ||
|
|
There was a problem hiding this comment.
The SYNOPSIS section is empty and contains placeholder examples with template text like '{{ Add title here }}' and '{{ Add code here }}'. According to the custom guideline 1000004, cmdlet reference documentation should avoid over-simplified descriptions. These should be properly documented with meaningful descriptions, real examples with actual code and output, and complete parameter descriptions before being included in a release.
| ## SYNOPSIS | ||
|
|
||
| ## SYNTAX | ||
|
|
||
| ``` | ||
| Test-ClusterMsiSupport [-ClusterNodeSession] <PSSession> | ||
| [<CommonParameters>] | ||
| ``` | ||
|
|
||
| ## DESCRIPTION | ||
|
|
||
| ## EXAMPLES | ||
|
|
||
| ### Example 1: {{ Add title here }} | ||
| ```powershell | ||
| {{ Add code here }} | ||
| ``` | ||
|
|
||
| ```output | ||
| {{ Add output here }} | ||
| ``` | ||
|
|
||
| {{ Add description here }} | ||
|
|
||
| ### Example 2: {{ Add title here }} | ||
| ```powershell | ||
| {{ Add code here }} | ||
| ``` | ||
|
|
||
| ```output | ||
| {{ Add output here }} | ||
| ``` | ||
|
|
||
| {{ Add description here }} | ||
|
|
There was a problem hiding this comment.
The SYNOPSIS section is empty and contains placeholder examples with template text like '{{ Add title here }}' and '{{ Add code here }}'. According to the custom guideline 1000004, cmdlet reference documentation should avoid over-simplified descriptions. These should be properly documented with meaningful descriptions, real examples with actual code and output, and complete parameter descriptions before being included in a release.
| Invoke-MSIFlow [-ClusterNodes] <Array> [-ClusterDNSSuffix] <String> [-ResourceId] <String> | ||
| [-RPAPIVersion] <String> [-TenantId] <String> [-Region] <String> [-ResourceGroupName] <String> | ||
| [-ResourceName] <String> [-SubscriptionId] <String> [-ClusterNodeSession] <PSSession> | ||
| [[-Credential] <PSCredential>] [[-Tag] <Hashtable>] [[-registrationOutput] <PSObject>] |
There was a problem hiding this comment.
The parameter name 'registrationOutput' does not follow PowerShell naming conventions. PowerShell cmdlet parameters should use PascalCase (also known as UpperCamelCase), not camelCase. This parameter should be named 'RegistrationOutput' to be consistent with PowerShell standards.
| Accept wildcard characters: False | ||
| ``` | ||
|
|
||
| ### -useStableAgentVersion |
There was a problem hiding this comment.
The parameter name 'useStableAgentVersion' does not follow PowerShell naming conventions. PowerShell cmdlet parameters should use PascalCase (also known as UpperCamelCase), not camelCase. This parameter should be named 'UseStableAgentVersion' to be consistent with other parameters in this cmdlet like 'UseDeviceAuthentication', 'IsWAC', 'RepairRegistration', etc.
| Accept wildcard characters: False | ||
| ``` | ||
|
|
||
| ### -useStableAgentVersion |
There was a problem hiding this comment.
The parameter name 'useStableAgentVersion' does not follow PowerShell naming conventions. PowerShell cmdlet parameters should use PascalCase (also known as UpperCamelCase), not camelCase. This parameter should be named 'UseStableAgentVersion' to be consistent with other parameters in this cmdlet.
| ## SYNOPSIS | ||
|
|
||
| ## SYNTAX | ||
|
|
||
| ``` | ||
| Test-ClusterArcEnabled [-ClusterNodes] <Array> [-ClusterDNSSuffix] <String> [-SubscriptionId] <String> | ||
| [-ArcResourceGroupName] <String> [[-Credential] <PSCredential>] | ||
| [<CommonParameters>] | ||
| ``` | ||
|
|
||
| ## DESCRIPTION | ||
|
|
||
| ## EXAMPLES | ||
|
|
||
| ### Example 1: {{ Add title here }} | ||
| ```powershell | ||
| {{ Add code here }} | ||
| ``` | ||
|
|
||
| ```output | ||
| {{ Add output here }} | ||
| ``` | ||
|
|
||
| {{ Add description here }} | ||
|
|
||
| ### Example 2: {{ Add title here }} | ||
| ```powershell | ||
| {{ Add code here }} | ||
| ``` | ||
|
|
||
| ```output | ||
| {{ Add output here }} | ||
| ``` | ||
|
|
||
| {{ Add description here }} |
There was a problem hiding this comment.
The SYNOPSIS section is empty and contains placeholder examples with template text like '{{ Add title here }}' and '{{ Add code here }}'. According to the custom guideline 1000004, cmdlet reference documentation should avoid over-simplified descriptions. These should be properly documented with meaningful descriptions, real examples with actual code and output, and complete parameter descriptions before being included in a release.
| ## SYNOPSIS | ||
|
|
||
| ## SYNTAX | ||
|
|
||
| ``` | ||
| Enable-ArcOnNodes [-ClusterNodes] <Array> [-ClusterDNSSuffix] <String> [-SubscriptionId] <String> | ||
| [-ResourceGroupName] <String> [-TenantId] <String> [-Location] <String> [-EnvironmentName] <String> | ||
| [-AccessToken] <String> [[-Credential] <PSCredential>] [[-UseStableAgent] <Boolean>] | ||
| [[-IsManagementNode] <Boolean>] [[-ComputerName] <String>] | ||
| [<CommonParameters>] | ||
| ``` | ||
|
|
||
| ## DESCRIPTION | ||
|
|
||
| ## EXAMPLES | ||
|
|
||
| ### Example 1: {{ Add title here }} | ||
| ```powershell | ||
| {{ Add code here }} | ||
| ``` | ||
|
|
||
| ```output | ||
| {{ Add output here }} | ||
| ``` | ||
|
|
||
| {{ Add description here }} | ||
|
|
||
| ### Example 2: {{ Add title here }} | ||
| ```powershell | ||
| {{ Add code here }} | ||
| ``` | ||
|
|
||
| ```output | ||
| {{ Add output here }} | ||
| ``` |
There was a problem hiding this comment.
The SYNOPSIS section is empty and contains placeholder examples with template text like '{{ Add title here }}' and '{{ Add code here }}'. According to the custom guideline 1000004, cmdlet reference documentation should avoid over-simplified descriptions. These should be properly documented with meaningful descriptions, real examples with actual code and output, and complete parameter descriptions before being included in a release.
| ### -ComputerName | ||
|
|
There was a problem hiding this comment.
The parameter description for 'ComputerName' is empty. According to custom guideline 1000004, cmdlet reference documentation should avoid over-simplified descriptions and provide meaningful parameter descriptions. This parameter should have a description explaining what the ComputerName represents and how it's used by this cmdlet.
| Accept wildcard characters: False | ||
| ``` | ||
|
|
||
| ### -registrationOutput |
There was a problem hiding this comment.
The parameter name 'registrationOutput' does not follow PowerShell naming conventions. PowerShell cmdlet parameters should use PascalCase (also known as UpperCamelCase), not camelCase. This parameter should be named 'RegistrationOutput' to be consistent with PowerShell standards and other parameters in this cmdlet like 'ClusterNodes', 'ResourceId', etc.
| Invoke-MSIFlow [-ClusterNodes] <Array> [-ClusterDNSSuffix] <String> [-ResourceId] <String> | ||
| [-RPAPIVersion] <String> [-TenantId] <String> [-Region] <String> [-ResourceGroupName] <String> | ||
| [-ResourceName] <String> [-SubscriptionId] <String> [-ClusterNodeSession] <PSSession> | ||
| [[-Credential] <PSCredential>] [[-Tag] <Hashtable>] [[-registrationOutput] <PSObject>] |
There was a problem hiding this comment.
The parameter name 'registrationOutput' does not follow PowerShell naming conventions. PowerShell cmdlet parameters should use PascalCase (also known as UpperCamelCase), not camelCase. This parameter should be named 'RegistrationOutput' to be consistent with PowerShell standards.
| [[-Credential] <PSCredential>] [[-Tag] <Hashtable>] [[-registrationOutput] <PSObject>] | |
| [[-Credential] <PSCredential>] [[-Tag] <Hashtable>] [[-RegistrationOutput] <PSObject>] |
1 participant
Description
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.