feat: PITR Recovery Period in Days (#99)

magreenbaum · web-flow · commit ee48ced57afb · 2025-05-17T11:38:41.000+02:00
diff --git a/README.md b/README.md
@@ -68,13 +68,13 @@ Users of Terragrunt can achieve similar results by using modules provided in the
 | Name | Version |
 |------|---------|
 | <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.0 |
-| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.72.1 |
+| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.98 |
 
 ## Providers
 
 | Name | Version |
 |------|---------|
-| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 5.72.1 |
+| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 5.98 |
 
 ## Modules
 
@@ -118,6 +118,7 @@ No modules.
 | <a name="input_name"></a> [name](#input\_name) | Name of the DynamoDB table | `string` | `null` | no |
 | <a name="input_on_demand_throughput"></a> [on\_demand\_throughput](#input\_on\_demand\_throughput) | Sets the maximum number of read and write units for the specified on-demand table | `any` | `{}` | no |
 | <a name="input_point_in_time_recovery_enabled"></a> [point\_in\_time\_recovery\_enabled](#input\_point\_in\_time\_recovery\_enabled) | Whether to enable point-in-time recovery | `bool` | `false` | no |
+| <a name="input_point_in_time_recovery_period_in_days"></a> [point\_in\_time\_recovery\_period\_in\_days](#input\_point\_in\_time\_recovery\_period\_in\_days) | Number of preceding days for which continuous backups are taken and maintained. Default 35 | `number` | `null` | no |
 | <a name="input_range_key"></a> [range\_key](#input\_range\_key) | The attribute to use as the range (sort) key. Must also be defined as an attribute | `string` | `null` | no |
 | <a name="input_read_capacity"></a> [read\_capacity](#input\_read\_capacity) | The number of read units for this table. If the billing\_mode is PROVISIONED, this field should be greater than 0 | `number` | `null` | no |
 | <a name="input_replica_regions"></a> [replica\_regions](#input\_replica\_regions) | Region names for creating replicas for a global DynamoDB table. | `any` | `[]` | no |
diff --git a/examples/autoscaling/README.md b/examples/autoscaling/README.md
@@ -20,7 +20,7 @@ Note that this example may create resources which can cost money (AWS Elastic IP
 | Name | Version |
 |------|---------|
 | <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.0 |
-| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.72.1 |
+| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.98 |
 | <a name="requirement_random"></a> [random](#requirement\_random) | >= 2.0 |
 
 ## Providers
diff --git a/examples/autoscaling/versions.tf b/examples/autoscaling/versions.tf
@@ -4,7 +4,7 @@ terraform {
   required_providers {
     aws = {
       source  = "hashicorp/aws"
-      version = ">= 5.72.1"
+      version = ">= 5.98"
     }
     random = {
       source  = "hashicorp/random"
diff --git a/examples/basic/README.md b/examples/basic/README.md
@@ -20,7 +20,7 @@ Note that this example may create resources which can cost money (AWS Elastic IP
 | Name | Version |
 |------|---------|
 | <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.0 |
-| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.72.1 |
+| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.98 |
 | <a name="requirement_random"></a> [random](#requirement\_random) | >= 2.0 |
 
 ## Providers
diff --git a/examples/basic/versions.tf b/examples/basic/versions.tf
@@ -4,7 +4,7 @@ terraform {
   required_providers {
     aws = {
       source  = "hashicorp/aws"
-      version = ">= 5.72.1"
+      version = ">= 5.98"
     }
     random = {
       source  = "hashicorp/random"
diff --git a/examples/global-tables/README.md b/examples/global-tables/README.md
@@ -20,15 +20,15 @@ Note that this example may create resources which can cost money (AWS Elastic IP
 | Name | Version |
 |------|---------|
 | <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.0 |
-| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.72.1 |
+| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.98 |
 | <a name="requirement_random"></a> [random](#requirement\_random) | >= 2.0 |
 
 ## Providers
 
 | Name | Version |
 |------|---------|
-| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 5.72.1 |
-| <a name="provider_aws.euwest2"></a> [aws.euwest2](#provider\_aws.euwest2) | >= 5.72.1 |
+| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 5.98 |
+| <a name="provider_aws.euwest2"></a> [aws.euwest2](#provider\_aws.euwest2) | >= 5.98 |
 | <a name="provider_random"></a> [random](#provider\_random) | >= 2.0 |
 
 ## Modules
diff --git a/examples/global-tables/versions.tf b/examples/global-tables/versions.tf
@@ -4,7 +4,7 @@ terraform {
   required_providers {
     aws = {
       source  = "hashicorp/aws"
-      version = ">= 5.72.1"
+      version = ">= 5.98"
     }
     random = {
       source  = "hashicorp/random"
diff --git a/examples/s3-import/README.md b/examples/s3-import/README.md
@@ -20,7 +20,7 @@ Note that this example may create resources which can cost money (AWS Elastic IP
 | Name | Version |
 |------|---------|
 | <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.0 |
-| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.72.1 |
+| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.98 |
 | <a name="requirement_random"></a> [random](#requirement\_random) | >= 2.0 |
 
 ## Providers
diff --git a/examples/s3-import/versions.tf b/examples/s3-import/versions.tf
@@ -4,7 +4,7 @@ terraform {
   required_providers {
     aws = {
       source  = "hashicorp/aws"
-      version = ">= 5.72.1"
+      version = ">= 5.98"
     }
     random = {
       source  = "hashicorp/random"
diff --git a/main.tf b/main.tf
@@ -26,7 +26,8 @@ resource "aws_dynamodb_table" "this" {
   }
 
   point_in_time_recovery {
-    enabled = var.point_in_time_recovery_enabled
+    enabled                 = var.point_in_time_recovery_enabled
+    recovery_period_in_days = var.point_in_time_recovery_period_in_days
   }
 
   dynamic "attribute" {
@@ -166,7 +167,8 @@ resource "aws_dynamodb_table" "autoscaled" {
   }
 
   point_in_time_recovery {
-    enabled = var.point_in_time_recovery_enabled
+    enabled                 = var.point_in_time_recovery_enabled
+    recovery_period_in_days = var.point_in_time_recovery_period_in_days
   }
 
   dynamic "attribute" {
@@ -310,7 +312,8 @@ resource "aws_dynamodb_table" "autoscaled_gsi_ignore" {
   }
 
   point_in_time_recovery {
-    enabled = var.point_in_time_recovery_enabled
+    enabled                 = var.point_in_time_recovery_enabled
+    recovery_period_in_days = var.point_in_time_recovery_period_in_days
   }
 
   dynamic "attribute" {
diff --git a/variables.tf b/variables.tf
@@ -52,6 +52,12 @@ variable "point_in_time_recovery_enabled" {
   default     = false
 }
 
+variable "point_in_time_recovery_period_in_days" {
+  description = "Number of preceding days for which continuous backups are taken and maintained. Default 35"
+  type        = number
+  default     = null
+}
+
 variable "ttl_enabled" {
   description = "Indicates whether ttl is enabled"
   type        = bool
diff --git a/versions.tf b/versions.tf
@@ -4,7 +4,7 @@ terraform {
   required_providers {
     aws = {
       source  = "hashicorp/aws"
-      version = ">= 5.72.1"
+      version = ">= 5.98"
     }
   }
 }
diff --git a/wrappers/main.tf b/wrappers/main.tf
@@ -24,6 +24,7 @@ module "wrapper" {
   name                                  = try(each.value.name, var.defaults.name, null)
   on_demand_throughput                  = try(each.value.on_demand_throughput, var.defaults.on_demand_throughput, {})
   point_in_time_recovery_enabled        = try(each.value.point_in_time_recovery_enabled, var.defaults.point_in_time_recovery_enabled, false)
+  point_in_time_recovery_period_in_days = try(each.value.point_in_time_recovery_period_in_days, var.defaults.point_in_time_recovery_period_in_days, null)
   range_key                             = try(each.value.range_key, var.defaults.range_key, null)
   read_capacity                         = try(each.value.read_capacity, var.defaults.read_capacity, null)
   replica_regions                       = try(each.value.replica_regions, var.defaults.replica_regions, [])
diff --git a/wrappers/versions.tf b/wrappers/versions.tf
@@ -4,7 +4,7 @@ terraform {
   required_providers {
     aws = {
       source  = "hashicorp/aws"
-      version = ">= 5.72.1"
+      version = ">= 5.98"
     }
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@ terraform {`
`4`	`4`	`required_providers {`
`5`	`5`	`aws = {`
`6`	`6`	`source = "hashicorp/aws"`
`7`		`- version = ">= 5.72.1"`
	`7`	`+ version = ">= 5.98"`
`8`	`8`	`}`
`9`	`9`	`random = {`
`10`	`10`	`source = "hashicorp/random"`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,8 @@ resource "aws_dynamodb_table" "this" {`
`26`	`26`	`}`
`27`	`27`
`28`	`28`	`point_in_time_recovery {`
`29`		`- enabled = var.point_in_time_recovery_enabled`
	`29`	`+ enabled = var.point_in_time_recovery_enabled`
	`30`	`+ recovery_period_in_days = var.point_in_time_recovery_period_in_days`
`30`	`31`	`}`
`31`	`32`
`32`	`33`	`dynamic "attribute" {`
`@@ -166,7 +167,8 @@ resource "aws_dynamodb_table" "autoscaled" {`
`166`	`167`	`}`
`167`	`168`
`168`	`169`	`point_in_time_recovery {`
`169`		`- enabled = var.point_in_time_recovery_enabled`
	`170`	`+ enabled = var.point_in_time_recovery_enabled`
	`171`	`+ recovery_period_in_days = var.point_in_time_recovery_period_in_days`
`170`	`172`	`}`
`171`	`173`
`172`	`174`	`dynamic "attribute" {`
`@@ -310,7 +312,8 @@ resource "aws_dynamodb_table" "autoscaled_gsi_ignore" {`
`310`	`312`	`}`
`311`	`313`
`312`	`314`	`point_in_time_recovery {`
`313`		`- enabled = var.point_in_time_recovery_enabled`
	`315`	`+ enabled = var.point_in_time_recovery_enabled`
	`316`	`+ recovery_period_in_days = var.point_in_time_recovery_period_in_days`
`314`	`317`	`}`
`315`	`318`
`316`	`319`	`dynamic "attribute" {`