Pahud Hsieh

Results 356 comments of Pahud Hsieh

I think you should not assign the `externalDbSg` to the fargate service. If your intention is allow the ingress from fargate to your DB, you should add an ingress of...

@fabianf92 CDK is pending on the CFN support. Please help the CFN team prioritize with 👍 on https://github.com/aws-cloudformation/cloudformation-coverage-roadmap/issues/2039 After CFN support that, CDK would be able to include that support...

OK I can see the violation messaegs now. ```ts export class DemoStack extends MyStack { constructor(scope: Construct, id: string, props: StackProps) { super(scope, id, props); // get a dummy func...

Hi @glitchassassin I'll open your PR branch later today and run the integ tests from my IDE. You can reach out to me on cdk.dev if you need any discussion...

Hi @glitchassassin I have created a PR to your branch to update the integ tests https://github.com/glitchassassin/aws-cdk/pull/2 Can you merge my PR to your branch and this PR should update immediately....

OK looks like your PR got new integ error ``` @aws-cdk-testing/framework-integ: CHANGED custom-resources/test/aws-custom-resource/integ.aws-custom-resource-dynamodb 1.258s @aws-cdk-testing/framework-integ: Resources @aws-cdk-testing/framework-integ: [~] AWS::Lambda::Function AWS679f53fac002430cb0da5b7982bd22872D164C4C @aws-cdk-testing/framework-integ: └─ [~] Code @aws-cdk-testing/framework-integ: └─ [~] .S3Key: @aws-cdk-testing/framework-integ: ├─...

The cli-integ-tests should be good now.

Update: CDK can't support this feature at this moment because CFN would not emit the outputs in `CONFIGURATION_COMPLETE` phase and dependent stacks would fail if they reference the outputs from...