Skip to content
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

Wrong documentation for iam_trusted_profile_claim_rule #3216

Closed
data-henrik opened this issue Oct 14, 2021 · 3 comments
Closed

Wrong documentation for iam_trusted_profile_claim_rule #3216

data-henrik opened this issue Oct 14, 2021 · 3 comments
Labels
documentation service/IAM Issues related to IAM

Comments

@data-henrik
Copy link
Contributor

https://registry.terraform.io/providers/IBM-Cloud/ibm/latest/docs/resources/iam_trusted_profile_claim_rule

The syntax for conditions is wrong, it does not work.

resource "ibm_iam_trusted_profile_claim_rule" "iam_trusted_profile_claim_rule" {
  conditions = { "claim" : "claim", "operator" : "operator", "value" : "value" }
  profile_id = "profile_id"
  type = "type"
}

It should be something like this:

resource "ibm_iam_trusted_profile_claim_rule" "iam_trusted_profile_claim_rule" {
  conditions { 
     "claim" = "claim"
     "operator"= "operator"
     "value" = "value" 
  }
  profile_id = "profile_id"
  type = "type"
}

But what about a truly working e2e sample?

@kavya498 kavya498 added the service/IAM Issues related to IAM label Oct 14, 2021
kavya498 added a commit to kavya498/terraform-provider-ibm that referenced this issue Oct 14, 2021
kavya498 added a commit to kavya498/terraform-provider-ibm that referenced this issue Oct 14, 2021
@data-henrik
Copy link
Contributor Author

FYI: I have a working example myself:


resource "ibm_iam_trusted_profile_claim_rule" "TP_AppID_claim_rule" {
  profile_id=ibm_iam_trusted_profile.TP_AppID_tp.profile_id
  type      ="Profile-SAML"
  realm_name= "appid://xxxx"
  expiration= 3600
  conditions {
    claim="emails" 
    operator="CONTAINS"
    value="\"[email protected]\""
    }
}

The point was that the actual documentation has no correct sample. :)

hkantare pushed a commit that referenced this issue Oct 19, 2021
@hkantare
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation service/IAM Issues related to IAM
Projects
None yet
Development

No branches or pull requests

3 participants