Merge branch 'stable-3.8' * stable-3.8: ConfigurationTest: Remove flaky invalidTargetArchiveFolder test Send Project Deleted Events also when archiving Change-Id: Ic64ac7317f430f520ef0350c0bbdb63846823086
diff --git a/src/main/java/com/googlesource/gerrit/plugins/deleteproject/fs/RepositoryDelete.java b/src/main/java/com/googlesource/gerrit/plugins/deleteproject/fs/RepositoryDelete.java index 5ed3b1a..a871817 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/deleteproject/fs/RepositoryDelete.java +++ b/src/main/java/com/googlesource/gerrit/plugins/deleteproject/fs/RepositoryDelete.java
@@ -92,7 +92,7 @@ Path repoPath = repository.getDirectory().toPath(); String projectName = project.get(); if (archiveDeletedRepos) { - archiveGitRepository(projectName, repoPath, archivedFolder); + archiveGitRepository(projectName, repoPath, archivedFolder, deletedListeners); } else { deleteGitRepository(projectName, repoPath, deletedListeners); } @@ -118,7 +118,7 @@ } private static void archiveGitRepository( - String projectName, Path repoPath, Optional<Path> archivedFolder) throws IOException { + String projectName, Path repoPath, Optional<Path> archivedFolder, DynamicSet<ProjectDeletedListener> deletedListeners) throws IOException { Path basePath = getBasePath(repoPath, projectName); if (archivedFolder.isEmpty()) { throw new IllegalArgumentException( @@ -131,6 +131,8 @@ MoreFiles.deleteRecursively(renamedProjectDir, ALLOW_INSECURE); } catch (IOException e) { log.atWarning().withCause(e).log("Error trying to archive %s", renamedProjectDir); + } finally { + sendProjectDeletedEvent(projectName, deletedListeners); } }