Skip to content

fix: Missing dependency when provisioning service with request scaling (off-ticket)#1289

Open
JohnStainsby wants to merge 1 commit intomainfrom
request-scaling-dependency-issue
Open

fix: Missing dependency when provisioning service with request scaling (off-ticket)#1289
JohnStainsby wants to merge 1 commit intomainfrom
request-scaling-dependency-issue

Conversation

@JohnStainsby
Copy link
Copy Markdown
Contributor

@JohnStainsby JohnStainsby commented Feb 17, 2026

  • Add dependency on dummy listener rule invocation when looking up load balancer for request scaling

This fixes an error in service Terraform when provisioning a service with request based autoscaling. Terraform tries to find the load balancer associated with the target group but this link isn't in place until the dummy listener rule lambda executes.

The order of execution should be:

Create target group -> Run lambda to associate with ALB -> Get ALB info -> Setup request autoscaling

Signed-off-by: DBT pre-commit check


Checklist:

Title:

Description:

  • Link to ticket included (unless it's a quick out of ticket thing)
  • Includes tests (or an explanation for why it doesn't)
  • If the work includes user interface changes, before and after screenshots included in description
  • Includes any applicable changes to the documentation in this code base
  • Includes link(s) to any applicable changes to the documentation in the DBT Platform Documentation (can be to a pull request)

Tasks:

Reviewer Checklist

  • I have reviewed the PR and ensured no secret values are present

… balancer for request scaling

Signed-off-by: DBT pre-commit check
@JohnStainsby JohnStainsby requested a review from a team as a code owner February 17, 2026 15:59
arn = one(aws_lb_target_group.target_group[0].load_balancer_arns)
depends_on = [
aws_lambda_invocation.dummy_listener_rule
]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could lines 369 and 370 be moved here instead?

aws_appautoscaling_policy.requests_autoscaling_policy depends on aws_lb.load_balancer, which if we now have the depends_on within the aws_alb block here removes the need for the depends_on within the aws_appautoscaling_policy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants