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 8b3ec451e8 Increase Health Check parameters for mssql integration
(#39877)
8b3ec451e8 is described below
commit 8b3ec451e8b6a29f0d3167fcb37ff8338575c0a2
Author: Jarek Potiuk <[email protected]>
AuthorDate: Mon May 27 17:06:14 2024 +0200
Increase Health Check parameters for mssql integration (#39877)
The values for mssql integration were too low, particularly the
timeout of 3s was far too low. Per documentation:
https://github.com/docker/cli/blob/5bf86c198488c7f78f2ffd05a8a2441f719d3b98/docs/reference/builder.md#healthcheck
Increasing the values shoudl bring stability back.
---
scripts/ci/docker-compose/integration-mssql.yml | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/scripts/ci/docker-compose/integration-mssql.yml
b/scripts/ci/docker-compose/integration-mssql.yml
index ffe171e050..7f990dffef 100644
--- a/scripts/ci/docker-compose/integration-mssql.yml
+++ b/scripts/ci/docker-compose/integration-mssql.yml
@@ -34,9 +34,9 @@ services:
healthcheck:
test: /opt/mssql-tools/bin/sqlcmd -S localhost,1433 -U sa -P
"airflow123" -Q "SELECT 1" -b -o /dev/null
interval: 10s
- timeout: 3s
- retries: 30
- start_period: 10s
+ timeout: 20s
+ retries: 20
+ start_period: 20s
airflow:
depends_on:
mssql:
