Merge remote-tracking branch 'origin/sebastian/prod-2981-set-up-catal…

…og-pipeline' into sebastian/prod-2981-set-up-catalog-pipeline
pluralsh · Dec 13, 2024 · b34eec3 · b34eec3
2 parents 200d83f + 365daa9
commit b34eec3
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 9 deletions.
diff --git a/catalogs/data/airbyte/terraform/aws/iam.tf b/catalogs/data/airbyte/terraform/aws/iam.tf
@@ -8,7 +8,7 @@ resource "aws_iam_policy" "airbyte" {
 resource "aws_iam_user" "airbyte" {
   name = "${data.plural_cluster.cluster.name}-airbyte"
 
-  depends_on = [ plural_cluster.cluster ]
+  depends_on = [ data.plural_cluster.cluster ]
 }
 
 resource "aws_iam_access_key" "airbyte" {
@@ -33,5 +33,5 @@ resource "aws_iam_policy_attachment" "airbyte-user" {
   users = [aws_iam_user.airbyte.name]
   policy_arn = aws_iam_policy.airbyte.arn
 
-  depends_on = [ plural_cluster.cluster ]
+  depends_on = [ data.plural_cluster.cluster ]
 }
diff --git a/catalogs/data/airbyte/terraform/aws/postgres.tf b/catalogs/data/airbyte/terraform/aws/postgres.tf
@@ -9,7 +9,7 @@ resource "random_password" "password" {
 data "aws_eks_cluster" "mgmt" {
   name = data.plural_cluster.cluster.name
 
-  depends_on = [ plural_cluster.cluster ]
+  depends_on = [ data.plural_cluster.cluster ]
 }
 
 data "aws_vpc" "mgmt" {

diff --git a/catalogs/data/airbyte/terraform/aws/s3.tf b/catalogs/data/airbyte/terraform/aws/s3.tf
@@ -12,9 +12,3 @@ resource "aws_s3_bucket_server_side_encryption_configuration" "airbyte" {
     }
   }
 }
-
-data "aws_iam_role" "postgres" {
-  name = "${data.plural_cluster.cluster.name}-postgres"
-
-  depends_on = [ plural_cluster.cluster ]
-}