From 81de8c8246ad2e63ff9e9baab9c19fadf873b32d Mon Sep 17 00:00:00 2001 From: Juee Himalbhai Desai Date: Wed, 29 Jan 2025 14:54:28 -0800 Subject: [PATCH] contrib/intel/jenkins: Use strings properly for send-mail Signed-off-by: Juee Himalbhai Desai --- contrib/intel/jenkins/Jenkinsfile | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/contrib/intel/jenkins/Jenkinsfile b/contrib/intel/jenkins/Jenkinsfile index 4132adac96e..c4cd099a918 100644 --- a/contrib/intel/jenkins/Jenkinsfile +++ b/contrib/intel/jenkins/Jenkinsfile @@ -122,19 +122,20 @@ def gather_logs(cluster, key, dest, source) { } def CI_summarize(verbose=false) { - cmd = """source ${CI_LOCATION}/${env.CI_MODULE}/venv/bin/activate;\ - python ${CI_LOCATION}/summarize.py \ - --log_directory=${env.LOG_DIR} - """ + def options = "" if (verbose) { - cmd = "${cmd} -v" + options = "${options} -v" } if (weekly || RELEASE) { - cmd = "${cmd} --send-mail" + options = "${options} --send-mail" } - sh "${cmd}" + sh """source ${CI_LOCATION}/${env.CI_MODULE}/venv/bin/activate;\ + python ${CI_LOCATION}/summarize.py \ + --log_directory=${env.LOG_DIR} \ + ${options} + """ } def summarize(item, verbose=false, release=false, send_mail=false) {