Skip to content

Commit 2575468

Browse files
authored
MAJOR: DEV-15150 remove unused variables (#28)
1 parent e5b6aee commit 2575468

File tree

5 files changed

+4
-22
lines changed

5 files changed

+4
-22
lines changed

CHANGELOG.md

+3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
# Tamr Terraform Template Repo
22

3+
## v4.0.0 - July 20th 2022
4+
* Remove unused variables security_group_name, vpc_id, and additional_cidrs
5+
36
## v3.1.2 - May 11th 2022
47
* Adds idle_in_transaction_session_timeout to parameter group
58

README.md

-3
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,6 @@ This terraform module will create:
5252
| rds\_subnet\_ids | VPC subnet IDs in subnet group | `list(string)` | n/a | yes |
5353
| security\_group\_ids | List of security group IDs to associate | `list(string)` | n/a | yes |
5454
| subnet\_group\_name | The name of the subnet group to add the RDS instance to | `string` | n/a | yes |
55-
| vpc\_id | VPC ID for the rds security group | `string` | n/a | yes |
56-
| additional\_cidrs | Additional CIDR to connect to RDS Postgres instance | `list(string)` | `[]` | no |
5755
| additional\_tags | [DEPRECATED: Use `tags` instead] Additional tags to set on the RDS instance. | `map(string)` | `{}` | no |
5856
| allocated\_storage | Allocate storage | `number` | `20` | no |
5957
| apply\_immediately | Apply immediately, do not set this to true for production | `bool` | `false` | no |
@@ -73,7 +71,6 @@ This terraform module will create:
7371
| parameter\_group\_family | The family of the DB parameter group | `string` | `"postgres12"` | no |
7472
| parameter\_group\_name | The name of the rds parameter group | `string` | `"rds-postgres-pg"` | no |
7573
| postgres\_name | The name of the postgres database to create on the DB instance | `string` | `"tamr_rds_db"` | no |
76-
| security\_group\_name | Name for the security group for the rds instance | `string` | `"tamr_rds_sg"` | no |
7774
| skip\_final\_snapshot | Skip final snapshot | `bool` | `true` | no |
7875
| storage\_type | Storage type (e.g. gp2, io1) | `string` | `"gp2"` | no |
7976
| tags | A map of tags to add to all resources. Replaces `additional_tags`. | `map(string)` | `{}` | no |

VERSION

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
3.1.2
1+
4.0.0

examples/minimal/main.tf

-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ module "rds_postgres" {
88
username = "exampleUsername"
99
password = "examplePassword" #tfsec:ignore:GEN003
1010

11-
vpc_id = var.vpc_id
1211
subnet_group_name = "${var.name_prefix}_example_subnet_group"
1312
# Network requirement: DB subnet group needs a subnet in at least two Availability Zones
1413
rds_subnet_ids = var.subnet_ids

variables.tf

-17
Original file line numberDiff line numberDiff line change
@@ -127,28 +127,11 @@ variable "tags" {
127127
default = {}
128128
}
129129

130-
variable "security_group_name" {
131-
description = "Name for the security group for the rds instance"
132-
type = string
133-
default = "tamr_rds_sg"
134-
}
135-
136130
variable "security_group_ids" {
137131
description = "List of security group IDs to associate"
138132
type = list(string)
139133
}
140134

141-
variable "vpc_id" {
142-
description = "VPC ID for the rds security group"
143-
type = string
144-
}
145-
146-
variable "additional_cidrs" {
147-
description = "Additional CIDR to connect to RDS Postgres instance"
148-
type = list(string)
149-
default = []
150-
}
151-
152135
variable "engine_version" {
153136
description = "Version of RDS Postgres"
154137
type = string

0 commit comments

Comments
 (0)