Skip to content

[v17] [tctl/top] Add raw metrics view#57634

Merged
okraport merged 1 commit intobranch/v17from
okraport/backport-57260-branch/v17
Aug 7, 2025
Merged

[v17] [tctl/top] Add raw metrics view#57634
okraport merged 1 commit intobranch/v17from
okraport/backport-57260-branch/v17

Conversation

@okraport
Copy link
Copy Markdown
Contributor

@okraport okraport commented Aug 7, 2025

Backport #57260 to branch/v17

changelog: tctl top can now display raw prometheus metrics

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
@github-actions github-actions bot requested review from creack and fspmarshall August 7, 2025 09:38
@github-actions github-actions bot added backport size/md tctl tctl - Teleport admin tool labels Aug 7, 2025
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
@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/v17 with commit 0257cd7 Aug 7, 2025
42 of 43 checks passed
@okraport okraport deleted the okraport/backport-57260-branch/v17 branch August 7, 2025 18:10
@doggydogworld doggydogworld mentioned this pull request Aug 18, 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