UPD: Adding input of backup-rule-cron

modules/storage/aws-backup/README.md

@@ -32,9 +32,9 @@ No modules.
 
 | Name | Description | Type | Default | Required |
 |------|-------------|------|---------|:--------:|
-| backup-plan-cron | n/a | `string` | `"cron(0 20 * * ? *)"` | no |
+| backup-plan-name | Backup plan name | `string` | n/a | yes |
-| backup-plan-name | n/a | `any` | n/a | yes |
+| backup-plan-retention | Backup retention period | `number` | n/a | yes |
-| backup-plan-retention | n/a | `any` | n/a | yes |
+| backup-rule-cron | Backup rule cron expression | `string` | n/a | yes |
 | opt-in-aurora | Opt in audora backup | `bool` | n/a | yes |
 | opt-in-dynamodb | Opt in dynamodb backup | `bool` | n/a | yes |
 | opt-in-ebs | Opt in ebs backup | `bool` | n/a | yes |

modules/storage/aws-backup/main.tf

@@ -69,7 +69,7 @@ resource "aws_backup_plan" "ab-plan" {
   rule {
     rule_name         = var.backup-plan-name
     target_vault_name = each.value.name
-    schedule          = var.backup-plan-cron
+    schedule          = var.backup-rule-cron
     start_window      = 60
     completion_window = 240
 

modules/storage/aws-backup/variables.tf

@@ -1,12 +1,17 @@
-variable "backup-plan-name" {}
+variable "backup-plan-name" {
-variable "backup-plan-cron" {
   type        = string
-  default = "cron(0 20 * * ? *)"
+  description = "Backup plan name"
-  # cron(Minutes Hours Day-of-month Month Day-of-week Year)
 }
-variable "backup-plan-retention" {}
 
+variable "backup-rule-cron" {
+  type        = string
+  description = "Backup rule cron expression"
+}
 
+variable "backup-plan-retention" {
+  type        = number
+  description = "Backup retention period"
+}
 
 variable "opt-in-aurora" {
   type        = bool