Skip to content
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

(WIP) VMDirectStorage descriptions - AI-generated #3953

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

JasonGerend
Copy link
Contributor

@JasonGerend JasonGerend commented Mar 20, 2025

PR Summary

EXPERIMENTAL: AI-generated draft content for VMDirectStorage module, which currently lacks all descriptions. Edits made by GitHub Copilot in Visual Studio Code using GPT 4.5 Preview and 4o, referencing the module source code. Will evaluate content quality and accuracy before publishing or using this method for any additional modules - this is an experiment and might serve no other function than to prompt human-authored content creation.

AI prompt:
Edit all files in the VMDirectStorage folder, replacing all sections that include {{ Fill }} with carefully written text. Also write descriptions for each object type in the ## INPUTS and ##OUTPUTS sections.
Do not edit the Untitled-1 text file. Instead use the Untitled text file as source material for your edits - this file contains the source code for all cmdlets. Also use web research for information on parameters and objects and #File:Get-physicaldisk.md as an example of good writing for descriptions. Write for an IT Pro audience using the Microsoft Writing Style Guide and the PowerShell style guide here: [https://learn.microsoft.com/en-us/powershell/scripting/community/contributing/powershell-style-guide?view=powershell-7.5]. Make sure that "cmdlets" is sentence case.
When you're finished, review all files again checking for anything you missed, and update the ms.date parameter with the date 3/20/2025. Also add to the end of the metadata section ai-usage: ai-generated. Add blank lines around headings and fenced code blocks.

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.

Copy link
Contributor

Learn Build status updates of commit 72f4dbd:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2025-ps/VMDirectStorage/Add-VMDirectVirtualDisk.md ✅Succeeded View (WindowsServer2025-ps)
docset/winserver2025-ps/VMDirectStorage/Get-VMDirectVirtualDisk.md ✅Succeeded View (WindowsServer2025-ps)
docset/winserver2025-ps/VMDirectStorage/Remove-VMDirectVirtualDisk.md ✅Succeeded View (WindowsServer2025-ps)
docset/winserver2025-ps/VMDirectStorage/VMDirectStorage.md ✅Succeeded View (WindowsServer2025-ps)

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

Learn Build status updates of commit 249dced:

✅ Validation status: passed

File Status Preview URL Details
docset/winserver2025-ps/VMDirectStorage/Add-VMDirectVirtualDisk.md ✅Succeeded View (WindowsServer2025-ps)
docset/winserver2025-ps/VMDirectStorage/Get-VMDirectVirtualDisk.md ✅Succeeded View (WindowsServer2025-ps)
docset/winserver2025-ps/VMDirectStorage/Remove-VMDirectVirtualDisk.md ✅Succeeded View (WindowsServer2025-ps)
docset/winserver2025-ps/VMDirectStorage/VMDirectStorage.md ✅Succeeded View (WindowsServer2025-ps)

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

@sdwheeler sdwheeler left a comment

Choose a reason for hiding this comment

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

See my notes and suggested changes.

```powershell
PS C:\> {{ Add example code here }}
PS C:\> Add-VMDirectVirtualDisk -VMName "VM01" -ControllerType SCSI -ControllerNumber 0 -ControllerLocation 1 -VirtualDiskUniqueId "12345678-ABCD-1234-ABCD-1234567890AB"
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't use the prompt in the example. For long command lines, use PowerShell splatting.

Suggested change
PS C:\> Add-VMDirectVirtualDisk -VMName "VM01" -ControllerType SCSI -ControllerNumber 0 -ControllerLocation 1 -VirtualDiskUniqueId "12345678-ABCD-1234-ABCD-1234567890AB"
$parameters = @{
VMName = "VM01"
ControllerType = 'SCSI'
ControllerNumber = 0
ControllerLocation = 1
VirtualDiskUniqueId = "12345678-ABCD-1234-ABCD-1234567890AB"
}
Add-VMDirectVirtualDisk @parameters

```
Add-VMDirectVirtualDisk [-VMDriveController] <VMDriveController> [-ControllerLocation <Int32>]
[-VirtualDiskUniqueId <String>] [<CommonParameters>]
```

## DESCRIPTION
{{ Fill in the Description }}

The `Add-VMDirectVirtualDisk` cmdlet attaches an existing virtual disk directly to a Hyper-V virtual machine. Direct-attached virtual disks provide improved storage performance by bypassing traditional virtual storage layers.
Copy link
Contributor

Choose a reason for hiding this comment

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

Wrap lines at 100 characters

Suggested change
The `Add-VMDirectVirtualDisk` cmdlet attaches an existing virtual disk directly to a Hyper-V virtual machine. Direct-attached virtual disks provide improved storage performance by bypassing traditional virtual storage layers.
The `Add-VMDirectVirtualDisk` cmdlet attaches an existing virtual disk directly to a Hyper-V virtual
machine. Direct-attached virtual disks provide improved storage performance by bypassing traditional
virtual storage layers.

```

