From a19dd43082869f213b25fdcc5c2ec617532e7b64 Mon Sep 17 00:00:00 2001 From: Guilherme Gallo Date: Tue, 19 Jul 2022 00:22:15 -0300 Subject: [PATCH] ci/lava: Remove MINIO_RESULTS_UPLOAD if requested When we don't want to communicate with minio, e.g. running lava_job_submitter script locally, MINIO_RESULTS_UPLOAD should be unset. But this variable is already set by generate-env script, so we need to remove it from the /set-job-env-vars.sh to avoid declaring it in unexpected scenarios. Signed-off-by: Guilherme Gallo Part-of: --- .gitlab-ci/lava/lava_job_submitter.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.gitlab-ci/lava/lava_job_submitter.py b/.gitlab-ci/lava/lava_job_submitter.py index dbc6414bf60..fe2d69a2998 100755 --- a/.gitlab-ci/lava/lava_job_submitter.py +++ b/.gitlab-ci/lava/lava_job_submitter.py @@ -160,7 +160,7 @@ def generate_lava_yaml(args): else: run_steps += [ "echo Could not find jwt file, disabling MINIO requests...", - "unset MINIO_RESULTS_UPLOAD", + "sed -i '/MINIO_RESULTS_UPLOAD/d' /set-job-env-vars.sh", ] run_steps += [