Skip to content
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

Add teleport_reverse_tunnels_connected Prometheus metric #9698

Merged
merged 6 commits into from
Feb 2, 2022

Conversation

rcanderson23
Copy link
Contributor

@rcanderson23 rcanderson23 commented Jan 7, 2022

Prometheus gauge that increments when the keepalive for the reverse tunnel is handled, decrements when marked when closed.

# HELP teleport_reverse_tunnels_connected Number of reverse SSH tunnels connected to proxy by Teleport instances
# TYPE teleport_reverse_tunnels_connected gauge
teleport_reverse_tunnels_connected{type="app"} 1
teleport_reverse_tunnels_connected{type="db"} 1
teleport_reverse_tunnels_connected{type="kube"} 2
teleport_reverse_tunnels_connected{type="node"} 1
teleport_reverse_tunnels_connected{type="windows_desktop"} 1

Part of https://github.com/gravitational/cloud/issues/946 and can help with #7844 if connected over reverse tunnel. A tunnel would be created per service so if a Teleport instance is configured with both DB and Kube, both would be incremented when those tunnels are created.

lib/reversetunnel/localsite.go Outdated Show resolved Hide resolved
docs/pages/setup/reference/metrics.mdx Outdated Show resolved Hide resolved
rcanderson23 and others added 2 commits January 18, 2022 11:56
Co-authored-by: Paul Gottschling <[email protected]>
Co-authored-by: Paul Gottschling <[email protected]>
@russjones
Copy link
Contributor

@ptgott Can you please re-review?

@rcanderson23 rcanderson23 enabled auto-merge (squash) February 2, 2022 20:31
@rcanderson23 rcanderson23 merged commit b384de6 into master Feb 2, 2022
@rcanderson23 rcanderson23 deleted the carson/reverse-tunnel-metric branch February 2, 2022 20:52
rcanderson23 added a commit that referenced this pull request Feb 8, 2022
Adds teleport_reverse_tunnels_connected Prometheus metric which tracks reverse tunnels connected to the proxy server by type.

* Update prometheus help

Co-authored-by: Paul Gottschling <[email protected]>

* Update metrics wording

Co-authored-by: Paul Gottschling <[email protected]>
rcanderson23 added a commit that referenced this pull request Feb 8, 2022
)

Adds teleport_reverse_tunnels_connected Prometheus metric which tracks reverse tunnels connected to the proxy server by type.

Co-authored-by: Paul Gottschling <[email protected]>
@webvictim webvictim mentioned this pull request Mar 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants