Skip to content

Conversation

@hariarla
Copy link
Contributor

@hariarla hariarla commented Nov 6, 2025

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Relates OR Closes #0000

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccXXX'
...
Screenshot 2025-11-06 at 2 48 09 PM Screenshot 2025-11-06 at 3 30 27 PM image

@hariarla hariarla changed the title feat(iampolicy): add role templates feat(iampolicy): add role templates [DO NOT MERGE] Nov 6, 2025
@hariarla hariarla force-pushed the enable_role_templates branch 3 times, most recently from b927359 to db04d2c Compare November 13, 2025 07:47
@hariarla hariarla force-pushed the enable_role_templates branch from db04d2c to 30150cb Compare November 18, 2025 07:36
@hariarla hariarla changed the title feat(iampolicy): add role templates [DO NOT MERGE] feat(iampolicy): add role templates Nov 18, 2025
@hariarla hariarla changed the title feat(iampolicy): add role templates feat(iampolicy): add role templates & assignments Nov 18, 2025
@hkantare
Copy link
Collaborator

Import stmts in docs also have mismatch with required fields it requires two fields as per d.SetId(...)

 Syntax
<pre>
$ terraform import ibm_iam_role_template_version.iam_role_template_version &lt;role_template_id&gt;
</pre>

Schema: map[string]*schema.Schema{
"target": &schema.Schema{
Type: schema.TypeList,
Computed: true,
Copy link
Collaborator

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
Copy link
Collaborator

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

Copy link
Collaborator

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"
}
Copy link
Collaborator

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"
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Templates is required parameter

@hkantare
Copy link
Collaborator

We have 4 new datasources but docs were added only for 2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants