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

Build lambda-sqs-worker-cdk before deploy #428

Merged
merged 4 commits into from
May 24, 2021
Merged

Build lambda-sqs-worker-cdk before deploy #428

merged 4 commits into from
May 24, 2021

Conversation

damienung1
Copy link
Contributor

Why?
Prior to this change, the built code was packaged with the docker image. Since the image is ECR cached on package.json and yarn.lock, stale code will be deployed if there are no changes to either file.

What?
This change adds a build command before CDK deploy to ensure the latest built code gets picked up and deployed. It also removes the unnecessary code build in the image.

So that freshly built code will be deployed
@damienung1 damienung1 requested a review from a team as a code owner May 22, 2021 07:17
@changeset-bot
Copy link

changeset-bot bot commented May 22, 2021

🦋 Changeset detected

Latest commit: d79fe4b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
skuba Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Member

@72636c 72636c left a comment

Choose a reason for hiding this comment

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

Nice one, thanks

template/lambda-sqs-worker-cdk/Dockerfile Outdated Show resolved Hide resolved
@72636c 72636c changed the title build before deploy Build lambda-sqs-worker-cdk before deploy May 24, 2021
@72636c 72636c enabled auto-merge (squash) May 24, 2021 02:52
@72636c 72636c merged commit 8842f92 into master May 24, 2021
@72636c 72636c deleted the fix-code-caching branch May 24, 2021 02:55
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