Update GitHub workflows to ignore specific test files and enhance RTT device handling

Author: PaulFirs

.github/workflows/test-erase.yml

@@ -12,6 +12,10 @@ on:
       - '.github/actions/setup-jlink-network/**'
       - '.github/workflows/test-erase.yml'
       - '.github/workflows/fw.*'
+    paths-ignore:
+      - '.github/workflows/test-scan.yml'
+      - '.github/workflows/test-flash.yml'
+      - '.github/workflows/test-rtt.yml'
   pull_request:
     paths:
       - 'src/bmlab_toolkit/erase_cli.py'
@@ -22,6 +26,10 @@ on:
       - '.github/actions/setup-jlink-network/**'
       - '.github/workflows/test-erase.yml'
       - '.github/workflows/fw.*'
+    paths-ignore:
+      - '.github/workflows/test-scan.yml'
+      - '.github/workflows/test-flash.yml'
+      - '.github/workflows/test-rtt.yml'
   workflow_dispatch:
 
 env:

.github/workflows/test-flash.yml

@@ -12,6 +12,10 @@ on:
       - '.github/actions/setup-jlink-network/**'
       - '.github/workflows/test-flash.yml'
       - '.github/workflows/fw.*'
+    paths-ignore:
+      - '.github/workflows/test-scan.yml'
+      - '.github/workflows/test-erase.yml'
+      - '.github/workflows/test-rtt.yml'
   pull_request:
     paths:
       - 'src/bmlab_toolkit/flashing.py'
@@ -22,6 +26,10 @@ on:
       - '.github/actions/setup-jlink-network/**'
       - '.github/workflows/test-flash.yml'
       - '.github/workflows/fw.*'
+    paths-ignore:
+      - '.github/workflows/test-scan.yml'
+      - '.github/workflows/test-erase.yml'
+      - '.github/workflows/test-rtt.yml'
   workflow_dispatch:
 
 env:

.github/workflows/test-rtt.yml

@@ -12,6 +12,10 @@ on:
       - '.github/actions/setup-jlink-network/**'
       - '.github/workflows/test-rtt.yml'
       - '.github/workflows/fw.*'
+    paths-ignore:
+      - '.github/workflows/test-scan.yml'
+      - '.github/workflows/test-flash.yml'
+      - '.github/workflows/test-erase.yml'
   pull_request:
     paths:
       - 'src/bmlab_toolkit/rtt_cli.py'
@@ -22,6 +26,10 @@ on:
       - '.github/actions/setup-jlink-network/**'
       - '.github/workflows/test-rtt.yml'
       - '.github/workflows/fw.*'
+    paths-ignore:
+      - '.github/workflows/test-scan.yml'
+      - '.github/workflows/test-flash.yml'
+      - '.github/workflows/test-erase.yml'
   workflow_dispatch:
 
 env:
@@ -93,15 +101,17 @@ jobs:
             exit 1
           fi
 
-      - name: Test 3 — RTT multiple devices by serial
+      - name: Test 3 — RTT multiple devices by serial (sequential)
         run: |
           . .venv/bin/activate
           serial1=$(sed -n '1p' serials.txt)
           serial2=$(sed -n '2p' serials.txt)
           serial3=$(sed -n '3p' serials.txt)
           
           mkdir -p rtt_logs_serial
-          timeout 10 bmlab-rtt --serial $serial1 $serial2 $serial3 --output-dir rtt_logs_serial --timeout 3 || true
+          # Serial devices are processed sequentially (USB driver limitation)
+          # Total timeout: 3 seconds per device * 3 devices + overhead
+          timeout 15 bmlab-rtt --serial $serial1 $serial2 $serial3 --mcu STM32F103RE --output-dir rtt_logs_serial --timeout 3 || true
           
           # Check that log files were created
           for s in $serial1 $serial2 $serial3; do

.github/workflows/test-scan.yml

@@ -11,6 +11,10 @@ on:
       - 'pyproject.toml'
       - '.github/actions/setup-jlink-network/**'
       - '.github/workflows/test-scan.yml'
+    paths-ignore:
+      - '.github/workflows/test-flash.yml'
+      - '.github/workflows/test-erase.yml'
+      - '.github/workflows/test-rtt.yml'
   pull_request:
     paths:
       - 'src/bmlab_toolkit/scan_cli.py'
@@ -20,6 +24,10 @@ on:
       - 'pyproject.toml'
       - '.github/actions/setup-jlink-network/**'
       - '.github/workflows/test-scan.yml'
+    paths-ignore:
+      - '.github/workflows/test-flash.yml'
+      - '.github/workflows/test-erase.yml'
+      - '.github/workflows/test-rtt.yml'
   workflow_dispatch:
 
 env:

src/bmlab_toolkit/rtt_cli.py

@@ -270,10 +270,13 @@ def rtt_device_task(device, output_dir, mcu, programmer_type, reset, timeout, ms
 
 
 def rtt_multiple_devices(devices, output_dir, mcu, programmer_type, reset, timeout, msg, msg_timeout, msg_retries, log_level):
-    """RTT for multiple devices in parallel."""
+    """RTT for multiple devices - parallel for IPs, sequential for serials."""
     # Create output directory
     Path(output_dir).mkdir(parents=True, exist_ok=True)
 
+    # Check if we have IP or serial devices
+    has_ip = any(dev['ip'] for dev in devices)
+    
     print(f"Starting RTT for {len(devices)} device(s)")
     print(f"Output directory: {output_dir}")
     if timeout == 0:
@@ -282,32 +285,49 @@ def rtt_multiple_devices(devices, output_dir, mcu, programmer_type, reset, timeo
         print(f"(Reading for {timeout} seconds)\n")
 
     results = []
-    threads = []
 
-    # Start threads for each device
-    for dev in devices:
-        device_id = dev['ip'] or f"serial {dev['serial']}" or "auto"
-        print(f"✓ Started RTT for {device_id}")
+    if has_ip:
+        # Parallel execution for IP devices using threads
+        threads = []
 
-        thread = threading.Thread(
-            target=lambda d=dev: results.append(
-                rtt_device_task(d, output_dir, mcu, programmer_type, reset, timeout, msg, msg_timeout, msg_retries, log_level)
+        for dev in devices:
+            device_id = dev['ip'] or f"serial {dev['serial']}" or "auto"
+            print(f"✓ Started RTT for {device_id}")
+            
+            thread = threading.Thread(
+                target=lambda d=dev: results.append(
+                    rtt_device_task(d, output_dir, mcu, programmer_type, reset, timeout, msg, msg_timeout, msg_retries, log_level)
+                )
             )
-        )
-        thread.start()
-        threads.append(thread)
-    
-    # Wait for all threads to complete
-    try:
-        for thread in threads:
-            thread.join()
-    except KeyboardInterrupt:
-        print("\n\nInterrupted by user. Waiting for threads to finish...")
-        for thread in threads:
-            thread.join(timeout=5)
+            thread.start()
+            threads.append(thread)
+        
+        # Wait for all threads to complete
+        try:
+            for thread in threads:
+                thread.join()
+        except KeyboardInterrupt:
+            print("\n\nInterrupted by user. Waiting for threads to finish...")
+            for thread in threads:
+                thread.join(timeout=5)
+    else:
+        # Sequential execution for serial devices (USB driver limitations)
+        print("Note: Serial devices are processed sequentially due to USB driver limitations\n")
+        
+        for dev in devices:
+            device_id = f"serial {dev['serial']}" if dev['serial'] else "auto"
+            print(f"✓ Processing RTT for {device_id}...")
+            
+            result = rtt_device_task(dev, output_dir, mcu, programmer_type, reset, timeout, msg, msg_timeout, msg_retries, log_level)
+            results.append(result)
+            
+            if result['success']:
+                print(f"  → Completed: {result['file']}\n")
+            else:
+                print(f"  → Failed: {result.get('error', 'Unknown error')}\n")
 
     # Summary
-    print(f"\n{'='*60}")
+    print(f"{'='*60}")
     print(f"RTT completed for {len(devices)} device(s)")
     print(f"{'='*60}\n")