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.

Sign up for GitHub

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

Jump to bottom

Single Lambda Layer for all lambdas #6

Closed
Atharva21 opened this issue Feb 11, 2022 · 2 comments
Closed

Single Lambda Layer for all lambdas #6

Atharva21 opened this issue Feb 11, 2022 · 2 comments
Labels
help wanted Extra attention is needed

Comments

@Atharva21
Copy link
Owner

Currently in deploy workflow, We are packaging single lambda layer python.zip which may be different for different lambdas.
@Atharva21 Atharva21 added help wanted Extra attention is needed question Further information is requested and removed question Further information is requested labels Feb 11, 2022
@ketan-10
Copy link
Contributor

In my opinion, In this case, a single layer makes more sense. As we don't have too many unshared heavy dependencies.
So keeping it common will have benefits of simplicity and less build time. and splitting it will not reduce the package size that much.
We can check if there is a substantial size reduction in the build-file after splitting.

@Atharva21
Copy link
Owner Author

Yeah I think it makes sense to have one layer for less number of lambdas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ketan-10 @Atharva21