Only publish when node is master AND we have done a reconciliation

Author: nicktindall

server/src/main/java/org/elasticsearch/cluster/routing/allocation/allocator/DesiredBalanceMetrics.java

@@ -399,15 +399,15 @@ private List<LongWithAttributes> getUndesiredAllocationsExcludingShuttingDownNod
 
     private List<LongWithAttributes> getIfPublishing(ToLongFunction<AllocationStats> value) {
         var currentStats = lastReconciliationAllocationStats;
-        if (currentStats != EMPTY_ALLOCATION_STATS) {
+        if (nodeIsMaster && currentStats != EMPTY_ALLOCATION_STATS) {
             return List.of(new LongWithAttributes(value.applyAsLong(currentStats)));
         }
         return List.of();
     }
 
     private List<DoubleWithAttributes> getUndesiredAllocationsRatioMetrics() {
         var currentStats = lastReconciliationAllocationStats;
-        if (currentStats != EMPTY_ALLOCATION_STATS) {
+        if (nodeIsMaster && currentStats != EMPTY_ALLOCATION_STATS) {
             return List.of(new DoubleWithAttributes(currentStats.undesiredAllocationsRatio()));
         }
         return List.of();