fix(CSI-297): nfsTargetIps override is handled incorreclty when empty

[sergeyberezansky]

TL;DR

Improved handling of empty NFS target IPs in WekaFS driver.

What changed?

• Modified GetNfsMountIp function to check if the first NFS target IP is empty before using it.
• Updated fromSecrets function to handle cases where nfsTargetIpsRaw is an empty string.

How to test?

1. Configure the WekaFS driver with empty NFS target IPs.
2. Attempt to mount a volume and verify that the driver handles the empty IP scenario gracefully.
3. Test with valid NFS target IPs to ensure existing functionality is not affected.

Why make this change?

This change prevents potential issues when empty NFS target IPs are provided in secret. It ensures that the driver doesn't attempt to use an empty IP address, which could lead to mounting failures or unexpected behavior. This improvement enhances the robustness of the WekaFS driver when dealing with various configuration scenarios.

[sergeyberezansky]

• fix(CSI-296): node registration fails after switch transport from NFS to Wekafs due to label conflict #375
• fix(CSI-297): nfsTargetIps override is handled incorreclty when empty #374 👈
• fix(CSI-292): parse NFS version 3.0 to correctly pass it to mountoption #372
• fix(CSI-294): caCertificate, NfsTargetIps, localContainerName are not hashed in API client #371
• docs: fix the example of static provisioning of directory-backed volume #370
• dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sergeyberezansky and the rest of your teammates on Graphite

[graphite-app]

Graphite Automations

"Request reviewers once CI passes" took an action on this PR • (10/23/24)

1 reviewer was added to this PR based on Sergey Berezansky's automation.

[sergeyberezansky]

Merge activity

• Oct 23, 9:36 AM EDT: A user started a stack merge that includes this pull request via Graphite.
• Oct 23, 9:46 AM EDT: Graphite rebased this pull request as part of a merge.
• Oct 23, 9:48 AM EDT: Graphite rebased this pull request as part of a merge.
• Oct 23, 9:49 AM EDT: A user merged this pull request with Graphite.