Support multi-tenant in controller publish and unpublish by OliverChen557 · Pull Request #431 · dell/csi-powerscale

OliverChen557 · 2025-08-07T19:06:18Z

Description

Support multi-tenant in controller publish and unpublish:

ControllerPublishVolume @hoppea2
If the volume context contains reference to AzNetwork, use node labels to find matching network interface and add export using a new function with the correct IP address for the export.
ControllerUnpublishVolume @OliverChen557
If the volume context contains reference to AzNetwork (get the context from the PV to determine if the new parameter is in the StorageClass), use node labels to find matching network interface and remove export using a new function with the correct IP address for the export.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #

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?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

Volume provision and checking debug logs. Logs are attached to story.

…ub.com/dell/csi-powerscale into usr/oliverchen557/controller-unpublish

Co-authored-by: Alexander Hoppe <Alexander.Hoppe@dell.com>

OliverChen557 and others added 10 commits August 7, 2025 11:48

Draft changes for ControllerPublishVolume

ec7130e

Added function to get IPs from az network node label

d4d5cb1

Added remove export by IPs

8671c4a

Added more debug for removing export for testing integrated changes

b428051

Return array from get ips from label

6f03e3e

Minor fixes

2336f89

Added GetNodeLabelsWithName

e1df558

Pass in node id from req

c52cedf

ControllerPublish changes

0e7550f

Updated comments

8bba144

OliverChen557 requested review from ChristianAtDell, HarishH-DELL, Sakshi-dell, alikdell, atye, bpjain2004, chimanjain, donatwork, meggm, nitesh3108, prablr79, rajkumar-palani, santhoshatdell, sharmilarama, shaynafinocchiaro and shefali-malhotra as code owners August 7, 2025 19:06