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

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

jedcunningham commented on PR #25647:
URL: https://github.com/apache/airflow/pull/25647#issuecomment-1245632718

   Unfortunately, adding mappings like this is a breaking change.
   
   For example, lets look at `dag_processing.last_duration.<dag_file>`. Right now that's `airflow_dag_processing_last_duration_hello_world` but will end up being `airflow_dag_processing_last_duration{dag_file="hello-world"}.
   
   Same thing with the `operator` vs `operator_name` thing - we can't change it for this release.
   
   So unfortunately, until we are ready for 2.0 of the chart we can't make these changes.
   
   I think https://github.com/apache/airflow/pull/25219 is closer to what we need for now as it opens up the possibility for users of the charts to provide their own mappings.


-- 
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