Fix: sys.exit(1) runs unconditionally due to indentation bug

[Jah-yee]

Description

The sys.exit(1) was at the same indentation level as the try block, causing it to run unconditionally after subprocess completes - even when the command succeeds. This fix moves sys.exit(1) inside the except block.

Bug Details

In the run_command function, when log_step is None (the else branch), the sys.exit(1) was placed outside the except block:

else:
    try:
        subprocess.run(command, shell=shell, check=True)
    except subprocess.CalledProcessError as e:
        logging.error(f"Error occurred while running command: {e}")
    sys.exit(1)  # BUG: Runs unconditionally!

The fix properly indents sys.exit(1) to be inside the except block.

Affected Files

• setup_env.py (line ~110)
• utils/e2e_benchmark.py (line ~23)

Related Issue

This PR addresses issue #447