-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Bug? in default policy created by CodeBuild with VPC config #2335
Closed
cjyclaire opened this issue
Apr 19, 2019
· 3 comments
· Fixed by #2506 or MechanicalRock/tech-radar#14 · May be fixed by MechanicalRock/cdk-constructs#5, MechanicalRock/cdk-constructs#6 or MechanicalRock/cdk-constructs#7
Closed
Bug? in default policy created by CodeBuild with VPC config #2335
cjyclaire opened this issue
Apr 19, 2019
· 3 comments
· Fixed by #2506 or MechanicalRock/tech-radar#14 · May be fixed by MechanicalRock/cdk-constructs#5, MechanicalRock/cdk-constructs#6 or MechanicalRock/cdk-constructs#7
Labels
Comments
Hey @cjyclaire, Would you be able to provide a minimal reproduction of your problem? It'll help us solve it faster. |
you should be able to reproduce with a vpc and any codebuild project place in the private subnets of the vpc, such as:
|
Thanks for reporting @cjyclaire . Confirming this is a bug on our side. |
skinny85
added a commit
to skinny85/aws-cdk
that referenced
this issue
May 9, 2019
…ment's condition when using a VPC. Fixes aws#2335
4 tasks
This was referenced Aug 22, 2019
This was referenced Dec 12, 2019
This was referenced Jan 20, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
https://github.com/awslabs/aws-cdk/blob/master/packages/%40aws-cdk/aws-codebuild/lib/project.ts#L900
`arn:aws:ec2:${Aws.region}:${Aws.accountId}:subnet/[[subnets]]`
instead of
[[subnets]]
, it need to provide actually subnet-ids such ascurrent policy generated will cause
I'm guessing the policy content is invalid as when I tried with exact subnet ids, deploy will succeed.
The text was updated successfully, but these errors were encountered: