-
Notifications
You must be signed in to change notification settings - Fork 7
[K9VULN-9412] [K9VULN-9413] docs(gcp): add gcp tf examples #239
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
Conversation
Co-authored-by: Moez Ezzeddine <[email protected]>
| description = "The Datadog site of your organization where scanner data will be sent (for example, datadoghq.com, datadoghq.eu, us3.datadoghq.com, us5.datadoghq.com, ap1.datadoghq.com, ddog-gov.com). See https://docs.datadoghq.com/getting_started/site/" | ||
| type = string | ||
| } | ||
|
|
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.
Got an error when trying to apply this terraform
╷
│ Error: Invalid count argument
│
│ on .terraform/modules/datadog_agentless_scanner/gcp/modules/vpc/main.tf line 6, in resource "random_id" "deployment_suffix":
│ 6: count = var.unique_suffix == "" ? 1 : 0
│
│ The "count" value depends on resource attributes that cannot be determined until apply, so Terraform cannot predict how many instances will be
│ created. To work around this, use the -target argument to first apply only the resources that the count depends on.
I fixed it by adding this here + passing the unique_suffix variable in main.tf (module datadog_agentless_scanner):
variable "unique_suffix" {
description = "Unique suffix to append to resource names. Must be alphanumeric only and maximum 6 characters (will be appended with region suffix)."
type = string
default = "foobar"
}
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.
This should fix the issue with TF plan
[K9VULN-8670] fix(gcp): fix gcp terraform plan
|
I merged the examples and the documentation. |
What?