Browse code

devmapper: udev sync is a requirement

closes #10664
closes #4036

Signed-off-by: Vincent Batts <vbatts@redhat.com>

Vincent Batts authored on 2015/03/17 02:18:40
Showing 1 changed files
... ...
@@ -963,9 +963,9 @@ func (devices *DeviceSet) initDevmapper(doInit bool) error {
963 963
 
964 964
 	// https://github.com/docker/docker/issues/4036
965 965
 	if supported := devicemapper.UdevSetSyncSupport(true); !supported {
966
-		logrus.Warnf("Udev sync is not supported. This will lead to unexpected behavior, data loss and errors")
966
+		logrus.Errorf("Udev sync is not supported. This will lead to unexpected behavior, data loss and errors")
967
+		return graphdriver.ErrNotSupported
967 968
 	}
968
-	logrus.Debugf("devicemapper: udev sync support: %v", devicemapper.UdevSyncSupported())
969 969
 
970 970
 	if err := os.MkdirAll(devices.metadataDir(), 0700); err != nil && !os.IsExist(err) {
971 971
 		return err