[GitHub] [airflow] Maor-Shmueli commented on pull request #25647: update statsd-exporter mappings

[GitBox <gi...@apache.org>]

Maor-Shmueli commented on PR #25647:
URL: https://github.com/apache/airflow/pull/25647#issuecomment-1224108659

   @dstandish @jedcunningham Just to clarify, 
   I ran into this file while me and my team set up statsd-exporer for airflow metrics.
   I noticed that it wasn't maintained for a while and was not aligned with the current version airflow metrics.
   
   So I did some tidying up, and basically:
   
   - I took all metrics that need a mapping (metrics with variable labels) from the [documentation](https://airflow.apache.org/docs/apache-airflow/stable/logging-monitoring/metrics.html), and created a mapping for each one of them.
   - The mappings are by the order of the metrics in docs, and I added the original metric name as a comment before the mapping.
   - Some of the labels was not aligned with the labels in docs (like operator → operator_name) so I changed them.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org