Skip to content

Conversation

acristin
Copy link
Contributor

@acristin acristin commented Jan 12, 2023

In some AWS regions, Fn::GetAtt Key.APIKeyId does not work, but Ref: Key also returns the API Key ID and seems to work even in these regions.

Issue #, if available

Description of changes

Description of how you validated changes

Manually deployed minimally modified test template in ap-southeast-4 both with and without this change; with change the stack deploys successfully and without, we see:

Waiter ChangeSetCreateComplete failed: Waiter encountered a terminal failure state: For expression "Status" we matched expected path: "FAILED" Status: FAILED. Reason: Template error: resource MyFirstApiKey does not support attribute type APIKeyId in Fn::GetAtt

Checklist

Examples?

Please reach out in the comments if you want to add an example. Examples will be
added to sam init through aws/aws-sam-cli-app-templates.

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

In some AWS regions, Fn::GetAtt Key.APIKeyId does not work, but Ref: Key also returns the API Key ID and seems to work even in these regions.
@acristin acristin requested a review from a team as a code owner January 12, 2023 22:12
@acristin acristin enabled auto-merge (squash) January 12, 2023 22:16
@acristin acristin merged commit ecb430d into aws:develop Jan 12, 2023
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.

3 participants