Skip to content

[v17] feat: List bot instances version and hostname sort#60521

Closed
nicholasmarais1158 wants to merge 2 commits intobranch/v17from
nicholasmarais1158/backport-59263-branch/v17
Closed

[v17] feat: List bot instances version and hostname sort#60521
nicholasmarais1158 wants to merge 2 commits intobranch/v17from
nicholasmarais1158/backport-59263-branch/v17

Conversation

@nicholasmarais1158
Copy link
Copy Markdown
Contributor

Backport #59263 to branch/v17

Updates: #57994

* Add version and hostname indexes to cache

* Add `ListBotInstancesV2` rpc and use request options

* Add v2 bot instance list endpoint

* Use v2 endpoint in web UI

* Pass signal through to support aborting requests

* Fix comment typo

* Rename util func

* Deprecate `ListBotInstances` rpc

* Encode hostname in cache key

* Address pre-release sorting in version numbers

* Rename bot instance cache utils

* Fix lint deprecation warnings

* Extract filter fields to message

* Replace `fmt.Sprintf("%06d", ...)`

* Update invalid sort field error

* Fallback to v1 endpoint if possible

* Use `strcase` for case-insensitive compare

* Backend results are filtered by bot name so no need to re-filter in `MatchBotInstance`

* Revert "Replace `fmt.Sprintf("%06d", ...)`"

This reverts commit 2fbd797.
# Conflicts:
#	web/packages/teleport/src/BotInstances/BotInstances.tsx
@nicholasmarais1158 nicholasmarais1158 self-assigned this Oct 23, 2025
@nicholasmarais1158 nicholasmarais1158 added machine-id no-changelog Indicates that a PR does not require a changelog entry labels Oct 23, 2025
@nicholasmarais1158 nicholasmarais1158 marked this pull request as ready for review October 24, 2025 14:42
@github-actions github-actions bot added backport size/md tctl tctl - Teleport admin tool ui labels Oct 24, 2025
@github-actions github-actions bot requested review from avatus and espadolini October 24, 2025 14:43
Copy link
Copy Markdown
Contributor

@espadolini espadolini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As previously mentioned in #59374 (comment), this backport must include #60557 or there will be a compatibility break in ListBotInstancesV2.

@nicholasmarais1158
Copy link
Copy Markdown
Contributor Author

Will backport all bot instance change together instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport machine-id no-changelog Indicates that a PR does not require a changelog entry size/md tctl tctl - Teleport admin tool ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants