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

Commit

Permalink
fix: use name instead of name prefix to avoid too long names (#4079)
Browse files Browse the repository at this point in the history
## Problme
In #4063 PR the EventBridge Schedule is introduces. For schedule groups
a name prefix is used. But since the name for the name prefix is already
unieque the name can be used instead of prefix to avoid too long prefix
names (30 chars plus).
  • Loading branch information
npalm authored Aug 19, 2024
1 parent a2fe8af commit 8bb6272
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions modules/runners/pool/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -199,8 +199,8 @@ resource "aws_iam_role" "scheduler" {
resource "aws_scheduler_schedule" "pool" {
for_each = { for i, v in var.config.pool : i => v }

name_prefix = "${var.config.prefix}-pool-${each.key}-rule"
group_name = aws_scheduler_schedule_group.pool.name
name = "${var.config.prefix}-pool-${each.key}-rule"
group_name = aws_scheduler_schedule_group.pool.name

flexible_time_window {
mode = "OFF"
Expand Down

0 comments on commit 8bb6272

Please sign in to comment.