-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Az.StackHCI Cmdlet registration changes for GA #13454
Conversation
Made changes to registration script to register the GA version of On-Premises Azure Stack HCI with Azure. - Supports registering with user provided certificate thumbprint. - Supports On-Premises Azure Stack HCI OS changes to use independent certificate on cluster nodes. - Cleans up resource group during unregistration. - Improves registration output and logging. Breaks the public preview registration of On-Premises Azure Stack HCI with Azure. - To register public preview On-Premises Azure Stack HCi with Azure, use 0.3.1 version of Az.StackHCI.
Added Test-AzStackHCIConnection Cmdlet and generated help files
Thank you for your contribution dineshreddy007! We will review the pull request and get back to you soon. |
Using UseDeviceAuthentication as paramater name instead of UseDeviceLogin. Checking existing resource location against given location.
# Test-AzStackHCIConnection | ||
|
||
## SYNOPSIS | ||
|
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.
Please fill in synopsis
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.
Filled in synopsis now.
--- | ||
external help file: Az.StackHCI-help.xml | ||
Module Name: Az.StackHCI | ||
online version: |
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.
Use
online version: | |
online version: https://docs.microsoft.com/powershell/module/az.stackhci/test-azstackhciconnection |
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.
updated online version
src/StackHCI/ChangeLog.md
Outdated
- Supports On-Premises Azure Stack HCI OS changes to use independent certificate on cluster nodes. | ||
- Cleans up resource group during unregistration. | ||
- Improves registration output and logging. | ||
* Breaks the public preview registration of On-Premises Azure Stack HCI with Azure. |
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.
More explicit
* Breaks the public preview registration of On-Premises Azure Stack HCI with Azure. | |
* [Breaking Change] Breaks the public preview registration of On-Premises Azure Stack HCI with Azure. |
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.
Mentioning explicitly now.
Added synopsis and corrected changelog as per review suggestion
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.
LGTM
/azp run |
Azure Pipelines successfully started running 3 pipeline(s). |
* Registration changes to support GA On-Premises Stack HCI Made changes to registration script to register the GA version of On-Premises Azure Stack HCI with Azure. - Supports registering with user provided certificate thumbprint. - Supports On-Premises Azure Stack HCI OS changes to use independent certificate on cluster nodes. - Cleans up resource group during unregistration. - Improves registration output and logging. Breaks the public preview registration of On-Premises Azure Stack HCI with Azure. - To register public preview On-Premises Azure Stack HCi with Azure, use 0.3.1 version of Az.StackHCI. * Added Test-AzStackHCIConnection Cmdlet Added Test-AzStackHCIConnection Cmdlet and generated help files * Using UseDeviceAuthentication param name Using UseDeviceAuthentication as paramater name instead of UseDeviceLogin. Checking existing resource location against given location. * Added synopsis and corrected changelog Added synopsis and corrected changelog as per review suggestion * try to fix static analysis issue Co-authored-by: Dinesh Reddy Gottam <digot@microsoft.com> Co-authored-by: Yeming Liu <Yeming.Liu@microsoft.com>
Description
Design review: https://github.com/Azure/azure-powershell-cmdlet-review-pr/issues/773
Made changes to registration script to register the GA version of On-Premises Azure Stack HCI with Azure.
Breaks the public preview registration of On-Premises Azure Stack HCI with Azure.
Checklist
CONTRIBUTING.md
ChangeLog.md
file(s) has been updated:ChangeLog.md
file can be found atsrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
## Upcoming Release
header -- no new version header should be added