GitList

Browse code

Log error from unmountVolumes on cleanup

Signed-off-by: Alexander Morozov <lk4d4@docker.com>

Alexander Morozov authored on 2015/11/03 06:18:07
Showing 3 changed files

daemon/container.go index f6250d3..b413173 100644
pkg/system/syscall_unix.go index 5005476..f1497c5 100644
pkg/system/syscall_windows.go index 3a3a55b..b3b94cb 100644

@@ -349,7 +349,9 @@ func (container *Container) cleanup() {
 		container.daemon.unregisterExecCommand(eConfig)
 	}
 
-	container.unmountVolumes(false)
+	if err := container.unmountVolumes(false); err != nil {
+		logrus.Warnf("%s cleanup: Failed to umount volumes: %v", container.ID, err)
+	}
 }
 
 // killSig sends the container the given signal. This wrapper for the

pkg/system/syscall_unix.go

History View file @ a20fea1

@@ -6,6 +6,6 @@ import "syscall"
                      // Unmount is a platform-specific helper function to call
                      // the unmount syscall.
                     -func Unmount(dest string) {
                     -	syscall.Unmount(dest, 0)
                     +func Unmount(dest string) error {
                     +	return syscall.Unmount(dest, 0)
+                     }

pkg/system/syscall_windows.go

History View file @ a20fea1

@@ -2,5 +2,6 @@ package system
                      // Unmount is a platform-specific helper function to call
                      // the unmount syscall. Not supported on Windows
                     -func Unmount(dest string) {
                     +func Unmount(dest string) error {
                     +	return nil
+                     }

...	...	@@ -349,7 +349,9 @@ func (container *Container) cleanup() {
349	349	container.daemon.unregisterExecCommand(eConfig)
350	350	}
351	351
352		- container.unmountVolumes(false)
	352	+ if err := container.unmountVolumes(false); err != nil {
	353	+ logrus.Warnf("%s cleanup: Failed to umount volumes: %v", container.ID, err)
	354	+ }
353	355	}
354	356
355	357	// killSig sends the container the given signal. This wrapper for the