fix(cleanup): fixed a bug with time comparisons

carldanley · carldanley · commit 088f81dea1e2 · 2023-09-29T23:11:53.000-04:00
diff --git a/internal/cleanup/execute.go b/internal/cleanup/execute.go
@@ -66,11 +66,9 @@ func Execute(cfg *Config, log *logrus.Logger) error {
 			return fmt.Errorf("could not get file info: %w", err)
 		}
 
-		if fileInfo.ModTime().After(maxRetentionDay) {
+		if fileInfo.ModTime().Before(maxRetentionDay) {
 			deletionQueue = append(deletionQueue, file)
 			mbToSave += (float64(fileInfo.Size()/ByteDivisor) / ByteDivisor)
-		} else {
-			log.Debugf("Skipping %s: mod time: %v", file.Name(), fileInfo.ModTime())
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -66,11 +66,9 @@ func Execute(cfg Config, log logrus.Logger) error {`
`66`	`66`	`return fmt.Errorf("could not get file info: %w", err)`
`67`	`67`	`}`
`68`	`68`
`69`		`- if fileInfo.ModTime().After(maxRetentionDay) {`
	`69`	`+ if fileInfo.ModTime().Before(maxRetentionDay) {`
`70`	`70`	`deletionQueue = append(deletionQueue, file)`
`71`	`71`	`mbToSave += (float64(fileInfo.Size()/ByteDivisor) / ByteDivisor)`
`72`		`- } else {`
`73`		`- log.Debugf("Skipping %s: mod time: %v", file.Name(), fileInfo.ModTime())`
`74`	`72`	`}`
`75`	`73`	`}`
`76`	`74`