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

fix: use Amazon Linux 2 provided runtime #457

Closed
wants to merge 7 commits into from

Conversation

scanlonp
Copy link
Contributor

See if automation passes with a non-main branch build.

SamStephens and others added 7 commits December 15, 2023 10:52
As the Go runtime is deprecated.
As part of this change, the compiled Go binary main is renamed
to bootstrap, as that is the name that the AL2023 Lambda runtime
expects.
We now build using the standard Go Docker image, as the AWS Lambda
Go image is deprecated.
It's important to note that the Dockerfile in the lambda/ folder
is not used as part of the runtime at all. It's only function is
to produce the /asset/bootstrap binary that is then uploaded to
Lambda and used with the AL2023 runtime. This was the case before
this change, except it used to produce /asset/main to use with
the GO runtime.
Now we're no longer building using the AWS build image, there's
no conflict with that image if we use GOPROXY.
The rename from main to bootstrap was missed from this step.
So we're tracking the latest 1.x Go version, like we were when
we were using public.ecr.aws/sam/build-go1.x:latest.
Update the major version to 3, as changing the base build image
is a breaking change. Run `npx projen`.
Copy link

mergify bot commented Dec 18, 2023

⚠️ The sha of the head commit of this PR conflicts with #424. Mergify cannot evaluate rules on this PR. ⚠️

@scanlonp scanlonp closed this Dec 18, 2023
auto-merge was automatically disabled December 18, 2023 13:37

Pull request was closed

@scanlonp scanlonp deleted the non-main-build branch December 18, 2023 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants