Skip to content

feat: add java8.al2023, java11.al2023, java17.al2023 runtime support - #556

Merged
ckawl merged 6 commits into
aws:masterfrom
ckawl:feat/java-al2023-runtimes
Jul 20, 2026
Merged

feat: add java8.al2023, java11.al2023, java17.al2023 runtime support#556
ckawl merged 6 commits into
aws:masterfrom
ckawl:feat/java-al2023-runtimes

Conversation

@ckawl

@ckawl ckawl commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:
Support for java8.al2023, java11.al2023, java17.al2023 Lambda managed runtimes according to the Lambda upcoming runtimes documentation.

Note: java8.al2023 does not include hello-pt-gradle or init.py , this is intentional, matching the base java8.al2 which also lacks both. The Powertools Gradle template was never created for java8.

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

@github-actions github-actions Bot added pr/external stage/needs-triage Automatically applied to new issues and PRs, indicating they haven't been looked at. labels Jul 14, 2026

@aws-sam-tooling-bot aws-sam-tooling-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review Results

Reviewed: 5b0b240..54afbc3
Files: ~30 (spot-checked across manifest, workflow, templates, Dockerfiles, poms, gradle, tests)
Comments: 1

Comment thread .github/workflows/build_test_invoke.yml
@ckawl
ckawl marked this pull request as ready for review July 14, 2026 17:10
@ckawl
ckawl requested a review from a team as a code owner July 14, 2026 17:10
ckawl added 4 commits July 14, 2026 15:31
Add init templates for Java AL2023 runtimes:
- Copy template directories from base runtimes (java8.al2, java11, java17)
- Update cookiecutter.json, template.yaml, and Dockerfiles with al2023 runtime/image refs
- Add entries to manifest-v2.json for both Zip and Image package types
- Add unit test files and CI workflow entries
@ckawl
ckawl force-pushed the feat/java-al2023-runtimes branch from 747b546 to 03b250b Compare July 14, 2026 22:31

@aws-sam-tooling-bot aws-sam-tooling-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review Results

Reviewed: 756d750..03b250b
Files: ~35 (spot-checked across manifest, workflow, templates, Dockerfiles, poms, tests)
Comments: 1

Comment thread tests/integration/unit_test/test_unit_test_java11_al2023.py

@vicheey vicheey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Two minor gaps to consider:

  1. Missing hello-pt-gradle for java8.al2023 -- java11.al2023 and java17.al2023 have both hello-pt-gradle and hello-pt-maven (Powertools). java8.al2023 only has hello-pt-maven. If this is intentional (Powertools Gradle doesn't target java8), a brief note in the PR description would help.

  2. Missing __init__.py for java8.al2023 -- java11.al2023 and java17.al2023 have a top-level __init__.py for Python test discovery. java8.al2023 does not -- this may cause test collection issues.

Otherwise LGTM. SAM Globals, Outputs, cookiecutter patterns all follow existing conventions.

vicheey

This comment was marked as duplicate.

@ckawl

ckawl commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Review Summary: ✅ Approve with minor comments

Mechanically sound -- templates are consistent copies of existing java11/java17 AL2 patterns with runtime string updated. Two minor gaps to consider:

  1. Missing hello-pt-gradle for java8.al2023 -- java11.al2023 and java17.al2023 have both hello-pt-gradle and hello-pt-maven (Powertools). java8.al2023 only has hello-pt-maven. If this is intentional (Powertools Gradle does not target java8), a brief note in the PR description would help.
  2. Missing __init__.py for java8.al2023 -- java11.al2023 and java17.al2023 have a top-level __init__.py for Python test discovery. java8.al2023 does not -- this may cause test collection issues in CI.

Otherwise LGTM. SAM Globals, Outputs, cookiecutter patterns all follow existing conventions.

Both are consistent with the base java8.al2, it doesn't have hello-pt-gradle or init.py either, so we kept the same template set for the al2023 copy. Good eye though, appreciate it!

@aws-sam-tooling-bot aws-sam-tooling-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review Results

Reviewed: 756d750..d58e6ea
Files: 448
Comments: 2

Comment thread tests/integration/build_invoke/java/test_java_11_al2023.py Outdated
Comment thread manifest-v2.json Outdated
- Fix 8-space indent to 4-space in test_java_11_al2023.py
- Add ': Maven' / ': Gradle' suffix to java11.al2023 Powertools
  displayNames to match java17.al2023 pattern
@ckawl
ckawl requested a review from vicheey July 14, 2026 23:26
@ckawl
ckawl merged commit 2633175 into aws:master Jul 20, 2026
109 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr/external stage/needs-triage Automatically applied to new issues and PRs, indicating they haven't been looked at.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants