This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new 73e692c63b Disable health check for mssql integration (#39885)
73e692c63b is described below

commit 73e692c63b0adccbe8d078b9e356b60d11964971
Author: Jarek Potiuk <[email protected]>
AuthorDate: Mon May 27 23:22:01 2024 +0200

    Disable health check for mssql integration (#39885)
    
    We already perform environment check inside Airlfow entrypoint_ci.sh
    so there is no need to run health-check in docker compose, the script
    will wait until mssql is available anyway.
---
 scripts/ci/docker-compose/integration-mssql.yml | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/scripts/ci/docker-compose/integration-mssql.yml 
b/scripts/ci/docker-compose/integration-mssql.yml
index fe16ab0d76..e572112589 100644
--- a/scripts/ci/docker-compose/integration-mssql.yml
+++ b/scripts/ci/docker-compose/integration-mssql.yml
@@ -31,16 +31,10 @@ services:
     restart: "on-failure"
     ports:
       - "${MSSQL_HOST_PORT}:1433"
-    healthcheck:
-      test: /opt/mssql-tools/bin/sqlcmd -S localhost,1433 -U sa -P 
"airflow123" -Q "SELECT 1" -b -o /dev/null
-      interval: 10s
-      timeout: 50s
-      retries: 20
-      start_period: 50s
   airflow:
     depends_on:
       mssql:
-        condition: service_healthy
+        condition: service_started
     environment:
       - INTEGRATION_MSSQL=true
 volumes:

Reply via email to