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

[v14] Add context around AWS join methods #43818

Merged
merged 1 commit into from
Sep 3, 2024

Conversation

ptgott
Copy link
Contributor

@ptgott ptgott commented Jul 3, 2024

Backport #43380 to branch/v14

@ptgott ptgott added the no-changelog Indicates that a PR does not require a changelog entry label Jul 3, 2024
@github-actions github-actions bot requested review from nklaassen and strideynet July 3, 2024 19:25
@ptgott ptgott enabled auto-merge July 9, 2024 17:21
@ptgott ptgott force-pushed the bot/backport-43380-branch/v14 branch from 217c91e to f013217 Compare July 9, 2024 17:21
Copy link
Contributor

@nklaassen nklaassen left a comment

Choose a reason for hiding this comment

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

My comments on the missing parens were not resolved
Screenshot 2024-07-09 at 10 42 17 AM

@ptgott ptgott force-pushed the bot/backport-43380-branch/v14 branch from f013217 to 36653c9 Compare August 14, 2024 22:22
@ptgott ptgott requested a review from mmcallister August 15, 2024 12:54
@ptgott ptgott force-pushed the bot/backport-43380-branch/v14 branch from 36653c9 to 73c52e3 Compare August 28, 2024 19:40
Closes #26893

Help users determine whether to use the EC2 or IAM join method by:

- Adding a "How it works" section to the top of each method's how-to
  guide explaining how the join method works.
- Ensuring that both guides include a brief discussion of alternative
  possibilities. Currently, only one of these guides does.
@ptgott ptgott force-pushed the bot/backport-43380-branch/v14 branch from 73c52e3 to 577b735 Compare August 30, 2024 13:21
@ptgott
Copy link
Contributor Author

ptgott commented Sep 3, 2024

@mmcallister Checking to see if you have a moment to review this one. Thanks!

@ptgott ptgott added this pull request to the merge queue Sep 3, 2024
Merged via the queue into branch/v14 with commit 3824909 Sep 3, 2024
27 checks passed
@ptgott ptgott deleted the bot/backport-43380-branch/v14 branch September 3, 2024 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 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.

5 participants