Kendra Neil comments

Results 136 comments of


                                            Kendra Neil

feat(ecs): add `maxSwap` and `swappiness` properties to LinuxParameters

@Mergifyio update

feat(ecs): add `maxSwap` and `swappiness` properties to LinuxParameters

@Mergifyio update

feat(ecs): add `maxSwap` and `swappiness` properties to LinuxParameters

@mrgrain Can you take another look? I've re-updated the tests.

fix(cli): large context causes E2BIG error during synthesis on Linux

@Mergifyio update

fix(ecs): iam policies are missing for firelens cloudwatch_logs log driver

> Works with me! None of these plugins have integ coverage AFAIK. > > I will add for the `cloudwatch` and `cloudwatch_logs` plugins since those are the primary focus of...

fix(ecs): iam policies are missing for firelens cloudwatch_logs log driver

@rhermes62 Just to clarify, I'd like the tests for `cloudwatch` and `cloudwatch_logs` added to this PR and the others in a separate PR. After reading my own response, I wasn't...

fix(ecs): iam policies are missing for firelens cloudwatch_logs log driver

> Still working on getting those cleaned up. Builds in flight likely to fail. > > These first integ tests for aws-cdk written so learning how this repo does them....

fix(ecs): iam policies are missing for firelens cloudwatch_logs log driver

@rhermes62 Actually, I just took a look and I can make myself useful here immediately. Just run `yarn integ --update-on-failed integ.firelens-log-driver.js` and it will update your test folder to the...

fix(ecs): iam policies are missing for firelens cloudwatch_logs log driver

> Sounds good! Thank you! We normally don't have direct access/credentials to do things like `cdk deploy` to AWS accounts since we have strict release processes/security so I was trying...

feat(custom-resource): allow AwsCustomResource to be placed in vpc

> While this is a feature and may need a README update, I wasn't sure how to go about documenting that naturally; most of the docs go over the features...