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

Incomplete list of Lambda runtimes #6441

Open
3 tasks done
skyzyx opened this issue Sep 5, 2024 · 5 comments
Open
3 tasks done

Incomplete list of Lambda runtimes #6441

skyzyx opened this issue Sep 5, 2024 · 5 comments
Assignees
Labels
bug This issue is a bug. closing-soon This issue will automatically close in 4 days unless further comments are made. p2 This is a standard priority issue service-api This issue is due to a problem in a service API, not the SDK implementation.

Comments

@skyzyx
Copy link

skyzyx commented Sep 5, 2024

Checkboxes for prior research

Describe the bug

The deprecated Lambda runtimes dotnet5.0 and dotnet7 are missing from the code, as well as the models.

https://docs.aws.amazon.com/lambda/latest/dg/lambda-runtimes.html

SDK version number

v3.646.0

Which JavaScript Runtime is this issue in?

Node.js

Details of the browser/Node.js/ReactNative version

N/A

Reproduction Steps

Using GitHub search in this repository for the terms dotnet5.0 or dotnet7 return 0 results. The terms dotnet6, dotnet8, and every other runtime listed on the official Lambda runtimes page returns expected results.

Observed Behavior

The dotnet5.0 and dotnet7 identifiers appear to be missing from the list of Lambda runtimes understood by this SDK.

Expected Behavior

I expected dotnet5.0 and dotnet7 to be supported.

Possible Solution

  • Perhaps the identifiers are missing from the source models.
  • Perhaps these get sorted out of the list when performing code generation.

Additional Information/Context

I worked at AWS on the SDK team from 2010-2014. At the time, "Coral" was used for modeling services. Nowadays, I see "Smithy" used everywhere. Not sure how they are/aren't related.

My real use-case is not in using the SDK, but rather in using the model files for code generation in a different programming language, and allowing me to programmatically keep certain patterns up-to-date. During my time at AWS, I was a champion for making the models available to end-users for precisely this purpose, so I'm glad it has come to fruition (some years ago at this point).

The JS SDK is the first one I've found that exposes JSON to work with (although I may have missed other SDKs).

@skyzyx skyzyx added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Sep 5, 2024
@kuhe
Copy link
Contributor

kuhe commented Sep 7, 2024

They would only be in the models if there was an enum of the runtimes that was involved in an SDK API operation to the AWS Lambda service. If no such operation exists, then there's no reason they would be in the models.

@kuhe
Copy link
Contributor

kuhe commented Sep 7, 2024

On second glance, there could be a modeling error. We receive these from the service teams rather than maintaining them ourselves. The maintainer(s) who triage the ticket may be able to inquire with the Lambda team about the discrepancy.

@kuhe kuhe added the service-api This issue is due to a problem in a service API, not the SDK implementation. label Sep 7, 2024
@kuhe
Copy link
Contributor

kuhe commented Sep 7, 2024

The two enum values share the suffix "container only", that probably has something to do with it.

@aBurmeseDev aBurmeseDev self-assigned this Sep 7, 2024
@aBurmeseDev
Copy link
Member

Hi @skyzyx - appreciate you for reporting it.

As mentioned above, service models are generated and managed by the upstream service teams which in this case is the Lambda team. If this issue requires an update to the service model itself, the Lambda team needs to address it as the SDK team does not have direct control over the service models.

I will reach out to the Lambda team on your behalf to gather more insights and have them investigate this issue further. I will keep you updated on any progress or response from the Lambda team. (ref: P153373156)

Best,
John

@aBurmeseDev aBurmeseDev added p2 This is a standard priority issue and removed needs-triage This issue or PR still needs to be triaged. labels Sep 7, 2024
@aBurmeseDev
Copy link
Member

Lambda team member responded:

The two specified runtimes in the issue are container image functions only, no managed runtime exists and as such when creating the function using the SDK you do not specify the runtime when creating a function which uses container images.

Let us know if you have further question before closing the issue.

@aBurmeseDev aBurmeseDev added the closing-soon This issue will automatically close in 4 days unless further comments are made. label Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. closing-soon This issue will automatically close in 4 days unless further comments are made. p2 This is a standard priority issue service-api This issue is due to a problem in a service API, not the SDK implementation.
Projects
None yet
Development

No branches or pull requests

3 participants