Skip to content
This repository has been archived by the owner on Jul 18, 2024. It is now read-only.

Move functions to Arm64 #187

Merged
merged 4 commits into from
Nov 18, 2021
Merged

Move functions to Arm64 #187

merged 4 commits into from
Nov 18, 2021

Conversation

nilskelleter
Copy link
Contributor

Issue #, if available: #184 Move functions to Arm64

Description of changes:
With this PR the functions of the delivery, delivery-pricing, orders, payment, platform, products, users and warehouse service are moved to Arm64.

With this PR the payment-3p service is not changed.
Also the pipeline is not changed.
Detail information in issue #184

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

nilskelleter and others added 2 commits November 3, 2021 08:44
…y, delivery-pricing, orders, payment, platform, products, users sand warehouse service.
@nmoutschen
Copy link
Contributor

Thanks a lot for this PR @nilskelleter ! I'm currently on sick leave but will review it next week.

@stefanullinger
Copy link

Hi, this sounds interesting! Can someone explain what the advantage of moving to arm64 is? Is it cheaper? Is it faster? Or can you share some link that explains it?

Thanks in advance!

@nilskelleter
Copy link
Contributor Author

@stefanullinger

This AWS blog post is a good starting point: https://aws.amazon.com/de/blogs/aws/aws-lambda-functions-powered-by-aws-graviton2-processor-run-your-functions-on-arm-and-get-up-to-34-better-price-performance/

@nmoutschen
Copy link
Contributor

After testing, the Python code works fine but there is an issue with the Lambda Insights extension. The current versions of the Lambda Insights layer only support x86_64.

Since they use Go binaries, this is a blocker to merge this PR. I've contacted the PM to let them know.

@nmoutschen
Copy link
Contributor

I've switch the CloudWatch Lambda Insights layer to the one that supports Arm64. All should be good now so merging this.

@nmoutschen nmoutschen merged commit f108019 into aws-samples:main Nov 18, 2021
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.

3 participants