Fix project deletion logs Log was showing project was deleted successfully even if it failed. Change-Id: I7be19991619ce1f205884a8b6658d9a7ed4109c7
diff --git a/src/main/java/com/googlesource/gerrit/plugins/replication/ReplicationQueue.java b/src/main/java/com/googlesource/gerrit/plugins/replication/ReplicationQueue.java index ebf9278..0f367aa 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/replication/ReplicationQueue.java +++ b/src/main/java/com/googlesource/gerrit/plugins/replication/ReplicationQueue.java
@@ -288,10 +288,8 @@ private void deleteProject(URIish replicateURI) { if (!replicateURI.isRemote()) { deleteLocally(replicateURI); - repLog.info("Deleted local repository: {}", replicateURI); } else if (isSSH(replicateURI)) { deleteRemoteSsh(replicateURI); - repLog.info("Deleted remote repository: {}", replicateURI); } else { repLog.warn( "Cannot delete project on remote site {}. " @@ -304,6 +302,7 @@ private static void deleteLocally(URIish uri) { try { recursivelyDelete(new File(uri.getPath())); + repLog.info("Deleted local repository: {}", uri); } catch (IOException e) { repLog.error("Error deleting local repository {}:\n", uri.getPath(), e); } @@ -333,6 +332,7 @@ OutputStream errStream = newErrorBufferStream(); try { executeRemoteSsh(uri, cmd, errStream); + repLog.info("Deleted remote repository: {}", uri); } catch (IOException e) { repLog.error( "Error deleting remote repository at {}:\n"