-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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(toolkit): conditionally emit AWS::CDK::Metadata resource #3692
Conversation
For relocatable stacks (those where no `region` is provided to the `core.Stack` constructor while synthesizing), generate a condition to only try to populate the `AWS::CDK::Metadata` resource in regions that are known to support it. Fixes #3648
Pull Request Checklist
|
Thanks so much for taking the time to contribute to the AWS CDK ❤️ We will shortly assign someone to review this pull request and help get it
|
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
@@ -4,6 +4,14 @@ import { Fact, FactName } from './fact'; | |||
* Information pertaining to an AWS region. | |||
*/ | |||
export class RegionInfo { | |||
/** | |||
* @returns the list of names of AWS regions for which there is at least one registered fact. This | |||
* may not be an exaustive list of all available AWS regions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exhaustive*
Thank you for contributing! Your pull request is now being automatically merged. |
1 similar comment
Thank you for contributing! Your pull request is now being automatically merged. |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
* feat(toolkit): conditionally emit AWS::CDK::Metadata resource For relocatable stacks (those where no `region` is provided to the `core.Stack` constructor while synthesizing), generate a condition to only try to populate the `AWS::CDK::Metadata` resource in regions that are known to support it. Fixes #3648 * fix typo * use higher level primitives * move condition name to a variable * fix broken unit test
For relocatable stacks (those where no
region
is provided to thecore.Stack
constructor while synthesizing), generate a condition toonly try to populate the
AWS::CDK::Metadata
resource in regions thatare known to support it.
Fixes #3648
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license