Skip to content
This repository was archived by the owner on Jan 16, 2025. It is now read-only.

Commit a8cba4e

Browse files
veronicatjannpalm
andauthored
fix: Add missing webhook_events_workflow_job_queue_policy to multi-runner 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 <[email protected]>
1 parent 0ddeefc commit a8cba4e

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

Diff for: modules/multi-runner/queues.tf

+6-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ data "aws_iam_policy_document" "deny_unsecure_transport" {
2626
}
2727
}
2828

29-
3029
resource "aws_sqs_queue" "queued_builds" {
3130
for_each = var.multi_runner_config
3231
name = "${var.prefix}-${each.key}-queued-builds${each.value.fifo ? ".fifo" : ""}"
@@ -71,6 +70,12 @@ resource "aws_sqs_queue_policy" "build_queue_dlq_policy" {
7170
policy = data.aws_iam_policy_document.deny_unsecure_transport.json
7271
}
7372

73+
resource "aws_sqs_queue_policy" "webhook_events_workflow_job_queue_policy" {
74+
count = var.enable_workflow_job_events_queue ? 1 : 0
75+
queue_url = aws_sqs_queue.webhook_events_workflow_job_queue[0].id
76+
policy = data.aws_iam_policy_document.deny_unsecure_transport.json
77+
}
78+
7479
resource "aws_sqs_queue" "webhook_events_workflow_job_queue" {
7580
count = var.enable_workflow_job_events_queue ? 1 : 0
7681
name = "${var.prefix}-webhook_events_workflow_job_queue"

0 commit comments

Comments
 (0)