JH-A-Kim commented on code in PR #70416: URL: https://github.com/apache/airflow/pull/70416#discussion_r3786917874
########## airflow-core/src/airflow/api/common/airflow_health.py: ########## @@ -16,77 +16,165 @@ # under the License. from __future__ import annotations -from typing import Any +from typing import TYPE_CHECKING, Any + +from sqlalchemy import select from airflow.jobs.dag_processor_job_runner import DagProcessorJobRunner +from airflow.jobs.job import Job, JobState from airflow.jobs.scheduler_job_runner import SchedulerJobRunner from airflow.jobs.triggerer_job_runner import TriggererJobRunner +from airflow.utils.session import NEW_SESSION, provide_session + +if TYPE_CHECKING: + from sqlalchemy.orm import Session HEALTHY = "healthy" UNHEALTHY = "unhealthy" +DEGRADED = "degraded" +DOWN = "down" + + +@provide_session +def get_jobs_health(job_runner_class, *, session: Session = NEW_SESSION) -> list[Job]: + """Return all running jobs for the runner class, ordered by latest heartbeat.""" + return list( + session.scalars( + select(Job) + .where( + Job.job_type == job_runner_class.job_type, + Job.state == JobState.RUNNING, Review Comment: > @JH-A-Kim Yup, that's the exact case. It's not part of this PR, but there should be a reaper to clean those up. Do you have time to open an Issue with the details so someone gets to it? yeah I should in a week be able to work on setting up an issue for that. Working on some stuff for the GitHub team for the past week that needs to be done by the end of next week Friday but I should be able to finish my work before then and then I can get to making an issue! -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
