Skip to content

Conversation

k-asm
Copy link
Contributor

@k-asm k-asm commented Apr 18, 2025

Description of changes

This pull request adds support for new AWS CodeBuild Lambda images for both ARM and x86_64 architectures.

reference: https://docs.aws.amazon.com/codebuild/latest/userguide/lambda-compute-images.html

Additions to LinuxArmLambdaBuildImage:

  • aws/codebuild/amazonlinux-aarch64-lambda-standard:nodejs22
  • aws/codebuild/amazonlinux-aarch64-lambda-standard:python3.13
  • aws/codebuild/amazonlinux-aarch64-lambda-standard:ruby3.4
  • aws/codebuild/amazonlinux-aarch64-lambda-standard:go1.24

Additions to LinuxLambdaBuildImage:

  • aws/codebuild/amazonlinux-x86_64-lambda-standard:nodejs22
  • aws/codebuild/amazonlinux-x86_64-lambda-standard:python3.13
  • aws/codebuild/amazonlinux-x86_64-lambda-standard:ruby3.4
  • aws/codebuild/amazonlinux-x86_64-lambda-standard:go1.24

Checklist


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@github-actions github-actions bot added beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK p2 labels Apr 18, 2025
@aws-cdk-automation aws-cdk-automation requested a review from a team April 18, 2025 04:58
Copy link
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(This review is outdated)

@k-asm
Copy link
Contributor Author

k-asm commented Apr 18, 2025

Exemption Request

All the tests for LinuxArmLambdaBuildImage and LinuxLambdaBuildImage use only one image (AMAZON_LINUX_2_NODE_18) , so this change doesn’t need any extra testing.

@aws-cdk-automation aws-cdk-automation added the pr-linter/exemption-requested The contributor has requested an exemption to the PR Linter feedback. label Apr 18, 2025
@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
  • Commit ID: e7a47df
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@aws-cdk-automation aws-cdk-automation added the pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. label Apr 18, 2025
@iankhou iankhou added pr-linter/exempt-readme The PR linter will not require README changes pr-linter/exempt-test The PR linter will not require test changes pr-linter/exempt-integ-test The PR linter will not require integ test changes and removed pr-linter/exemption-requested The contributor has requested an exemption to the PR Linter feedback. labels Apr 19, 2025
@iankhou iankhou self-assigned this Apr 19, 2025
@aws-cdk-automation aws-cdk-automation removed the pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. label Apr 19, 2025
@aws-cdk-automation aws-cdk-automation dismissed their stale review April 19, 2025 23:43

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

Copy link
Contributor

mergify bot commented Apr 19, 2025

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit 5a265d1 into aws:main Apr 19, 2025
50 of 53 checks passed
Copy link
Contributor

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK p2 pr-linter/exempt-integ-test The PR linter will not require integ test changes pr-linter/exempt-readme The PR linter will not require README changes pr-linter/exempt-test The PR linter will not require test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants