.ci/aws: Make failures happen in correct stage

Signed-off-by: Seth Zegelstein <[email protected]>
a-szegel · Aug 15, 2024 · 4e9c301 · 4e9c301
1 parent 32daa37
commit 4e9c301
Showing 1 changed file with 5 additions and 25 deletions.
diff --git a/.ci/aws/Jenkinsfile b/.ci/aws/Jenkinsfile
@@ -6,9 +6,6 @@ if (buildNumber > 1) milestone(buildNumber - 1)
 milestone(buildNumber)
 
 
-import groovy.transform.Field
-@Field boolean build_ok = true
-
 def get_portafiducia_download_path() {
     /* Stable Portafiducia tarball */
     def AWS_ACCOUNT_ID = sh (
@@ -84,16 +81,11 @@ def run_test_orchestrator_once(run_name, build_tag, os, instance_type, instance_
 
     def cluster_name = get_cluster_name(build_tag, os, instance_type)
     def args = "--os ${os} --odcr ${odcr} --instance-type ${instance_type} --instance-count ${instance_count} --region ${region} --cluster-name ${cluster_name} ${addl_args} --junit-xml outputs/${cluster_name}.xml"
-    def ret = sh (
-                    script: ". venv/bin/activate; ./PortaFiducia/tests/test_orchestrator.py ${args}",
-                    returnStatus: true
-                  )
-    if (ret == 65)
-        unstable('Scripts exited with status 65')
-    else if (ret != 0)
-        build_ok = false
-    catchError(buildResult: 'SUCCESS', stageResult: 'FAILURE') {
-        sh "exit ${ret}"
+
+    try {
+        sh ". venv/bin/activate; ./PortaFiducia/tests/test_orchestrator.py ${args}",
+    } catch (Exception e) {
+        currentBuild.result = "FAILURE"
     }
 }
 
@@ -256,18 +248,6 @@ pipeline {
                 }
             }
         }
-        stage('check build_ok') {
-            steps {
-                script {
-                    if (build_ok) {
-                        currentBuild.result = "SUCCESS"
-                    }
-                    else {
-                        currentBuild.result = "FAILURE"
-                    }
-                }
-            }
-        }
     }
     post {
         always {