Skip to content

[v16] [tctl/top] Add raw metrics view#57633

Merged
okraport merged 1 commit intobranch/v16from
okraport/backport-57260-branch/v16
Aug 7, 2025
Merged

[v16] [tctl/top] Add raw metrics view#57633
okraport merged 1 commit intobranch/v16from
okraport/backport-57260-branch/v16

Conversation

@okraport
Copy link
Copy Markdown
Contributor

@okraport okraport commented Aug 7, 2025

Backport #57260 to branch/v16

changelog: tctl top can now display raw prometheus metrics

@github-actions github-actions bot added backport size/md tctl tctl - Teleport admin tool labels Aug 7, 2025
@github-actions github-actions bot requested review from creack and fspmarshall August 7, 2025 09:37
@okraport okraport force-pushed the okraport/backport-57260-branch/v16 branch from 62f1ca9 to 2f80f25 Compare August 7, 2025 09:46
This commit adds a new tab to the `tctl top` utility
which renders prometheus metrics as a list which supports
live updates and filtering.

It makes use of the bubbles/list charm which supports
fuzzy searching via github.com/sahilm/fuzzy.

Refactores the main model to split the refresh tick
into seperate tea commands to allow tapping the raw
metrics concurrently to generating the report.
Keybinds are now stored as part of the model and used
directly for handling KeyMsg events.

Note that `list.Model` does not support variable height
list item entries and for pagination purposes this value
needs to be fixed, for this reason histogram metrics will
only render sum and count.

changelog: tctl top can now display raw prometheus metrics

* apply review suggestions
@okraport okraport force-pushed the okraport/backport-57260-branch/v16 branch from 2f80f25 to 2eb4904 Compare August 7, 2025 10:27
Comment thread go.mod
@okraport okraport enabled auto-merge August 7, 2025 13:24
@okraport okraport added this pull request to the merge queue Aug 7, 2025
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from fspmarshall August 7, 2025 14:43
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 7, 2025
@okraport okraport added this pull request to the merge queue Aug 7, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 7, 2025
@okraport okraport added this pull request to the merge queue Aug 7, 2025
Merged via the queue into branch/v16 with commit b336348 Aug 7, 2025
41 checks passed
@okraport okraport deleted the okraport/backport-57260-branch/v16 branch August 7, 2025 18:09
@fheinecke fheinecke mentioned this pull request Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport size/md tctl tctl - Teleport admin tool

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants