Skip to content

fix(integ): remove CapacityProviderName exceeding 64-char limit#3887

Merged
roger-zhangg merged 1 commit intodevelopfrom
fix-integ-test-cp
Feb 24, 2026
Merged

fix(integ): remove CapacityProviderName exceeding 64-char limit#3887
roger-zhangg merged 1 commit intodevelopfrom
fix-integ-test-cp

Conversation

@vicheey
Copy link
Contributor

@vicheey vicheey commented Feb 23, 2026

Issue #, if available

N/A

Description of changes

Removed CapacityProviderName: !Sub "${AWS::StackName}-cp" from integration test templates function_lmi_custom.yaml and function_lmi_default.yaml. The resolved name exceeded the 64-character limit enforced by Lambda's CapacityProvider API, causing deployment failures (e.g. SamTranslatorLambda-sam-integ-stack-function-lmi-custom-eplzcyobezoy-cp is 72 chars). Since CapacityProviderName is optional, CloudFormation will auto-generate a valid name.

Description of how you validated changes

  • Confirmed no integration test assertions depend on the CapacityProviderName value — the tests only use it as a lookup key via get_physical_id_by_logical_id, which works with auto-generated names.
  • Verified both templates are syntactically valid after the change.

Checklist

Examples?

N/A

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@vicheey vicheey requested a review from a team as a code owner February 23, 2026 23:49
@roger-zhangg roger-zhangg merged commit 81da5ee into develop Feb 24, 2026
7 checks passed
@roger-zhangg roger-zhangg deleted the fix-integ-test-cp branch February 24, 2026 00:09
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.

4 participants