Code Review
/
releng
/
global-jjb.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Fix shellcheck errors in scripts
[releng/global-jjb.git]
/
shell
/
tox-run.sh
diff --git
a/shell/tox-run.sh
b/shell/tox-run.sh
index
8be8459
..
498ddc7
100644
(file)
--- a/
shell/tox-run.sh
+++ b/
shell/tox-run.sh
@@
-10,11
+10,15
@@
##############################################################################
echo "---> tox-run.sh"
##############################################################################
echo "---> tox-run.sh"
+# do not use -o pipefail
+set -eux
+
ARCHIVE_TOX_DIR="$WORKSPACE/archives/tox"
ARCHIVE_TOX_DIR="$WORKSPACE/archives/tox"
+ARCHIVE_DOC_DIR="$WORKSPACE/archives/docs"
mkdir -p "$ARCHIVE_TOX_DIR"
cd "$WORKSPACE/$TOX_DIR" || exit 1
mkdir -p "$ARCHIVE_TOX_DIR"
cd "$WORKSPACE/$TOX_DIR" || exit 1
-if [
-d "/opt/pyenv"
]; then
+if [
[ -d /opt/pyenv ]
]; then
echo "---> Setting up pyenv"
export PYENV_ROOT="/opt/pyenv"
export PATH="$PYENV_ROOT/bin:$PATH"
echo "---> Setting up pyenv"
export PYENV_ROOT="/opt/pyenv"
export PATH="$PYENV_ROOT/bin:$PATH"
@@
-24,8
+28,8
@@
if [ -d "/opt/pyenv" ]; then
fi
PARALLEL="${PARALLEL:-true}"
fi
PARALLEL="${PARALLEL:-true}"
-if [
"${PARALLEL}" = true
]; then
- if [
-n "$TOX_ENVS"
]; then
+if [
[ ${PARALLEL} = true ]
]; then
+ if [
[ -n ${TOX_ENVS:-} ]
]; then
tox -e "$TOX_ENVS" --parallel auto --parallel-live | tee -a "$ARCHIVE_TOX_DIR/tox.log"
tox_status="${PIPESTATUS[0]}"
else
tox -e "$TOX_ENVS" --parallel auto --parallel-live | tee -a "$ARCHIVE_TOX_DIR/tox.log"
tox_status="${PIPESTATUS[0]}"
else
@@
-33,7
+37,7
@@
if [ "${PARALLEL}" = true ]; then
tox_status="${PIPESTATUS[0]}"
fi
else
tox_status="${PIPESTATUS[0]}"
fi
else
- if [
-n "$TOX_ENVS"
]; then
+ if [
[ -n ${TOX_ENVS:-} ]
]; then
tox -e "$TOX_ENVS" | tee -a "$ARCHIVE_TOX_DIR/tox.log"
tox_status="${PIPESTATUS[0]}"
else
tox -e "$TOX_ENVS" | tee -a "$ARCHIVE_TOX_DIR/tox.log"
tox_status="${PIPESTATUS[0]}"
else
@@
-42,13
+46,26
@@
else
fi
fi
fi
fi
+echo "---> Completed tox runs"
+
# Disable SC2116 as we want to echo a space separated list of TOX_ENVS
# shellcheck disable=SC2116
for i in .tox/*/log; do
tox_env=$(echo "$i" | awk -F'/' '{print $2}')
# Disable SC2116 as we want to echo a space separated list of TOX_ENVS
# shellcheck disable=SC2116
for i in .tox/*/log; do
tox_env=$(echo "$i" | awk -F'/' '{print $2}')
- cp -r "$i" "$ARCHIVE_TOX_DIR/$tox_env"
+ # defend against glob finding no matches
+ if ! cp -r "$i" "$ARCHIVE_TOX_DIR/$tox_env"; then
+ echo "WARN: no logs found to archive"
+ break
+ fi
done
done
-echo "Completed tox runs."
+# If docs are generated push them to archives.
+DOC_DIR="${DOC_DIR:-docs/_build/html}"
+if [[ -d $DOC_DIR ]]; then
+ echo "---> Archiving generated docs"
+ mv "$DOC_DIR" "$ARCHIVE_DOC_DIR"
+fi
+
+echo "---> tox-run.sh ends"
test "$tox_status" -eq 0 || exit "$tox_status"
test "$tox_status" -eq 0 || exit "$tox_status"