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

ELB priority in use #324

Closed
AndreyMarchuk opened this issue Sep 5, 2018 · 7 comments
Closed

ELB priority in use #324

AndreyMarchuk opened this issue Sep 5, 2018 · 7 comments
Labels
hacktoberfest https://hacktoberfest.digitalocean.com/ help wanted

Comments

@AndreyMarchuk
Copy link
Contributor

On 'mu svc deploy devenv' ELB PriorityInUse issue happens.

ElbHttpsHostListenerRule (AWS::ElasticLoadBalancingV2::ListenerRule) CREATE_IN_PROGRESS Resource creation Initiated AwaitFinalStatus ▶ ERROR mu-service-XXX-devenv: ElbHttpHostListenerRule (AWS::ElasticLoadBalancingV2::ListenerRule) CREATE_FAILED Priority '26' is currently in use (Service: AmazonElasticLoadBalancingV2; Status Code: 400; Error Code: PriorityInUse; Request ID: 82936918-b09e-11e8-8810-afb2085f3d7e)

@brentley
Copy link
Contributor

brentley commented Sep 5, 2018

https://docs.aws.amazon.com/cli/latest/reference/elbv2/set-rule-priorities.html

...as long as there are no priority conflicts...

You can't duplicate priority values for rules. This error seems to indicate that there is already a rule given a priority of "26" so building a second rule with the same priority is failing.

@AndreyMarchuk
Copy link
Contributor Author

That is clear, but in source code it seems to be getting the largest rule id + 1.
So I wonder why this issue is happening.

@brentley
Copy link
Contributor

brentley commented Sep 6, 2018

Does it work if you configure the priority yourself?

@AndreyMarchuk
Copy link
Contributor Author

It was coming from config, commented out and it's fine.
Would it make sense to validate this within mu before CloudFormation fails on it?

@brentley
Copy link
Contributor

brentley commented Sep 7, 2018

The config is independent of the other potential projects that are deployed, or not yet deployed. Incrementing by one, or manually keeping track "architecturally" seems like the only options.

@AndreyMarchuk
Copy link
Contributor Author

Right, but it can fail faster if the check is added into mu where Rules are iterated. For example fail if some Rule.priority == workflow.priority even before CF stack is executed.

@cplee
Copy link
Contributor

cplee commented Oct 24, 2018

fixed in #373

@cplee cplee closed this as completed Oct 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest https://hacktoberfest.digitalocean.com/ help wanted
Projects
None yet
Development

No branches or pull requests

3 participants