Skip to content

[docs] troubleshooting for AWS Access SSM sessions#34074

Merged
greedy52 merged 2 commits intomasterfrom
STeve/docs_troubleshot_tsh_aws_ssm
Nov 1, 2023
Merged

[docs] troubleshooting for AWS Access SSM sessions#34074
greedy52 merged 2 commits intomasterfrom
STeve/docs_troubleshot_tsh_aws_ssm

Conversation

@greedy52
Copy link
Copy Markdown
Contributor

@greedy52 greedy52 commented Oct 31, 2023

Related:

Add this troubleshooting section as this problem appears in both Zendesk and github issues.

Note that the above changes only fix tsh aws xxx. For tsh proxy aws, user still needs the workaround using NO_PROXY.

Not sure if it's a good idea link github issues though.

@greedy52 greedy52 added documentation backport/branch/v12 no-changelog Indicates that a PR does not require a changelog entry labels Oct 31, 2023
@greedy52 greedy52 requested review from Tener and smallinsky October 31, 2023 15:40
@greedy52 greedy52 self-assigned this Oct 31, 2023
@greedy52 greedy52 temporarily deployed to vercel October 31, 2023 15:40 — with GitHub Actions Inactive
@github-actions
Copy link
Copy Markdown
Contributor

🤖 Vercel preview here: https://docs-71w7onr5h-goteleport.vercel.app/docs/ver/preview

Copy link
Copy Markdown
Contributor

@Tener Tener left a comment

Choose a reason for hiding this comment

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

I think the links are useful are are likely to remain live for quite some time.

@greedy52 greedy52 temporarily deployed to vercel October 31, 2023 17:46 — with GitHub Actions Inactive
@greedy52 greedy52 requested review from ptgott, r0mant and zmb3 October 31, 2023 17:46
@github-actions
Copy link
Copy Markdown
Contributor

🤖 Vercel preview here: https://docs-dcqdimrjf-goteleport.vercel.app/docs/ver/preview

### `remote error: tls: bad certificate` error during SSM sessions

You may encounter the `remote error: tls: bad certificate` error when starting
System Session Manager (SSM) sessions using `tsh aws ssm start-session` or `tsh
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
System Session Manager (SSM) sessions using `tsh aws ssm start-session` or `tsh
System Session Manager (SSM) sessions using the `tsh aws ssm start-session` or `tsh

Comment on lines +387 to +390
implemented for [`tsh aws ssm
start-session`](https://github.com/gravitational/teleport/pull/30510) and [`tsh
aws ecs
execute-command`](https://github.com/gravitational/teleport/pull/33705).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would make it more explicit that the links are to PRs. Otherwise, a user could think they link to another docs page and ignore them. I would remove the links here and add the following at the end of this H3 section:

For more information on the `tsh` workarounds, see the 
pull requests that introduced them:

- [https://github.com/gravitational/teleport/pull/30510](https://github.com/gravitational/teleport/pull/30510)
- [https://github.com/gravitational/teleport/pull/30510](https://github.com/gravitational/teleport/pull/33705)

@greedy52 greedy52 temporarily deployed to vercel November 1, 2023 17:24 — with GitHub Actions Inactive
@greedy52 greedy52 enabled auto-merge November 1, 2023 17:24
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Nov 1, 2023

🤖 Vercel preview here: https://docs-jf8jv8axv-goteleport.vercel.app/docs/ver/preview

@greedy52 greedy52 added this pull request to the merge queue Nov 1, 2023
Merged via the queue into master with commit 6aa1aac Nov 1, 2023
@greedy52 greedy52 deleted the STeve/docs_troubleshot_tsh_aws_ssm branch November 1, 2023 17:50
@public-teleport-github-review-bot
Copy link
Copy Markdown

@greedy52 See the table below for backport results.

Branch Result
branch/v12 Create PR
branch/v13 Create PR
branch/v14 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation no-changelog Indicates that a PR does not require a changelog entry size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants