Send Project Deleted Events also when archiving
When projects are either deleted fully, or preserved by setting as
hidden and moving to f.i. Deleted-Projects, we send a Project Deleted
Event. We do not do the same if we archive the repository, even though
this is effectively also a deleted project (more so than hidden is) from
a Gerrit perspective.
Not sending this event means that archived repositories are not properly
removed from replicas when replicateRepositoryDeletions is set.
This makes sure that we still send the projectDeletionEvent while
archiving is set.
Release-Notes: Plugin Delete-Project archiveDeletedRepos option now also emits a Project Deletion event.
Change-Id: I0829a1a2016d67ac789b92f67d1d7c6a13b65c65
diff --git a/src/main/java/com/googlesource/gerrit/plugins/deleteproject/fs/FilesystemDeleteHandler.java b/src/main/java/com/googlesource/gerrit/plugins/deleteproject/fs/FilesystemDeleteHandler.java
index 053b2fc..bbfa633 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/deleteproject/fs/FilesystemDeleteHandler.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/deleteproject/fs/FilesystemDeleteHandler.java
@@ -88,6 +88,8 @@
MoreFiles.deleteRecursively(renamedProjectDir, ALLOW_INSECURE);
} catch (IOException e) {
log.atWarning().withCause(e).log("Error trying to archive %s", renamedProjectDir);
+ } finally {
+ sendProjectDeletedEvent(projectName);
}
}
