Fix modules to support new Apigee X environment types (#1841)
* Update main.tf * Update variables.tf * Update main.tf Updated environment members to be alphabetical order * fixed linting and terraform fmt * removed venv * removed venv directory --------- Co-authored-by: Teo De Las Heras <teodlh@google.com>
This commit is contained in:
parent
a877f0541b
commit
0f502a8cfb
|
@ -1,3 +1,5 @@
|
||||||
|
venv/*
|
||||||
|
*/venv/*
|
||||||
**/.terraform
|
**/.terraform
|
||||||
**/terraform.tfstate*
|
**/terraform.tfstate*
|
||||||
**/terraform.tfvars
|
**/terraform.tfvars
|
||||||
|
|
|
@ -354,19 +354,18 @@ module "apigee" {
|
||||||
}
|
}
|
||||||
# tftest modules=1 resources=10
|
# tftest modules=1 resources=10
|
||||||
```
|
```
|
||||||
|
|
||||||
<!-- BEGIN TFDOC -->
|
<!-- BEGIN TFDOC -->
|
||||||
## Variables
|
## Variables
|
||||||
|
|
||||||
| name | description | type | required | default |
|
| name | description | type | required | default |
|
||||||
|---|---|:---:|:---:|:---:|
|
|---|---|:---:|:---:|:---:|
|
||||||
| [project_id](variables.tf#L125) | Project ID. | <code>string</code> | ✓ | |
|
| [project_id](variables.tf#L126) | Project ID. | <code>string</code> | ✓ | |
|
||||||
| [addons_config](variables.tf#L17) | Addons configuration. | <code title="object({ advanced_api_ops = optional(bool, false) api_security = optional(bool, false) connectors_platform = optional(bool, false) integration = optional(bool, false) monetization = optional(bool, false) })">object({…})</code> | | <code>null</code> |
|
| [addons_config](variables.tf#L17) | Addons configuration. | <code title="object({ advanced_api_ops = optional(bool, false) api_security = optional(bool, false) connectors_platform = optional(bool, false) integration = optional(bool, false) monetization = optional(bool, false) })">object({…})</code> | | <code>null</code> |
|
||||||
| [endpoint_attachments](variables.tf#L29) | Endpoint attachments. | <code title="map(object({ region = string service_attachment = string }))">map(object({…}))</code> | | <code>{}</code> |
|
| [endpoint_attachments](variables.tf#L29) | Endpoint attachments. | <code title="map(object({ region = string service_attachment = string }))">map(object({…}))</code> | | <code>{}</code> |
|
||||||
| [envgroups](variables.tf#L39) | Environment groups (NAME => [HOSTNAMES]). | <code>map(list(string))</code> | | <code>{}</code> |
|
| [envgroups](variables.tf#L39) | Environment groups (NAME => [HOSTNAMES]). | <code>map(list(string))</code> | | <code>{}</code> |
|
||||||
| [environments](variables.tf#L46) | Environments. | <code title="map(object({ display_name = optional(string) description = optional(string, "Terraform-managed") deployment_type = optional(string) api_proxy_type = optional(string) node_config = optional(object({ min_node_count = optional(number) max_node_count = optional(number) })) iam = optional(map(list(string)), {}) iam_bindings = optional(map(object({ role = string members = list(string) })), {}) iam_bindings_additive = optional(map(object({ role = string member = string })), {}) envgroups = optional(list(string), []) }))">map(object({…}))</code> | | <code>{}</code> |
|
| [environments](variables.tf#L46) | Environments. | <code title="map(object({ display_name = optional(string) description = optional(string, "Terraform-managed") deployment_type = optional(string) api_proxy_type = optional(string) type = optional(string) node_config = optional(object({ min_node_count = optional(number) max_node_count = optional(number) })) iam = optional(map(list(string)), {}) iam_bindings = optional(map(object({ role = string members = list(string) })), {}) iam_bindings_additive = optional(map(object({ role = string member = string })), {}) envgroups = optional(list(string), []) }))">map(object({…}))</code> | | <code>{}</code> |
|
||||||
| [instances](variables.tf#L72) | Instances ([REGION] => [INSTANCE]). | <code title="map(object({ name = optional(string) display_name = optional(string) description = optional(string, "Terraform-managed") runtime_ip_cidr_range = optional(string) troubleshooting_ip_cidr_range = optional(string) disk_encryption_key = optional(string) consumer_accept_list = optional(list(string)) enable_nat = optional(bool, false) environments = optional(list(string), []) }))">map(object({…}))</code> | | <code>{}</code> |
|
| [instances](variables.tf#L73) | Instances ([REGION] => [INSTANCE]). | <code title="map(object({ name = optional(string) display_name = optional(string) description = optional(string, "Terraform-managed") runtime_ip_cidr_range = optional(string) troubleshooting_ip_cidr_range = optional(string) disk_encryption_key = optional(string) consumer_accept_list = optional(list(string)) enable_nat = optional(bool, false) environments = optional(list(string), []) }))">map(object({…}))</code> | | <code>{}</code> |
|
||||||
| [organization](variables.tf#L97) | Apigee organization. If set to null the organization must already exist. | <code title="object({ display_name = optional(string) description = optional(string, "Terraform-managed") authorized_network = optional(string) runtime_type = optional(string, "CLOUD") billing_type = optional(string) database_encryption_key = optional(string) analytics_region = optional(string, "europe-west1") retention = optional(string) disable_vpc_peering = optional(bool, false) })">object({…})</code> | | <code>null</code> |
|
| [organization](variables.tf#L98) | Apigee organization. If set to null the organization must already exist. | <code title="object({ display_name = optional(string) description = optional(string, "Terraform-managed") authorized_network = optional(string) runtime_type = optional(string, "CLOUD") billing_type = optional(string) database_encryption_key = optional(string) analytics_region = optional(string, "europe-west1") retention = optional(string) disable_vpc_peering = optional(bool, false) })">object({…})</code> | | <code>null</code> |
|
||||||
|
|
||||||
## Outputs
|
## Outputs
|
||||||
|
|
||||||
|
|
|
@ -40,11 +40,11 @@ resource "google_apigee_envgroup" "envgroups" {
|
||||||
|
|
||||||
resource "google_apigee_environment" "environments" {
|
resource "google_apigee_environment" "environments" {
|
||||||
for_each = var.environments
|
for_each = var.environments
|
||||||
name = each.key
|
|
||||||
display_name = each.value.display_name
|
|
||||||
description = each.value.description
|
|
||||||
deployment_type = each.value.deployment_type
|
|
||||||
api_proxy_type = each.value.api_proxy_type
|
api_proxy_type = each.value.api_proxy_type
|
||||||
|
deployment_type = each.value.deployment_type
|
||||||
|
description = each.value.description
|
||||||
|
display_name = each.value.display_name
|
||||||
|
name = each.key
|
||||||
dynamic "node_config" {
|
dynamic "node_config" {
|
||||||
for_each = try(each.value.node_config, null) != null ? [""] : []
|
for_each = try(each.value.node_config, null) != null ? [""] : []
|
||||||
content {
|
content {
|
||||||
|
@ -53,6 +53,7 @@ resource "google_apigee_environment" "environments" {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
org_id = local.org_id
|
org_id = local.org_id
|
||||||
|
type = each.value.type
|
||||||
lifecycle {
|
lifecycle {
|
||||||
ignore_changes = [
|
ignore_changes = [
|
||||||
node_config["current_aggregate_node_count"]
|
node_config["current_aggregate_node_count"]
|
||||||
|
|
|
@ -50,6 +50,7 @@ variable "environments" {
|
||||||
description = optional(string, "Terraform-managed")
|
description = optional(string, "Terraform-managed")
|
||||||
deployment_type = optional(string)
|
deployment_type = optional(string)
|
||||||
api_proxy_type = optional(string)
|
api_proxy_type = optional(string)
|
||||||
|
type = optional(string)
|
||||||
node_config = optional(object({
|
node_config = optional(object({
|
||||||
min_node_count = optional(number)
|
min_node_count = optional(number)
|
||||||
max_node_count = optional(number)
|
max_node_count = optional(number)
|
||||||
|
|
Loading…
Reference in New Issue