-
Notifications
You must be signed in to change notification settings - Fork 711
feat(iampolicy): add role templates & assignments #6539
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
base: master
Are you sure you want to change the base?
Conversation
b927359 to
db04d2c
Compare
Signed-off-by: Hari K Arla <[email protected]>
db04d2c to
30150cb
Compare
|
Import stmts in docs also have mismatch with required fields it requires two fields as per d.SetId(...) |
| Schema: map[string]*schema.Schema{ | ||
| "target": &schema.Schema{ | ||
| Type: schema.TypeList, | ||
| Computed: true, |
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.
Is this not a required argument? In create we do d.Get("target")
With this change the testcases should have failed
|
|
||
| ```hcl | ||
| data "ibm_iam_role_template" "iam_role_template" { | ||
| account_id = ibm_iam_role_template.iam_role_template_instance.account_id |
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.
account_id is n't defined in the schema for datasource "ibm_iam_role_template"
We are fetching account_id from the logged in user context
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.
In Schema we don't have any optional/required arguments only computed.
| ```hcl | ||
| data "ibm_iam_role_template_version" "iam_role_template_version" { | ||
| role_template_id = "role_template_id" | ||
| } |
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.
version is also required in Schema
| target { | ||
| type = "Account" | ||
| id = "id" | ||
| } |
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.
Templates is required parameter
|
We have 4 new datasources but docs were added only for 2 |
Community Note
Relates OR Closes #0000
Output from acceptance testing: