-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
docs: simplify machine-id getting started #47552
docs: simplify machine-id getting started #47552
Conversation
@@ -49,17 +49,8 @@ Before you create a bot user, you need to determine which role(s) you want to | |||
assign to it. You can use the `tctl` command below to examine what roles exist | |||
on your system. | |||
|
|||
<Tabs> |
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.
👍 Yeah - this is much simpler. I don't know why we ever had this split....
🤖 Vercel preview here: https://docs-iay16j13z-goteleport.vercel.app/docs/ver/preview |
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.
Looks good - thank you. Part of me is thinking we should do another PR to simplify this guide further by removing the references to iam
joining - this made sense when we only had IAM joining, but now we have a full range of join methods, it doesn't make any sense to me to pick out AWS as the special case to include in this getting started guide.
I did a brief to explain why you could want a non-token method. Probably worth keeping that then. pls take a look |
🤖 Vercel preview here: https://docs-ftrh4cljm-goteleport.vercel.app/docs/ver/preview |
Actually - if you drop that last commit - I figured it made sense for me to raise a PR with a few changes in regards to talking about joining in this guide: #47554 |
This reverts commit 1731a26. this is being done in a separate pr on joining methods
Sure, dropped! |
🤖 Vercel preview here: https://docs-pohfvft7l-goteleport.vercel.app/docs/ver/preview |
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.
Approved.
@stevenGravy See the table below for backport results.
|
* docs: simplify machine-id getting started * docs: update token joining desc for machine-id getting started * Revert "docs: update token joining desc for machine-id getting started" This reverts commit 1731a26. this is being done in a separate pr on joining methods
Removed separation of cloud vs self-hosted. As a getting started wasn't much value or diff. This was referring to parameters like
--auth-server
that weren't used that can cause confusion getting started.