Skip to content

[v16] [api] Add ListDatabases#58112

Merged
okraport merged 2 commits intobranch/v16from
okraport/backport-57805-branch/v16
Aug 27, 2025
Merged

[v16] [api] Add ListDatabases#58112
okraport merged 2 commits intobranch/v16from
okraport/backport-57805-branch/v16

Conversation

@okraport
Copy link
Copy Markdown
Contributor

Backport #57805 to branch/v16
Regenerated protos with make grpc for backport.

Note that this backport, does not include the
Range* helpers as those are not supported in go1.23.

Changelog: Add paginated API ListDatabases, deprecate GetDatabases

@okraport okraport marked this pull request as ready for review August 20, 2025 10:55
@github-actions github-actions bot added backport size/md tctl tctl - Teleport admin tool labels Aug 20, 2025
@okraport okraport marked this pull request as draft August 20, 2025 13:09
@okraport
Copy link
Copy Markdown
Contributor Author

Putting this on draft to wait for #57861

@okraport
Copy link
Copy Markdown
Contributor Author

Putting this on draft to wait for #57861

The api module is on go1.23, and backend does not have the ranging iterator, we have decided to go ahead with manual backports to v17 & v16 for the time being.

This commits deperecates GetDatabases, which
should be replaced by a paginated counterpart:
ListDatabases

The commit also adds TakeWhile for conditional
iter stream reads.

Note that this backport, does not include the
`Range*` helpers as those are not supported in go1.23.

Towards: gravitational/teleport.e/issues/6759
Changelog: Add paginated API ListDatabases, deprecate GetDatabases
@okraport okraport force-pushed the okraport/backport-57805-branch/v16 branch from 2b3d446 to 42413a6 Compare August 21, 2025 13:20
@okraport okraport marked this pull request as ready for review August 21, 2025 13:20
@okraport
Copy link
Copy Markdown
Contributor Author

This is ready for a stamp, @fspmarshall could you have a look please?

@rosstimothy rosstimothy requested a review from zmb3 August 27, 2025 15:05
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from fspmarshall August 27, 2025 15:17
@okraport okraport added this pull request to the merge queue Aug 27, 2025
Merged via the queue into branch/v16 with commit 16b8a5f Aug 27, 2025
44 checks passed
@okraport okraport deleted the okraport/backport-57805-branch/v16 branch August 27, 2025 16:18
@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