Skip to content

Airflow metrics#381

Merged
LDiazN merged 15 commits intomainfrom
airflow-metrics
Mar 20, 2026
Merged

Airflow metrics#381
LDiazN merged 15 commits intomainfrom
airflow-metrics

Conversation

@LDiazN
Copy link
Contributor

@LDiazN LDiazN commented Mar 20, 2026

This PR will implement airflow metrics collection. It adds:

  1. A new prometheus_statsd exporter Ansible role. Several services use statsd, so I made an ansible role for them when it's necessary
  2. Added this role to the airflow role
  3. Set up airflow to push metrics to the exporter

closes #363

@LDiazN LDiazN requested a review from hellais March 20, 2026 12:21
@LDiazN LDiazN self-assigned this Mar 20, 2026
@github-actions
Copy link

Ansible Run Output 🤖

Ansible Playbook Recap 🔍



Ansible playbook output 📖success

Show Execution

$ ansible-playbook playbook.yml --check --diff -i ../tf/modules/ansible_inventory/inventories/inventory-dev.ini
[WARNING]: provided hosts list is empty, only localhost is available. Note that the implicit localhost does not match 'all'
[ERROR]: the role 'geerlingguy.docker' was not found in /home/runner/work/devops/devops/ansible/roles:/home/runner/.ansible/roles:/usr/share/ansible/roles:/etc/ansible/roles:/home/runner/work/devops/devops/ansible
Origin: /home/runner/work/devops/devops/ansible/deploy-testlists.yml:16:7

14         node_exporter_host: "0.0.0.0"
15         node_exporter_options: ""
16     - role: geerlingguy.docker
         ^ column 7

Pusher @LDiazN
Action pull_request
Working Directory
Workflow .github/workflows/check_ansible.yml
Last updated Fri, 20 Mar 2026 12:22:28 GMT

Copy link
Member

@hellais hellais left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@LDiazN LDiazN merged commit d219350 into main Mar 20, 2026
2 checks passed
@LDiazN LDiazN deleted the airflow-metrics branch March 20, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add monitoring to airflow tasks

2 participants