Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v17] Add client tools auto update tctl commands #50971

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

vapopov
Copy link
Contributor

@vapopov vapopov commented Jan 11, 2025

Backport #47692 to branch/v17

Related: #50966

@vapopov vapopov added backport no-changelog Indicates that a PR does not require a changelog entry labels Jan 11, 2025
@vapopov vapopov marked this pull request as ready for review January 15, 2025 20:04
@github-actions github-actions bot requested review from r0mant and tcsc January 15, 2025 20:05
@github-actions github-actions bot added size/md tctl tctl - Teleport admin tool labels Jan 15, 2025
Base automatically changed from vapopov/auth-client-lazy-loading-v17 to branch/v17 January 15, 2025 21:40
@vapopov vapopov requested review from hugoShaka and tigrato January 15, 2025 23:04
* Add client tools auto update tctl commands

* Always print version for watch command
Restrict update empty target version
Rename command to upsert

* Add alias on/off for tools mode
Rename update command to configure

* Add semantic version validation

* Drop watch command for autoupdate

* Replace Upsert with Update/Create
Add format option for output json/yaml

* Change update message

* Use get/set naming for client-tools

* Add mode to response

* Change sub-command help messages
Leave only aliases for enabled/disabled

* Reorganize tctl commands to have commands not required auth client

* Propagate insecure flag with global config to commands by context

* Fix autoupdate command without auth client

* Change commands to enable/disable/target

* Add retry in case of the parallel request

* Add more than one retry
Code review changes

* Update tool/tctl/common/autoupdate_command.go

Co-authored-by: rosstimothy <[email protected]>

---------

Co-authored-by: rosstimothy <[email protected]>
@vapopov vapopov force-pushed the vapopov/add-auto-updates-tctl-commands-v17 branch from 361ea7f to b36afe2 Compare January 15, 2025 23:10
@vapopov vapopov added this pull request to the merge queue Jan 16, 2025
Merged via the queue into branch/v17 with commit 1513a14 Jan 16, 2025
39 checks passed
@vapopov vapopov deleted the vapopov/add-auto-updates-tctl-commands-v17 branch January 16, 2025 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport no-changelog Indicates that a PR does not require a changelog entry size/md tctl tctl - Teleport admin tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants