Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Airflow Metrics Page Misses Some Metrics #10091

Closed
rafidka opened this issue Aug 1, 2020 · 6 comments · Fixed by #13708
Closed

Airflow Metrics Page Misses Some Metrics #10091

rafidka opened this issue Aug 1, 2020 · 6 comments · Fixed by #13708
Assignees
Labels

Comments

@rafidka
Copy link

rafidka commented Aug 1, 2020

Apache Airflow version:

1.10.11rc2

Kubernetes version (if you are using kubernetes) (use kubectl version):

N/A

Environment:

N/A

  • Cloud provider or hardware configuration:
  • OS (e.g. from /etc/os-release):
  • Kernel (e.g. uname -a):
  • Install tools:
  • Others:

What happened:

By inspecting the code of Airflow (1.10.11rc2, though I believe previous versions apply as well), I noticed the metrics documentation page misses the following metrics:

  • {job_name}_heartbeat_failure
  • dag_file_refresh_error
  • scheduler.tasks.executable
  • scheduler.tasks.pending
  • scheduler.tasks.running
  • scheduler.tasks.starving
  • scheduler.tasks.without_dagrun
  • task_removed_from_dag.{dag_id}
  • task_restored_to_dag.{dag_id}
  • task_instance_created-{operator_name}

Any reason why these are not documented?

Additionally, the following metrics are incorrectly documented:

  • dag_processing.processor_timeouts: documented as a guage but it is actually a counter
  • dag_processing.last_runtime.{dag_filename}: documented as a gauge but it is actually a timer

What you expected to happen:

  • The Metrics page should document all metrics being emitted by Airflow.
  • The Metrics page should correctly document the type of the metric.

How to reproduce it:

  • Clone the latest Airflow 1.x code base.
  • Search the code base for Stats., i.e. lines that publish metrics using the Stats object.
  • Compare the result with Metrics page in Airflow documentation.

Anything else we need to know:

I am happy to send a PR to update the documentation.

@rafidka rafidka added the kind:bug This is a clearly a bug label Aug 1, 2020
@boring-cyborg
Copy link

boring-cyborg bot commented Aug 1, 2020

Thanks for opening your first issue here! Be sure to follow the issue template!

@mik-laj
Copy link
Member

mik-laj commented Aug 2, 2020

Does this problem also occur on the master branch?
https://airflow.readthedocs.io/en/latest/metrics.html

@rafidka
Copy link
Author

rafidka commented Aug 2, 2020

Yes, I just checked and apart from a couple of the metrics in my list above which seem to be documented, the rest are still undocumented. Also, the two metrics which are wrongly documented as gauges are still so in the page you linked.

@rafidka
Copy link
Author

rafidka commented Aug 12, 2020

Any update on this?

@mik-laj
Copy link
Member

mik-laj commented Aug 12, 2020

@rafidka Can you create a PR? This way, it is easier to discuss specific changes. I only know now that not all documentation changes may be available in Airflow 1.10 and this is still a problem, but a less important one. We are now trying to focus on developing the documentation for the master branch as we want to release Airflow 2.0 soon. If there is a problem, I think it is worth making the necessary improvements.

@rafidka
Copy link
Author

rafidka commented Aug 14, 2020

@mik-laj sure, I will do.

kaxil added a commit to astronomer/airflow that referenced this issue Jan 15, 2021
@kaxil kaxil self-assigned this Jan 15, 2021
kaxil added a commit that referenced this issue Jan 16, 2021
kaxil added a commit that referenced this issue Jan 21, 2021
closes #10091

(cherry picked from commit 74b2cd7)
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Sep 16, 2021
closes apache/airflow#10091

(cherry picked from commit 74b2cd7364df192a8b53d4734e33b07e69864acc)

GitOrigin-RevId: e3ad69e0020d19cff5b7758ff04085e07366fb20
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Sep 17, 2021
closes apache/airflow#10091

GitOrigin-RevId: 74b2cd7364df192a8b53d4734e33b07e69864acc
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Sep 23, 2021
closes apache/airflow#10091

GitOrigin-RevId: 74b2cd7364df192a8b53d4734e33b07e69864acc
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Nov 27, 2021
closes apache/airflow#10091

GitOrigin-RevId: 74b2cd7364df192a8b53d4734e33b07e69864acc
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Mar 10, 2022
closes apache/airflow#10091

GitOrigin-RevId: 74b2cd7364df192a8b53d4734e33b07e69864acc
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Jun 4, 2022
closes apache/airflow#10091

GitOrigin-RevId: 74b2cd7364df192a8b53d4734e33b07e69864acc
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Jun 7, 2022
closes apache/airflow#10091

(cherry picked from commit 74b2cd7364df192a8b53d4734e33b07e69864acc)

GitOrigin-RevId: e3ad69e0020d19cff5b7758ff04085e07366fb20
kosteev pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Jul 9, 2022
closes apache/airflow#10091

GitOrigin-RevId: 74b2cd7364df192a8b53d4734e33b07e69864acc
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Aug 27, 2022
closes apache/airflow#10091

GitOrigin-RevId: 74b2cd7364df192a8b53d4734e33b07e69864acc
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Oct 4, 2022
closes apache/airflow#10091

GitOrigin-RevId: 74b2cd7364df192a8b53d4734e33b07e69864acc
aglipska pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Oct 7, 2022
closes apache/airflow#10091

GitOrigin-RevId: 74b2cd7364df192a8b53d4734e33b07e69864acc
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Dec 7, 2022
closes apache/airflow#10091

GitOrigin-RevId: 74b2cd7364df192a8b53d4734e33b07e69864acc
leahecole pushed a commit to GoogleCloudPlatform/composer-airflow that referenced this issue Jan 27, 2023
closes apache/airflow#10091

GitOrigin-RevId: 74b2cd7364df192a8b53d4734e33b07e69864acc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants