add new parameters support in apigee module

modules/apigee/README.md

@@ -25,14 +25,18 @@ module "apigee" {
   }
   environments = {
     apis-test = {
-      display_name = "APIs test"
-      description  = "APIs Test"
-      envgroups    = ["test"]
+      display_name    = "APIs test"
+      description     = "APIs Test"
+      deployment_type = "ARCHIVE"
+      api_proxy_type  = "PROGRAMMABLE"
+      envgroups       = ["test"]
     }
     apis-prod = {
-      display_name = "APIs prod"
-      description  = "APIs prod"
-      envgroups    = ["prod"]
+      display_name    = "APIs prod"
+      description     = "APIs prod"
+      deployment_type = "PROXY"
+      api_proxy_type  = "CONFIGURABLE"
+      envgroups       = ["prod"]
       iam = {
         "roles/viewer" = ["group:devops@myorg.com"]
       }
@@ -169,12 +173,12 @@ module "apigee" {
 
 | name | description | type | required | default |
 |---|---|:---:|:---:|:---:|
-| [project_id](variables.tf#L75) | Project ID. | <code>string</code> | ✓ |  |
+| [project_id](variables.tf#L77) | Project ID. | <code>string</code> | ✓ |  |
 | [endpoint_attachments](variables.tf#L17) | Endpoint attachments. | <code title="map&#40;object&#40;&#123;&#10;  region             &#61; string&#10;  service_attachment &#61; string&#10;&#125;&#41;&#41;">map&#40;object&#40;&#123;&#8230;&#125;&#41;&#41;</code> |  | <code>null</code> |
 | [envgroups](variables.tf#L26) | Environment groups (NAME => [HOSTNAMES]). | <code>map&#40;list&#40;string&#41;&#41;</code> |  | <code>null</code> |
-| [environments](variables.tf#L32) | Environments. | <code title="map&#40;object&#40;&#123;&#10;  display_name &#61; optional&#40;string&#41;&#10;  description  &#61; optional&#40;string, &#34;Terraform-managed&#34;&#41;&#10;  node_config &#61; optional&#40;object&#40;&#123;&#10;    min_node_count &#61; optional&#40;number&#41;&#10;    max_node_count &#61; optional&#40;number&#41;&#10;  &#125;&#41;&#41;&#10;  iam       &#61; optional&#40;map&#40;list&#40;string&#41;&#41;&#41;&#10;  envgroups &#61; list&#40;string&#41;&#10;&#125;&#41;&#41;">map&#40;object&#40;&#123;&#8230;&#125;&#41;&#41;</code> |  | <code>null</code> |
-| [instances](variables.tf#L47) | Instances. | <code title="map&#40;object&#40;&#123;&#10;  display_name         &#61; optional&#40;string&#41;&#10;  description          &#61; optional&#40;string, &#34;Terraform-managed&#34;&#41;&#10;  region               &#61; string&#10;  environments         &#61; list&#40;string&#41;&#10;  psa_ip_cidr_range    &#61; string&#10;  disk_encryption_key  &#61; optional&#40;string&#41;&#10;  consumer_accept_list &#61; optional&#40;list&#40;string&#41;&#41;&#10;&#125;&#41;&#41;">map&#40;object&#40;&#123;&#8230;&#125;&#41;&#41;</code> |  | <code>null</code> |
-| [organization](variables.tf#L61) | Apigee organization. If set to null the organization must already exist. | <code title="object&#40;&#123;&#10;  display_name            &#61; optional&#40;string&#41;&#10;  description             &#61; optional&#40;string, &#34;Terraform-managed&#34;&#41;&#10;  authorized_network      &#61; optional&#40;string&#41;&#10;  runtime_type            &#61; optional&#40;string, &#34;CLOUD&#34;&#41;&#10;  billing_type            &#61; optional&#40;string&#41;&#10;  database_encryption_key &#61; optional&#40;string&#41;&#10;  analytics_region        &#61; optional&#40;string, &#34;europe-west1&#34;&#41;&#10;&#125;&#41;">object&#40;&#123;&#8230;&#125;&#41;</code> |  | <code>null</code> |
+| [environments](variables.tf#L32) | Environments. | <code title="map&#40;object&#40;&#123;&#10;  display_name    &#61; optional&#40;string&#41;&#10;  description     &#61; optional&#40;string, &#34;Terraform-managed&#34;&#41;&#10;  deployment_type &#61; optional&#40;string&#41;&#10;  api_proxy_type  &#61; optional&#40;string&#41;&#10;  node_config &#61; optional&#40;object&#40;&#123;&#10;    min_node_count &#61; optional&#40;number&#41;&#10;    max_node_count &#61; optional&#40;number&#41;&#10;  &#125;&#41;&#41;&#10;  iam       &#61; optional&#40;map&#40;list&#40;string&#41;&#41;&#41;&#10;  envgroups &#61; list&#40;string&#41;&#10;&#125;&#41;&#41;">map&#40;object&#40;&#123;&#8230;&#125;&#41;&#41;</code> |  | <code>null</code> |
+| [instances](variables.tf#L49) | Instances. | <code title="map&#40;object&#40;&#123;&#10;  display_name         &#61; optional&#40;string&#41;&#10;  description          &#61; optional&#40;string, &#34;Terraform-managed&#34;&#41;&#10;  region               &#61; string&#10;  environments         &#61; list&#40;string&#41;&#10;  psa_ip_cidr_range    &#61; string&#10;  disk_encryption_key  &#61; optional&#40;string&#41;&#10;  consumer_accept_list &#61; optional&#40;list&#40;string&#41;&#41;&#10;&#125;&#41;&#41;">map&#40;object&#40;&#123;&#8230;&#125;&#41;&#41;</code> |  | <code>null</code> |
+| [organization](variables.tf#L63) | Apigee organization. If set to null the organization must already exist. | <code title="object&#40;&#123;&#10;  display_name            &#61; optional&#40;string&#41;&#10;  description             &#61; optional&#40;string, &#34;Terraform-managed&#34;&#41;&#10;  authorized_network      &#61; optional&#40;string&#41;&#10;  runtime_type            &#61; optional&#40;string, &#34;CLOUD&#34;&#41;&#10;  billing_type            &#61; optional&#40;string&#41;&#10;  database_encryption_key &#61; optional&#40;string&#41;&#10;  analytics_region        &#61; optional&#40;string, &#34;europe-west1&#34;&#41;&#10;&#125;&#41;">object&#40;&#123;&#8230;&#125;&#41;</code> |  | <code>null</code> |
 
 ## Outputs
 

modules/apigee/main.tf

@@ -40,10 +40,12 @@ resource "google_apigee_envgroup" "envgroups" {
 }
 
 resource "google_apigee_environment" "environments" {
-  for_each     = local.environments
-  name         = each.key
-  display_name = each.value.display_name
-  description  = each.value.description
+  for_each        = local.environments
+  name            = each.key
+  display_name    = each.value.display_name
+  description     = each.value.description
+  deployment_type = each.value.deployment_type != null ? each.value.deployment_type : null
+  api_proxy_type  = each.value.api_proxy_type != null ? each.value.api_proxy_type : null
   dynamic "node_config" {
     for_each = try(each.value.node_config, null) != null ? [""] : []
     content {

modules/apigee/variables.tf

@@ -32,8 +32,10 @@ variable "envgroups" {
 variable "environments" {
   description = "Environments."
   type = map(object({
-    display_name = optional(string)
-    description  = optional(string, "Terraform-managed")
+    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)

tests/modules/apigee/fixture/test.env_only_with_api_proxy_type.tfvars

@@ -0,0 +1,13 @@
+project_id = "my-project"
+environments = {
+  apis-test = {
+    display_name   = "APIs test"
+    description    = "APIs Test"
+    api_proxy_type = "PROGRAMMABLE"
+    envgroups      = ["test"]
+    node_config = {
+      min_node_count = 2
+      max_node_count = 5
+    }
+  }
+}

tests/modules/apigee/fixture/test.env_only_with_deployment_type.tfvars

@@ -0,0 +1,13 @@
+project_id = "my-project"
+environments = {
+  apis-test = {
+    display_name    = "APIs test"
+    description     = "APIs Test"
+    deployment_type = "ARCHIVE"
+    envgroups       = ["test"]
+    node_config = {
+      min_node_count = 2
+      max_node_count = 5
+    }
+  }
+}

tests/modules/apigee/fixture/variables.tf

@@ -32,8 +32,10 @@ variable "envgroups" {
 variable "environments" {
   description = "Environments."
   type = map(object({
-    display_name = optional(string)
-    description  = optional(string, "Terraform-managed")
+    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)

tests/modules/apigee/test_plan.py

@@ -54,6 +54,18 @@ def test_env_only(plan_runner):
         'google_apigee_envgroup_attachment.envgroup_attachments': 1,
     }
 
+def test_env_only_with_deployment_type(plan_runner):
+    "Test that creates an environment in an existing environment group, with deployment_type set."
+    _, resources = plan_runner(tf_var_file='test.env_only_with_deployment_type.tfvars')
+    assert [r['values'].get('deployment_type') for r in resources
+            ] == [None, 'ARCHIVE']
+
+def test_env_only_with_api_proxy_type(plan_runner):
+    "Test that creates an environment in an existing environment group, with api_proxy_type set."
+    _, resources = plan_runner(tf_var_file='test.env_only_with_api_proxy_type.tfvars')
+    assert [r['values'].get('api_proxy_type') for r in resources
+            ] == [None, 'PROGRAMMABLE']
+
 def test_instance_only(plan_runner):
     "Test that creates only an instance."
     _, resources = plan_runner(tf_var_file='test.instance_only.tfvars')