Skip to content

Commit

Permalink
fix: Add missing webhook_events_workflow_job_queue_policy to multi-ru…
Browse files Browse the repository at this point in the history
…nner queue (#3848)

Add missing
[aws_sqs_queue_policy.webhook_events_workflow_job_queue_policy](https://github.com/philips-labs/terraform-aws-github-runner/blob/main/main.tf#L54-L58)
to multi-runner module queue.tf:


```
resource "aws_sqs_queue_policy" "webhook_events_workflow_job_queue_policy" {
  count     = var.enable_workflow_job_events_queue ? 1 : 0
  queue_url = aws_sqs_queue.webhook_events_workflow_job_queue[0].id
  policy    = data.aws_iam_policy_document.deny_unsecure_transport.json
}
```

---------

Co-authored-by: Niek Palm <npalm@users.noreply.github.com>
  • Loading branch information
veronicatjan and npalm committed Apr 18, 2024
1 parent 0ddeefc commit a8cba4e
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion modules/multi-runner/queues.tf
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ data "aws_iam_policy_document" "deny_unsecure_transport" {
}
}


resource "aws_sqs_queue" "queued_builds" {
for_each = var.multi_runner_config
name = "${var.prefix}-${each.key}-queued-builds${each.value.fifo ? ".fifo" : ""}"
Expand Down Expand Up @@ -71,6 +70,12 @@ resource "aws_sqs_queue_policy" "build_queue_dlq_policy" {
policy = data.aws_iam_policy_document.deny_unsecure_transport.json
}

resource "aws_sqs_queue_policy" "webhook_events_workflow_job_queue_policy" {
count = var.enable_workflow_job_events_queue ? 1 : 0
queue_url = aws_sqs_queue.webhook_events_workflow_job_queue[0].id
policy = data.aws_iam_policy_document.deny_unsecure_transport.json
}

resource "aws_sqs_queue" "webhook_events_workflow_job_queue" {
count = var.enable_workflow_job_events_queue ? 1 : 0
name = "${var.prefix}-webhook_events_workflow_job_queue"
Expand Down

0 comments on commit a8cba4e

Please sign in to comment.