[MESOS-9729] Unpublishing a volume that is failed to publish crashes the agent with CSI v1.

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 1.8.0
• Fix Version/s: 1.8.0
• Labels: mesosphere mesosphere-dss-ga storage
• Target Version/s: 1.8.1
• Epic Link: CSI Spec v1.0 Support
• Sprint: Storage: RI-13 Sprint 44
• Story Points: 1
• Assignee: Chun-Hung Hsiao
• Reporter: Chun-Hung Hsiao
• Shepherd: Benjamin Bannier
• Created: 12/Apr/19 20:57
• Updated: 12/Apr/19 22:23
• Resolved: 12/Apr/19 22:23

Description

The CSI v1 volume manager recovers a failed `publishVolume` call through `unpublishVolume`, which mistakenly assume that the target path, which is supposed to be created by the CSI plugin after a successful publishing, always exists. If volume publishing fails, a subsequent unpublishing would crash the agent with the following message:

F0412 20:20:12.254420  7540 v1_volume_manager.cpp:1161] Check failed: os::exists(targetPath)