azure-stack-docs icon indicating copy to clipboard operation
azure-stack-docs copied to clipboard

updated 'deploy-gpu-node-pool.md' with instructions for using Headless OS

Open mgodfre3 opened this issue 1 year ago • 13 comments

The instructions for uninstalling drivers was written for a Desktop Experience Server, not Azure Stack HCI, which is intended OS, I updated docs to include PowerShell examples.

mgodfre3 avatar Jun 11 '24 14:06 mgodfre3

@mgodfre3 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

prmerger-automator[bot] avatar Jun 11 '24 14:06 prmerger-automator[bot]

Learn Build status updates of commit 0018650:

:white_check_mark: Validation status: passed

File Status Preview URL Details
AKS-Hybrid/deploy-gpu-node-pool.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

@sethmanheim

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team

Court72 avatar Jun 11 '24 15:06 Court72

Learn Build status updates of commit 412c583:

:white_check_mark: Validation status: passed

File Status Preview URL Details
AKS-Hybrid/deploy-gpu-node-pool.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Learn Build status updates of commit 03d48f0:

:white_check_mark: Validation status: passed

File Status Preview URL Details
AKS-Hybrid/deploy-gpu-node-pool.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Would it be possible to avoid mix and match legacy shell commands with PowerShell? See line 73 piping to findstr. I am no expert but believe PowerShell can do this natively.

Any idea @JamesLiang29 ?

Get-PnpDevice | Where-Object FriendlyName -Like '3D Video*' | Select-Object Status, FriendlyName, InstanceId

JamesLiang29 avatar Jun 20 '24 09:06 JamesLiang29

I was asking for your help as I cannot test the result.

Would you mind to create a request for change of line 74?

Karl-WE avatar Jun 20 '24 18:06 Karl-WE

Added the requested changes and added a new GPU command for additional GPUs.

mgodfre3 avatar Jun 21 '24 17:06 mgodfre3

Learn Build status updates of commit ca8cf70:

:white_check_mark: Validation status: passed

File Status Preview URL Details
AKS-Hybrid/deploy-gpu-node-pool.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Learn Build status updates of commit 8da0cdd:

:white_check_mark: Validation status: passed

File Status Preview URL Details
AKS-Hybrid/deploy-gpu-node-pool.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Learn Build status updates of commit 0557b31:

:white_check_mark: Validation status: passed

File Status Preview URL Details
AKS-Hybrid/deploy-gpu-node-pool.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Learn Build status updates of commit b447080:

:white_check_mark: Validation status: passed

File Status Preview URL Details
AKS-Hybrid/deploy-gpu-node-pool.md :white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

@mgodfre3 Sorry for the delay here. This PR has an open discussion that hasn't been resolved -- can you please advise as to current status of this PR? There is also a merge conflict that must be resolved -- I'd suggest closing this PR and opening a new one, once the discussions are resolved. Thank you for the contribution!

sethmanheim avatar May 07 '25 20:05 sethmanheim

@mgodfre3

Can you respond to the comment above?

@MicrosoftDocs/public-repo-pr-review-team

v-dirichards avatar Jun 13 '25 15:06 v-dirichards

@mgodfre3 The article has been substantially updated or deleted, and this change no longer applies. If you'd still like to update the article, please open a new PR. Or if you believe this PR was closed by mistake, email the PublicPRs alias.

#please-close @MicrosoftDocs/public-repo-pr-review-team

v-dirichards avatar Jun 23 '25 21:06 v-dirichards