xBis7 commented on code in PR #67328:
URL: https://github.com/apache/airflow/pull/67328#discussion_r3379939500
##########
providers/edge3/docs/edge_executor.rst:
##########
@@ -217,3 +217,30 @@ Current Limitations Edge Executor
- Multi-team isolation is logical only — all teams share a single
authentication secret. A worker
administrator could change the team name and access another team's jobs.
See
:ref:`edge_executor:multi_team` for details and planned improvements.
+
+
+Metrics Export Compatibility
+-----------------------------
+
+The Edge Worker integrates with Airflow's statsd metrics system to export
runtime metrics. Compatibility
+between Edge provider versions and Airflow versions varies due to changes in
the metrics initialization
+pipeline. The table below documents known compatibility issues and workarounds:
+
+.. list-table::
+ :header-rows: 1
+
+ * - Provider version
+ - Airflow 3.3
+ - Airflow 3.2
+ - Airflow 3.1
+ * - >= 3.6.0
+ - Working
+ - Broken: webserver statsd initialization is missing
(DualStatsManager removed)
Review Comment:
These are 2 separate issues.
Removing the `DualStatsManager` led to missing legacy metric names.
If `stats` initialization is skipped, then we won't get stats at all because
`stats` won't know which backend to use and will default to a no-op logger.
--
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]
