Description

PowerStore CSI driver incorrectly labelling nodes for NFS protocol

GitHub Issues

List the GitHub issues impacted by this PR:

Checklist:

• I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
• I have verified that new and existing unit tests pass locally with my changes
• I have not allowed coverage numbers to degenerate
• I have maintained at least 90% code coverage
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works
• Backward compatibility is not broken

How Has This Been Tested?

• Installed Driver with 1 NFS Enabled Array and nfs label applies as expected and able to provision volumes.
• Installed Driver with 1 NFS Disabled Array and nfs label not applies as expected and able to provision volumes.
• Installed Driver with Multi-Array (Array1: NFS Disabled & Array2: NFS Enabled):
  • NFS label not added for Array1 to csinode.
  • NFS label added for Array2 to csinode.
  • Provisioning failed for NFS volumes for Array1.
  • Able to provision for NFS volumes for Array2.
• Upgrade test from 2.14 to latest driver
  • Provisioning failed for NFS volumes.
  • Provisioning worked for non-NFS volumes.
  • Tested against non-NFS volumes created before the upgrade:
    • Validated volume can be re-mounted to the same node as well as a different node.
    • Validated Snapshot:
      • Write data to the volume.
      • Validated that a snapshot can be taken.
      • Validated volume creation from snapshot.
    • Validated that volume cloning is possible.
      • Validated that the cloned volume can be mounted.