-
Notifications
You must be signed in to change notification settings - Fork 5
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
Move IAM policy/role creation to after selecting collector platform #237
Conversation
@@ -145,6 +145,7 @@ | |||
|
|||
.navTabs { | |||
border-bottom: 1px solid #ddd; | |||
margin-left: 0px !important; |
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.
export const IAMRoleCreation = () => { | ||
const tabs = [ | ||
[ 'cli', 'CLI' ], | ||
[ 'console', 'console'], |
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.
I believe aws is calling the dashboard "console", you can find it in this doc too (it has "console", "AWS CLI", "AWS API"):
https://docs.aws.amazon.com/IAM/latest/UserGuide/access_policies_create.html
I first thought that "console" is not so obvious to talk about the UI dashboard thingy, but I guess it's okay if it's what aws uses?
d1a09db
to
ac7a2dd
Compare
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.
I think we'll need to update paths in the install docs component when incorporating this into the in-app docs, but looks good.
4391933
to
0f0fde9
Compare
With current flow, setting up IAM policy/role comes before the collector platform selection.
This PR removes step 3, then add these steps inside of each platform's section after selected. Also, add the "CLI" and "console" option for IAM policy/role creation so that the user can use CLI to do such thing.
It might be better to review this with review app to see how it looks like. When you do so, please go from step 2 of aws section and see if it makes sense:
https://pganalyze-we-new-aws-in-xuapf2.herokuapp.com/docs/install/amazon_rds/02_create_monitoring_user