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

feat: Use nodejs18 in custom-resources and aws-sdk v3 #12445

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

Conversation

xoapit
Copy link

@xoapit xoapit commented Apr 23, 2024

Closes: #12133

Based on changes from #12146 by svoychik. He is not available to resolve the conflicts so I have recreated the PR based on that changes.

@tonivdv
Copy link

tonivdv commented Apr 24, 2024

@xoapit thanks for taking the time and hope it will be merged asap :)

Just little question, is there a specific reason to not already bump the runtime to nodejs20 ? I see that in the v4 branch it was already set to 20 so I guess it should work?!

@xoapit
Copy link
Author

xoapit commented Apr 24, 2024

@tonivdv nodejs20 runtime is not available in AWS Cloud (US) https://docs.aws.amazon.com/govcloud-us/latest/UserGuide/govcloud-lambda.html so I want to keep it as 18.x to make sure it works for all partitions

@jeffrey1226
Copy link

When will you merge it? Node 16 is deprecated soon. So we need to fix it asap.

@xoapit
Copy link
Author

xoapit commented May 2, 2024

No one from the serverless team reviews and merges the PR, no releases from Nov 2023. Hmm, It looks like they do not want to maintain version 3.

@jeffrey1226
Copy link

Hi @xoapit , so should I use v4?

@xoapit
Copy link
Author

xoapit commented May 2, 2024

V.4 introduces a new pricing model, please refer to https://www.serverless.com/blog/serverless-framework-v4-a-new-model

@jeffrey1226
Copy link

Hi @xoapit , Thank you for your reply. I see. It's not good for me. Can you please help me how to download your PR to my local project? I can't wait. And it seems like they stopped to support this v3.

@xoapit
Copy link
Author

xoapit commented May 3, 2024

@jeffrey1226 It won't work with your local project. You can apply a patch like this #12133 (comment)

@garethmcc
Copy link
Contributor

@xoapit Thanks for the PR. We will be looking into this in the coming days. V3 is still being maintained even with the team hard at work on V.4 beta coming out in a few days.

@garethmcc
Copy link
Contributor

Some additional clarity as well regarding AWS timelines; while AWS is marking June, 2024 as when Node 16 is deprecated, this will have no impact on deployments at that stage. AWS will only be blocking function create from February, 2025.

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.

Upgrade Custom Resource functions to use Node.js 18
5 participants