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

Support Arm64 architecture in Lambda handlers #44

Closed
wants to merge 2 commits into from

Conversation

SeriousBug
Copy link
Contributor

Adds support for Arm64 architecture in Lambda handlers. X86_64 is used by default, as they were before this PR.

Changing to Arm64 will reduce costs, but node packages that rely on native components may not work without additional effort as they will have to be compiled for Arm. However majority of node packages are javascript-only, so most handlers can switch to Arm with no additional effort.

The support is added using an architecture field, which can be defined either in the function definition to pick an architecture for that handler, or in the stack definition defaults to set it globally.

@github-actions
Copy link

github-actions bot commented Dec 13, 2022

Unit Test Results

  1 files  ±0    9 suites  ±0   47s ⏱️ +8s
30 tests ±0  30 ✔️ ±0  0 💤 ±0  0 ±0 
31 runs  ±0  31 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 9b97ef7. ± Comparison against base commit d9af446.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Dec 13, 2022

Codecov Report

Merging #44 (9b97ef7) into main (4ad6fb1) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #44      +/-   ##
==========================================
+ Coverage   82.69%   82.77%   +0.08%     
==========================================
  Files          24       24              
  Lines         624      627       +3     
  Branches      142      144       +2     
==========================================
+ Hits          516      519       +3     
  Misses         99       99              
  Partials        9        9              
Impacted Files Coverage Δ
constructs/lambda-service.ts 70.09% <ø> (ø)
handlers/handler.ts 100.00% <ø> (ø)
constructs/base-function.ts 84.61% <100.00%> (+2.00%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@SeriousBug
Copy link
Contributor Author

Superseded by #46

@SeriousBug SeriousBug closed this Mar 3, 2023
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