Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

Commit

Permalink
ci: Not set 'KATA_DEV_MODE' to 'false' explicitly
Browse files Browse the repository at this point in the history
In many of our CI scripts, we assume the 'KATA_DEV_MODE' environment
variable to be not set at all when we are not in the 'kata dev
mode'. Setting its value explicitly to 'false' would fail the check '[-n
$KATA_DEV_MODE]' and make the script believe we ARE in the 'kata dev
mode', which would skip some installation. For example,
'install_rust.sh' is totally skipped in CLH's CI, which caused our
current regression.

Fixes: #2863

Signed-off-by: Bo Chen <chen.bo@intel.com>
  • Loading branch information
likebreath committed Sep 19, 2020
1 parent 0bff89b commit 52f33c3
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion .ci/jenkins_job_build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,12 @@ init_ci_flags() {
# Make jobs work like in CI
# CI disables non-working tests
export CI="true"
export KATA_DEV_MODE="false"

# Many checks assume this environment variable to be not set
# (e.g. [ -n $KATA_DEV_MODE ]). As a result, even its value is
# set to 'false', the check would think we are in "kata_dev_mode".
# export KATA_DEV_MODE="false"

# Install crio
export CRIO="no"
# Install cri-containerd
Expand Down

0 comments on commit 52f33c3

Please # to comment.