mik-laj commented on a change in pull request #9162:
URL: https://github.com/apache/airflow/pull/9162#discussion_r436514184
##########
File path: scripts/ci/in_container/entrypoint_ci.sh
##########
@@ -163,22 +148,19 @@ ssh-keyscan -H localhost >> ~/.ssh/known_hosts 2>/dev/null
if [[ ${CI:=} == "true" && ${RUN_TESTS} == "true" ]] ; then
echo
echo "
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!"
- echo " Setting default parallellism to 2 because we can run out of memory
during tests on CI"
+ echo " Setting default parallelism to 2 because we can run out of memory
during tests on CI"
echo "
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!"
echo
- export AIRFLOW__CORE__PARALELLISM=2
+ export AIRFLOW__CORE__PARALLELISM=2
fi
+cd "${AIRFLOW_SOURCES}"
+
set +u
# If we do not want to run tests, we simply drop into bash
-if [[ "${RUN_TESTS}" == "false" ]]; then
- if [[ ${#ARGS} == 0 ]]; then
- exec /bin/bash
- else
- exec /bin/bash -c "$(printf "%q " "${ARGS[@]}")"
- fi
+if [[ "${RUN_TESTS:=false}" != "true" ]]; then
+ exec /bin/bash "${@}"
Review comment:
Are you sure of it? Should not be added --c here?
----------------------------------------------------------------
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:
[email protected]