-
Notifications
You must be signed in to change notification settings - Fork 616
Update add-clustervirtualmachinerole.md #1197
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
Conversation
@DanieleDeAn Thank you for the valuable find. The PR is being rolled into review. Much appreciated. |
@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks. |
@dariomws306 @get-itips @tseward Peer review needed for this PR. Thanks. |
Looks ok, but I think we can improve it. Maybe two different notes? Note: This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. What do you think? |
Agree with @dariomws306 |
@@ -37,7 +37,12 @@ By creating clustered virtual machines, you can consolidate multiple servers on | |||
Instead, if that server, or cluster node, fails or requires scheduled maintenance, then another node begins to run the virtual machines instead through a process known as failover. | |||
The virtual hard disk (VHD) file for the clustered virtual machine must be on the clustered disk used by that virtual machine. | |||
|
|||
Note: This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. | |||
|
|||
> **Note:** This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. Also this cmdlet generate a .TMP file under the user temp folder, and you can't have more than 65535 .TMP files generated by this cmdlet otherwise you will receive an exception "The File Exist". |
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.
> **Note:** This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. Also this cmdlet generate a .TMP file under the user temp folder, and you can't have more than 65535 .TMP files generated by this cmdlet otherwise you will receive an exception "The File Exist". | |
> [!Note] | |
> This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. | |
> [!Note] | |
> This cmdlet generates a .TMP file under the user temp folder, and you can't have more than 65535 .TMP files generated by this cmdlet, otherwise you will receive an exception "The File Exist". For more information, please go to: https://techcommunity.microsoft.com/t5/ITOps-Talk-Blog/TQA-Add-ClusterVirtualMachineRole-fails-with-the-error-quot-The/ba-p/713344 |
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.
> **Note:** This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. Also this cmdlet generate a .TMP file under the user temp folder, and you can't have more than 65535 .TMP files generated by this cmdlet otherwise you will receive an exception "The File Exist". | |
> [!NOTE} | |
> This cmdlet generate a .TMP file in the user temp folder, and if more than 65535 .TMP files are generated you will get the error: "The File Exists". For more information, see this [support article](https://techcommunity.microsoft.com/t5/ITOps-Talk-Blog/TQA-Add-ClusterVirtualMachineRole-fails-with-the-error-quot-The/ba-p/713344). |
Note: This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. | ||
|
||
> **Note:** This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. Also this cmdlet generate a .TMP file under the user temp folder, and you can't have more than 65535 .TMP files generated by this cmdlet otherwise you will receive an exception "The File Exist". | ||
https://techcommunity.microsoft.com/t5/ITOps-Talk-Blog/TQA-Add-ClusterVirtualMachineRole-fails-with-the-error-quot-The/ba-p/713344 |
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.
https://techcommunity.microsoft.com/t5/ITOps-Talk-Blog/TQA-Add-ClusterVirtualMachineRole-fails-with-the-error-quot-The/ba-p/713344 |
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.
Copy Review - Two notes in accordance with Dario's suggestion.
|
||
> **Note:** This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. Also this cmdlet generate a .TMP file under the user temp folder, and you can't have more than 65535 .TMP files generated by this cmdlet otherwise you will receive an exception "The File Exist". | ||
https://techcommunity.microsoft.com/t5/ITOps-Talk-Blog/TQA-Add-ClusterVirtualMachineRole-fails-with-the-error-quot-The/ba-p/713344 | ||
|
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.
> **Note:** This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. Also this cmdlet generate a .TMP file under the user temp folder, and you can't have more than 65535 .TMP files generated by this cmdlet otherwise you will receive an exception "The File Exist". | ||
https://techcommunity.microsoft.com/t5/ITOps-Talk-Blog/TQA-Add-ClusterVirtualMachineRole-fails-with-the-error-quot-The/ba-p/713344 | ||
|
||
|
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.
@@ -37,7 +37,12 @@ By creating clustered virtual machines, you can consolidate multiple servers on | |||
Instead, if that server, or cluster node, fails or requires scheduled maintenance, then another node begins to run the virtual machines instead through a process known as failover. | |||
The virtual hard disk (VHD) file for the clustered virtual machine must be on the clustered disk used by that virtual machine. | |||
|
|||
Note: This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. | |||
|
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.
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.
Remove extra lines
Also, I think the same note should be added to W2012 and 2012 R2 versions. |
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.
Hi @JohanFreelancer9 i like the two notes, but why you are using the syntax "[!Note]"?
in the preview this syntax "**Note:**" bold text and create a better effect in my honest opinion.
Note:
This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer.
Note:
This cmdlet generates a .TMP file under the user temp folder, and you can't have more than 65535 .TMP files generated by this cmdlet, otherwise you will receive an exception "The File Exist". For more information, please go to: https://techcommunity.microsoft.com/t5/ITOps-Talk-Blog/TQA-Add-ClusterVirtualMachineRole-fails-with-the-error-quot-The/ba-p/713344
[!NOTE] stands out in documentation and is the standard for adding NOTES. See https://docs.microsoft.com/en-us/sharepoint/upgrade-and-update/install-a-software-update as one example near the top in the purple-ish section of a note. |
Hi @tseward many thanks for the info, now is clear, but is also clear that the preview do not read correctly "[!NOTE]" syntax, or may be I wrong something :). Many thanks |
Final Version based on suggestions
@JohanFreelancer9, @nenonix Hi, I have modified the file based on your suggestion but probably I need to study more Github! :) because now i see that your info are outdated, sorry guys but this is a new world for me :), If I need to change something please let me know ;) |
docset/windows/failoverclusters/add-clustervirtualmachinerole.md
Outdated
Show resolved
Hide resolved
@@ -37,7 +37,11 @@ By creating clustered virtual machines, you can consolidate multiple servers on | |||
Instead, if that server, or cluster node, fails or requires scheduled maintenance, then another node begins to run the virtual machines instead through a process known as failover. | |||
The virtual hard disk (VHD) file for the clustered virtual machine must be on the clustered disk used by that virtual machine. | |||
|
|||
Note: This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. | |||
|
|||
> [!NOTE]This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. |
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.
> [!NOTE]This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. | |
> [!NOTE] This cmdlet cannot be run remotely without Credential Security Service Provider (CredSSP) authentication on the server computer. |
Many thanks Johan Co-Authored-By: JohanFreelancer9 <48568725+JohanFreelancer9@users.noreply.github.com>
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.
Copy Review - Please insert spaces as indicated.
Added spaces
Done and many tnx ;) |
Hi @e0i we have a version ready to be merged in the master branch, I need to do something? :) |
Hello @DanieleDeAn, It seems that the notes need to be added for other supported Windows versions, via dariomws306's comment below,
The corresponding folders for these two Windows versions can be found under Thank you. |
Modified the File for 2012 and 2012R2
Hi @e0i, you are correct ! I miss this info , I have modified also the version for 2012 and 2012R2, and many thanks to @dariomws306 ;). |
@DanieleDeAn Thank you for your valuable contributions and taking the time to follow-up with reviewer's suggestions. The PR will now be merged. |
@AndreaBarr The PR is now ready to be merged. Thanks. |
I have found an issue related to this Cmdlet in a large enterprise, we can add a note and maybe the link of the blog to explain better.