Merge pull request #5 from trussworks/am-updating-variables

removing .terraform directory and updated variable max_db_snapshot as optional

.circleci/config.yml

@@ -2,7 +2,7 @@ version: 2
 jobs:
   validate:
     docker:
-      - image: trussworks/circleci-docker-primary:e66fbea875bcb788b29b1b5f59142e8231961ec5
+      - image: trussworks/circleci-docker-primary:683f06a3c6b20ba8696a09632c1c26a532df5bf5
     steps:
       - checkout
       - restore_cache:

README.md

@@ -33,6 +33,12 @@ module "rds-snapshot-cleaner" {
 ```
 
 <!-- BEGINNING OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
+## Requirements
+
+| Name | Version |
+|------|---------|
+| terraform | >= 0.12 |
+
 ## Providers
 
 | Name | Version |
@@ -42,10 +48,10 @@ module "rds-snapshot-cleaner" {
 ## Inputs
 
 | Name | Description | Type | Default | Required |
-|------|-------------|------|---------|:-----:|
+|------|-------------|------|---------|:--------:|
 | cleaner\_db\_instance\_identifier | The RDS database instance identifier. | `string` | n/a | yes |
 | cleaner\_dry\_run | Don't make any changes and log what would have happened. | `string` | n/a | yes |
-| cleaner\_max\_db\_snapshot\_count | The maximum number of manual snapshots allowed. This takes precedence over -retention-days. | `string` | n/a | yes |
+| cleaner\_max\_db\_snapshot\_count | The maximum number of manual snapshots allowed. This takes precedence over -retention-days. | `string` | `""` | no |
 | cleaner\_retention\_days | The maximum retention age in days. | `string` | n/a | yes |
 | cloudwatch\_logs\_retention\_days | Number of days to keep logs in AWS CloudWatch. | `string` | `90` | no |
 | environment | Environment tag, e.g prod. | `any` | n/a | yes |

variables.tf

@@ -11,6 +11,7 @@ variable "cleaner_dry_run" {
 variable "cleaner_max_db_snapshot_count" {
   description = "The maximum number of manual snapshots allowed. This takes precedence over -retention-days."
   type        = string
+  default     = ""
 }
 
 variable "cleaner_retention_days" {
@@ -49,4 +50,3 @@ variable "version_to_deploy" {
   description = "The version the Lambda function to deploy."
   type        = string
 }
-