Skip to content

Fix serverless endpoint configuration field name conversion #308

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

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

Conversation

MarcusKoolaard
Copy link

@MarcusKoolaard MarcusKoolaard commented Jun 11, 2025

Fixes field name conversion bug where acronyms in snake_case were not properly capitalized when converted to PascalCase for AWS API calls:

  • memory_size_in_mb now converts to MemorySizeInMB (not MemorySizeInMb)
  • supported_response_mime_types now converts to SupportedResponseMIMETypes (not SupportedResponseMimeTypes)

This resolves parameter validation failures when creating serverless endpoint configurations and inference specifications.

Also removed duplicate snake_to_pascal function definition.

#225

Test plan

  • Verified field conversions work correctly
  • All existing tests pass (63/63)
  • Serverless endpoint configuration now serialises correctly
  • No linting errors

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

Fixes field name conversion bug where acronyms in snake_case were not
properly capitalized when converted to PascalCase for AWS API calls:

- memory_size_in_mb now converts to MemorySizeInMB (not MemorySizeInMb)
- supported_response_mime_types now converts to SupportedResponseMIMETypes
  (not SupportedResponseMimeTypes)

This resolves parameter validation failures when creating serverless
endpoint configurations and inference specifications.

Also removed duplicate snake_to_pascal function definition.
@MarcusKoolaard MarcusKoolaard requested a review from a team as a code owner June 11, 2025 04:06
@MarcusKoolaard MarcusKoolaard requested a review from chad119 June 11, 2025 04:06
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.

1 participant