Skip to content
This repository has been archived by the owner on May 22, 2021. It is now read-only.

Commit

Permalink
[AIRFLOW-6511] Remove BATS docker containers (apache#7103)
Browse files Browse the repository at this point in the history
The containers were not removed and you have to remove them
with `dockery system prune`. The --rm flag is added.
  • Loading branch information
potiuk authored and galuszkak committed Mar 5, 2020
1 parent 9d3670e commit 8deb730
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions scripts/ci/_utils.sh
Original file line number Diff line number Diff line change
Expand Up @@ -663,10 +663,10 @@ function run_flake8() {
function run_bats_tests() {
FILES=("$@")
if [[ "${#FILES[@]}" == "0" ]]; then
docker run --workdir /airflow -v "$(pwd):/airflow" \
docker run --workdir /airflow -v "$(pwd):/airflow" --rm \
bats/bats:latest --tap -r /airflow/tests/bats | tee -a "${OUTPUT_LOG}"
else
docker run --workdir /airflow -v "$(pwd):/airflow" \
docker run --workdir /airflow -v "$(pwd):/airflow" --rm \
bats/bats:latest --tap -r "${FILES[@]}" | tee -a "${OUTPUT_LOG}"
fi
}
Expand Down

0 comments on commit 8deb730

Please sign in to comment.