jedcunningham commented on PR #36305:
URL: https://github.com/apache/airflow/pull/36305#issuecomment-1905003845
That removal is fine, thanks for doing that digging. But moving existing
metrics to use tags would be breaking for existing deployments no?
e.g.
```
Before:
airflow_pool_queued_slots_default_pool 0
After:
airflow_pool_queued_slots{pool="default_pool"} 0
```
Unfortunately, this is the spot we are in. Our next opportunity to revamp
the default mapping will be in 2.0 of the chart.
In the meantime, taking advantage of `statsd.overrideMappings` is what folks
should use in their own env.
--
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-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
