Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 21 Dec 2016 20:06:32 +0100
From:      Ed Schouten <ed@nuxi.nl>
To:        Adrian Chadd <adrian.chadd@gmail.com>
Cc:        Ed Schouten <ed@freebsd.org>,  "src-committers@freebsd.org" <src-committers@freebsd.org>,  "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>,  "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>
Subject:   Re: svn commit: r310354 - in head/usr.sbin: . prometheus_sysctl_exporter
Message-ID:  <CABh_MKkQo72NB=U-5nAHgxEeCcsmQekzPTN2NaZp9aiD1BOrMA@mail.gmail.com>
In-Reply-To: <CAJ-VmoksTgmNY1AeYRJV5UN7%2Bew%2BqDgL63_=1Aaa7AQbyrLOhQ@mail.gmail.com>
References:  <201612210829.uBL8Tijt041811@repo.freebsd.org> <CAJ-VmoksTgmNY1AeYRJV5UN7%2Bew%2BqDgL63_=1Aaa7AQbyrLOhQ@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
Hi Adrian,

2016-12-21 18:32 GMT+01:00 Adrian Chadd <adrian.chadd@gmail.com>:
> Maybe just "sysctl_export" would be fine.

Good point. Almost all Prometheus metrics exporters use this naming scheme:

~80%: ${foo}_exporter
~20%: prometheus_${foo}_exporter

Examples;

- https://github.com/prometheus/node_exporter
- https://github.com/ewr/elasticsearch_exporter
- https://github.com/jonnenauha/prometheus_varnish_exporter

The reason why I went for the 'prometheus_' prefix was mainly because
of the following totally subjective reasons:

- I didn't want to wreck tab completion. Right now if you type in
'sysc' and press tab, it tends to complete to 'sysctl'.
- This tool is part of the base system, meaning that it will be
present even if the user did nothing special. This is why I wanted the
name to be a bit more self-documenting.
- Thinking ahead: what if we're going to add exporters for other
things as well? E.g., for Sendmail, sshd, ftpd, HAST, etc. In that
case it would be nicer if those exporters shared a common prefix.

That said, if people truly think it's a good idea to call this
sysctl_exporter instead, then I wouldn't mind renaming it.

Thoughts? Opinions?

-- 
Ed Schouten <ed@nuxi.nl>
Nuxi, 's-Hertogenbosch, the Netherlands
KvK-nr.: 62051717



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CABh_MKkQo72NB=U-5nAHgxEeCcsmQekzPTN2NaZp9aiD1BOrMA>