commit 08f1240b467326f337197fd93758bae9c21b311e
author David Pursehouse <dpursehouse@digital.ai> Tue Apr 28 13:55:16 2020 +0900
committer David Pursehouse <dpursehouse@digital.ai> Tue Apr 28 13:55:16 2020 +0900
tree 78d40481aafb087e8f26fb952adbc79f2016f7bc
parent 841ed8fe0417a8aef985d1dd4bfd368d9a00b355
parent e1af5cb3239913d7bdc9116e6994c93a61c83306

Merge branch 'stable-3.1'

* stable-3.1:
  PushOne: Log when skipping deletion of ref

Change-Id: I708f1aa7c93aa4b655ed882fd87f32ed42fc94f2

src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java b/src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java
index 36f52fa..72cb8fb 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/replication/PushOne.java
@@ -556,12 +556,14 @@
 
     if (config.isMirror()) {
       for (Ref ref : remote.values()) {
-        if (!Constants.HEAD.equals(ref.getName())) {
-          RefSpec spec = matchDst(ref.getName());
-          if (spec != null && !local.containsKey(spec.getSource())) {
-            // No longer on local side, request removal.
-            delete(cmds, spec);
-          }
+        if (Constants.HEAD.equals(ref.getName())) {
+          repLog.atFine().log("Skipping deletion of %s", ref.getName());
+          continue;
+        }
+        RefSpec spec = matchDst(ref.getName());
+        if (spec != null && !local.containsKey(spec.getSource())) {
+          // No longer on local side, request removal.
+          delete(cmds, spec);
         }
       }
     }