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

Improve aws-lambda.adoc regarding Java requirements and SSL #45721

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Jan 20, 2025

Fixes #45616 (hopefully)

@gsmet
Copy link
Member Author

gsmet commented Jan 20, 2025

@zakkak could you have a look at the changes I made to the native + SSL part?

Copy link

🎊 PR Preview 47a1a8c has been successfully built and deployed to https://quarkus-pr-main-45721-preview.surge.sh/version/main/guides/

  • Images of blog posts older than 3 months are not available.
  • Newsletters older than 3 months are not available.

Copy link

quarkus-bot bot commented Jan 20, 2025

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 5ac3904.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

Copy link
Contributor

@zakkak zakkak left a comment

Choose a reason for hiding this comment

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

@gsmet the changes look good to me, but when I try it locally it fails with:

❯ sam local invoke --template target/sam.native.yaml --event payload.json
No current session found, using default AWS::AccountId                                          
Invoking not.used.in.provided.runtime (provided)                                                
Decompressing /tmp/lambda/target/function.zip                                                   
Unknown 404 - Unable to check if base image is current.                                         
                                                                                                
Possible incompatible Docker engine clone employed. Consider `--skip-pull-image` for improved   
speed, the tradeoff being not running the latest image.                                         
Removing rapid images for repo public.ecr.aws/sam/emulation-provided                            
Building image........................
Using local image: public.ecr.aws/lambda/provided:alami-rapid-x86_64.                           
                                                                                                
Mounting /tmp/tmpdpycsvel as /var/task:ro,delegated, inside runtime container                   
WARN[0673] Requested oom_score_adj=0 is lower than the current one, changing to 100 
START RequestId: a8cbd9f6-9bba-4e27-b4f4-3e4c00cdd73e Version: $LATEST
21 Jan 2025 09:18:57,698 [ERROR] (rapid) Init failed error=fork/exec /var/task/bootstrap: permission denied InvokeID=
21 Jan 2025 09:18:57,698 [ERROR] (rapid) Invoke failed error=fork/exec /var/task/bootstrap: permission denied InvokeID=048917ac-4587-44bb-b418-4a185a8bff82
21 Jan 2025 09:18:57,698 [ERROR] (rapid) Invoke DONE failed: Runtime.InvalidEntrypoint


1. A custom `bootstrap` script
2. `libsunec.so` must be added to `function.zip`
Copy link
Contributor

@zakkak zakkak Jan 21, 2025

Choose a reason for hiding this comment

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

Note that the generated boostrap-example.sh still referes to libsunec.so

# Make sure to copy Graal's libsunec.o and cacerts into src/main/zip.native/

@gsmet
Copy link
Member Author

gsmet commented Jan 21, 2025

Yeah tbh I didn't try, I just made the doc more accurate. Dunno how to properly fix this thing.

@zakkak
Copy link
Contributor

zakkak commented Jan 22, 2025

Yeah tbh I didn't try, I just made the doc more accurate. Dunno how to properly fix this thing.

Unfortunately I am not knowledgeable enough on the topic either, so we will need someone else to review (and possibly further update the docs).

@gsmet
Copy link
Member Author

gsmet commented Jan 22, 2025

Yeah, same here. I thought it could be an easy win but apparently there's a lot more to fix. Bummer.

Gonna mark as draft in case someone is interested in pursuing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

aws-lambda guide refers to AWS Java 11 image and mentions Java 8/11 version
2 participants