Skip to content

chore: update spectrumxrailpoolconfig CRD - #2997

Merged
e0ne merged 1 commit into
Mellanox:masterfrom
rollandf:vf-follow
Aug 10, 2026
Merged

chore: update spectrumxrailpoolconfig CRD#2997
e0ne merged 1 commit into
Mellanox:masterfrom
rollandf:vf-follow

Conversation

@rollandf

@rollandf rollandf commented Aug 9, 2026

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Fred Rolland <frolland@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Aug 9, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@greptile-apps

greptile-apps Bot commented Aug 9, 2026

Copy link
Copy Markdown

Greptile Summary

The PR updates the SpectrumXRailPoolConfig CRD embedded in the Spectrum-X operator state.

  • Adds the optional spec.vfStateFollow boolean field with a documented default of true.
  • Reformats the generated CRD YAML without changing the remaining schema semantics.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete blocking or non-blocking issues identified.

The sole functional change adds a documented, optional boolean CRD field with a matching Kubernetes default, while the remaining diff preserves the existing schema.

Important Files Changed

Filename Overview
manifests/state-spectrum-x-operator/0001-spectrumx.nvidia.com_spectrumxrailpoolconfigs.yaml Adds the vfStateFollow schema property and otherwise applies generated YAML formatting changes; no actionable defect was identified.

Reviews (1): Last reviewed commit: "chore: update spectrumxrailpoolconfig CR..." | Re-trigger Greptile

@e0ne
e0ne merged commit dd4f707 into Mellanox:master Aug 10, 2026
17 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