Merge pull request #3662 from The-OpenROAD-Project-staging/ci-upload-metrics-fix

CI: Ensures builds with failed tests proceed to cleanup and reporting

Author: vvbandeira

jenkins/public_nightly.Jenkinsfile

@@ -55,18 +55,22 @@ node {
         buildBins(DOCKER_IMAGE, '--local --no_init --latest');
     }
 
-    stage('Run Tests') {
-        if (env.CHANGE_BRANCH && env.CHANGE_BRANCH.contains('ci-dev')) {
-            runTests(DOCKER_IMAGE, 'dev');
-        } else {
-            runTests(DOCKER_IMAGE, 'nightly');
+    try {
+        stage('Run Tests') {
+            if (env.CHANGE_BRANCH && env.CHANGE_BRANCH.contains('ci-dev')) {
+                runTests(DOCKER_IMAGE, 'dev');
+            } else {
+                runTests(DOCKER_IMAGE, 'nightly');
+            }
+        }
+    } catch (e) {
+        throw e
+    } finally {
+        stage ('Cleanup and Reporting') {
+            env.CHANGE_BRANCH = 'nightly'
+            env.BRANCH_NAME = 'nightly'
+            finalReport(DOCKER_IMAGE);
         }
-    }
-
-    stage ('Cleanup and Reporting') {
-        env.CHANGE_BRANCH = 'nightly'
-        env.BRANCH_NAME = 'nightly'
-        finalReport(DOCKER_IMAGE);
     }
 
 }

jenkins/public_tests_all.Jenkinsfile

@@ -52,16 +52,20 @@ node {
         buildBins(DOCKER_IMAGE);
     }
 
-    stage('Run Tests') {
-        if (env.CHANGE_BRANCH && env.CHANGE_BRANCH.contains('ci-dev')) {
-            runTests(DOCKER_IMAGE, 'dev');
-        } else {
-            runTests(DOCKER_IMAGE, 'pr');
+    try {
+        stage('Run Tests') {
+            if (env.CHANGE_BRANCH && env.CHANGE_BRANCH.contains('ci-dev')) {
+                runTests(DOCKER_IMAGE, 'dev');
+            } else {
+                runTests(DOCKER_IMAGE, 'pr');
+            }
+        }
+    } catch (e) {
+        throw e
+    } finally {
+        stage ('Cleanup and Reporting') {
+            finalReport(DOCKER_IMAGE);
         }
-    }
-
-    stage ('Cleanup and Reporting') {
-        finalReport(DOCKER_IMAGE);
     }
 
 }