Skip to content

[v17] feat: Bot instances advanced filter#60557

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

[v17] feat: Bot instances advanced filter#60557
nicholasmarais1158 wants to merge 2 commits intonicholasmarais1158/backport-59263-branch/v17from
nicholasmarais1158/backport-59374-branch/v17

Conversation

@nicholasmarais1158
Copy link
Copy Markdown
Contributor

Backport #59374 and #59819 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

* Add expression parser

* Contribute `to_string` function to default parser

* Add API support for `query` filter

* Fix `SearchPanel` submit with advanced toggle

* Add advanced search to web UI

* 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`

* Use `t.Context()`

* Remove expression methods

* Remove unnecessary fallback comments

* Return early if only bot name filter is required (backend only)

* Fix lint

* replace `to_string` with `equals` (version type only)

* Fix comment

* Remove unnecessary `to_string` tests

* Switch to a true equals function
# Conflicts:
#	lib/cache/bot_instance.go
#	web/packages/teleport/src/BotInstances/List/BotInstancesList.tsx
@nicholasmarais1158 nicholasmarais1158 self-assigned this Oct 24, 2025
@nicholasmarais1158 nicholasmarais1158 added machine-id no-changelog Indicates that a PR does not require a changelog entry labels Oct 24, 2025
@nicholasmarais1158 nicholasmarais1158 marked this pull request as ready for review October 24, 2025 14:42
@nicholasmarais1158
Copy link
Copy Markdown
Contributor Author

Will backport all bot instance changes together.

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

Labels

machine-id no-changelog Indicates that a PR does not require a changelog entry size/lg ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants