apache/airflow

Five "otel_*" options appear twice in the "Configuration Reference"

Open

#43,366 创建于 2024年10月24日

在 GitHub 查看
 (7 评论) (0 反应) (0 负责人)Python (16,781 fork)batch import
good first issuekind:bugkind:documentationneeds-triage

仓库指标

Star
 (44,809 star)
PR 合并指标
 (平均合并 7天 18小时) (30 天内合并 834 个 PR)

描述

What do you see as an issue?

In https://github.com/apache/airflow/commit/ccf12026b44685f2d9917cd313676784e63c81af five configuration options have been moved from the "metrics" section to the "traces" section. Unfortunately, they have also been left in "metrics" untouched, and now they are effectively duplicated in the documentation. This is unfortunate, because users cannot easily tell what's the difference between them and which option should they prefer to use.

Here's a list of the duplicated options:

  • otel_debugging_on
  • otel_host
  • otel_on
  • otel_port
  • otel_ssl_active

If you search for them on this page: https://airflow.apache.org/docs/apache-airflow/2.10.2/configurations-ref.html, you'll see that each of them occurs twice.

Solving the problem

The problem can be solved by removing the [metrics] otel_* options from the airflow/config_templates/config.yml file (and keeping the [traces] otel_* options intact).

Anything else

The newer options also have descriptions that do not render as nicely as the old ones in the documentation (the markdown highlights are gone). It may be a good moment to fix this.

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

贡献者指南