Update some comments of the change

bindings/python/CMakeLists.txt

@@ -86,7 +86,7 @@ if (NOT WIN32 AND NOT OPEN_FOR_IDE)
   add_fdbclient_test(
     NAME multi_process_fdbcli_tests
     PROCESS_NUMBER 5
-    TEST_TIMEOUT 120
+    TEST_TIMEOUT 120 # The test can take near to 1 minutes sometime, set timeout to 2 minutes to be safe
     COMMAND ${CMAKE_SOURCE_DIR}/bindings/python/tests/fdbcli_tests.py
             ${CMAKE_BINARY_DIR}/bin/fdbcli
             @CLUSTER_FILE@

bindings/python/tests/fdbcli_tests.py

@@ -396,7 +396,7 @@ def exclude(logger):
 
 if __name__ == '__main__':
     # fdbcli_tests.py <path_to_fdbcli_binary> <path_to_fdb_cluster_file> <process_number>
-    assert len(sys.argv) == 4, "Please pass arguments: <path_to_fdbcli_binary> <path_to_fdb_cluster_file> <is_multi_process_cluster>"
+    assert len(sys.argv) == 4, "Please pass arguments: <path_to_fdbcli_binary> <path_to_fdb_cluster_file> <process_number>"
     # shell command template
     command_template = [sys.argv[1], '-C', sys.argv[2], '--exec']
     # tests for fdbcli commands

tests/TestRunner/local_cluster.py

@@ -99,6 +99,7 @@ logdir = {logdir}
                     # If a port number is given and process_number > 1, we will use subsequent numbers
                     # E.g., port = 4000, process_number = 5
                     # Then 4000,4001,4002,4003,4004 will be used as ports
+                    # If port number is not given, we will randomly pick free ports
                     for index, _ in enumerate(range(process_number)):
                         f.write('[fdbserver.{server_port}]\n'.format(server_port=self.port))
                         self.port = get_free_port() if port is None else str(int(self.port) + 1)