Skip to content
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

aws-s3: eventBridgeEnabled creates redundant resources #29932

Closed
blaxx opened this issue Apr 23, 2024 · 3 comments
Closed

aws-s3: eventBridgeEnabled creates redundant resources #29932

blaxx opened this issue Apr 23, 2024 · 3 comments
Labels
@aws-cdk/aws-s3 Related to Amazon S3 bug This issue is a bug. duplicate This issue is a duplicate. effort/medium Medium work item – several days of effort p2

Comments

@blaxx
Copy link

blaxx commented Apr 23, 2024

Describe the bug

When creating a new Bucket with eventBridgeEnabled = true, redundant resources are created as well.

Expected Behavior

The snippet
const bucket = new s3.Bucket(this, 'MyEventBridgeBucket', { eventBridgeEnabled: true, });
creates a new AWS::S3::Bucket resource with eventBridgeEnabled = true only.

Current Behavior

The snippet
const bucket = new s3.Bucket(this, 'MyEventBridgeBucket', { eventBridgeEnabled: true, });
creates not only a AWS::S3::Bucket, but also the following resources:

  • Custom::S3BucketNotifications
  • AWS::IAM::Role
  • AWS::IAM::Policy
  • AWS::Lambda::Function

Reproduction Steps

const bucket = new s3.Bucket(this, 'MyEventBridgeBucket', { eventBridgeEnabled: true, });

Possible Solution

No response

Additional Information/Context

Was also reported in #26772 but closed without fix.

CDK CLI Version

2.137.0 (build bb90b4c)

Framework Version

No response

Node.js Version

v20.12.1

OS

Mac OS 12.7.4 (21H1123)

Language

TypeScript

Language Version

TypeScript (5.4.5)

Other information

No response

@blaxx blaxx added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Apr 23, 2024
@github-actions github-actions bot added the @aws-cdk/aws-s3 Related to Amazon S3 label Apr 23, 2024
@khushail khushail added duplicate This issue is a duplicate. p2 effort/medium Medium work item – several days of effort and removed needs-triage This issue or PR still needs to be triaged. labels Apr 23, 2024
@pahud
Copy link
Contributor

pahud commented Apr 23, 2024

Just reopened #26772
Closing in favor of #26772

@pahud pahud closed this as completed Apr 23, 2024
Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

@aws-cdk-automation
Copy link
Collaborator

Comments on closed issues and PRs are hard for our team to see. If you need help, please open a new issue that references this one.

@aws aws locked as resolved and limited conversation to collaborators Jul 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
@aws-cdk/aws-s3 Related to Amazon S3 bug This issue is a bug. duplicate This issue is a duplicate. effort/medium Medium work item – several days of effort p2
Projects
None yet
Development

No branches or pull requests

4 participants