jscheffl commented on PR #67328: URL: https://github.com/apache/airflow/pull/67328#issuecomment-4653452211
> > @AutomationDev85 you saw metrics in your metrics backend without the worker label or metrics completely gone? > > Because I'm on Airflow 3.2.2, edge3 provider 3.7.0, statsd_exporter v0.28.0 and we did saw all edge worker metrics completely gone, but due to bug I've described in #68077. After implementing a fix I do see all the metrics back including the worker label, using the statsd mapping: > > ```yaml > > - match: airflow.edge_worker.concurrency.* > > name: "airflow_edge_worker_concurrency" > > labels: > > worker_name: "$1" > > ``` > > I noticed the issue with metrics missing the worker label. In our Airflow deployment, we had already patched this locally to address the same problem you encountered. Since the fix is already on main, we took the fastest path to run Airflow 3.2 with the latest Edge Worker package. We are now waiting for the Airflow 3.3 release so we can remove our local workaround. @diogosilva30 @AutomationDev85 do you think in regards of this metrics "trouble" and changes, do we need to add some sectiont o docs to describe what metrics template to adjust to make it working in different Airflow versions such that others do not need to reverse-engineer the same problems? Some small note in RST docs? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
