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

Load Balancer Name Clash #424

Open
lukasniemeier-zalando opened this issue Dec 12, 2016 · 3 comments
Open

Load Balancer Name Clash #424

lukasniemeier-zalando opened this issue Dec 12, 2016 · 3 comments

Comments

@lukasniemeier-zalando
Copy link
Contributor

Currently senza generates the load balancer name from stack name and version. In case the name is too long, the stack name component is cut to 32 chars, see https://github.com/zalando-stups/senza/blob/master/senza/components/elastic_load_balancer.py#L18

This can lead to a situation in which senza tries to create two load balancers for the different stacks with the same name.

senza could fallback to generate random name in case the name-version combo is already taken.

@hjacobs
Copy link
Contributor

hjacobs commented Dec 12, 2016

@lukasniemeier-zalando there is already a workaround for this by using the NameSuffix property on the LB component. Example: https://github.com/zalando-incubator/kubernetes-on-aws/blob/master/cluster/senza-definition.yaml#L99

@hjacobs
Copy link
Contributor

hjacobs commented Dec 12, 2016

I agree that Senza should handle this automatically (e.g. by adding a increasing number if more than one LB was defined).

@lukasniemeier-zalando
Copy link
Contributor Author

Using the NameSuffix property solves my issue, thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants