Port or deprecate Icinga airflow checks
Closed, ResolvedPublicGoal
Actions

Assigned To

Authored By

	fgiunchedi
	Jan 20 2025, 2:29 PM

Description

The following checks are in icinga for airflow unit status, my understanding is that as airflow is moving (has moved?) to k8s (T362788) the checks will be come obsolete. Would you confirm/agree @BTullis @brouberol ?

check_airflow-kerberos@INSTANCE_status
check_airflow-kerberos@INSTANCE_status
check_airflow-scheduler@INSTANCE_status
check_airflow-scheduler@INSTANCE_status
check_airflow-webserver@INSTANCE_status
check_airflow-webserver@analytics_status
check_airflow@INSTANCE_status
check_airflow@INSTANCE_status

! MIGRATION TABLE !

Migrated? (Y/N)	Title	Resource Type	Command	File	Profiles
N	check_airflow-X@analytics_status	Nrpe::Monitor_service	/usr/local/lib/nagios/plugins/check_systemd_unit_status	modules/systemd/manifests/monitor.pp:23	profile::airflow

Details

Related Changes in Gerrit:

Subject	Repo	Branch	Lines +/-
airflow: remove nrpe definitions	operations/puppet	production	+0 -28
hiera: disable monitoring for legacy profile::airflow::instances	operations/puppet	production	+1 -1
airflow: disable icinga nrpe checks	operations/puppet	production	+14 -12

Customize query in gerrit

Related Objects
Search...

Status	Subtype	Assigned	Task
Open		tappof	T395441 Port all Icinga checks to Prometheus/Alertmanager: preparation
Open		tappof	T321808 Port all Icinga checks to Prometheus/Alertmanager
Resolved	Goal	colewhite	T384214 Port or deprecate Icinga airflow checks

Event Timeline

fgiunchedi created this task.Jan 20 2025, 2:29 PM

Yes, these checks will be obsolete once https://phabricator.wikimedia.org/T362788 is closed. As it stands, we still have 3 airflow instances to migrate.

Change #1155619 had a related patch set uploaded (by Tiziano Fogli; author: Tiziano Fogli):

[operations/puppet@production] monitoring services: add migration task T384214 to instances

https://gerrit.wikimedia.org/r/1155619

gerritbot added a project: Patch-For-Review.Jun 11 2025, 9:19 AM

Change #1155619 merged by Tiziano Fogli:

[operations/puppet@production] monitoring services: add migration task T384214 to instances

https://gerrit.wikimedia.org/r/1155619

Maintenance_bot removed a project: Patch-For-Review.Jun 23 2025, 1:31 PM

tappof triaged this task as Low priority.Jun 24 2025, 1:12 PM

tappof changed the subtype of this task from "Task" to "Goal".Sep 2 2025, 1:34 PM

These checks can be deprecated without any impact. Sorry, I should have mentioned that much earlier.

tappof updated the task description. (Show Details)Sep 2 2025, 2:04 PM