aws-cdk icon indicating copy to clipboard operation
aws-cdk copied to clipboard

feat: allow notification resource handler to get deployed within a vpc (closes #20903)

Open falkvonohlen opened this issue 2 years ago • 10 comments

In an monitored AWS account it might be necessary to deploy any lambda inside a VPC. This causes the CDK deployment to fail, if Bucket.addEventNotification() is called, as it creates a utility lambda without a VPC attachment. This pull requests adds an optional attribute to the S3Bucket that can be used to provided a VPC to which the lambda gets deployed.

Fixes https://github.com/aws/aws-cdk/issues/20903

All Submissions:

Adding new Unconventional Dependencies:

  • [ ] This PR adds new unconventional dependencies following the process described here

New Features

  • [ ] Have you added the new feature to an integration test?
    • [ ] Did you use yarn integ to deploy the infrastructure and generate the snapshot (i.e. yarn integ without --dry-run)?

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

falkvonohlen avatar Aug 04 '22 13:08 falkvonohlen

gitpod-io[bot] avatar Aug 04 '22 13:08 gitpod-io[bot]

Hi,

I don`t see why the build is failing. It has something to do with the dependencies outside of the areas I touched. Could someone please have a look on the build result and let me know what I need to!

Thanks Falk

falkvonohlen avatar Aug 11 '22 15:08 falkvonohlen

@mergifyio update

mrgrain avatar Aug 11 '22 19:08 mrgrain

@falkvonohlen Let's see if updating with the latest changes helps

mrgrain avatar Aug 11 '22 19:08 mrgrain

update

✅ Branch has been successfully updated

mergify[bot] avatar Aug 11 '22 19:08 mergify[bot]

I actually added the ec2 dependency in a separate commit as I thought it would solve my issue, but reverted it now. However I still don`t see why the build is failing.

falkvonohlen avatar Aug 24 '22 07:08 falkvonohlen

@Mergifyio update

TheRealAmazonKendra avatar Aug 31 '22 12:08 TheRealAmazonKendra

update

✅ Branch has been successfully updated

mergify[bot] avatar Aug 31 '22 12:08 mergify[bot]

This PR has been in the BUILD FAILING state for 3 weeks, and looks abandoned. To keep this PR from being closed, please continue work on it. If not, it will automatically be closed in a week.

aws-cdk-automation avatar Sep 22 '22 00:09 aws-cdk-automation

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
  • Commit ID: decc223d35d4075d1581c2bedf327ace332f3948
  • Result: FAILED
  • Build Logs (available for 30 days)

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

aws-cdk-automation avatar Sep 27 '22 07:09 aws-cdk-automation

This PR has been deemed to be abandoned, and will be closed. Please create a new PR for these changes if you think this decision has been made in error.

TheRealAmazonKendra avatar Oct 02 '22 04:10 TheRealAmazonKendra