1
76%
main_v2: 70%

Ran 19 Jun 2026 09:18PM UTC

Files 34

Run time 0s

Badge

Embed ▾

Committed 19 Jun 2026 09:13PM UTC coverage: 75.54% (-0.4%) from 75.907%

Job # 27848737582.1

Build Type

push

github

Committed by

jmclong

Commit Message

feat(azuredisk): skip detach for already-unattached disk on ControllerUnpublishVolume

During mass detach (e.g. when many spot/preemptible nodes are deleted at
once), ControllerUnpublishVolume calls DetachDisk per volume, which
serializes on the global VMManagementType mutex and can stall the
controller.

Add a fast-path that does a cheap disk read first: if the disk no longer
exists (HTTP 404) or exists but has no owning VM (ManagedBy and
ManagedByExtended both empty), the volume is already detached from every
node, so we return success without taking the node-centric DetachDisk
path. checkDiskExists honors GetDisk throttling and a timeout, so any read
error simply falls through to the normal detach.

The behavior is gated by a new CLI flag --skip-detach-for-unattached-disk
(default true). Adds unit tests for diskHasNoOwner and isDiskNotFoundError.

Coverage Stats

4688 of 6206 relevant lines covered (75.54%)

11.28 hits per line

jmclong / azuredisk-csi-driver / 27848737582 / 1
76%
main_v2: 70%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 27848737582.1

jmclong / azuredisk-csi-driver / 27848737582 / 1 76% main_v2: 70%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 27848737582.1

jmclong / azuredisk-csi-driver / 27848737582 / 1
76%
main_v2: 70%

README BADGES
x