ashb commented on a change in pull request #7678: [AIRFLOW-7029] Use separate
docker image for running license check
URL: https://github.com/apache/airflow/pull/7678#discussion_r392412861
##########
File path: scripts/ci/ci_check_license.sh
##########
@@ -22,21 +22,39 @@ export PYTHON_VERSION=${PYTHON_VERSION:-3.6}
. "$( dirname "${BASH_SOURCE[0]}" )/_script_init.sh"
function run_check_license() {
- docker run "${EXTRA_DOCKER_FLAGS[@]}" -t \
- --entrypoint "/usr/local/bin/dumb-init" \
- --env PYTHONDONTWRITEBYTECODE \
- --env VERBOSE \
- --env VERBOSE_COMMANDS \
- --env HOST_USER_ID="$(id -ur)" \
- --env HOST_GROUP_ID="$(id -gr)" \
- --rm \
- "${AIRFLOW_CI_IMAGE}" \
- "--" "/opt/airflow/scripts/ci/in_container/run_check_licence.sh" \
- | tee -a "${OUTPUT_LOG}"
-}
+ echo
+ echo "Running Licence check"
+ echo
+
+ # This is the target of a symlink in airflow/www/static/docs -
+ # and rat exclude doesn't cope with the symlink target doesn't exist
+ mkdir -p docs/_build/html/
-prepare_build
+ echo "Running license checks. This can take a while."
-rebuild_ci_image_if_needed
+ if ! docker run "${EXTRA_DOCKER_FLAGS[@]}" -t \
+ --user "$(id -ur):$(id -gr)" \
+ --rm \
+ ashb/apache-rat:0.13-1 \
Review comment:
I'm not really sure which project it's part of, it seems like a side project?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