{{ Add example description here }}
This command attaches a virtual disk identified by its unique ID directly to the SCSI controller 0 at location 1 on the virtual machine named VM01.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This command attaches a virtual disk identified by its unique ID directly to the SCSI controller 0 at location 1 on the virtual machine named VM01.
This command attaches a virtual disk identified by its unique ID directly to the SCSI controller 0
at location 1 on the virtual machine named VM01.

@@ -64,7 +72,8 @@ Accept wildcard characters: False
```

### -ControllerLocation
{{ Fill ControllerLocation Description }}

Specifies the location on the controller where the virtual disk is attached. Valid values are from 0 to 63.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Specifies the location on the controller where the virtual disk is attached. Valid values are from 0 to 63.
Specifies the location on the controller where the virtual disk is attached. Valid values are from 0
to 63.

@@ -170,19 +185,34 @@ Accept wildcard characters: False
```

### CommonParameters

This cmdlet supports the common parameters: -Debug, -ErrorAction, -ErrorVariable, -InformationAction, -InformationVariable, -OutVariable, -OutBuffer, -PipelineVariable, -Verbose, -WarningAction, and -WarningVariable. For more information, see [about_CommonParameters](http://go.microsoft.com/fwlink/?LinkID=113216).
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This cmdlet supports the common parameters: -Debug, -ErrorAction, -ErrorVariable, -InformationAction, -InformationVariable, -OutVariable, -OutBuffer, -PipelineVariable, -Verbose, -WarningAction, and -WarningVariable. For more information, see [about_CommonParameters](http://go.microsoft.com/fwlink/?LinkID=113216).
This cmdlet supports the common parameters: -Debug, -ErrorAction, -ErrorVariable,
-InformationAction, -InformationVariable, -OutVariable, -OutBuffer, -PipelineVariable, -Verbose,
-WarningAction, and -WarningVariable. For more information, see
[about_CommonParameters](http://go.microsoft.com/fwlink/?LinkID=113216).

Comment on lines +49 to +50
PS C:\> $disk = Get-VMDirectVirtualDisk -VMName "VM01"
PS C:\> Remove-VMDirectVirtualDisk -VirtualDisk $disk
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
PS C:\> $disk = Get-VMDirectVirtualDisk -VMName "VM01"
PS C:\> Remove-VMDirectVirtualDisk -VirtualDisk $disk
$disk = Get-VMDirectVirtualDisk -VMName "VM01"
Remove-VMDirectVirtualDisk -VirtualDisk $disk

```

{{ Add example description here }}
This command retrieves all direct-attached virtual disks on the virtual machine named VM01 and removes them.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This command retrieves all direct-attached virtual disks on the virtual machine named VM01 and removes them.
This command retrieves all direct-attached virtual disks on the virtual machine named VM01 and
removes them.

@@ -131,17 +152,30 @@ Accept wildcard characters: False
```

### CommonParameters

This cmdlet supports the common parameters: -Debug, -ErrorAction, -ErrorVariable, -InformationAction, -InformationVariable, -OutVariable, -OutBuffer, -PipelineVariable, -Verbose, -WarningAction, and -WarningVariable. For more information, see [about_CommonParameters](http://go.microsoft.com/fwlink/?LinkID=113216).
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This cmdlet supports the common parameters: -Debug, -ErrorAction, -ErrorVariable, -InformationAction, -InformationVariable, -OutVariable, -OutBuffer, -PipelineVariable, -Verbose, -WarningAction, and -WarningVariable. For more information, see [about_CommonParameters](http://go.microsoft.com/fwlink/?LinkID=113216).
This cmdlet supports the common parameters: -Debug, -ErrorAction, -ErrorVariable,
-InformationAction, -InformationVariable, -OutVariable, -OutBuffer, -PipelineVariable, -Verbose,
-WarningAction, and -WarningVariable. For more information, see
[about_CommonParameters](http://go.microsoft.com/fwlink/?LinkID=113216).

Comment on lines +180 to +181
[Add-VMDirectVirtualDisk](Add-VMDirectVirtualDisk.md)
[Get-VMDirectVirtualDisk](Get-VMDirectVirtualDisk.md)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
[Add-VMDirectVirtualDisk](Add-VMDirectVirtualDisk.md)
[Get-VMDirectVirtualDisk](Get-VMDirectVirtualDisk.md)
[Add-VMDirectVirtualDisk](Add-VMDirectVirtualDisk.md)
[Get-VMDirectVirtualDisk](Get-VMDirectVirtualDisk.md)

## Description
{{ Fill in the Description }}

The VMDirectStorage module provides cmdlets for managing direct-attached virtual disks in Hyper-V virtual machines. These cmdlets allow administrators to add, retrieve, and remove direct-attached virtual disks, enabling efficient storage management and improved performance for virtualized workloads.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The VMDirectStorage module provides cmdlets for managing direct-attached virtual disks in Hyper-V virtual machines. These cmdlets allow administrators to add, retrieve, and remove direct-attached virtual disks, enabling efficient storage management and improved performance for virtualized workloads.
The VMDirectStorage module provides cmdlets for managing direct-attached virtual disks in Hyper-V
virtual machines. These cmdlets allow administrators to add, retrieve, and remove direct-attached
virtual disks, enabling efficient storage management and improved performance for virtualized
workloads.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants