Merge pull request #1864 from AllenInstitute/feature/1864-pipeline-tw…

…eaks

Add timeout statements for quicker error reporting

.github/workflows/build-pr.yml

@@ -105,7 +105,7 @@ jobs:
       installer_artifact_name: BuildInstaller-dev-assets
       installer_flags: ${{ fromJSON('["-x skipHardwareXOPs",""]')[matrix.hardware] }} -s ${{ matrix.source }}
       artifact_name: CompilationTest-${{ matrix.source }}-${{ fromJSON('["no-hardware","hardware"]')[matrix.hardware] }}-assets
-      expensive_checks: "1"
+      timeout_minutes: 60
 
   Documentation:
     name: 👷 Documentation
@@ -151,6 +151,7 @@ jobs:
       artifact_name: test-without-hw-assets
       expensive_checks: "1"
       instrument_tests: ${{ fromJson('["0", "1"]')[inputs.do_instrumentation] }}
+      timeout_minutes: 60
 
   TestNI:
     name: 🧪 Test NI ${{ matrix.name }}
@@ -173,6 +174,7 @@ jobs:
       artifact_name: test-ni-assets
       expensive_checks: "1"
       instrument_tests: ${{ fromJson('["0", "1"]')[inputs.do_instrumentation] }}
+      timeout_minutes: 180
 
   TestITC:
     name: 🧪 Test ITC ${{ matrix.name }}
@@ -195,6 +197,7 @@ jobs:
       artifact_name: test-itc-assets
       expensive_checks: "1"
       instrument_tests: ${{ fromJson('["0", "1"]')[inputs.do_instrumentation] }}
+      timeout_minutes: 180
 
   ValidateNwb:
     name: 👮🏼 Validate NWBv2

.github/workflows/test-igor-workflow.yml

@@ -51,6 +51,11 @@ on:
         default: "0"
         type: string
         description: Sets environment variable CI_INSTRUMENT_TESTS
+      timeout_minutes:
+        required: false
+        default: 360
+        type: number
+        description: Defines the job timeout in minutes
 defaults:
   run:
     shell: bash
@@ -59,6 +64,7 @@ jobs:
   Test:
     name: ${{ inputs.overwrite_job_name && inputs.job_name || 'Run' }}
     runs-on: ${{ fromJson(inputs.target) }}
+    timeout-minutes: ${{ inputs.timeout_minutes }}
     env:
       CI_EXPENSIVE_CHECKS: ${{ inputs.expensive_checks }}
       CI_INSTRUMENT_TESTS: ${{ inputs.instrument_tests }}