Role Definition: support for Data Actions #1971
Conversation
``` $ acctests azurerm TestAccAzureRMRoleDefinition_complete === RUN TestAccAzureRMRoleDefinition_complete --- PASS: TestAccAzureRMRoleDefinition_complete (46.46s) PASS ok github.com/terraform-providers/terraform-provider-azurerm/azurerm 47.349s ``` Fixes #1538
There was a problem hiding this comment.
My only concern here is that both fields should IMO be TypeSet since ordering is not significant.
I don't know the API's behaviour but I worry that the current implementation would cause spurious diffs, hence I'm making it a blocker, although I understand this may need fixing in other places too.
|
Ignoring an existing failing test - the tests pass:
|
|
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks! |
ghost
locked and limited conversation to collaborators
Fixes #1538