fix(test): mitigate subprocess fork race conditions in logging test

Use subprocess.run() instead of Popen() to mitigate race conditions in s3client fork handlers.

A stronger fix might be needed if this still causes race conditions, as run() still uses fork multiprocessing.

Author: jet-tong

s3torchconnectorclient/python/tst/integration/test_logging.py

@@ -1,6 +1,7 @@
 #  Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
 #  // SPDX-License-Identifier: BSD
 import os.path
+import platform
 import sys
 import tempfile
 from typing import List
@@ -180,7 +181,7 @@ def _start_subprocess(
     debug_logs_config: str = "",
     logs_directory: str = "",
 ):
-    process = subprocess.Popen(
+    result = subprocess.run(
         [
             sys.executable,
             "-c",
@@ -190,11 +191,10 @@ def _start_subprocess(
             debug_logs_config,
             logs_directory,
         ],
-        stdout=subprocess.PIPE,
-        stderr=subprocess.PIPE,
+        capture_output=True,
         text=True,
     )
-    return process.communicate()
+    return result.stdout, result.stderr
 
 
 def _read_log_file(log_file: str):