dstandish commented on code in PR #74046:
URL: https://github.com/apache/airflow/pull/74046#discussion_r4298268941
##########
airflow-core/docs/administration-and-deployment/logging-monitoring/metrics.rst:
##########
@@ -329,6 +361,12 @@ On earlier versions, use the ``Stats`` class instead:
These metrics are silently dropped unless a backend is enabled (see `Setup
- StatsD`_
or `Setup - OpenTelemetry`_).
+.. note::
+
+ With OpenTelemetry, a metric emitted from inside a task leaves the task's
process as a delta
+ value under ``service.instance.id=task-sdk``. A backend that only accepts
cumulative values,
+ such as Prometheus, needs the Collector to convert it back, see
`Short-lived processes`_.
Review Comment:
similar to above comments. maybe we should talk about "shoulds" because user
has control over delta temporality. and, we should check whether you can set
service.instance.id to a constant via env var and if so, we should probably
just recommend that the user do that rather than doing it for them.
--
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]
