Skip to content

[v18] feat: Bot instances advanced filter#60558

Closed
nicholasmarais1158 wants to merge 2 commits intonicholasmarais1158/backport-59263-branch/v18from
nicholasmarais1158/backport-59374-branch/v18
Closed

[v18] feat: Bot instances advanced filter#60558
nicholasmarais1158 wants to merge 2 commits intonicholasmarais1158/backport-59263-branch/v18from
nicholasmarais1158/backport-59374-branch/v18

Conversation

@nicholasmarais1158
Copy link
Copy Markdown
Contributor

Backport #59374 and #59819 to branch/v18

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
@espadolini
Copy link
Copy Markdown
Contributor

If these backports have to land at the same time (and they have to, this PR breaks "logical" compatibility in ListBotInstancesV2) I'd rather see both in the same PR rather than risk merging one but not the other in the intended release.

@nicholasmarais1158
Copy link
Copy Markdown
Contributor Author

Superseded by #60761.

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

Labels

hold-for-18.4 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.

3 participants