Correctly evaluate UPDATE_CLOUD_IMAGE 50/64250/4 v0.55.0
authorAndrew Grimberg <agrimberg@linuxfoundation.org>
Wed, 10 Jun 2020 16:01:13 +0000 (09:01 -0700)
committerAndrew Grimberg <agrimberg@linuxfoundation.org>
Wed, 10 Jun 2020 16:15:17 +0000 (09:15 -0700)
commit5fc616e4b1231808c1f11608608415312b08aa02
tree25c6520a4d9e53e99776192e09bba949c15f5172
parent1bebe2af0f9dbe2db7fa1a22091144a0076b3c5f
Correctly evaluate UPDATE_CLOUD_IMAGE

While trying to get a clean packer build on an AWS based Jenkins I kept
running into the packer job failing. Digging into the code and turning
on debugging allowed me to spot that

if [[ ${UPDATE_CLOUD_IMAGE} ]]; then

Was translating to

[[ -n false ]]

During runs where $UPDATE_CLOUD_IMAGE wasn't set. This seemed strange to
me and then looking a bit further I realized that the boolean variable
in Jenkins is converting to a string ENV var of either 'true' or 'false'
depending the set. This means for the if statement to properly execute
it must evaluate as a string comparison as the variable is not actually
a boolean.

Change-Id: Ie98b51eb22412705c8c264b61be3c76b4e5ff985
Issue: RELENG-2985
Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>
releasenotes/notes/fix_packer_build-3d4cc3519be3c2a7.yaml [new file with mode: 0644]
shell/packer-build.sh