fix(lambda-python-alpha): pass architecture property to function#28443
fix(lambda-python-alpha): pass architecture property to function#28443mapk-amazon wants to merge 1 commit intoaws:mainfrom mapk-amazon:mapk-amazon-patch-architecture-for-lambda-alpha
Conversation
By default Lambda will take x86 architecture, which might get into conflict with layers/code.
aws-cdk-automation
left a comment
There was a problem hiding this comment.
The pull request linter has failed. See the aws-cdk-automation comment below for failure reasons. If you believe this pull request should receive an exemption, please comment and provide a justification.
A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed add Clarification Request to a comment.
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
kaizencc
left a comment
There was a problem hiding this comment.
Thanks @mapk-amazon, we need a unit test to go along with this change to ensure that it works. Are you up for that?
|
The pull request linter fails with the following errors: PRs must pass status checks before we can provide a meaningful review. If you would like to request an exemption from the status checks or clarification on feedback, please leave a comment on this PR containing |
By default Lambda will take x86 architecture, which might get into conflict with layers/code. With this change the architecture information is passed along and the correct architecture for the docker image is chosen.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license