mik-laj commented on a change in pull request #16159:
URL: https://github.com/apache/airflow/pull/16159#discussion_r641954469



##########
File path: scripts/ci/testing/ci_run_single_airflow_test_in_docker.sh
##########
@@ -82,6 +82,24 @@ function run_airflow_testing_in_docker() {
     echo
     echo "Semaphore grabbed. Running tests for ${TEST_TYPE}"
     echo
+    local backend_docker_compose=("-f" 
"${SCRIPTS_CI_DIR}/docker-compose/backend-${BACKEND}.yml")
+    if [[ ${BACKEND} == "mssql" ]]; then
+        local docker_filesystem
+        docker_filesystem=$(stat "-f" "-c" "%T" /var/lib/docker || echo 
"unknown")
+        if [[ ${docker_filesystem} == "tmpfs" ]]; then
+            # In case of tmpfs backend for docker, mssql fails because TMPFS 
does not support
+            # O_DIRECT parameter for direct writing to the filesystem
+            # https://github.com/microsoft/mssql-docker/issues/13
+            # so we need to mount an external volume for its db location
+            # the external db must allow for parallel testing so TEST_TYPE is 
added to the
+            # volume name (see the backend-mssql-bind-volume.yml file)
+            mkdir -p "${HOME}/tmp-mssql-volume-${TEST_TYPE}"

Review comment:
       Should we clean this directory between tries?




-- 
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]


Reply via email to