Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG]: Gobrick does not clean wwids from /etc/multipath/wwids after removing multipath devices #1447

Closed
AkshaySainiDell opened this issue Sep 3, 2024 · 2 comments
Assignees
Labels
area/csi-powermax Issue pertains to the CSI Driver for Dell EMC PowerMax area/csi-powerstore Issue pertains to the CSI Driver for Dell EMC PowerStore area/csi-unity Issue pertains to the CSI Driver for Dell EMC Unity type/bug Something isn't working. This is the default label associated with a bug issue.
Milestone

Comments

@AkshaySainiDell
Copy link
Contributor

Bug Description

Gobrick executes the following set of commands for removing a multipath device:

multipath -f /dev/dm-2
echo 1 > /sys/class/block/sdf/device/delete
multipathd del path /dev/sdf

These commands do not remove entries from the /etc/multipath/wwids file, resulting in stale entries being left behind. The growing count of entries could potentially impact the performance of multipath if the file becomes exceptionally large.

To address this issue, Gobrick should be enhanced to remove a device from the wwids file using below command:

multipath -w mpathX (or WWID)

Logs

N/A

Screenshots

image

Additional Environment Information

No response

Steps to Reproduce

Using any CSM Driver:

  1. Create a PVC.
  2. Create a POD that consumes the PVC.
  3. Note the WWIDs on the worker node where the POD is scheduled:
    wc -l /etc/multipath/wwids
  4. Delete the POD.
  5. Check the WWIDs again:
    wc -l /etc/multipath/wwids
    Observation: The count will not decrease.

Expected Behavior

Stale devices from wwids file should be removed

CSM Driver(s)

All CSM drivers

Installation Type

No response

Container Storage Modules Enabled

No response

Container Orchestrator

N/A

Operating System

N/A

@AkshaySainiDell AkshaySainiDell added the type/bug Something isn't working. This is the default label associated with a bug issue. label Sep 3, 2024
@AkshaySainiDell AkshaySainiDell self-assigned this Sep 3, 2024
@AkshaySainiDell
Copy link
Contributor Author

link: 27849

@suryagupta4 suryagupta4 added this to the v1.12.0 milestone Sep 3, 2024
@gallacher
Copy link
Contributor

@AkshaySainiDell, please add labels for all impacted CSM components. Thanks.

@AkshaySainiDell AkshaySainiDell added area/csi-powerstore Issue pertains to the CSI Driver for Dell EMC PowerStore area/csi-unity Issue pertains to the CSI Driver for Dell EMC Unity area/csi-powermax Issue pertains to the CSI Driver for Dell EMC PowerMax labels Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/csi-powermax Issue pertains to the CSI Driver for Dell EMC PowerMax area/csi-powerstore Issue pertains to the CSI Driver for Dell EMC PowerStore area/csi-unity Issue pertains to the CSI Driver for Dell EMC Unity type/bug Something isn't working. This is the default label associated with a bug issue.
Projects
None yet
Development

No branches or pull requests

3 participants