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

[v15] Update metrics.mdx with better clarity for heartbeat_connections_received_total #43216

Merged
merged 4 commits into from
Jun 18, 2024

Conversation

pschisa
Copy link
Contributor

@pschisa pschisa commented Jun 18, 2024

Backport #42632 to branch/v15

pschisa and others added 3 commits June 18, 2024 17:47
…ived_total

adding clarity to heartbeat_connections_received_total due to confusion around what a heartbeat connection represented. 

Confirmed that the metric counts the number of times an agent has connected to the endpoint on auth used to periodically send heartbeats. So it represents the number of agents actively heartbeating rather than the number of heartbeats received over the lifespan of that particular auth instance.
Co-authored-by: Paul Gottschling <paul.gottschling@goteleport.com>
change heartbeating to heart beating
@pschisa pschisa added the no-changelog Indicates that a PR does not require a changelog entry label Jun 18, 2024
@pschisa pschisa enabled auto-merge June 18, 2024 19:50
Copy link

🤖 Vercel preview here: https://docs-ljylqifti-goteleport.vercel.app/docs/ver/preview

@pschisa pschisa added this pull request to the merge queue Jun 18, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 18, 2024
Copy link

🤖 Vercel preview here: https://docs-gubsx3qai-goteleport.vercel.app/docs/ver/preview

@pschisa pschisa added this pull request to the merge queue Jun 18, 2024
Merged via the queue into branch/v15 with commit bad27ac Jun 18, 2024
31 checks passed
@pschisa pschisa deleted the bot/backport-42632-branch/v15 branch June 18, 2024 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport documentation no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants