Fixes the active shard count check in the case of (#19760)

ActiveShardCount.ALL by checking for active shards,
not just started shards, as a shard could be active
but in the relocating state (i.e. not in the started
state).

Author: Ali Beyad

core/src/main/java/org/elasticsearch/action/support/ActiveShardCount.java

@@ -160,12 +160,15 @@ public boolean enoughShardsActive(final ClusterState clusterState, final String
      * to meet the required shard count represented by this instance.
      */
     public boolean enoughShardsActive(final IndexShardRoutingTable shardRoutingTable) {
+        final int activeShardCount = shardRoutingTable.activeShards().size();
         if (this == ActiveShardCount.ALL) {
-            return shardRoutingTable.allShardsStarted();
+            // adding 1 for the primary in addition to the total number of replicas,
+            // which gives us the total number of shard copies
+            return activeShardCount == shardRoutingTable.replicaShards().size() + 1;
         } else if (this == ActiveShardCount.DEFAULT) {
-            return shardRoutingTable.primaryShard().started();
+            return activeShardCount >= 1;
         } else {
-            return shardRoutingTable.activeShards().size() >= value;
+            return activeShardCount >= value;
         }
     }