[BugFix] fix job status in master CI (#3962)

* [BugFix] fix job status in master CI * finalize

[BugFix] fix job status in master CI (#3962)
* [BugFix] fix job status in master CI * finalize
56614f32 · Rhett Ying · GitHub · 529fd107 · 56614f32 · 56614f32
Unverified Commit 56614f32 authored Apr 29, 2022 by Rhett Ying Committed by GitHub Apr 29, 2022
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 2 deletions

Jenkinsfile Jenkinsfile +3 -1

tests/scripts/ci_report/status.py tests/scripts/ci_report/status.py +1 -1

No files found.
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -490,7 +490,9 @@ pipeline {

              def comment = sh(returnStdout: true, script: "python3 status.py").trim()
              echo(comment)
-              pullRequest.comment(comment)
+              if ((env.BRANCH_NAME).startsWith('PR-')) {
+                pullRequest.comment(comment)
+              }
            }
          }
        }

--- a/tests/scripts/ci_report/status.py
+++ b/tests/scripts/ci_report/status.py
@@ -8,7 +8,7 @@ job_link = os.environ["BUILD_URL"]
 response = requests.get('{}wfapi'.format(job_link)).json()
 status = "✅ CI test succeeded"
 for v in response['stages']:
-    if 'FAILED' in v['status']:
+    if v['status'] not in ['SUCCESS', 'IN_PROGRESS']:
        status = "❌ CI test failed in Stage [{}].".format(v['name'])
        break