Skip to content

Conversation

@raj-prince
Copy link
Contributor

@raj-prince raj-prince commented Jul 14, 2025

Issue:

  • Misleading error logs while unmount GCSFuse in GKE world, where mount is created externally and passed via GCSfuse in /dev/fd/N format.
"Failed to unmount in response to SIGTERM: exec: \"fusermount\": executable file not found in $PATH
  • This happens, as jacobsa/fuse library tries to unmount using fusermount binary, which is not present in GKE container environment and leads to failure.

Proposal:

  • Return a custom error during fuse.Unmount() so that file-system implementation can handle appropriately.

Testing:
Working as expected.

@raj-prince raj-prince force-pushed the unmount_err_improvement branch from 776f1ac to 5d98039 Compare July 14, 2025 09:02
@raj-prince raj-prince changed the title Suppress unmount error for /dev/fd/N mount point Suppress unmount error for /dev/fuse/N mount point Jul 14, 2025
@raj-prince raj-prince force-pushed the unmount_err_improvement branch 2 times, most recently from a9006a4 to e5d83fd Compare July 14, 2025 12:33
@raj-prince raj-prince changed the title Suppress unmount error for /dev/fuse/N mount point Suppress unmount error for /dev/fd/N mount point Jul 14, 2025
@raj-prince raj-prince force-pushed the unmount_err_improvement branch 2 times, most recently from 3852305 to 333348d Compare July 14, 2025 12:59
@raj-prince raj-prince changed the title Suppress unmount error for /dev/fd/N mount point Returning custome error while unmounting /dev/fd/N kind of mount point Jul 14, 2025
@raj-prince raj-prince force-pushed the unmount_err_improvement branch from 333348d to aef91ce Compare July 14, 2025 13:16
@raj-prince raj-prince changed the title Returning custome error while unmounting /dev/fd/N kind of mount point Returning custom error while unmounting /dev/fd/N kind of mount point Jul 14, 2025
@raj-prince raj-prince requested a review from stapelberg July 16, 2025 04:58
@raj-prince raj-prince requested a review from stapelberg July 17, 2025 08:09
@stapelberg stapelberg merged commit 15503eb into jacobsa:master Jul 17, 2025
3 checks passed
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.

2 participants