Merge pull request #70 from observeinc/OB-20481-tf-initial-snapshot

feat: add initial snapshot invoke for lambda

modules/snapshot/README.md

@@ -115,6 +115,7 @@ No modules.
 | [aws_cloudwatch_event_target.target](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/cloudwatch_event_target) | resource |
 | [aws_iam_policy.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/iam_policy) | resource |
 | [aws_iam_role_policy_attachment.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/iam_role_policy_attachment) | resource |
+| [aws_lambda_invocation.snapshot](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/lambda_invocation) | resource |
 | [aws_lambda_permission.this](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/lambda_permission) | resource |
 | [aws_arn.function](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/arn) | data source |
 | [aws_arn.role](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/arn) | data source |
@@ -130,6 +131,7 @@ No modules.
 | <a name="input_exclude"></a> [exclude](#input\_exclude) | List of actions to exclude from being executed on snapshot request. | `list(string)` | `[]` | no |
 | <a name="input_iam_name_prefix"></a> [iam\_name\_prefix](#input\_iam\_name\_prefix) | Prefix used for all created IAM roles and policies | `string` | `""` | no |
 | <a name="input_include"></a> [include](#input\_include) | List of actions to include in snapshot request. | `list(string)` | `[]` | no |
+| <a name="input_invoke_snapshot_on_start_enabled"></a> [invoke\_snapshot\_on\_start\_enabled](#input\_invoke\_snapshot\_on\_start\_enabled) | Toggle invocation of snapshot from Cloudformation. | `string` | `true` | no |
 | <a name="input_lambda"></a> [lambda](#input\_lambda) | Observe Lambda module | <pre>object({<br>    lambda_function = object({<br>      arn  = string<br>      role = string<br>    })<br>  })</pre> | n/a | yes |
 | <a name="input_overrides"></a> [overrides](#input\_overrides) | List of configuration overrides. | <pre>list(object({<br>    action = string<br>    config = map(any)<br>  }))</pre> | `[]` | no |
 | <a name="input_resources"></a> [resources](#input\_resources) | List of resources to scope policy to. | `list(string)` | <pre>[<br>  "*"<br>]</pre> | no |

modules/snapshot/main.tf

@@ -60,3 +60,12 @@ resource "aws_lambda_permission" "this" {
   function_name       = trimprefix(data.aws_arn.function.resource, "function:")
   source_arn          = aws_cloudwatch_event_rule.trigger.arn
 }
+
+resource "aws_lambda_invocation" "snapshot" {
+  count = var.invoke_snapshot_on_start_enabled ? 1 : 0
+  # Note: Use function name from the arn to prevent requiring function_name
+  function_name = trimprefix(data.aws_arn.function.resource, "function:")
+
+  input = aws_cloudwatch_event_target.target.input
+}
+

modules/snapshot/variables.tf

@@ -31,11 +31,11 @@ variable "eventbridge_name_prefix" {
 
 variable "action" {
   description = <<-EOF
-    List of actions allowed by policy and periodically triggered. By default,
-    this list contains all policies which the lambda can act upon. You should
-    only override this list if you do not want to execute more actions as they
-    become available in future lambda upgrades. If you instead wish to extend
-    this list, or ignore a subset of actions, use \"include\" and \"exclude\".
+  List of actions allowed by policy and periodically triggered. By default,
+  this list contains all policies which the lambda can act upon. You should
+  only override this list if you do not want to execute more actions as they
+  become available in future lambda upgrades. If you instead wish to extend
+  this list, or ignore a subset of actions, use \"include\" and \"exclude\".
   EOF
   type        = list(string)
   nullable    = false
@@ -130,3 +130,10 @@ variable "eventbridge_schedule_expression" {
   nullable    = false
   default     = "rate(3 hours)"
 }
+
+variable "invoke_snapshot_on_start_enabled" {
+  description = "Toggle invocation of snapshot from Cloudformation."
+  type        = string
+  nullable    = false
+  default     = true
+}