feat(CSI-249): optimize NFS mounter to use multiple targets

[sergeyberezansky]

TL;DR

Added a new function to determine the node's IP address based on routing to a target host.

What changed?

• Introduced a new function GetNodeIpAddressByRouting in utils.go that determines the node's IP address by creating a UDP connection to a target host.
• Updated the doMount function in nfsmount.go to use the new GetNodeIpAddressByRouting function, falling back to GetNodeIpAddress if it fails.
• Added unit tests for the new GetNodeIpAddressByRouting function in utils_test.go.

How to test?

1. Run the new unit tests in utils_test.go to verify the functionality of GetNodeIpAddressByRouting.
2. Test the NFS mount process with different network configurations to ensure it correctly determines the node's IP address.
3. Verify that the mount process falls back to the original GetNodeIpAddress function if GetNodeIpAddressByRouting fails.

Why make this change?

This change improves the accuracy of determining the node's IP address for NFS mounts, especially in complex network environments. By using routing information to determine the IP address, we can ensure that the correct network interface is used for NFS communications, potentially resolving issues related to multi-homed systems or specific network configurations.

---

feat(deps): implement InterfaceGroup.GetRandomIpAddress()

feat(deps): implement InterfaceGroup.GetRandomIpAddress()

refactor(deps): allow multimount for NFS

refactor(deps): unmount underlying FS straight after publishing volume

[sergeyberezansky]

• chore(deps): update sidecar versions #326
• docs(CSI-254): update official docs link in Helm templates and README #323
• feat(CSI-252): implement kubelet PVC stats #322
• refactor(CSI-250): do not maintain redundant active mounts from node server after publishing volume #320
• feat(CSI-247): implement InterfaceGroup.GetRandomIpAddress() #319
• feat(CSI-249): optimize NFS mounter to use multiple targets #318 👈
• fix(CSI-241): fix unmountWithOptions to use map key rather than options.String() #317
• feat(CSI-245): allow specifying client group for NFS #316
• chore(deps): update packages to latest versions and Go to 1.22.5 #314
• chore(deps): combine chmod with ADD in Dockerfile #313
• feat(CSI-239): moveToTrash does not return error to upper layers #312
• fix(CSI-243): service accounts for CSI plugin assume ImagePullSecret and cause error messages. #311
• feat(CSI-244): match subnets if existing in client rule #315
• fix(CSI-241): conflict in metrics between node and controller #325
• feat(CSI-213): support NFS transport #299
• fix(CSI-241): disregard sync_on_close in mountmap per FS #310
• feat(CSI-253): support custom CA certificate in API secret #324
• 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 • (09/05/24)

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

[sergeyberezansky]

Merge activity

• Sep 12, 6:10 AM EDT: @sergeyberezansky started a stack merge that includes this pull request via Graphite.
• Sep 12, 7:43 AM EDT: Graphite couldn't merge this pull request because a downstack PR feat(CSI-245): allow specifying client group for NFS #316 failed to merge.
• Sep 12, 8:43 AM EDT: @sergeyberezansky started a stack merge that includes this pull request via Graphite.
• Sep 12, 8:45 AM EDT: The Graphite merge of this pull request was cancelled.