Revert "Added input variable for postgres version"

Author: ashwinimanoj

README.md

@@ -86,8 +86,6 @@ aws ssm put-parameter --name "/rds/POSTGRES_DB_NAME" --value "value" --type "Sec
 | <a name="input_multi_az"></a> [multi\_az](#input\_multi\_az) | Specifies if the RDS instance is multi-AZ | `bool` | `false` | no |
 | <a name="input_performance_insights_enabled"></a> [performance\_insights\_enabled](#input\_performance\_insights\_enabled) | Whether to enable performance insights | `bool` | `true` | no |
 | <a name="input_performance_insights_retention_period"></a> [performance\_insights\_retention\_period](#input\_performance\_insights\_retention\_period) | The retention period for performance insights | `number` | `7` | no |
-| <a name="input_postgres_engine_version"></a> [postgres\_engine\_version](#input\_postgres\_engine\_version) | PostgreSQL engine version for the RDS instance (e.g., 15.4, 16.3). Defaults to latest supported. | `number` | `16.3` | no |
-| <a name="input_postgres_major_engine_version"></a> [postgres\_major\_engine\_version](#input\_postgres\_major\_engine\_version) | Major PostgreSQL engine version (e.g., 15, 16). Used for parameter group family naming. | `number` | `16` | no |
 | <a name="input_private_subnet_ids"></a> [private\_subnet\_ids](#input\_private\_subnet\_ids) | List of private subnet IDs for database and Kong ECS deployment | `list(string)` | n/a | yes |
 | <a name="input_public_subnet_ids"></a> [public\_subnet\_ids](#input\_public\_subnet\_ids) | List of public subnet IDs for public-facing load balancers | `list(string)` | n/a | yes |
 | <a name="input_rds_db_tags"></a> [rds\_db\_tags](#input\_rds\_db\_tags) | List of tags | `map(string)` | `{}` | no |

examples/complete/README.md

@@ -88,8 +88,6 @@ No resources.
 | <a name="input_multi_az"></a> [multi\_az](#input\_multi\_az) | Specifies if the RDS instance is multi-AZ | `bool` | n/a | yes |
 | <a name="input_performance_insights_enabled"></a> [performance\_insights\_enabled](#input\_performance\_insights\_enabled) | Whether to enable performance insights | `bool` | n/a | yes |
 | <a name="input_performance_insights_retention_period"></a> [performance\_insights\_retention\_period](#input\_performance\_insights\_retention\_period) | The retention period for performance insights | `number` | n/a | yes |
-| <a name="input_postgres_engine_version"></a> [postgres\_engine\_version](#input\_postgres\_engine\_version) | The version of the Postgres engine | `number` | n/a | yes |
-| <a name="input_postgres_major_engine_version"></a> [postgres\_major\_engine\_version](#input\_postgres\_major\_engine\_version) | The major version of the Postgres engine | `number` | n/a | yes |
 | <a name="input_private_subnet_ids"></a> [private\_subnet\_ids](#input\_private\_subnet\_ids) | List of private subnet IDs | `list(string)` | n/a | yes |
 | <a name="input_public_subnet_ids"></a> [public\_subnet\_ids](#input\_public\_subnet\_ids) | List of public subnet IDs | `list(string)` | n/a | yes |
 | <a name="input_rds_db_tags"></a> [rds\_db\_tags](#input\_rds\_db\_tags) | List of tags | `map(string)` | n/a | yes |

examples/complete/main.tf

@@ -28,6 +28,4 @@ module "kong" {
   memory_for_kong_task           = var.memory_for_kong_task
   desired_count_for_kong_service = var.desired_count_for_kong_service
   force_new_deployment           = var.force_new_deployment
-  postgres_engine_version        = var.postgres_engine_version
-  postgres_major_engine_version  = var.postgres_major_engine_version
 }

examples/complete/variables.tf

@@ -122,13 +122,3 @@ variable "force_new_deployment" {
   description = "Whether to force new deployment"
   type        = bool
 }
-
-variable "postgres_engine_version" {
-  description = "The version of the Postgres engine"
-  type        = number
-}
-
-variable "postgres_major_engine_version" {
-  description = "The major version of the Postgres engine"
-  type        = number
-}

locals.tf

@@ -14,16 +14,15 @@ locals {
     engine               = "postgres"
     storage_encrypted    = true
     storage_type         = "gp3"
-    engine_version       = var.postgres_engine_version
-    engine_family        = "postgres${var.postgres_major_engine_version}"
-    major_engine_version = var.postgres_major_engine_version
-
-    port              = 5432
-    sg_name           = "kong-postgres"
-    sg_description    = "Allow all traffic within vpc"
-    postgres_username = data.aws_ssm_parameter.rds["POSTGRES_USERNAME"].value
-    postgres_password = data.aws_ssm_parameter.rds["POSTGRES_PASSWORD"].value
-    postgres_db_name  = data.aws_ssm_parameter.rds["POSTGRES_DB_NAME"].value
+    engine_version       = 16.3
+    engine_family        = "postgres16"
+    major_engine_version = 16
+    port                 = 5432
+    sg_name              = "kong-postgres"
+    sg_description       = "Allow all traffic within vpc"
+    postgres_username    = data.aws_ssm_parameter.rds["POSTGRES_USERNAME"].value
+    postgres_password    = data.aws_ssm_parameter.rds["POSTGRES_PASSWORD"].value
+    postgres_db_name     = data.aws_ssm_parameter.rds["POSTGRES_DB_NAME"].value
   }
 
   ecs = {

variables.tf

@@ -161,23 +161,3 @@ variable "force_new_deployment" {
   type        = bool
   default     = true
 }
-
-variable "postgres_engine_version" {
-  description = "PostgreSQL engine version for the RDS instance (e.g., 15.4, 16.3). Defaults to latest supported."
-  type        = number
-  default     = 16.3
-  validation {
-    condition     = var.postgres_engine_version >= 16
-    error_message = "The PostgreSQL engine version must be 16 or higher."
-  }
-}
-
-variable "postgres_major_engine_version" {
-  description = "Major PostgreSQL engine version (e.g., 15, 16). Used for parameter group family naming."
-  type        = number
-  default     = 16
-  validation {
-    condition     = var.postgres_major_engine_version >= 16
-    error_message = "The major PostgreSQL engine version must be 16 or higher."
-  }
-}