You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/07/21 14:24:49 UTC

[GitHub] [airflow] SuperQ commented on pull request #25219: Update statsd-exporter setup

SuperQ commented on PR #25219:
URL: https://github.com/apache/airflow/pull/25219#issuecomment-1191551156

   Thinking about this some more, what if we completely eliminated the airflow image fork and embedded mappings config. We could move the current mappings to `values.yaml` as `statsd.defaultMappings`. This would make it even easier to override things in the default setup.
   
   We can still keep the `statsd.extraMappings` for easy extension of the defaults.


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