potiuk commented on a change in pull request #9162:
URL: https://github.com/apache/airflow/pull/9162#discussion_r436526224
##########
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:
Right. I think it's not needed after recent HIVE mocking. Let's see what
our tests have to say about it.
----------------------------------------------------------------
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]
