kubernetes-sigs / blob-csi-driver / 30063194211

83%

master: 83%

Build Type

Pull #2573

github

Committed by andyzhangx

Commit Message

fix: use syscall.Statfs instead of ReadDir for mount health probe

ReadDir(1) on a blobfuse mount translates to a BlockBlob.List() (ListBlobs)
REST call against Azure Blob Storage. It's fine for correctness but is
unnecessarily heavy for a mount-liveness check on containers with many
blobs, and it runs on every NodeStageVolume (and used to run on every
NodePublishVolume republish before #2572).

Replace it with syscall.Statfs on the mount root:

  - Statfs is served by the kernel FUSE layer and does not issue any
    data-plane request to Azure Blob Storage.
  - It still detects the failure modes we care about: ENOTCONN when the
    blobfuse process died, ESTALE, EIO on a broken mount.
  - Reference: GCS FUSE CSI driver uses the same kernel-only pattern for
    mount health probes.

Pull Request Pull Request #2573: fix: use syscall.Statfs instead of ReadDir for mount health probe

Coverage Stats

9 of 12 new or added lines in 1 file covered. (75.0%)

43 existing lines in 1 file now uncovered.

2994 of 3625 relevant lines covered (82.59%)

11.53 hits per line