-
Notifications
You must be signed in to change notification settings - Fork 1.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
DLP inspect template resource not allowing "default" per_info_type limit #18149
Closed
AdamMAtWork opened this issue
May 15, 2024
· 1 comment
· Fixed by GoogleCloudPlatform/magic-modules#10818, hashicorp/terraform-provider-google-beta#7438, #18285 or GoogleCloudPlatform/terraform-google-conversion#2322
Closed
DLP inspect template resource not allowing "default" per_info_type limit #18149
AdamMAtWork opened this issue
May 15, 2024
· 1 comment
· Fixed by GoogleCloudPlatform/magic-modules#10818, hashicorp/terraform-provider-google-beta#7438, #18285 or GoogleCloudPlatform/terraform-google-conversion#2322
Labels
Comments
github-actions
bot
added
forward/review
In review; remove label to forward
service/dlp
labels
May 15, 2024
Received in buganizer on DLP end; will be fixing shortly |
This was referenced May 31, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Community Note
Terraform Version & Provider Version(s)
Terraform v1.8.3
on darwin_amd64
Affected Resource(s)
google_data_loss_prevention_inspect_template
Terraform Configuration
Debug Output
No response
Expected Behavior
According to the API documentaion for InfoTypeLimit:
This describes a method to set a default limit for all
info_type
entries.The google terraform provider includes the resource
google_data_loss_prevention_inspect_template
, which support theinspect_config.limits
block as an argument. Thislimits
block accepts amax_findings_per_info_type
block, which correlates to the above API object.Via the API explorer, I am able to create an inspect template with a
limits
block containing amaxFindingsPerInfoType
entry consisting of only amaxFindings
- no infoType is required. This matches the documentation.I'm expecting to be able to do the same via the terraform provider.
Actual Behavior
Rather than accept a
maxFindingsPerInfoType
limit with noinfo_type
block (to set the default per_info_type limit) the provider exits with the following errorThe terraform provider is not allowing me to set the default "per_info_type_limit".
Steps to reproduce
terraform plan
with the terraform config attached to this ticketImportant Factoids
No response
References
No response
b/341993420
The text was updated successfully, but these errors were encountered: