[FIXED JENKINS-25655] Incorrect caching from entrySet sometimes caused incorrect results.

Caused a failure in hudson.plugins.promoted_builds.conditions.ManualConditionTest.testManualPromotionProcessViaWebClient when run against 1.580.1.

core/src/main/java/jenkins/model/lazy/AbstractLazyLoadRunMap.java

@@ -585,6 +585,8 @@ public abstract class AbstractLazyLoadRunMap<R> extends AbstractMap<Integer,R> i
             numberOnDisk = a;
         }
 
+        entrySet.clearCache();
+
         return unwrap(old);
     }
 
@@ -742,6 +744,8 @@ public abstract class AbstractLazyLoadRunMap<R> extends AbstractMap<Integer,R> i
         BuildReference<R> old = copy.byId.remove(getIdOf(run));
         this.index = copy;
 
+        entrySet.clearCache();
+
         return unwrap(old)!=null;
     }
 

core/src/main/java/jenkins/model/lazy/LazyLoadRunMapEntrySet.java

@@ -32,6 +32,10 @@ class LazyLoadRunMapEntrySet<R> extends AbstractSet<Entry<Integer,R>> {
         return all;
     }
 
+    synchronized void clearCache() {
+        all = null;
+    }
+
     @Override
     public int size() {
         return all().size();

core/src/test/java/jenkins/model/lazy/AbstractLazyLoadRunMapTest.java

@@ -414,6 +414,13 @@ public class AbstractLazyLoadRunMapTest extends Assert {
         assertEquals(0, b.entrySet().size());
     }
 
+    @Issue("JENKINS-25655")
+    @Test public void entrySetChanges() {
+        assertEquals(3, a.entrySet().size());
+        a.put(new Build(7, "D"));
+        assertEquals(4, a.entrySet().size());
+    }
+
     @Issue("JENKINS-18065")
     @Test
     public void entrySetContains() {