Skip to content

fix(cli): handle attributes of AWS::Events::EventBus when hotswapping#18834

Merged
mergify[bot] merged 2 commits intoaws:masterfrom
skinny85:hotswap-cfn-attributes
Feb 9, 2022
Merged

fix(cli): handle attributes of AWS::Events::EventBus when hotswapping#18834
mergify[bot] merged 2 commits intoaws:masterfrom
skinny85:hotswap-cfn-attributes

Conversation

@skinny85
Copy link
Copy Markdown
Contributor

@skinny85 skinny85 commented Feb 4, 2022

Without them, customers referencing an AWS::Events::EventBus in their StepFunctions StateMachine definition would get this message:

Could not perform a hotswap deployment, because the CloudFormation template could not be resolved:
We don't support attributes of the 'AWS::Events::EventBus' resource. This is a CDK limitation.
Please report it at https://github.com/aws/aws-cdk/issues/new/choose
Falling back to doing a full deployment

when hotswapping.

Fixes #18831


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

@skinny85 skinny85 requested a review from rix0rrr February 4, 2022 23:42
@gitpod-io
Copy link
Copy Markdown

gitpod-io bot commented Feb 4, 2022

@skinny85 skinny85 changed the title fix(cl): handle attributes of AWS::Events::EventBus when hotswapping fix(cli): handle attributes of AWS::Events::EventBus when hotswapping Feb 4, 2022
@github-actions github-actions bot added the package/tools Related to AWS CDK Tools or CLI label Feb 4, 2022
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Feb 4, 2022
@skinny85 skinny85 force-pushed the hotswap-cfn-attributes branch from 160fe5b to 842b679 Compare February 4, 2022 23:44
@skinny85 skinny85 force-pushed the hotswap-cfn-attributes branch from 842b679 to f8bf792 Compare February 4, 2022 23:47
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Feb 9, 2022

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@aws-cdk-automation
Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
  • Commit ID: 11412dd
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@mergify mergify bot merged commit a30a32a into aws:master Feb 9, 2022
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Feb 9, 2022

Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@skinny85 skinny85 deleted the hotswap-cfn-attributes branch February 11, 2022 00:39
moelasmar pushed a commit to moelasmar/aws-cdk that referenced this pull request Feb 15, 2022
…aws#18834)

Without them, customers referencing an `AWS::Events::EventBus` in their StepFunctions StateMachine definition would get this message:

```
Could not perform a hotswap deployment, because the CloudFormation template could not be resolved:
We don't support attributes of the 'AWS::Events::EventBus' resource. This is a CDK limitation.
Please report it at https://github.com/aws/aws-cdk/issues/new/choose
Falling back to doing a full deployment
```

when hotswapping.

Fixes aws#18831

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
TikiTDO pushed a commit to TikiTDO/aws-cdk that referenced this pull request Feb 21, 2022
…aws#18834)

Without them, customers referencing an `AWS::Events::EventBus` in their StepFunctions StateMachine definition would get this message:

```
Could not perform a hotswap deployment, because the CloudFormation template could not be resolved:
We don't support attributes of the 'AWS::Events::EventBus' resource. This is a CDK limitation.
Please report it at https://github.com/aws/aws-cdk/issues/new/choose
Falling back to doing a full deployment
```

when hotswapping.

Fixes aws#18831

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution/core This is a PR that came from AWS. package/tools Related to AWS CDK Tools or CLI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(cli): support hotswap for EventBus attributes

3 participants