From 9ac0ae7cba36eb28fea0d0730cd2eaf069f6675d Mon Sep 17 00:00:00 2001 From: Jobbins Date: Thu, 5 Feb 2026 08:56:42 -0700 Subject: [PATCH] [composablekernel] fix failure status (#4351) ## Motivation Pipelines were failing on Math CI status check. ## Technical Details For the success case, we just changed the config in Jenkins to use a proper app token and no code changes were required. However, the failure case would not have worked as coded, so we needed to move that outside of the `rocmnode()` block. ## Test Plan I removed all of the CI in one of the commits to quickly test, and then added it back. Got a successful "success" message and "failure" message produced --- Jenkinsfile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Jenkinsfile b/Jenkinsfile index 363caed12e..2a6b8df1bd 100644 --- a/Jenkinsfile +++ b/Jenkinsfile @@ -1965,10 +1965,10 @@ pipeline { description: 'All checks have passed' } failure { + githubNotify context: 'Math CI Summary', + status: 'FAILURE', + description: 'Some checks have failed' node(rocmnode("nogpu")) { - githubNotify context: 'Math CI Summary', - status: 'FAILURE', - description: 'Some checks have failed' script { sendFailureNotifications() }