Page MenuHomePhabricator

Feature: enable prometheus-nginx-exporter for nginx metrics
Open, LowPublic

Description

Debian just recently included nginxinc's prometheus exporter (https://packages.qa.debian.org/p/prometheus-nginx-exporter/news/20181113T140938Z.html).

We used to collect these metrics via Diamond, but it was deprecated in T183454. For places we're using nginx, it may be useful to collect those metrics.

Event Timeline

colewhite updated the task description. (Show Details)
fgiunchedi added a subscriber: fgiunchedi.

Boldly declining as we're still using nginx but it is on its way out (frontend caches already off nginx, internal usage should be replaced with envoy)

Boldly declining as we're still using nginx but it is on its way out (frontend caches already off nginx, internal usage should be replaced with envoy)

Acme Chief, Aptly, Dumps, Toolforge, Cloud VPS, OpenStack, Quarry, and a few other things in ops/puppet.git are using nginx::site. None of those are the CDN edge TLS termination or obviously replaceable by envoy. The core MediaWiki monitoring workflow is certainly important, but we do have many systems and services outside of that system.

+1. nginx is in use on install servers, puppetmaster, puppetdb, mirrors, docker registry. But if there is a goal to remove it maybe we should do another ticket for that specifically, with checkboxes.

I'm reopening the task because we indeed have many nginx instances deployed which should have metrics (however basic, from the non-commercial nginx).