Skip to content
This repository was archived by the owner on Jan 15, 2023. It is now read-only.

Adding AWS Lambda Builders library to Build images#128

Closed
sanathkr wants to merge 1 commit intolambci:masterfrom
sanathkr:builderlib
Closed

Adding AWS Lambda Builders library to Build images#128
sanathkr wants to merge 1 commit intolambci:masterfrom
sanathkr:builderlib

Conversation

@sanathkr
Copy link
Copy Markdown
Contributor

I am working on adding a sam build command to AWS SAM CLI. As part of this, I created a separated project called aws-lambda-builders that contains a collection of builder workflows for various programming languages + dependency manager combinations. We have already implemented Python+PIP. We are in the process of adding support for other languages. My goal is to work with the community and add support in this library to build Lambda functions written in many combination of Programming Language (ex: Python) + DependencyManager (ex: PIP) + ApplicationFramework (Ex: Chalice).

I want to contribute to the Docker-Lambda build container by adding the “Lambda Builders” library into it. This library has a CLI wrapper that will allow customers to directly run a build inside the container using one of the built-in workflows without writing scripts. sam build command will also use this container to perform higher-fidelity builds.

This PR adds it the library to the base build image. For now I am using a development preview version 0.0.1-dev to the images build fine. But I hope to release a stable 0.1.0 version soon which we can release to Dockerhub with.

@mhart
Copy link
Copy Markdown
Member

mhart commented Nov 18, 2018

Hey @sanathkr – thanks for this! I ended up adding this in the final layer of each runtime's build image, instead of in the base build image – so that only the last layer needs to be updated each time we need to bump the version or similar.

I also added AWS SAM, and ensured the build images have the latest AWS CLI

@mhart mhart closed this Nov 18, 2018
@mhart
Copy link
Copy Markdown
Member

mhart commented Nov 18, 2018

See 6276809 for the changes I made

@sanathkr
Copy link
Copy Markdown
Contributor Author

This looks even better :-) thanks for accepting the patch!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants