Skip to content

New commendlet to get nfs connect value#390

Merged
Malika133 merged 3 commits into
Azure:mainfrom
Malika133:mchhibber/NFSConnect
May 30, 2026
Merged

New commendlet to get nfs connect value#390
Malika133 merged 3 commits into
Azure:mainfrom
Malika133:mchhibber/NFSConnect

Conversation

@Malika133
Copy link
Copy Markdown
Contributor

@Malika133 Malika133 commented Apr 16, 2026

This PR closes #

The changes in this PR are as follows:

  • Added new Get-NFSDatastoreNConnectValue function module that retrieves NFS NConnect (TCP connections) configuration for a specified datastore across all hosts in a cluster
  • Updated module manifest Microsoft.AVS.NFS.psd1 to export the new function
  • Added comprehensive Pester tests for the new function in Microsoft.AVS.NFS.Tests.ps1

I have read the contributor guidelines and have completed the following:

  • Formatted the code using VSCode default formatter for PowerShell.
  • Tested the code end-to-end against an SDDC.
  • Documented the functions using standard PowerShell markup and applied AVSAttribute to newly exported functions.
  1. Check Datastore exists or NOT and Check Cluster exists for not
image
  1. Validate datastore is of Type NFS only
image
  1. Successful Get nConnect Command
image

@Malika133 Malika133 changed the title added new run command get nfs connect value New run command get nfs connect value Apr 16, 2026
@Malika133 Malika133 requested review from et1975 and prtyag April 17, 2026 04:22
@et1975 et1975 requested a review from Copilot April 17, 2026 11:33
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new AVS Run Command function to the Microsoft.AVS.NFS module for retrieving NFS nConnect (TCP connections) configuration per host for a given datastore/cluster, along with module export wiring and Pester coverage.

Changes:

  • Added Get-NFSDatastoreNConnectValue to query NFS nConnect/connection details across connected hosts in a cluster.
  • Exported the new function via Microsoft.AVS.NFS.psd1.
  • Added initial Pester tests for module loading and several validation/error paths.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
Microsoft.AVS.NFS/Microsoft.AVS.NFS.psm1 Adds the new Get-NFSDatastoreNConnectValue implementation and sets NamedOutputs.
Microsoft.AVS.NFS/Microsoft.AVS.NFS.psd1 Exports the new function in FunctionsToExport.
tests/Microsoft.AVS.NFS.Tests.ps1 Introduces Pester tests covering import/export + parameter and failure-mode validation for the new cmdlet.

Comment thread Microsoft.AVS.NFS/Microsoft.AVS.NFS.psm1 Outdated
Comment thread tests/Microsoft.AVS.NFS.Tests.ps1
Comment thread tests/Microsoft.AVS.NFS.Tests.ps1
@et1975 et1975 changed the title New run command get nfs connect value New commendlet to get nfs connect value May 7, 2026
mchhibber and others added 3 commits May 30, 2026 13:38
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Malika133 Malika133 force-pushed the mchhibber/NFSConnect branch from decf95e to 1042583 Compare May 30, 2026 08:08
@Malika133 Malika133 merged commit db6a03d into Azure:main May 30, 2026
9 checks passed
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.

3 participants