Skip to content

Fix NodeStageVolume for Metro NVMe#582

Merged
AkshaySainiDell merged 1 commit into
mainfrom
fix-metro-nvme
Aug 29, 2025
Merged

Fix NodeStageVolume for Metro NVMe#582
AkshaySainiDell merged 1 commit into
mainfrom
fix-metro-nvme

Conversation

@WilsonRadadia20

Copy link
Copy Markdown
Contributor

Description

This PR addresses an issue where DataVolumes remain stuck in the ImportScheduled state due to importer pods failing to initialize. The root cause was traced to a failure in the CSI driver's NodeStageVolume operation, which logs:

Warning FailedMapVolume (x8 over 55s) kubelet MapVolume.SetUpDevice failed for volume "techops-f679e8fb01" : rpc error: code = Internal desc = Unable to find device after multiple discovery attempts: at least one NVMe target required

Although NVMeTCP targets from the remote array are successfully discovered and cached, the driver attempts to connect using targets from the local array, resulting in device connection failure.

This update ensures correct target selection during volume staging.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
https://github.com/dell/csm/issues/1896

Checklist:

  • Have you run format,vet & lint checks against your submission?
  • Have you made sure that the code compiles?
  • Did you run the unit & integration tests successfully?
  • Have you maintained at least 90% code coverage?
  • Have you commented your code, particularly in hard-to-understand areas
  • Have you done corresponding changes to the documentation
  • Did you run tests in a real Kubernetes cluster?
  • Backward compatibility is not broken

How Has This Been Tested?

  • Installed PowerMax driver and then Virtualization using a metro SC
image image

@github-actions

Copy link
Copy Markdown
Contributor

Merging this branch will not change overall coverage

Impacted Packages Coverage Δ 🤖
github.com/dell/csi-powermax/service 0.00% (ø)

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/dell/csi-powermax/service/node.go 0.00% (ø) 0 0 0

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

@AkshaySainiDell AkshaySainiDell merged commit b6696a3 into main Aug 29, 2025
6 checks passed
@AkshaySainiDell AkshaySainiDell deleted the fix-metro-nvme branch August 29, 2025 09:38
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