Skip to content

Conversation

@arudell
Copy link
Member

@arudell arudell commented Jan 27, 2026

This pull request updates the logic for parsing VFP port state properties in Get-SdnVfpPortState to better handle properties that are output without explicit key/value pairs, and adds support for the SriovEnabled property in the VfpPortState class. The main changes focus on improving how enabled features are detected and mapped to the state object.

Enhancements to VFP Port State Parsing:

  • Updated the parsing logic in Get-SdnVfpPortState to handle lines that only contain a property name (without a value), assuming these properties are enabled by default when present in the output. This ensures properties like "SR-IOV Enabled" and various offload features are correctly set to $true when detected.

  • Moved the mapping of several port state and offload properties (such as "DTLS Offload Enabled", "GFT Offload Enabled", and various NVGRE/VXLAN offload features) from the key/value parsing block to the new single-property-line parsing block, improving robustness and accuracy. [1] [2]

VfpPortState Class Update:

  • Added a new [boolean]$SriovEnabled property to the VfpPortState class to track the SR-IOV enablement state.

Change type

  • Bug fix (non-breaking change)
  • Code style update (formatting, local variables)
  • New Feature (non-breaking change that adds new functionality without impacting existing)
  • Breaking change (fix or feature that may cause functionality impact)
  • Other

Checklist:

  • My code follows the style and contribution guidelines of this project.
  • I have tested and validated my code changes.

@arudell arudell requested a review from a team as a code owner January 27, 2026 21:51
@arudell arudell merged commit ee8e42a into main Jan 27, 2026
8 checks passed
@arudell arudell deleted the fix-class-properties-for-portstate branch January 27, 2026 22:23
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