search

IBM / ubiquity

Ubiquity
Apache License 2.0
90 stars 26 forks source link

Fix/ub 1349 can attach two pods to one pvc #227

Closed olgashtivelman closed 6 years ago

olgashtivelman commented 6 years ago

This PR is to not allow attachement of 2 pods to one PVC. currently this is allowed due to a fix of another idempotent issue. we are checking if a PVC is already attached to another pod using the slinks - in the mount process we check if the /ubiquity/volume is already linked to a pod that is not the current pod - if so then we will fail the mount process.

This change is Reviewable

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-5.3%) to 52.012% when pulling 2658568b4dfff217bd232a3b96a6ea5360971cbf on fix/UB-1349_can_attach_two_pods_to_one_pvc into 7611848ebb777a726ab2ad2ad78617bf1c0c6331 on dev.