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

Resolve deployment timeout issues on Elastic Beanstalk #43

Merged
merged 3 commits into from
Apr 23, 2020
Merged

Conversation

liangyuanruo
Copy link
Contributor

@liangyuanruo liangyuanruo commented Apr 22, 2020

Problem

Deployments to Elastic Beanstalk were failing with an inexplicable timeout issue, preventing deployments to staging and production environments.

Solution

The initial suspect was cast on the Elastic Beanstalk version due to certain filesystem mounting warnings in eb-activity. However upgrading the EB platform did not resolve this issue as discovered by @yong-jie and @kylerwsm . Further investigation revealed that EB deployments have a deployment timeout that if exceeded, would terminate a deployment.

The solution was to move the image build step out of the EB deployment pipeline into the Travis pre-deploy pipeline instead. Images would be built in Travis and pushed to the AWS Elastic Container Registry before triggering a deploy in Elastic Beanstalk. EB would only have to download the image instead of spending long periods time building it during the deployment.

Improvements

Added a section on deploying with Travis to the README.

Tests

Repeated deploys of this branch to edge.

@liangyuanruo liangyuanruo requested a review from yong-jie April 22, 2020 10:41
Copy link
Member

@yong-jie yong-jie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified that it does indeed fix the issue. LGTM!

@liangyuanruo liangyuanruo merged commit 765d0e4 into develop Apr 23, 2020
@liangyuanruo liangyuanruo deleted the images branch April 23, 2020 03:36
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

Successfully merging this pull request may close these issues.

2 participants