]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-build.git/commit
jjb: be verbose and fail early on errors 21/head
authorKen Dreyer <kdreyer@redhat.com>
Fri, 14 Nov 2014 14:27:53 +0000 (07:27 -0700)
committerKen Dreyer <kdreyer@redhat.com>
Fri, 14 Nov 2014 14:27:53 +0000 (07:27 -0700)
commitbcb21eca8961d905ef19fe2ef11b66fb93024874
tree0e0f4b5cfd1542547bc29db9b13f7fc315a41b5a
parent6377f82bbfedf42dd6bd62d25b6a800fa93e60f5
jjb: be verbose and fail early on errors

Prior to this commit, if the "job-builder update" command failed,
Jenkins would still consider the overall job to have succeeded, because
the jjb/config/config script was exiting with a zero exit code. (This
was because the for/ls loop were the final commands to run, and they
were exiting with zero.)

When I modified the Jenkins UID as a part of the overall Jenkins
authentication changes yesterday, the "job-builder update" command no
longer worked, because the password in jenkins_builder.ini was no longer
valid. However, Jenkins still considered the jjb job to be passing, as
explained above.

Tell bash to be verbose about the commands it runs ("set -x") and fail
early if there are any problems ("set -e").

With this change, if there is a problem with "job-builder update" or any
other command in the script, the job will fail right away and Jenkins
will bring it to our attention sooner.
jjb/config/config