Remove extra env var

Author: bobbyiliev

README.md

@@ -100,21 +100,6 @@ disk_support_config = {
 
 The `materialize_instances` variable is a list of objects that define the configuration for each Materialize instance.
 
-### `environmentd_extra_env`
-
-Optional list of extra environment variables to pass to the `environmentd` container. This allows you to pass any additional configuration supported by Materialize.
-
-Each entry should be an object with `name` and `value` fields:
-
-```hcl
-environmentd_extra_env = [
-  {
-    name  = "MZ_LOG_FILTER"
-    value = "materialized::coord=debug"
-  }
-]
-```
-
 ### `environmentd_extra_args`
 
 Optional list of additional command-line arguments to pass to the `environmentd` container. This can be used to override default system parameters or enable specific features.
@@ -203,7 +188,7 @@ These flags configure default limits for clusters, connections, and tables. You
 | <a name="input_install_metrics_server"></a> [install\_metrics\_server](#input\_install\_metrics\_server) | Whether to install the metrics-server for the Materialize Console | `bool` | `true` | no |
 | <a name="input_kubernetes_namespace"></a> [kubernetes\_namespace](#input\_kubernetes\_namespace) | The Kubernetes namespace for the Materialize resources | `string` | `"materialize-environment"` | no |
 | <a name="input_log_group_name_prefix"></a> [log\_group\_name\_prefix](#input\_log\_group\_name\_prefix) | Prefix for the CloudWatch log group name (will be combined with environment name) | `string` | `"materialize"` | no |
-| <a name="input_materialize_instances"></a> [materialize\_instances](#input\_materialize\_instances) | Configuration for Materialize instances. Due to limitations in Terraform, `materialize_instances` cannot be defined on the first `terraform apply`. | <pre>list(object({<br/>    name                             = string<br/>    namespace                        = optional(string)<br/>    database_name                    = string<br/>    environmentd_version             = optional(string)<br/>    cpu_request                      = optional(string, "1")<br/>    memory_request                   = optional(string, "1Gi")<br/>    memory_limit                     = optional(string, "1Gi")<br/>    create_database                  = optional(bool, true)<br/>    create_nlb                       = optional(bool, true)<br/>    internal_nlb                     = optional(bool, true)<br/>    enable_cross_zone_load_balancing = optional(bool, true)<br/>    in_place_rollout                 = optional(bool, false)<br/>    request_rollout                  = optional(string)<br/>    force_rollout                    = optional(string)<br/>    balancer_memory_request          = optional(string, "256Mi")<br/>    balancer_memory_limit            = optional(string, "256Mi")<br/>    balancer_cpu_request             = optional(string, "100m")<br/>    license_key                      = optional(string)<br/>    environmentd_extra_env = optional(list(object({<br/>      name  = string<br/>      value = string<br/>    })), [])<br/>    environmentd_extra_args = optional(list(string), [])<br/>  }))</pre> | `[]` | no |
+| <a name="input_materialize_instances"></a> [materialize\_instances](#input\_materialize\_instances) | Configuration for Materialize instances. Due to limitations in Terraform, `materialize_instances` cannot be defined on the first `terraform apply`. | <pre>list(object({<br/>    name                             = string<br/>    namespace                        = optional(string)<br/>    database_name                    = string<br/>    environmentd_version             = optional(string)<br/>    cpu_request                      = optional(string, "1")<br/>    memory_request                   = optional(string, "1Gi")<br/>    memory_limit                     = optional(string, "1Gi")<br/>    create_database                  = optional(bool, true)<br/>    create_nlb                       = optional(bool, true)<br/>    internal_nlb                     = optional(bool, true)<br/>    enable_cross_zone_load_balancing = optional(bool, true)<br/>    in_place_rollout                 = optional(bool, false)<br/>    request_rollout                  = optional(string)<br/>    force_rollout                    = optional(string)<br/>    balancer_memory_request          = optional(string, "256Mi")<br/>    balancer_memory_limit            = optional(string, "256Mi")<br/>    balancer_cpu_request             = optional(string, "100m")<br/>    license_key                      = optional(string)<br/>    environmentd_extra_args          = optional(list(string), [])<br/>  }))</pre> | `[]` | no |
 | <a name="input_metrics_retention_days"></a> [metrics\_retention\_days](#input\_metrics\_retention\_days) | Number of days to retain CloudWatch metrics | `number` | `7` | no |
 | <a name="input_namespace"></a> [namespace](#input\_namespace) | Namespace for all resources, usually the organization or project name | `string` | n/a | yes |
 | <a name="input_network_id"></a> [network\_id](#input\_network\_id) | The ID of the VPC in which resources will be deployed. Only used if create\_vpc is false. | `string` | `""` | no |

docs/header.md

@@ -99,21 +99,6 @@ disk_support_config = {
 
 The `materialize_instances` variable is a list of objects that define the configuration for each Materialize instance.
 
-### `environmentd_extra_env`
-
-Optional list of extra environment variables to pass to the `environmentd` container. This allows you to pass any additional configuration supported by Materialize.
-
-Each entry should be an object with `name` and `value` fields:
-
-```hcl
-environmentd_extra_env = [
-  {
-    name  = "MZ_LOG_FILTER"
-    value = "materialized::coord=debug"
-  }
-]
-```
-
 ### `environmentd_extra_args`
 
 Optional list of additional command-line arguments to pass to the `environmentd` container. This can be used to override default system parameters or enable specific features.

examples/simple/main.tf

@@ -150,10 +150,6 @@ variable "materialize_instances" {
     balancer_memory_limit   = optional(string, "256Mi")
     balancer_cpu_request    = optional(string, "100m")
     license_key             = optional(string)
-    environmentd_extra_env = optional(list(object({
-      name  = string
-      value = string
-    })), [])
     environmentd_extra_args = optional(list(string), [])
   }))
   default = []

main.tf

@@ -267,7 +267,6 @@ locals {
       create_nlb                       = instance.create_nlb
       internal_nlb                     = instance.internal_nlb
       enable_cross_zone_load_balancing = instance.enable_cross_zone_load_balancing
-      environmentd_extra_env           = instance.environmentd_extra_env
       environmentd_extra_args          = instance.environmentd_extra_args
 
       metadata_backend_url = format(

variables.tf

@@ -367,11 +367,7 @@ variable "materialize_instances" {
     balancer_memory_limit            = optional(string, "256Mi")
     balancer_cpu_request             = optional(string, "100m")
     license_key                      = optional(string)
-    environmentd_extra_env = optional(list(object({
-      name  = string
-      value = string
-    })), [])
-    environmentd_extra_args = optional(list(string), [])
+    environmentd_extra_args          = optional(list(string), [])
   }))
   default = []