Sfoglia il codice sorgente

Add more detailed logging to aufs init

Addresses some comments on 276b44608b04f08bdf46ce7c816b1f744bf24b7d

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
Brian Goff 7 anni fa
parent
commit
0a98025d4b
1 ha cambiato i file con 14 aggiunte e 7 eliminazioni
  1. 14 7
      daemon/graphdriver/aufs/aufs.go

+ 14 - 7
daemon/graphdriver/aufs/aufs.go

@@ -141,19 +141,26 @@ func Init(root string, options []string, uidMaps, gidMaps []idtools.IDMap) (grap
 			return nil, err
 			return nil, err
 		}
 		}
 	}
 	}
+	logger := logrus.WithFields(logrus.Fields{
+		"module": "graphdriver",
+		"driver": "aufs",
+	})
 
 
 	for _, path := range []string{"mnt", "diff"} {
 	for _, path := range []string{"mnt", "diff"} {
 		p := filepath.Join(root, path)
 		p := filepath.Join(root, path)
-		dirs, err := ioutil.ReadDir(p)
+		entries, err := ioutil.ReadDir(p)
 		if err != nil {
 		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
 			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")
 				}
 				}
 			}
 			}
 		}
 		}