Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ALB weighted target groups #11477

Closed
houstonj1 opened this issue Jan 4, 2020 · 6 comments
Closed

ALB weighted target groups #11477

houstonj1 opened this issue Jan 4, 2020 · 6 comments
Labels
enhancement Requests to existing resources that expand the functionality or scope. service/elbv2 Issues and PRs that pertain to the elbv2 service.

Comments

@houstonj1
Copy link

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

There are new features on ALBs that allow for multiple target groups on a listener with a weighted priority. The potential configuration would allow for multiple action blocks on a aws_lb_listener_rule with a priority included.

AWS Reference: https://docs.aws.amazon.com/elasticloadbalancing/latest/application/load-balancer-target-groups.html

New or Affected Resource(s)

  • aws_lb_listener_rule

Potential Terraform Configuration

resource "aws_alb_listener_rule" "my_lb_listener_rule" {
  listener_arn = ...
  priority     = ...

  action {
    type             = "forward"
    target_group_arn = arn_1
    priority = 8                            # 80% to target 1
  }

  action {
    type             = "forward"
    target_group_arn = arn_2
    priority = 2                            # 20% to target 2
  }

  condition {
    ....
  }
}

References

@houstonj1 houstonj1 added the enhancement Requests to existing resources that expand the functionality or scope. label Jan 4, 2020
@ghost ghost added the service/elbv2 Issues and PRs that pertain to the elbv2 service. label Jan 4, 2020
@github-actions github-actions bot added the needs-triage Waiting for first response or review from a maintainer. label Jan 4, 2020
@DrMerlin
Copy link

DrMerlin commented Jan 6, 2020

Using 8 for 80% and 2 for 20% seems odd to me. Is that from the AWS API? Or just something that seemed good to you, James?

@houstonj1
Copy link
Author

Using 8 for 80% and 2 for 20% seems odd to me. Is that from the AWS API? Or just something that seemed good to you, James?

Great question @DrMerlin! AWS offers values 0-999 for target weights, so there is a wide range of granularity to use. My example above was for simplicity :)

@bflad
Copy link
Contributor

bflad commented Jan 6, 2020

Hi @houstonj1 👋 Thank you for submitting this. This appears to be a duplicate or very similar to #10942, so to consolidate any discussions and efforts, I'm going to close this issue in preference of the earlier one. Please upvote and follow that issue for further updates.

@bflad bflad closed this as completed Jan 6, 2020
@houstonj1
Copy link
Author

houstonj1 commented Jan 6, 2020

Thanks @bflad, I see there was a PR merged related to that, which was closed. The purpose of this was to make it visible again. Thank you!

@bflad
Copy link
Contributor

bflad commented Jan 6, 2020

#10942 is still open -- one of the referenced pull requests (#8268) was indeed merged with similar, but unrelated functionality.

@ghost
Copy link

ghost commented Mar 27, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!

@ghost ghost locked and limited conversation to collaborators Mar 27, 2020
@breathingdust breathingdust removed the needs-triage Waiting for first response or review from a maintainer. label Sep 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Requests to existing resources that expand the functionality or scope. service/elbv2 Issues and PRs that pertain to the elbv2 service.
Projects
None yet
Development

No branches or pull requests

4 participants