Removing the duplicate process check based on truncated name and rely…

…ing on cmdline check

On linux, the process name is truncated to 15 characters and hence it matches both
opensearch-benchmark and opensearch-benchmarkd. This results in killing the daemon
process and breaks the distributed run setup. Removing the name based check on the
truncated name and relying on command line which provides the full command.

Keeping the name based check on full name as is.

Signed-off-by: Prabhakar Sithanandam <[email protected]>

osbenchmark/utils/process.py

@@ -133,12 +133,7 @@ def run_subprocess_with_logging(command_line, header=None, level=logging.INFO, s
 
 def is_benchmark_process(p):
     cmdline = p.cmdline()
-    # On Linux, /proc/PID/status truncates the command name to 15 characters which matches both opensearch-benchmark
-    # and opensearch-benchmarkd. Hence, checking command line to exclude opensearch-benchmarkd process.
     return p.name() == "opensearch-benchmark" or \
-        (p.name() == "opensearch-benc" and
-         len(cmdline) > 1 and
-         os.path.basename(cmdline[1]) != "opensearch-benchmarkd") or \
         (len(cmdline) > 1 and
          os.path.basename(cmdline[0].lower()).startswith("python") and
          os.path.basename(cmdline[1]) == "opensearch-benchmark")