Fix InternalTestCluster StopRandomNode Assertion (#44258) (#44265)

* The assertion added in #44214 is tripped by tests running dedicated
test clusters per test needlessly.This breaks existing tests like the one in #44245.
* Closes #44245

Author: original-brownbear

server/src/test/java/org/elasticsearch/action/admin/indices/shards/IndicesShardStoreRequestIT.java

@@ -71,7 +71,6 @@ public void testEmpty() {
         assertThat(rsp.getStoreStatuses().size(), equalTo(0));
     }
 
-    @AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/44245")
     public void testBasic() throws Exception {
         String index = "test";
         internalCluster().ensureAtLeastNumDataNodes(2);

test/framework/src/main/java/org/elasticsearch/test/InternalTestCluster.java

@@ -1632,7 +1632,9 @@ public synchronized void stopRandomNode(final Predicate<Settings> filter) throws
         ensureOpen();
         NodeAndClient nodeAndClient = getRandomNodeAndClient(nc -> filter.test(nc.node.settings()));
         if (nodeAndClient != null) {
-            if (nodeAndClient.nodeAndClientId() < sharedNodesSeeds.length && nodeAndClient.isMasterEligible() && autoManageMasterNodes
+            if (nodePrefix.equals(ESIntegTestCase.SUITE_CLUSTER_NODE_PREFIX) && nodeAndClient.nodeAndClientId() < sharedNodesSeeds.length
+                && nodeAndClient.isMasterEligible()
+                && autoManageMasterNodes
                 && nodes.values().stream()
                         .filter(NodeAndClient::isMasterEligible)
                         .filter(n -> n.nodeAndClientId() < sharedNodesSeeds.length)