ci: export RESULTS_DIR in crosvm-script.sh
Export the RESULTS_DIR environment variable in crosvm-script.sh to ensure it points to the correct directory. Without this, artifacts are not generated because the results directory is created in the wrong location after the directory changes in the VM. Also fix https://www.shellcheck.net/wiki/SC2129 shellcheck error. Suggested-by: Eric Engestrom <eric@igalia.com> Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com> Reviewed-by: Eric Engestrom <None> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/33023>
This commit is contained in:
@@ -80,9 +80,12 @@ ${SCRIPTS_DIR}/common/generate-env.sh | tee ${VM_TEMP_DIR}/crosvm-env.sh
|
||||
cp ${SCRIPTS_DIR}/setup-test-env.sh ${VM_TEMP_DIR}/setup-test-env.sh
|
||||
|
||||
# Set the crosvm-script as the arguments of the current script
|
||||
echo "export SCRIPTS_DIR=${SCRIPTS_DIR}" > ${VM_TEMP_DIR}/crosvm-script.sh
|
||||
echo ". ${VM_TEMP_DIR}/setup-test-env.sh" >> ${VM_TEMP_DIR}/crosvm-script.sh
|
||||
echo "$@" >> ${VM_TEMP_DIR}/crosvm-script.sh
|
||||
{
|
||||
echo "export SCRIPTS_DIR=${SCRIPTS_DIR}"
|
||||
echo "export RESULTS_DIR=${RESULTS_DIR}"
|
||||
echo ". ${VM_TEMP_DIR}/setup-test-env.sh"
|
||||
echo "$@"
|
||||
} > ${VM_TEMP_DIR}/crosvm-script.sh
|
||||
|
||||
# Setup networking
|
||||
/usr/sbin/iptables-legacy -w -t nat -A POSTROUTING -o eth0 -j MASQUERADE
|
||||
|
||||
Reference in New Issue
Block a user