MB-68093: Fix autofailover trim_needed check

It should take into account the current count

Change-Id: I28d3e229003156c1ea56b8f6416cf5af13d1ecef
Reviewed-on: https://review.couchbase.org/c/ns_server/+/232306
Well-Formed: Build Bot <[email protected]>
Reviewed-by: Neelima Premsankar <[email protected]>
Tested-by: Build Bot <[email protected]>

Author: boparai11

apps/ns_server/src/auto_failover.erl

@@ -628,8 +628,8 @@ process_action({failover, NodesWithUUIDs}, S, DownNodes, NodeStatuses,
 
 trim_needed(_Nodes, #state{disable_max_count = true}) ->
     false;
-trim_needed(Nodes, #state{max_count = Max}) ->
-    length(Nodes) > Max.
+trim_needed(Nodes, #state{count = Count, max_count = Max}) ->
+    Count  + length(Nodes) > Max.
 
 trim_nodes(Nodes, #state{count = Count, max_count = Max}) ->
     lists:sublist(Nodes, Max - Count).