fix(ec2): naming collisions when using ec2.InitFile.fromAsset() on …

…multiple instances in the same stack (#27468) Closes #16891 If a user creates more than one EC2 instance in the same stack and defines an InitConfig for each instance using `ec2.InitFile.fromAsset()`, the user will get an error if they pass in the same `targetFilePath` or `targetDirectory`. This bug is due to [how the asset is uploaded to S3](https://github.com/aws/aws-cdk/blob/main/packages/aws-cdk-lib/aws-ec2/lib/cfn-init-elements.ts#L427). This PR fixes this issue by adding a hash to the `targetFileName` before being uploaded as an s3 asset. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
aws · Oct 23, 2023 · fc3be31 · fc3be31
1 parent 10b3221
commit fc3be31
Show file tree

Hide file tree

Showing 17 changed files with 3,255 additions and 2 deletions.
diff --git a/...ltiple.js.snapshot/asset.04c44be8722f4360fe0830af22298ad0870480310d7bd6eedbbd2391aecf59a8 b/...ltiple.js.snapshot/asset.04c44be8722f4360fe0830af22298ad0870480310d7bd6eedbbd2391aecf59a8
diff --git a/.../asset.1be0bac6581864b510bdbf0a114f1d3429244758da7657cc365f73d371fe70ed/__entrypoint__.js b/.../asset.1be0bac6581864b510bdbf0a114f1d3429244758da7657cc365f73d371fe70ed/__entrypoint__.js
diff --git a/....snapshot/asset.1be0bac6581864b510bdbf0a114f1d3429244758da7657cc365f73d371fe70ed/index.js b/....snapshot/asset.1be0bac6581864b510bdbf0a114f1d3429244758da7657cc365f73d371fe70ed/index.js
diff --git a/...ltiple.js.snapshot/asset.b9ca21e9930115b91ab7e6877bec3bc97b410a6d41fd58816b56aba6db2643bd b/...ltiple.js.snapshot/asset.b9ca21e9930115b91ab7e6877bec3bc97b410a6d41fd58816b56aba6db2643bd
diff --git a/...esting/framework-integ/test/aws-ec2/test/integ.instance-init-multiple.js.snapshot/cdk.out b/...esting/framework-integ/test/aws-ec2/test/integ.instance-init-multiple.js.snapshot/cdk.out
diff --git a/...nstance-init-multiple.js.snapshot/instancetestDefaultTestDeployAssert5516EAF1.assets.json b/...nstance-init-multiple.js.snapshot/instancetestDefaultTestDeployAssert5516EAF1.assets.json
diff --git a/...tance-init-multiple.js.snapshot/instancetestDefaultTestDeployAssert5516EAF1.template.json b/...tance-init-multiple.js.snapshot/instancetestDefaultTestDeployAssert5516EAF1.template.json
diff --git a/...nteg.instance-init-multiple.js.snapshot/integ-ec2-multiple-instances-in-stack.assets.json b/...nteg.instance-init-multiple.js.snapshot/integ-ec2-multiple-instances-in-stack.assets.json