-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Connect My Computer: Generate the config file #29010
Conversation
Please do not review it yet, I misunderstood the point of |
5140e39
to
fa24094
Compare
Ready for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't have much time to review this, I'll come back to it tomorrow.
fa24094
to
fefda87
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't tried it out in the dev version yet, but the code looks good overall.
f95729c
to
e3f1c9f
Compare
… `TokenProvisioner`
fefda87
to
5587ce2
Compare
# Conflicts: # lib/teleterm/daemon/daemon.go
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good!
Contributes to #27881
This PR adds generating the agent config file. To do this, we first have to create the join token for a node (this is done via
CreateConnectMyComputerNodeToken
). Created token is then passed (with a few other params) tocreateAgentConfigFile
function that creates the spawnsteleport configure
process.RFD section https://github.com/gravitational/teleport/blob/master/rfd/0133-connect-my-computer.md#generating-the-config-file
I didn't run token removal (although I added a gRPC handler for it). I left a TODO for myself to do this along with joining the cluster.