-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Provide human understandable target group names #870
Comments
one option is by using the |
I have been planing to change the targetGroup name to k8s-{namespace:8}-{name:8}-{uuid:10}. For Ingress controller, the |
Is the {name} the name of the k8s service configured in the Ingress Controller? If so, that should be sufficient being able to identify what the target group is associated with.
… On Mar 8, 2019, at 5:02 PM, M00nF1sh ***@***.***> wrote:
I have been planing to change the targetGroup name to k8s-{namespace:8}-{name:8}-{uuid:10}.
uuid is hash consisted of clusterName, targetType, protocol, etc. Which should be good enough for general use cases.
(actually in my NLB PR kubernetes/kubernetes#74910, i did that name change for NLB targetGroups).
For Ingress controller, the --alb-name-prefix have forbids me to do so, needs to figure out the real use case of such flag.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Do we have a solution for this? |
@rverma-nikiai |
@M00nF1sh I would prefer k8-{ing/svc name}-{namespace}-{UUID:10}. Your suggestion is very good too. |
Also it would be nice that once we deploy the fix it should take care of deleting the existing random names and creating new target groups |
@M00nF1sh Looking forward to a release with this feature. Thanks |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Rotten issues close after 30d of inactivity. Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
@fejta-bot: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/reopen Please? |
@kerma: You can't reopen an issue/PR unless you authored it or you are a collaborator. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Custom ALB names are now supported via annotations. Can this be provided for target group names as well? Or at least use the same custom ALB name when creating the target group? |
Should we create a dedicated issue for the ability to name the target group as well? |
I would like to use CloudWatch to gather metrics dealing with various Target Groups behind an ELB but is very difficult to at-a-glance understand which Target Group is having issues due to the UUIDs/hashes being generated as the names currently. It would be nice to have some sort of configurable option to specify the name or at least a prefix to denote what Kubernetes service you are using (default value to a combination of servicer name + namespace would be nice).
The text was updated successfully, but these errors were encountered: