Skip to content

Add --current-device capabilities to tsh and tctl#30636

Merged
codingllama merged 9 commits intomasterfrom
codingllama/dt-currentdevice
Aug 18, 2023
Merged

Add --current-device capabilities to tsh and tctl#30636
codingllama merged 9 commits intomasterfrom
codingllama/dt-currentdevice

Conversation

@codingllama
Copy link
Copy Markdown
Contributor

Add the ability to act on the current device more easily.

  1. tsh device asset-tag is a debug command we planned for, but never executed. It's a simpler version of tsh device collect.

  2. tsh device enroll --current-device attempts to enroll the current device. It's focused on device admins, as it requires various permissions end-users typically won't have.

  3. tctl has a --current-device flag added to various commands. It substitutes the need to provide --asset-tag and --os.

The changes above are all focused on "day 1" experience, thus are catered to powerful Teleport users - powerful as in with various administrative permissions.

@codingllama codingllama requested a review from zmb3 August 17, 2023 19:39
@codingllama
Copy link
Copy Markdown
Contributor Author

FYI @sshahcodes.

@github-actions github-actions Bot added size/md tctl tctl - Teleport admin tool tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels Aug 17, 2023
Comment thread lib/devicetrust/enroll/enroll.go Outdated
Comment thread tool/tsh/common/device.go Outdated
Comment thread tool/tctl/common/devices.go Outdated
Comment thread tool/tsh/common/device.go Outdated
Comment thread tool/tsh/common/device.go Outdated
Comment thread tool/tsh/common/device.go Outdated
Comment thread lib/devicetrust/enroll/enroll.go Outdated
Comment thread lib/devicetrust/enroll/enroll.go Outdated
Comment thread tool/tsh/common/device.go Outdated
Comment thread tool/tsh/common/device.go Outdated
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from greedy52 August 18, 2023 08:23
@codingllama codingllama force-pushed the codingllama/dt-currentdevice branch from 3574a6d to c7462d5 Compare August 18, 2023 14:10
@codingllama
Copy link
Copy Markdown
Contributor Author

PTAL, comments addressed.

@codingllama codingllama added this pull request to the merge queue Aug 18, 2023
Merged via the queue into master with commit 5c8f91a Aug 18, 2023
@codingllama codingllama deleted the codingllama/dt-currentdevice branch August 18, 2023 15:37
@public-teleport-github-review-bot
Copy link
Copy Markdown

@codingllama See the table below for backport results.

Branch Result
branch/v13 Failed

codingllama added a commit that referenced this pull request Aug 18, 2023
* Add the `tsh device asset-tag` hidden command

* Implement registration in the fake device service

* Add `--current-device` to `tsh device enroll`
github-merge-queue Bot pushed a commit that referenced this pull request Aug 21, 2023
* Add the `tsh device asset-tag` hidden command

* Implement registration in the fake device service

* Add `--current-device` to `tsh device enroll`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/md tctl tctl - Teleport admin tool tsh tsh - Teleport's command line tool for logging into nodes running Teleport.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants