Skip to content

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

Merged
merged 28 commits into from
Oct 18, 2022
Merged

docs: add eks auth #4332

merged 28 commits into from
Oct 18, 2022

Conversation

olegsu
Copy link
Contributor

@olegsu olegsu commented Sep 28, 2022

What does this PR do?

Document the options for cloudbeat the authenticate when running in EKS cluster.

This will close elastic/cloudbeat#403

@olegsu olegsu added the docs label Sep 28, 2022
@olegsu olegsu requested a review from a team as a code owner September 28, 2022 11:11
@olegsu olegsu force-pushed the kspm-aws branch 2 times, most recently from d45a8af to 49da2cd Compare September 28, 2022 11:12
@elasticmachine
Copy link

elasticmachine commented Sep 28, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-10-18T07:14:48.678+0000

  • Duration: 14 min 53 sec

Test stats 🧪

Test Results
Failed 0
Passed 2
Skipped 0
Total 2

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@kfirpeled
Copy link
Contributor

kfirpeled commented Sep 28, 2022

@tinnytintin10 can you review this one? this PR contains only a change in the documentation

@oren-zohar
Copy link
Contributor

/test

@tinnytintin10 tinnytintin10 self-requested a review October 3, 2022 17:51
Copy link
Contributor

@tinnytintin10 tinnytintin10 left a 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.

@tinnytintin10
Copy link
Contributor

@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.

Copy link
Contributor

@benironside benironside left a 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.

Copy link
Contributor

@benironside benironside left a 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

Copy link
Contributor

@benironside benironside left a comment

Choose a reason for hiding this comment

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

More comments

@tinnytintin10
Copy link
Contributor

@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!

Copy link
Contributor Author

@olegsu olegsu left a 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

@elasticmachine
Copy link

elasticmachine commented Oct 13, 2022

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (0/0) 💚
Files 100.0% (0/0) 💚 2.759
Classes 100.0% (0/0) 💚 2.759
Methods 25.0% (1/4) 👎 -65.249
Lines 100.0% (0/0) 💚 8.499
Conditionals 100.0% (0/0) 💚

Copy link
Contributor

@benironside benironside left a 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

@tinnytintin10
Copy link
Contributor

A few minor revisions

Thanks, @benironside! I have applied your suggestion.

olegsu and others added 6 commits October 18, 2022 10:12
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>
tinnytintin10 and others added 22 commits October 18, 2022 10:12
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>
@olegsu olegsu merged commit ad08aaa into elastic:main Oct 18, 2022
@kfirpeled kfirpeled added the Team:Cloud Security Cloud Security team [elastic/cloud-security-posture] label Jun 26, 2023
@andrewkroh andrewkroh added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Team:Cloud Security Cloud Security team [elastic/cloud-security-posture]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Validate AWS auth options
7 participants