Skip to content

fix(serverless): fix AWS Lambda deployment with Serverless Framework#453

Merged
Shillaker merged 1 commit intoBoavizta:mainfrom
Shillaker:fix/serverless-deployment
Jan 21, 2026
Merged

fix(serverless): fix AWS Lambda deployment with Serverless Framework#453
Shillaker merged 1 commit intoBoavizta:mainfrom
Shillaker:fix/serverless-deployment

Conversation

@Shillaker
Copy link
Copy Markdown
Collaborator

@Shillaker Shillaker commented Jan 21, 2026

Summary

This PR fixes the serverless deployment to AWS Lambda, which was previously broken (see #153).

Changes

  • Switched from serverless-python-requirements plugin to Docker-based deployment. Using layers and Python requirements exceeded the file size limit, and made the setup more complicated (e.g. needing to have an up-to-date requirements.txt).
  • Simplified serverless.yml and removed requirements.txt. Now that we're using the existing Dockerfile, the serverless.yml file is simpler, and we don't need the requirements.txt file (it is still needed for Clever Cloud, but we have a separate CI task that generates it on the fly).
  • Updated README to fix up Serverless instructions.

Demo

I deployed an instance to my AWS account here: https://k9wllbzcc2.execute-api.eu-west-1.amazonaws.com/

@Shillaker Shillaker requested a review from demeringo January 21, 2026 10:37
@Shillaker Shillaker force-pushed the fix/serverless-deployment branch from 6276478 to 91a6650 Compare January 21, 2026 10:41
Copy link
Copy Markdown
Collaborator

@jonperron jonperron left a comment

Choose a reason for hiding this comment

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

Lgtm from a code point of view 👍

@Shillaker Shillaker force-pushed the fix/serverless-deployment branch from 91a6650 to 269cce0 Compare January 21, 2026 13:50
@demeringo
Copy link
Copy Markdown
Collaborator

Thank you for this PR 💙

Copy link
Copy Markdown
Collaborator

@demeringo demeringo left a comment

Choose a reason for hiding this comment

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

Works like a charm 👍

@Shillaker Shillaker merged commit 43b0833 into Boavizta:main Jan 21, 2026
4 checks passed
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.

3 participants