-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
New Resource: azurerm_policy_assignment
#1051
Conversation
``` $ acctests azurerm TestAccAzureRMPolicyAssignment_ === RUN TestAccAzureRMPolicyAssignment_basic --- PASS: TestAccAzureRMPolicyAssignment_basic (262.01s) === RUN TestAccAzureRMPolicyAssignment_complete --- PASS: TestAccAzureRMPolicyAssignment_complete (257.42s) PASS ok github.com/terraform-providers/terraform-provider-azurerm/azurerm 519.467s ```
``` $ acctests azurerm TestAccAzureRMPolicyDefinition_ === RUN TestAccAzureRMPolicyDefinition_importBasic --- PASS: TestAccAzureRMPolicyDefinition_importBasic (116.69s) === RUN TestAccAzureRMPolicyDefinition_basic --- PASS: TestAccAzureRMPolicyDefinition_basic (112.03s) PASS ok github.com/terraform-providers/terraform-provider-azurerm/azurerm 228.755s ```
Policy Definitions tests pass:
|
``` $ acctests azurerm TestAccAzureRMPolicyAssignment_importComplete === RUN TestAccAzureRMPolicyAssignment_importComplete --- PASS: TestAccAzureRMPolicyAssignment_importComplete (300.40s) PASS ok github.com/terraform-providers/terraform-provider-azurerm/azurerm 300.433s ```
Import tests pass:
|
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.
👍 Looks good.
Thanks for this usefull resource. By replacing this sentence
By
|
Hello @JamesDLD, Do you have a reference for this information? In the portal I can only select a subscription or resource group and a quick google doesn't find anything. Thanks! |
Hello @katbyte The First samples shows how to assign a policy to a Resource type : Resource Group. Here I did it on several objects like : Resource Group, Virtual Network, Recovery Services Vault : https://github.com/JamesDLD/terraform |
@katbyte @JamesDLD in this case I'd suggest we say @JamesDLD would you mind opening a new issue specifically for this? Thanks! :) |
Look great to me @tombuildsstuff @katbyte , just did create this ticket #1420 accordingly. |
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 👉 [email protected]. Thanks! |
Fixes #46