Skip to content

Added openVPN exporter #2655

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

Fadi-hamwi
Copy link

OpenVPN Metrics Exporter where I expose per-client traffic, connection times, and server status metrics from OpenVPN's management interface. not the status file for real-time monitoring.
OpenVPN exporter Repo

Signed-off-by: Fadi Hamwi <[email protected]>
Signed-off-by: Fadi Hamwi <[email protected]>
@juliusv
Copy link
Member

juliusv commented May 28, 2025

Hi, the whole docs site and its structure just got changed with the launch of the new prometheus.io website yesterday - the same doc now lives at https://github.com/prometheus/docs/blob/main/docs/instrumenting/exporters.md, could you rebase the PR and make the change against that file instead?

I would also note that the exporters list is (mostly, although sadly not perfectly) alphabetically sorted, so it would be nice to sort the new entry in alphabetically.

Regarding the metrics that the exporter produces:

  • openvpn_client_bytes_received and openvpn_client_bytes_sent should have a _total suffix since they are counters, see https://prometheus.io/docs/practices/naming/.
  • I think the connection_time label contains the "connected since" timestamp, right? So you have that both as a label on all metrics, and as a separate metric? Just wondering whether it'd be better to omit that on the individual metrics and only join it in when needed.

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.

2 participants