-
Notifications
You must be signed in to change notification settings - Fork 1
fix: update iam permission for cloud monitoring DA #82
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
Changes from 7 commits
6ba71c6
9889067
779e835
efa442d
a69c108
ef88ed9
cad598a
a9fbf7a
406f37c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -217,23 +217,30 @@ | |
| "role_crns": [ | ||
| "crn:v1:bluemix:public:iam::::role:Administrator" | ||
| ], | ||
| "service_name": "iam-identity", | ||
| "notes": "[Optional] Required if Cloud automation for account configuration is enabled." | ||
| "service_name": "All Account Management services", | ||
| "notes": "[Optional] Administrator access is required to create and delete resource groups." | ||
| }, | ||
| { | ||
| "role_crns": [ | ||
| "crn:v1:bluemix:public:iam::::serviceRole:Manager", | ||
| "crn:v1:bluemix:public:iam::::role:Editor" | ||
| "crn:v1:bluemix:public:iam::::role:Viewer" | ||
| ], | ||
| "service_name": "sysdig-monitor", | ||
| "notes": "[Optional] Required for creating a new instance of cloud monitoring." | ||
| "service_name": "Resource group only", | ||
| "notes": "Viewer access is required in the resource group you want to provision in." | ||
| }, | ||
| { | ||
| "role_crns": [ | ||
| "crn:v1:bluemix:public:iam::::role:Administrator" | ||
| ], | ||
| "service_name": "All Identity and Access enabled services", | ||
| "notes": "Required to create IAM authorization policy between Metrics Router and Cloud Monitoring." | ||
|
||
| }, | ||
| { | ||
| "role_crns": [ | ||
| "crn:v1:bluemix:public:iam::::serviceRole:Manager", | ||
| "crn:v1:bluemix:public:iam::::role:Editor" | ||
| ], | ||
| "service_name": "metrics-router", | ||
| "notes": "[Optional] Required if metrics routing to cloud monitoring instance is enabled." | ||
| "service_name": "sysdig-monitor", | ||
| "notes": "Required for creating a new instance of cloud monitoring." | ||
iamar7 marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| } | ||
| ], | ||
| "architecture": { | ||
|
|
@@ -257,7 +264,7 @@ | |
| "dependencies": [ | ||
| { | ||
| "name": "deploy-arch-ibm-account-infra-base", | ||
| "description": "Enable to create a resource groups by default where all the resources will be provisioned and, when you enable the “with Account Settings” option, it also applies baseline security and governance settings. When disabled, provide your own resource group via the `existing_resource_group_name` input.", | ||
| "description": "Organize your IBM Cloud account with preconfigured resource groups. If not selected, the default resource group is used. Optionally, expand to apply recommended security controls via \"with Account Settings\" variation.", | ||
| "id": "63641cec-6093-4b4f-b7b0-98d2f4185cd6-global", | ||
| "version": "v3.0.7", | ||
| "flavors": [ | ||
|
|
||
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.
Does this fit well here ?
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.
Yes, I will update the notes to that