chore: use null_resource for flyway ecs task trigger

bcgov · Jan 27, 2025 · 72fe664 · 72fe664
1 parent e453cae
commit 72fe664
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 11 deletions.
diff --git a/infrastructure/migration/ecs.tf b/infrastructure/migration/ecs.tf
@@ -32,10 +32,6 @@ data "aws_ecs_cluster" "ecs_cluster" {
   cluster_name = "ecs-cluster-${var.app_name}"
 }
 
-resource "terraform_data" "trigger_deployment" {
-  input = "${timestamp()}"
-}
-
 module "flyway_task" {
   source = "../modules/ecs-flyway-task"
 

diff --git a/infrastructure/modules/ecs-flyway-task/main.tf b/infrastructure/modules/ecs-flyway-task/main.tf
@@ -1,3 +1,9 @@
+resource "null_resource" "trigger_deployment" {
+  triggers = {
+    deployment_timestamp = timestamp()
+  }
+}
+
 resource "aws_ecs_task_definition" "flyway_task" {
   family                   = "${var.app_name}-flyway-task"
   network_mode             = "awsvpc"
@@ -28,7 +34,7 @@ resource "aws_ecs_task_definition" "flyway_task" {
   ])
 
   lifecycle {
-    replace_triggered_by = [terraform_data.trigger_deployment]
+    replace_triggered_by = [null_resource.trigger_deployment]
   }
 
   provisioner "local-exec" {

diff --git a/infrastructure/modules/ecs-flyway-task/variables.tf b/infrastructure/modules/ecs-flyway-task/variables.tf
@@ -8,12 +8,6 @@ variable "flyway_image" {
   type        = string
 }
 
-variable "trigger_deployment" {
-  description = "Trigger for replacing the task definition"
-  type        = any
-  default     = null
-}
-
 variable "cpu" {
   description = "Task CPU"
   type        = string