aws-load-balancer-controller icon indicating copy to clipboard operation
aws-load-balancer-controller copied to clipboard

Merge ALB rules by the same host

Open michaelkoro opened this issue 2 years ago • 5 comments

Is your feature request related to a problem? We would like to be able to merge rules in the ingress resource, on the ALB level.

For example, when creating the following -

spec:
  rules:
    - http:
        paths:
          - path: /test1
            pathType: ImplementationSpecific
            backend:
              service:
                name: service_name
                port:
                  number: 80
          - path: /test2
            pathType: ImplementationSpecific
            backend:
              service:
                name: service_name
                port:
                  number: 80
    - http:
        paths:
          - path: /*
            pathType: ImplementationSpecific
            backend:
              service:
                name: service_name
                port:
                  number: 80

We want the controller to merge the two paths in the first rule, and create one rule in the ALB. We are aware of the conditions annotation, but unfortunately working with it dynamically is very complicated. Is there an easier way?

michaelkoro avatar Jun 06 '22 10:06 michaelkoro

@michaelkoro, we can optimize in limited situations - for consecutive rules so as to honor the ingress group order. Also the current ALB limit is max 5 conditions per rule.

Are you hitting the rules quotas or have other concerns?

/kind feature

kishorj avatar Aug 11 '22 22:08 kishorj

+1, maybe this feature can prevent hit AWS ELB listener rule api throttle

acgs771126 avatar Oct 24 '22 03:10 acgs771126

The Kubernetes project currently lacks enough contributors to adequately respond to all PRs.

This bot triages PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the PR is closed

You can:

  • Mark this PR as fresh with /remove-lifecycle stale
  • Close this PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Feb 08 '23 10:02 k8s-triage-robot

+1 would be great to see this in v2.6.0

benjefferies avatar Oct 03 '23 10:10 benjefferies