Skip to content

fix: bump all Lambda nodejs runtimes to 12.x#2237

Merged
mergify[bot] merged 3 commits intoaws:mainlinefrom
efekarakus:upgrade-lambda-runtimes
Apr 29, 2021
Merged

fix: bump all Lambda nodejs runtimes to 12.x#2237
mergify[bot] merged 3 commits intoaws:mainlinefrom
efekarakus:upgrade-lambda-runtimes

Conversation

@efekarakus
Copy link
Copy Markdown
Contributor

@efekarakus efekarakus commented Apr 29, 2021

Lambda is on the path to deprecate nodejs10.x, bumping the runtime version to 12.x.
Tested by deploying an application with copilotv1.5 and then deployed the service again:

  • the environment gets upgraded to v1.3.1
  • service deployment is successful

Resolves #2235

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

Lambda is on the path to deprecate nodejs10.x, bumping the runtime
version to the latest 14.x

Resolves aws#2235
@efekarakus efekarakus requested a review from a team as a code owner April 29, 2021 16:32
@efekarakus efekarakus requested a review from huanjani April 29, 2021 16:32
# SPDX-License-Identifier: Apache-2.0
Description: CloudFormation environment template for infrastructure shared among Copilot workloads.
Metadata:
Version: 'v1.3.1'
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

There is no difference in this template besides the version here, we had to create a new version so that the Lambda runtime version updates can be picked up by env upgrade

@efekarakus efekarakus added the do-not-merge Pull requests that mergify shouldn't merge until the requester allows it. label Apr 29, 2021
@efekarakus
Copy link
Copy Markdown
Contributor Author

Welp I just tried deploying with Autoscaling and get this error 😭 :

  Resource handler returned message: "Invalid request provided: The ZipFile attribute is not supported for the runtime 'nodejs14.x'"

I'm going try to change all the Lambdas to nodejs12.x and see if ZipFile attribute is supported

@efekarakus efekarakus removed the do-not-merge Pull requests that mergify shouldn't merge until the requester allows it. label Apr 29, 2021
@efekarakus efekarakus changed the title fix: bump all Lambda nodejs runtimes to 14.x fix: bump all Lambda nodejs runtimes to 12.x Apr 29, 2021
@mergify mergify Bot merged commit d233d07 into aws:mainline Apr 29, 2021
@efekarakus efekarakus deleted the upgrade-lambda-runtimes branch April 29, 2021 17:34
thrau pushed a commit to localstack/copilot-cli-local that referenced this pull request Dec 9, 2022
Lambda is on the path to deprecate nodejs10.x, bumping the runtime version to 12.x.
Tested by deploying an application with copilotv1.5 and then deployed the service again:
- [x] the environment gets upgraded to v1.3.1
- [x] service deployment is successful 

Resolves aws#2235


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
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.

Runtime Nodejs10 deprecated

4 participants