Great project. Very much enjoy using it!
Is it possible to set the Load Balancers Name as it appears in the AWS Console as part of the manifests? We produce several of these LB's using Helm charts, the ingress names and namespaces are identical in each cluster, making it inconvenient to determine which LB is on which cluster. We're currently using the tags to find the right cluster but it would make things marginally easier.
Currently this is not possible. There is a controller parameter where you can set alb prefix: https://github.com/kubernetes-sigs/aws-alb-ingress-controller/blob/e5c5d6e293e2b81167791ca9b2cea39fc5af7078/internal/ingress/controller/config/config.go#L75
..but it's not really useful in case where you need to find the ALB programmatically.
I'm thinking of trying to take a stab at this, as the workarounds are not that nice. Would be good to get a confirmation from the maintainers that they'd accept the PR.
@M00nF1sh could you kindly chime in?
I'm thinking of trying to take a stab at this, as the workarounds are not that nice. Would be good to get a confirmation from the maintainers that they'd accept the PR.
I've been wondering about this too, is there a way the controller can write to the ingress status:
1.- ALB name.
2.- ALB arn.
maybe target group arn?
It will improve the UX, in our case we have large multi tenant clusters, our users don't have access to the AWS accounts, every time they need the ALB name or ARN they have to contact us.
If this is a desirable approach I might be able to submit a PR if @M00nF1sh doesn't beat me to :).
Thanks!
IMO it could be as simple as removing the prefix. The prefix is already an empty string by default. I think the code should avoid generating a prefix if it is by default not provided.
With "namespace" + "ingressName" + md5sum, it is already quite sufficient to avoid collision. Though I assume the reason to have a prefix is exactly to avoid a collision.. so maybe we will need to have some collision handling code.
Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale
/remove-lifecycle stale
This also has the potential to make consuming ALB logs and metrics way easier.
The whole problem boils down to a scenario that might be pretty common:
Hi guys,
with the new v2.0.0 controller, we have applied AWS tags to resources, which will allow to track resources.
Also, we removed support for name prefix and used a general pattern for load balancer name: "k8s-namespace-ingName-hash".
We'll add support to customize ALB name soon, tracking it at https://github.com/kubernetes-sigs/aws-load-balancer-controller/issues/1483
@jescarri we'll add ALB ARN into Ingress annotations, tracking it at https://github.com/kubernetes-sigs/aws-load-balancer-controller/issues/1573
closing this issue.
Thanks @M00nF1sh 馃槃
Most helpful comment
Currently this is not possible. There is a controller parameter where you can set alb prefix: https://github.com/kubernetes-sigs/aws-alb-ingress-controller/blob/e5c5d6e293e2b81167791ca9b2cea39fc5af7078/internal/ingress/controller/config/config.go#L75
..but it's not really useful in case where you need to find the ALB programmatically.
I'm thinking of trying to take a stab at this, as the workarounds are not that nice. Would be good to get a confirmation from the maintainers that they'd accept the PR.