Added version protection for correct pre/post TCM behaviours

Author: bereng

bootstrap_test.py

@@ -250,11 +250,12 @@ def test_read_from_bootstrapped_node(self):
         """
         cluster = self.cluster
         cluster.set_environment_variable('CASSANDRA_TOKEN_PREGENERATION_DISABLED', 'True')
-        # Nodes need to be started one by one, not all at once. See CASSANDRA-19097
         cluster.populate(3)
-        cluster.nodelist()[0].start(wait_for_binary_proto=True)
-        cluster.nodelist()[1].start(wait_for_binary_proto=True)
-        cluster.nodelist()[2].start(wait_for_binary_proto=True)
+        if cluster.version() <= '4.1':
+            # Nodes need to be started one by one pre-TCM, not all at once. See CASSANDRA-19097
+            cluster.nodelist()[0].start(wait_for_binary_proto=True)
+            cluster.nodelist()[1].start(wait_for_binary_proto=True)
+            cluster.nodelist()[2].start(wait_for_binary_proto=True)
         cluster.start()
 
         node1 = cluster.nodes['node1']
@@ -522,10 +523,11 @@ def test_manual_bootstrap(self):
         """
         cluster = self.cluster
         cluster.set_environment_variable('CASSANDRA_TOKEN_PREGENERATION_DISABLED', 'True')
-        # Nodes need to be started one by one, not all at once. See CASSANDRA-19097
         cluster.populate(2)
-        cluster.nodelist()[0].start(wait_for_binary_proto=True)
-        cluster.nodelist()[1].start(wait_for_binary_proto=True)
+        if cluster.version() <= '4.1':
+            # Nodes need to be started one by one pre-TCM, not all at once. See CASSANDRA-19097
+            cluster.nodelist()[0].start(wait_for_binary_proto=True)
+            cluster.nodelist()[1].start(wait_for_binary_proto=True)
         cluster.start()
         (node1, node2) = cluster.nodelist()
 
@@ -627,11 +629,12 @@ def _wiped_node_cannot_join_test(self, gently):
         """
         cluster = self.cluster
         cluster.set_environment_variable('CASSANDRA_TOKEN_PREGENERATION_DISABLED', 'True')
-        # Nodes need to be started one by one, not all at once. See CASSANDRA-19097
         cluster.populate(3)
-        cluster.nodelist()[0].start(wait_for_binary_proto=True)
-        cluster.nodelist()[1].start(wait_for_binary_proto=True)
-        cluster.nodelist()[2].start(wait_for_binary_proto=True)
+        if cluster.version() <= '4.1':
+            # Nodes need to be started one by one pre-TCM, not all at once. See CASSANDRA-19097
+            cluster.nodelist()[0].start(wait_for_binary_proto=True)
+            cluster.nodelist()[1].start(wait_for_binary_proto=True)
+            cluster.nodelist()[2].start(wait_for_binary_proto=True)
         cluster.start()
 
         stress_table = 'keyspace1.standard1'
@@ -666,11 +669,12 @@ def test_decommissioned_wiped_node_can_join(self):
         """
         cluster = self.cluster
         cluster.set_environment_variable('CASSANDRA_TOKEN_PREGENERATION_DISABLED', 'True')
-        # Nodes need to be started one by one, not all at once. See CASSANDRA-19097
         cluster.populate(3)
-        cluster.nodelist()[0].start(wait_for_binary_proto=True)
-        cluster.nodelist()[1].start(wait_for_binary_proto=True)
-        cluster.nodelist()[2].start(wait_for_binary_proto=True)
+        if cluster.version() <= '4.1':
+            # Nodes need to be started one by one pre-TCM, not all at once. See CASSANDRA-19097
+            cluster.nodelist()[0].start(wait_for_binary_proto=True)
+            cluster.nodelist()[1].start(wait_for_binary_proto=True)
+            cluster.nodelist()[2].start(wait_for_binary_proto=True)
         cluster.start()
 
         stress_table = 'keyspace1.standard1'