Skip to content

Conversation

peterschen
Copy link
Contributor

PR Summary

Add missing -ManagementPointNetworkType parameter to the New-Cluster cmdlet documentation, which is available since Windows Server 2019.

PR Checklist

  • Descriptive Title: This PR's title is a synopsis of the changes it proposes.
  • Summary: This PR's summary describes the scope and intent of the change.
  • Contributor's Guide: I have read the contributors guide.
  • Style: This PR adheres to the style guide.

@peterschen
Copy link
Contributor Author

@microsoft-github-policy-service agree

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit eebb2d2:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2022-ps/failoverclusters/New-Cluster.md ✅Succeeded View (WindowsServer2022-ps)

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

@JohanFreelancer9 JohanFreelancer9 left a comment

Choose a reason for hiding this comment

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

Copy review - @peterschen, kindly commit the minor suggested change. Many thanks.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit a07d3f6:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2022-ps/failoverclusters/New-Cluster.md ✅Succeeded View (WindowsServer2022-ps)

For more details, please refer to the build report.

For any questions, please:

@alexjmoore
Copy link

Hi @JohanFreelancer9 - just wondering if there was an update to this PR?

@JohanFreelancer9
Copy link
Contributor

Hi @JohanFreelancer9 - just wondering if there was an update to this PR?

@scanum, can you please help to get this PR reviewed and merged? Thanks.

@scanum scanum assigned ghost , get-itips and scanum Aug 11, 2023
Copy link
Contributor

@get-itips get-itips left a comment

Choose a reason for hiding this comment

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

Can you please make the same additions for the Windows Server 2019 version of the article? Thank you

@alexjmoore
Copy link

Hi @get-itips - in fact these additions are already in the 2019 docs, but were omitted from the 2022, which this PR aims to correct.

@get-itips
Copy link
Contributor

Thanks, didn't notice @alexjmoore

@scanum
Copy link

scanum commented Aug 23, 2023

@tseward hope all is well. Could you please help us review? Thanks!

@alexjmoore
Copy link

Any update on this one?

@scanum
Copy link

scanum commented Sep 1, 2023

@tiburd
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@alexjmoore
Copy link

Any update?

@scanum
Copy link

scanum commented Sep 25, 2023

@tiburd
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 6dea426:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2022-ps/failoverclusters/New-Cluster.md ✅Succeeded View (WindowsServer2022-ps)

For more details, please refer to the build report.

For any questions, please:

@tiburd
Copy link
Contributor

tiburd commented Sep 25, 2023

close > reopen

@tiburd tiburd closed this Sep 25, 2023
@tiburd tiburd reopened this Sep 25, 2023
@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 6dea426:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2022-ps/failoverclusters/New-Cluster.md ✅Succeeded View (WindowsServer2022-ps)

For more details, please refer to the build report.

For any questions, please:

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 8631be5:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2022-ps/failoverclusters/New-Cluster.md ✅Succeeded View (WindowsServer2022-ps)

For more details, please refer to the build report.

For any questions, please:

@tiburd tiburd merged commit c702ccb into MicrosoftDocs:main Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants