From 0a98025d4b2910c2089325b87d28c32d05803e13 Mon Sep 17 00:00:00 2001
From: Brian Goff <cpuguy83@gmail.com>
Date: Tue, 22 Aug 2017 14:17:19 -0400
Subject: [PATCH] Add more detailed logging to aufs init

Addresses some comments on 276b44608b04f08bdf46ce7c816b1f744bf24b7d

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
---
 daemon/graphdriver/aufs/aufs.go | 21 ++++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/daemon/graphdriver/aufs/aufs.go b/daemon/graphdriver/aufs/aufs.go
index 7b574f9f64..05822a70c9 100644
--- a/daemon/graphdriver/aufs/aufs.go
+++ b/daemon/graphdriver/aufs/aufs.go
@@ -141,19 +141,26 @@ func Init(root string, options []string, uidMaps, gidMaps []idtools.IDMap) (grap
 			return nil, err
 		}
 	}
+	logger := logrus.WithFields(logrus.Fields{
+		"module": "graphdriver",
+		"driver": "aufs",
+	})
 
 	for _, path := range []string{"mnt", "diff"} {
 		p := filepath.Join(root, path)
-		dirs, err := ioutil.ReadDir(p)
+		entries, err := ioutil.ReadDir(p)
 		if err != nil {
-			logrus.WithError(err).WithField("dir", p).Error("error reading dir entries")
+			logger.WithError(err).WithField("dir", p).Error("error reading dir entries")
 			continue
 		}
-		for _, dir := range dirs {
-			if strings.HasSuffix(dir.Name(), "-removing") {
-				logrus.WithField("dir", dir.Name()).Debug("Cleaning up stale layer dir")
-				if err := system.EnsureRemoveAll(filepath.Join(p, dir.Name())); err != nil {
-					logrus.WithField("dir", dir.Name()).WithError(err).Error("Error removing stale layer dir")
+		for _, entry := range entries {
+			if !entry.IsDir() {
+				continue
+			}
+			if strings.HasSuffix(entry.Name(), "-removing") {
+				logger.WithField("dir", entry.Name()).Debug("Cleaning up stale layer dir")
+				if err := system.EnsureRemoveAll(filepath.Join(p, entry.Name())); err != nil {
+					logger.WithField("dir", entry.Name()).WithError(err).Error("Error removing stale layer dir")
 				}
 			}
 		}