Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Speedup liveness probe for scheduler and triggerer #20833

Merged
merged 1 commit into from
Jan 13, 2022

Conversation

dstandish
Copy link
Contributor

Speedup liveness probe for scheduler and triggerer
Liveness probe is submitted through /entrypoint which by default runs airflow db check.
This can be slow.
We can disable by setting CONNECTION_CHECK_MAX_COUNT=0.

@dstandish dstandish changed the title Sched trig liveness speedup Speedup liveness probe for scheduler and triggerer Jan 12, 2022
Liveness probe is submitted through `/entrypoint` which by default runs `airflow db check`.

This can be slow.

We can disable by setting `CONNECTION_CHECK_MAX_COUNT=0`.
@dstandish dstandish force-pushed the sched-trig-liveness-speedup branch from f2a6e1f to 1f9a1b0 Compare January 12, 2022 18:10
@github-actions github-actions bot added the okay to merge It's ok to merge this PR as it does not require more tests label Jan 12, 2022
@github-actions
Copy link

The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest main or amend the last commit of the PR, and push it with --force-with-lease.

Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@kaxil kaxil added this to the Airflow Helm Chart 1.5.0 milestone Jan 12, 2022
@dstandish dstandish merged commit 83b1e36 into apache:main Jan 13, 2022
@dstandish dstandish deleted the sched-trig-liveness-speedup branch January 13, 2022 00:51
dstandish added a commit to astronomer/airflow that referenced this pull request Jan 25, 2022
PR apache#20833 tried to speed up the liveness probe by setting variable CONNECTION_CHECK_MAX_COUNT=0 which disables a connectivity check in `/entrypoint` (which turns out to be slow).

Unfortunately the approach taken doesn't work; we have to use `sh -c exec` instead.
jedcunningham pushed a commit that referenced this pull request Jan 26, 2022
PR #20833 tried to speed up the liveness probe by setting variable CONNECTION_CHECK_MAX_COUNT=0 which disables a connectivity check in `/entrypoint` (which turns out to be slow).

Unfortunately the approach taken doesn't work; we have to use `sh -c exec` instead.
kosteev pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this pull request Jul 10, 2022
PR apache/airflow#20833 tried to speed up the liveness probe by setting variable CONNECTION_CHECK_MAX_COUNT=0 which disables a connectivity check in `/entrypoint` (which turns out to be slow).

Unfortunately the approach taken doesn't work; we have to use `sh -c exec` instead.

GitOrigin-RevId: bca5caf3611fc8659b8bd5fdcddc04dc5b104344
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this pull request Aug 27, 2022
PR apache/airflow#20833 tried to speed up the liveness probe by setting variable CONNECTION_CHECK_MAX_COUNT=0 which disables a connectivity check in `/entrypoint` (which turns out to be slow).

Unfortunately the approach taken doesn't work; we have to use `sh -c exec` instead.

GitOrigin-RevId: bca5caf3611fc8659b8bd5fdcddc04dc5b104344
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this pull request Oct 4, 2022
PR apache/airflow#20833 tried to speed up the liveness probe by setting variable CONNECTION_CHECK_MAX_COUNT=0 which disables a connectivity check in `/entrypoint` (which turns out to be slow).

Unfortunately the approach taken doesn't work; we have to use `sh -c exec` instead.

GitOrigin-RevId: bca5caf3611fc8659b8bd5fdcddc04dc5b104344
aglipska pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this pull request Oct 7, 2022
PR apache/airflow#20833 tried to speed up the liveness probe by setting variable CONNECTION_CHECK_MAX_COUNT=0 which disables a connectivity check in `/entrypoint` (which turns out to be slow).

Unfortunately the approach taken doesn't work; we have to use `sh -c exec` instead.

GitOrigin-RevId: bca5caf3611fc8659b8bd5fdcddc04dc5b104344
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this pull request Dec 7, 2022
PR apache/airflow#20833 tried to speed up the liveness probe by setting variable CONNECTION_CHECK_MAX_COUNT=0 which disables a connectivity check in `/entrypoint` (which turns out to be slow).

Unfortunately the approach taken doesn't work; we have to use `sh -c exec` instead.

GitOrigin-RevId: bca5caf3611fc8659b8bd5fdcddc04dc5b104344
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this pull request Jan 27, 2023
PR apache/airflow#20833 tried to speed up the liveness probe by setting variable CONNECTION_CHECK_MAX_COUNT=0 which disables a connectivity check in `/entrypoint` (which turns out to be slow).

Unfortunately the approach taken doesn't work; we have to use `sh -c exec` instead.

GitOrigin-RevId: bca5caf3611fc8659b8bd5fdcddc04dc5b104344
kosteev pushed a commit to kosteev/composer-airflow-test-copybara that referenced this pull request Sep 12, 2024
PR apache/airflow#20833 tried to speed up the liveness probe by setting variable CONNECTION_CHECK_MAX_COUNT=0 which disables a connectivity check in `/entrypoint` (which turns out to be slow).

Unfortunately the approach taken doesn't work; we have to use `sh -c exec` instead.

GitOrigin-RevId: bca5caf3611fc8659b8bd5fdcddc04dc5b104344
kosteev pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this pull request Sep 17, 2024
PR apache/airflow#20833 tried to speed up the liveness probe by setting variable CONNECTION_CHECK_MAX_COUNT=0 which disables a connectivity check in `/entrypoint` (which turns out to be slow).

Unfortunately the approach taken doesn't work; we have to use `sh -c exec` instead.

GitOrigin-RevId: bca5caf3611fc8659b8bd5fdcddc04dc5b104344
kosteev pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this pull request Nov 7, 2024
PR apache/airflow#20833 tried to speed up the liveness probe by setting variable CONNECTION_CHECK_MAX_COUNT=0 which disables a connectivity check in `/entrypoint` (which turns out to be slow).

Unfortunately the approach taken doesn't work; we have to use `sh -c exec` instead.

GitOrigin-RevId: bca5caf3611fc8659b8bd5fdcddc04dc5b104344
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:CLI area:helm-chart Airflow Helm Chart area:providers area:secrets okay to merge It's ok to merge this PR as it does not require more tests provider:amazon-aws AWS/Amazon - related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants