weka / csi-wekafs

WekaFS Container Storage Interface (CSI) Plugin
Apache License 2.0
35 stars 6 forks source link

fix(CSI-234): leak of wekafs mounts due to bad IsMounted() check #293

Closed sergeyberezansky closed 3 months ago

sergeyberezansky commented 3 months ago

TL;DR

Refactored the PathIsWekaMount method to remove reliance on the os/exec package and instead read directly from /proc/mounts for determining Weka mount paths.

What changed?

How to test?

Ensure that the PathIsWekaMount method functions correctly to identify Weka mount paths using /proc/mounts.

Why make this change?

Improves performance and reliability by avoiding the use of shell commands to determine Weka mount paths.


sergeyberezansky commented 3 months ago

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

Join @sergeyberezansky and the rest of your teammates on Graphite Graphite

graphite-app[bot] commented 3 months ago

Graphite Automations

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

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

sergeyberezansky commented 3 months ago

Merge activity