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

Improve the usability of the Local Users guide #12001

Merged
merged 2 commits into from
Apr 18, 2022
Merged

Conversation

ptgott
Copy link
Contributor

@ptgott ptgott commented Apr 15, 2022

See #11841

  • Define "local users" at the beginning of the guide
  • Split the "Adding and deleting users" section and move the "Deleting
    users" section after the "Editing users" section. This way, readers
    can follow all commands in the guide. Otherwise, the new user is
    deleted before being edited.
  • Hide scope-irrelevant details
  • Remove a "tsh ssh" command that a reader may not necessarily have
    set up Teleport to run properly.

| `bob` | `bob` | Teleport user `bob` can log in into member Nodes only as OS user `bob` |
| `ross` | | If no OS login is specified, it defaults to the same name as the Teleport user `ross`. |
| `joe` | `joe`, `root` | Teleport user `joe` can log in to member Nodes as user `joe` or `root` on the OS. |
| `bob` | `bob` | Teleport user ``bob` can log in to member Nodes only as OS user `bob`. |
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
| `bob` | `bob` | Teleport user ``bob` can log in to member Nodes only as OS user `bob`. |
| `bob` | `bob` | Teleport user `bob` can log in to member Nodes only as OS user `bob`. |

@ptgott ptgott force-pushed the paul.gottschling/11841-2 branch from 807173d to 1b6a000 Compare April 15, 2022 20:25
ptgott added 2 commits April 18, 2022 15:47
See #11841

- Define "local users" at the beginning of the guide
- Split the "Adding and deleting users" section and move the "Deleting
  users" section after the "Editing users" section. This way, readers
  can follow all commands in the guide. Otherwise, the new user is
  deleted before being edited.
- Hide scope-irrelevant details
- Remove a "tsh ssh" command that a reader may not necessarily have
  set up Teleport to run properly.
@ptgott ptgott force-pushed the paul.gottschling/11841-2 branch from de85db4 to 4a71d88 Compare April 18, 2022 19:49
@ptgott ptgott enabled auto-merge (squash) April 18, 2022 19:49
@ptgott ptgott merged commit d9a0584 into master Apr 18, 2022
@ptgott ptgott deleted the paul.gottschling/11841-2 branch April 18, 2022 19:56
ptgott added a commit that referenced this pull request Apr 21, 2022
* Improve the usability of the Local Users guide

See #11841

- Define "local users" at the beginning of the guide
- Split the "Adding and deleting users" section and move the "Deleting
  users" section after the "Editing users" section. This way, readers
  can follow all commands in the guide. Otherwise, the new user is
  deleted before being edited.
- Hide scope-irrelevant details
- Remove a "tsh ssh" command that a reader may not necessarily have
  set up Teleport to run properly.

* Respond to PR feedback
ptgott added a commit that referenced this pull request Apr 21, 2022
* Improve the usability of the Local Users guide

See #11841

- Define "local users" at the beginning of the guide
- Split the "Adding and deleting users" section and move the "Deleting
  users" section after the "Editing users" section. This way, readers
  can follow all commands in the guide. Otherwise, the new user is
  deleted before being edited.
- Hide scope-irrelevant details
- Remove a "tsh ssh" command that a reader may not necessarily have
  set up Teleport to run properly.

* Respond to PR feedback
ptgott added a commit that referenced this pull request Apr 21, 2022
* Improve the usability of the Local Users guide

See #11841

- Define "local users" at the beginning of the guide
- Split the "Adding and deleting users" section and move the "Deleting
  users" section after the "Editing users" section. This way, readers
  can follow all commands in the guide. Otherwise, the new user is
  deleted before being edited.
- Hide scope-irrelevant details
- Remove a "tsh ssh" command that a reader may not necessarily have
  set up Teleport to run properly.

* Respond to PR feedback
@webvictim webvictim mentioned this pull request Jun 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants