canonical-hw-cert team mailing list archive

[Kevin Yeh <mp+432786@xxxxxxxxxxxxxxxxxx>]

Kevin Yeh has proposed merging ~kevinyeh/hwcert-jenkins-jobs:name-job-with-systemid into hwcert-jenkins-jobs:master.

Commit message:
Change: name job with system-id instead of project name.

Requested reviews:
  hwcert-jenkins (hwcert-jenkins)

For more details, see:
https://code.launchpad.net/~kevinyeh/hwcert-jenkins-jobs/+git/hwcert-jenkins-jobs/+merge/432786

In the past we use project name as jenkins job since each DUT has own job yaml.
Now we put all DUT which test on same release in one yaml file, so project name is always the same.
So we start using system-id for some jobs(impish,jammy,raspi), but I think it can apply to all jenkins jobs now.  
-- 
Your team hwcert-jenkins is requested to review the proposed merge of ~kevinyeh/hwcert-jenkins-jobs:name-job-with-systemid into hwcert-jenkins-jobs:master.

diff --git a/jobs/sru/run-sru.sh b/jobs/sru/run-sru.sh
index fed0583..223c19e 100644
--- a/jobs/sru/run-sru.sh
+++ b/jobs/sru/run-sru.sh
@@ -267,11 +267,7 @@ git -C hwcert-data pull -q || git clone -q git+ssh://ce-certification-qa@xxxxxxx
 
 # Create a card if this is a proper SRU run
 if [ "{trello_reporting}" == "True" ]; then
-if [[ "{srutype}" =~ "raspi" ]] || [[ "{series}" == "impish" ]] || [[ "{series}" == "jammy" ]]; then
     flock /tmp/TRELLO-UPDATE.LOCK certification-dashboard-manager.deb-board-updater --config hwcert-data/debs.yaml -a {arch} -n {system-id} -k {kernel-to-check} -s {series} -t {srutype} --cardonly /dev/null
-  else
-    flock /tmp/TRELLO-UPDATE.LOCK certification-dashboard-manager.deb-board-updater --config hwcert-data/debs.yaml -a {arch} -n {name} -k {kernel-to-check} -s {series} -t {srutype} --cardonly /dev/null
-  fi
 fi
 
 JOB_ID=$(testflinger submit -q job.yaml)