Skip to content

add a scheduler health check command#9352

Closed
aneesh-joseph wants to merge 1 commit intoapache:masterfrom
aneesh-joseph:scheduler-healthcheck-cli
Closed

add a scheduler health check command#9352
aneesh-joseph wants to merge 1 commit intoapache:masterfrom
aneesh-joseph:scheduler-healthcheck-cli

Conversation

@aneesh-joseph
Copy link
Contributor


Make sure to mark the boxes below before creating PR: [x]

  • Description above provides context of the change
  • Unit tests coverage for changes (not needed for documentation changes)
  • Target Github ISSUE in description if exists
  • Commits follow "How to write a good git commit message"
  • Relevant documentation is updated including usage instructions.
  • I will engage committers as explained in Contribution Workflow Example.

In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.
Read the Pull Request Guidelines for more information.

Add a scheduler health check command which could probably be used for kubernetes livenessProbe etc

airflow scheduler_health
airflow scheduler_health --silent

@mik-laj
Copy link
Member

mik-laj commented Jun 17, 2020

@aneesh-joseph
Copy link
Contributor Author

does the command syntax look alright..If so I could try adding a test for the same..The helm chart validates the hostname as well, so maybe we could add an extra flag --validate-host which additionally validates if the hostname matches?

@mik-laj mik-laj mentioned this pull request Jun 17, 2020
6 tasks
@aneesh-joseph aneesh-joseph force-pushed the scheduler-healthcheck-cli branch from 4202fdf to 62baa5d Compare June 18, 2020 03:09
@aneesh-joseph aneesh-joseph force-pushed the scheduler-healthcheck-cli branch from 62baa5d to 88b0121 Compare June 18, 2020 06:25
@stale
Copy link

stale bot commented Aug 2, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Stale PRs per the .github/workflows/stale.yml policy file label Aug 2, 2020
@stale stale bot closed this Aug 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:CLI stale Stale PRs per the .github/workflows/stale.yml policy file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants