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

[v16] Add client tools auto update tctl commands #50972

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

vapopov
Copy link
Contributor

@vapopov vapopov commented Jan 11, 2025

Backport #47692 to branch/v16

Related: #50968

@vapopov vapopov added backport no-changelog Indicates that a PR does not require a changelog entry labels Jan 11, 2025
@vapopov vapopov changed the title [v16] Add client tools auto update tctl commands #50971 [v16] Add client tools auto update tctl commands Jan 15, 2025
@vapopov vapopov marked this pull request as ready for review January 15, 2025 20:04
@github-actions github-actions bot added size/md tctl tctl - Teleport admin tool labels Jan 15, 2025
@github-actions github-actions bot requested review from avatus and mvbrock January 15, 2025 20:05
Base automatically changed from vapopov/auth-client-lazy-loading-v16 to branch/v16 January 23, 2025 19:23
* 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-v16 branch from 5764070 to 59a7420 Compare January 23, 2025 19:59
@vapopov vapopov enabled auto-merge January 23, 2025 20:10
@vapopov vapopov added this pull request to the merge queue Jan 23, 2025
Merged via the queue into branch/v16 with commit 8ac0f4d Jan 23, 2025
39 checks passed
@vapopov vapopov deleted the vapopov/add-auto-updates-tctl-commands-v16 branch January 23, 2025 20:35
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.

4 participants