Skip to content

Commit db9dd7a

Browse files
author
Aaron Schlesinger
committed
fix(pkg/cleaner/cleaner.go): remove errant debugging, lock the right value
1 parent 6fe24f5 commit db9dd7a

1 file changed

Lines changed: 9 additions & 15 deletions

File tree

pkg/cleaner/cleaner.go

Lines changed: 9 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -90,39 +90,33 @@ func Run(gitHome string, nsLister k8s.NamespaceLister, repoLock sshd.RepositoryL
9090
for i, ns := range nsList.Items {
9191
lst[i] = strings.ToLower(ns.Name)
9292
}
93-
log.Printf("Cleaner found namespaces %s", lst)
9493
}
9594

9695
gitDirs, err := localDirs(gitHome, dirHasGitSuffix)
9796
if err != nil {
9897
log.Printf("Cleaner error listing local git directories (%s)", err)
9998
continue
100-
} else {
101-
log.Printf("Cleaner found local git directories in %s: %s", gitHome, gitDirs)
10299
}
103100

104101
gitDirs = stripSuffixes(gitDirs, dotGitSuffix)
105102

106103
appsToDelete := getDiff(nsList.Items, gitDirs)
107-
if len(appsToDelete) > 0 {
108-
log.Printf("Cleaner found the following %d apps to delete: %v", len(appsToDelete), appsToDelete)
109-
} else {
110-
log.Printf("Cleaner found no apps to delete")
111-
}
104+
112105
for _, appToDelete := range appsToDelete {
113-
if err := repoLock.Lock(appToDelete, time.Duration(0)); err != nil {
114-
log.Printf("Cleaner error locking repository %s for deletion (%s)", appToDelete, err)
106+
dirToDelete := appToDelete + dotGitSuffix
107+
// this value must be the same as what's in sshd/server.go. search for repoName := parts[0]
108+
lockValue := "/" + filepath.Base(dirToDelete)
109+
if err := repoLock.Lock(lockValue, time.Duration(0)); err != nil {
110+
log.Printf("Cleaner error locking directory %s for deletion (%s)", dirToDelete, err)
115111
continue
116112
}
117-
dirToDelete := appToDelete + dotGitSuffix
118-
log.Printf("Cleaner deleting %s", dirToDelete)
113+
119114
if err := os.RemoveAll(dirToDelete); err != nil {
120115
log.Printf("Cleaner error removing deleted app %s (%s)", dirToDelete, err)
121116
}
122-
log.Printf("Cleaner deleted %s", dirToDelete)
123117

124-
if err := repoLock.Unlock(appToDelete, time.Duration(0)); err != nil {
125-
log.Printf("Cleaner error unlocking repository %s for deletion (%s)", appToDelete, err)
118+
if err := repoLock.Unlock(lockValue, time.Duration(0)); err != nil {
119+
log.Printf("Cleaner error unlocking directory %s for deletion (%s)", dirToDelete, err)
126120
continue
127121
}
128122
}

0 commit comments

Comments
 (0)