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

docs: add warning on using non-uuid name for openssh nodes #50906

Merged
merged 5 commits into from
Jan 14, 2025

Conversation

stevenGravy
Copy link
Contributor

Using a non-uuid can result in the web ui showing this on access attempts

image

Copy link

github-actions bot commented Jan 9, 2025

Amplify deployment status

Branch Commit Job ID Status Preview Updated (UTC)
stevenGravy/docs/nonuuidwarning 8c1a444 5 ✅SUCCEED stevengravy-docs-nonuuidwarning 2025-01-09 17:05:40

Copy link
Contributor

@rosstimothy rosstimothy left a comment

Choose a reason for hiding this comment

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

If we fix the current error, then half of this warning seems moot. Can we instead make the wording in the section above more forceful about using a UUID?

@stevenGravy
Copy link
Contributor Author

If we fix the current error, then half of this warning seems moot. Can we instead make the wording in the section above more forceful about using a UUID?

Please see update for the name.

@stevenGravy
Copy link
Contributor Author

stevenGravy commented Jan 9, 2025 via email

@stevenGravy stevenGravy added this pull request to the merge queue Jan 14, 2025
Merged via the queue into master with commit 3c79278 Jan 14, 2025
42 checks passed
@stevenGravy stevenGravy deleted the stevenGravy/docs/nonuuidwarning branch January 14, 2025 16:37
@public-teleport-github-review-bot

@stevenGravy See the table below for backport results.

Branch Result
branch/v15 Failed
branch/v16 Failed
branch/v17 Create PR

mvbrock pushed a commit that referenced this pull request Jan 18, 2025
* docs: add warning on using non-uuid name for openssh nodes

* docs: update verbiage on using non-uuid

* docs: move warning

* docs: update nodename instrs

Co-authored-by: rosstimothy <[email protected]>

* docs: update verbiage for metadata.name

Co-authored-by: rosstimothy <[email protected]>

---------

Co-authored-by: rosstimothy <[email protected]>
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