osmith has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ci/+/42648?usp=email )

Change subject: jobs/gerrit-verifications: fix hwtest param
......................................................................

jobs/gerrit-verifications: fix hwtest param

This must be a bool instead of a string. Without this patch, the
condition checking for the bool is always true.

Fixes: 986c2648 ("jobs/gerrit: run osmo-ccid-firmware-hwtest")
Change-Id: I3117ddab888dd226ed57b62b9b9e672f1cb9127d
---
M jobs/gerrit-verifications.yml
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  fixeria: Looks good to me, but someone else must approve
  osmith: Looks good to me, approved
  Jenkins Builder: Verified
  jolly: Looks good to me, but someone else must approve




diff --git a/jobs/gerrit-verifications.yml b/jobs/gerrit-verifications.yml
index 09c465b..faae9b9 100644
--- a/jobs/gerrit-verifications.yml
+++ b/jobs/gerrit-verifications.yml
@@ -591,7 +591,7 @@
           name: PIPELINE_BUILD
           description: Enable the build job (runs contrib/jenkins.sh)
           default: '{obj:pipeline_build}'
-      - string:
+      - bool:
           name: PIPELINE_HWTEST
           description: |
             Enable a hardware test job that uses artifacts from the build job.

--
To view, visit https://gerrit.osmocom.org/c/osmo-ci/+/42648?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: I3117ddab888dd226ed57b62b9b9e672f1cb9127d
Gerrit-Change-Number: 42648
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: jolly <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>

Reply via email to