fix: use count instead of for_each for responder_user_ids in pagerdut…

…y_team_membership( DEC-2926 ) (#28)
honestbank · Mar 20, 2023 · 3e2d9e0 · 3e2d9e0
1 parent 248c646
commit 3e2d9e0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/.github/workflows/semantic-pr.yaml b/.github/workflows/semantic-pr.yaml
@@ -1,4 +1,4 @@
-name: "Semantic Pull Request"
+name: "honest-public-semantic-pull-request-workflow"
 
 on:
   pull_request:
@@ -10,8 +10,8 @@ on:
 permissions: read-all
 
 jobs:
-  main:
-    name: Semantic Pull Request
+  semantic-pull-request:
+    name: semantic-pull-request
     runs-on: ubuntu-latest
     steps:
       - uses: amannn/action-semantic-pull-request@v4

diff --git a/modules/pagerduty-team/membership.tf b/modules/pagerduty-team/membership.tf
@@ -6,9 +6,9 @@ locals {
 }
 
 resource "pagerduty_team_membership" "responders" {
-  for_each = toset(var.responder_user_ids)
+  count = length(var.responder_user_ids)
 
-  user_id = each.key
+  user_id = var.responder_user_ids[count.index]
   team_id = pagerduty_team.team.id
   role    = local.CONST_RESPONDER_ROLE
 }