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

r/azurerm_xyz_policy_assignment add support for non_compliance_message #14518

Merged

Conversation

matt-FFFFFF
Copy link
Contributor

@matt-FFFFFF matt-FFFFFF commented Dec 7, 2021

This PR:

Acceptance tests

Acceptance Tests for Policies


❯ TF_ACC=1 /usr/local/go/bin/go test -v ./internal/services/policy -run=TestAccSubscriptionPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage -timeout 30m -ldflags="-X=github.com/hashicorp/terraform-provider-azurerm/version.ProviderVersion=acc"
=== RUN   TestAccSubscriptionPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage
=== PAUSE TestAccSubscriptionPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage
=== CONT  TestAccSubscriptionPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage
--- PASS: TestAccSubscriptionPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage (843.31s)
PASS
ok  	github.com/hashicorp/terraform-provider-azurerm/internal/services/policy	844.474s

❯ TF_ACC=1 /usr/local/go/bin/go test -v ./internal/services/policy -run=TestAccResourcePolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage -timeout 30m -ldflags="-X=github.com/hashicorp/terraform-provider-azurerm/version.ProviderVersion=acc"
=== RUN   TestAccResourcePolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage
=== PAUSE TestAccResourcePolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage
=== CONT  TestAccResourcePolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage
--- PASS: TestAccResourcePolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage (860.95s)
PASS
ok  	github.com/hashicorp/terraform-provider-azurerm/internal/services/policy	862.068s

❯ TF_ACC=1 /usr/local/go/bin/go test -v ./internal/services/policy -run=TestAccResourceGroupPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage -timeout 30m -ldflags="-X=github.com/hashicorp/terraform-provider-azurerm/version.ProviderVersion=acc"
=== RUN   TestAccResourceGroupPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage
=== PAUSE TestAccResourceGroupPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage
=== CONT  TestAccResourceGroupPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage
--- PASS: TestAccResourceGroupPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage (848.64s)
PASS
ok  	github.com/hashicorp/terraform-provider-azurerm/internal/services/policy	850.804s

❯ TF_ACC=1 /usr/local/go/bin/go test -v ./internal/services/policy -run=TestAccManagementGroupPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage -timeout 30m -ldflags="-X=github.com/hashicorp/terraform-provider-azurerm/version.ProviderVersion=acc"
=== RUN   TestAccManagementGroupPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage
=== PAUSE TestAccManagementGroupPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage
=== CONT  TestAccManagementGroupPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage
--- PASS: TestAccManagementGroupPolicyAssignment_basicWithBuiltInPolicyNonComplianceMessage (689.18s)
PASS
ok  	github.com/hashicorp/terraform-provider-azurerm/internal/services/policy	690.368s

Acceptance Tests for Policy Sets


❯ TF_ACC=1 /usr/local/go/bin/go test -v ./internal/services/policy -run=TestAccSubscriptionPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage -timeout 30m -ldflags="-X=github.com/hashicorp/terraform-provider-azurerm/version.ProviderVersion=acc"
=== RUN   TestAccSubscriptionPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage
=== PAUSE TestAccSubscriptionPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage
=== CONT  TestAccSubscriptionPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage
--- PASS: TestAccSubscriptionPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage (819.12s)
PASS
ok  	github.com/hashicorp/terraform-provider-azurerm/internal/services/policy	820.257s

❯ TF_ACC=1 /usr/local/go/bin/go test -v ./internal/services/policy -run=TestAccResourcePolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage -timeout 30m -ldflags="-X=github.com/hashicorp/terraform-provider-azurerm/version.ProviderVersion=acc"
=== RUN   TestAccResourcePolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage
=== PAUSE TestAccResourcePolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage
=== CONT  TestAccResourcePolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage
--- PASS: TestAccResourcePolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage (1052.03s)
PASS
ok  	github.com/hashicorp/terraform-provider-azurerm/internal/services/policy	1056.193s

❯ TF_ACC=1 /usr/local/go/bin/go test -v ./internal/services/policy -run=TestAccResourceGroupPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage -timeout 30m -ldflags="-X=github.com/hashicorp/terraform-provider-azurerm/version.ProviderVersion=acc"
=== RUN   TestAccResourceGroupPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage
=== PAUSE TestAccResourceGroupPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage
=== CONT  TestAccResourceGroupPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage
--- PASS: TestAccResourceGroupPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage (949.83s)
PASS
ok  	github.com/hashicorp/terraform-provider-azurerm/internal/services/policy	952.996s

❯ TF_ACC=1 /usr/local/go/bin/go test -v ./internal/services/policy -run=TestAccManagementGroupPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage -timeout 30m -ldflags="-X=github.com/hashicorp/terraform-provider-azurerm/version.ProviderVersion=acc"
=== RUN   TestAccManagementGroupPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage
=== PAUSE TestAccManagementGroupPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage
=== CONT  TestAccManagementGroupPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage
--- PASS: TestAccManagementGroupPolicyAssignment_basicWithBuiltInPolicySetNonComplianceMessage (943.61s)
PASS
ok  	github.com/hashicorp/terraform-provider-azurerm/internal/services/policy	945.769s

The following has been done:

  • Documentation updated
  • Acceptance tests updated
  • Linting passed

@matt-FFFFFF matt-FFFFFF force-pushed the azure-policy-noncompliancemessages branch from 020f8a6 to 596aab0 Compare December 7, 2021 15:09
@matt-FFFFFF matt-FFFFFF changed the title Add Azure Policy Non-Compliance Messages to azurerm_xyz_policy_assignment r/azurerm_xyz_policy_assignment add non_compliance_message Dec 7, 2021
@matt-FFFFFF matt-FFFFFF changed the title r/azurerm_xyz_policy_assignment add non_compliance_message r/azurerm_xyz_policy_assignment add support for non_compliance_message Dec 7, 2021
Copy link
Collaborator

@katbyte katbyte left a comment

Choose a reason for hiding this comment

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

Thanks @matt-FFFFFF - couple comments about the schema whilst passing through

@matt-FFFFFF
Copy link
Contributor Author

Thanks @matt-FFFFFF - couple comments about the schema whilst passing through

Hi @katbyte,

I've implemented the changes now and the checks pass. Are you able to take another look please?

😄

Copy link
Member

@mbfrahry mbfrahry left a comment

Choose a reason for hiding this comment

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

LGTM! The tests all pass and I really appreciate testing across all the policies we are touching here. Thanks @matt-FFFFFF!

@mbfrahry mbfrahry added this to the v2.90.0 milestone Dec 16, 2021
@mbfrahry mbfrahry merged commit 4d1f858 into hashicorp:main Dec 16, 2021
mbfrahry added a commit that referenced this pull request Dec 16, 2021
@github-actions
Copy link

This functionality has been released in v2.90.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

@matt-FFFFFF matt-FFFFFF deleted the azure-policy-noncompliancemessages branch December 21, 2021 17:25
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for nonComplianceMessages in azurerm_policy_assignment
3 participants