-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
Add some basic metrics to the Triggerer #18214
Conversation
a012cd6
to
6c2c2b4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add them to https://github.com/apache/airflow/blob/main/docs/apache-airflow/logging-monitoring/metrics.rst please
The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, or amend the last commit of the PR, and push it with --force-with-lease. |
6c2c2b4
to
0e965e6
Compare
@andrewgodwin Can you add docs to https://github.com/apache/airflow/blob/main/docs/apache-airflow/logging-monitoring/metrics.rst, please? |
Follow-on from apache#18214
Follow-on from apache/airflow#18214 GitOrigin-RevId: 29d700dd3de7ede6b5c426f397418deb3cc7a916
Follow-on from apache/airflow#18214 GitOrigin-RevId: 29d700dd3de7ede6b5c426f397418deb3cc7a916
Follow-on from apache/airflow#18214 GitOrigin-RevId: 29d700dd3de7ede6b5c426f397418deb3cc7a916
Follow-on from apache/airflow#18214 GitOrigin-RevId: 29d700dd3de7ede6b5c426f397418deb3cc7a916
Follow-on from apache/airflow#18214 GitOrigin-RevId: 29d700dd3de7ede6b5c426f397418deb3cc7a916
Follow-on from apache/airflow#18214 GitOrigin-RevId: 29d700dd3de7ede6b5c426f397418deb3cc7a916
Follow-on from apache/airflow#18214 GitOrigin-RevId: 29d700dd3de7ede6b5c426f397418deb3cc7a916
Follow-on from apache/airflow#18214 GitOrigin-RevId: 29d700dd3de7ede6b5c426f397418deb3cc7a916
This adds some basic metrics to the Triggerer:
triggers.running
: A gauge of how many are currently runningtriggers.succeeded
andtriggers.failed
: Counters of how many triggers have fired successfully/failed, respectively.