-
Notifications
You must be signed in to change notification settings - Fork 474
docs: add eks auth #4332
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 eks auth #4332
Conversation
d45a8af
to
49da2cd
Compare
@tinnytintin10 can you review this one? this PR contains only a change in the documentation |
/test |
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.
@olegsu I removed the kustomize example you used out of the readme. We'll cover it in the getting started guide of our docs.
@benironside can you please read through the readme here and provide feedback? The goal here is not to include an in-depth getting started guide but instead to point them to the docs that we're updating. |
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 great overall, left some suggestions for your consideration, I hope they're helpful.
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.
Left one more comment
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.
More comments
@oren-zohar @olegsu can y'all review the "Requirments" section of the read me for accuracy? @benironside I made some final updates to the top of the issue- can you review it one last time? Thanks all! |
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.
Few notes about the default attached role
🌐 Coverage report
|
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.
A few minor revisions
Thanks, @benironside! I have applied your suggestion. |
suggestion by Ben Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
suggestion by Ben Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
suggestion by Ben Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
suggestion by Ben Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
suggestion by Ben Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
suggestion by Ben Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
2. Adding links to the documentation for the various pages we mention
Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
What does this PR do?
Document the options for cloudbeat the authenticate when running in EKS cluster.
This will close elastic/cloudbeat#403