events: Fix reversed logic in clean()
This is a bug preventing cleanup of the build directory under normal
operation. We want the logic to only skip cleaning the build directory
when there was more than 'max' items cleaned in the delete directory.
Change-Id: Ib457a8b47c2c025dc7859094836ea6624f4ae4c3
diff --git a/src/main/java/com/googlesource/gerrit/plugins/events/fsstore/FsTransaction.java b/src/main/java/com/googlesource/gerrit/plugins/events/fsstore/FsTransaction.java
index 6884daa..cb7192b 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/events/fsstore/FsTransaction.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/events/fsstore/FsTransaction.java
@@ -68,7 +68,7 @@
public boolean clean(FileTime expiry, int max) {
try {
return Fs.tryRecursiveDeleteEntriesOlderThan(delete, expiry, max)
- || renameAndDeleteEntriesOlderThan(build, delete, expiry, max);
+ && renameAndDeleteEntriesOlderThan(build, delete, expiry, max);
} catch (IOException e) {
// If we knew if it was a repeat offender, we could consider logging it.
return true; // Don't keep retrying failures.
