← Back to team overview

canonical-hw-cert team mailing list archive

[Merge] ~kevinyeh/hwcert-jenkins-jobs:fix-remove-useless-job into hwcert-jenkins-jobs:master

 

Kevin Yeh has proposed merging ~kevinyeh/hwcert-jenkins-jobs:fix-remove-useless-job into hwcert-jenkins-jobs:master.

Commit message:
Fix: fix jenkins job: remove-useless-job
     1. fix wrong job name, should with a prefix cert.
     2. fix missing a double quote.

Requested reviews:
  hwcert-jenkins (hwcert-jenkins)

For more details, see:
https://code.launchpad.net/~kevinyeh/hwcert-jenkins-jobs/+git/hwcert-jenkins-jobs/+merge/433970
-- 
Your team hwcert-jenkins is requested to review the proposed merge of ~kevinyeh/hwcert-jenkins-jobs:fix-remove-useless-job into hwcert-jenkins-jobs:master.
diff --git a/jobs/infrastructure/remove-jobs/remove-hwcert-jobs.sh b/jobs/infrastructure/remove-jobs/remove-hwcert-jobs.sh
index 71dfd28..11e86a2 100644
--- a/jobs/infrastructure/remove-jobs/remove-hwcert-jobs.sh
+++ b/jobs/infrastructure/remove-jobs/remove-hwcert-jobs.sh
@@ -28,7 +28,7 @@ url=http://10.102.156.15:8080
 query_plugins_info=False
 EOF
 
-echo "Action: $JJB_FUNCTION 
+echo "Action: $JJB_FUNCTION"
 jenkins-jobs --conf jjb.conf list -p production.yaml:jobs/ -r > local_jobs
 jenkins-jobs --conf jjb.conf list > remote_jobs
 diff local_jobs remote_jobs | grep "^>" |  sed 's/^> //' > removeable_job.log
diff --git a/jobs/infrastructure/remove-jobs/remove-hwcert-jobs.yaml b/jobs/infrastructure/remove-jobs/remove-hwcert-jobs.yaml
index 9696b7d..6a4ebfd 100644
--- a/jobs/infrastructure/remove-jobs/remove-hwcert-jobs.yaml
+++ b/jobs/infrastructure/remove-jobs/remove-hwcert-jobs.yaml
@@ -1,7 +1,7 @@
 - project:
     name: remove-useless
     jobs:
-        - '{name}-test-jobs'
+        - 'cert-{name}-test-jobs'
 
 - job-template:
     name: 'cert-{name}-test-jobs'

Follow ups