fix!: Fixing case and Name parameter value for AMI's Distribution Configuration

fix!: Fix casing and Name parameter value for AMI's Distribution Configuration#

Author: hsohail94

package-lock.json

@@ -242,9 +242,11 @@ export class ImagePipeline extends Construct {
         const distributionConfig: any = {
           region: distributionRegion,
           amiDistributionConfiguration: {
-            name: `${props.imageRecipe}-${distributionRegion}`,
-            description: `copy AMI ${props.imageRecipe} to ${distributionRegion}`,
-            targetAccountIds: props.distributionAccountIDs,
+            //Capital case here because it's an object of type any, but capital case is what is expected in CloudFormation
+            //https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-imagebuilder-distributionconfiguration-amidistributionconfiguration.html
+            Name: `${props.imageRecipe}-${distributionRegion}-{{imagebuilder:buildDate}}`,
+            Description: `copy AMI ${props.imageRecipe} to ${distributionRegion}`,
+            TargetAccountIds: props.distributionAccountIDs,
           },
         };
         distributionsList.push(distributionConfig);

src/index.ts

@@ -56,8 +56,8 @@ const propsWithVolumeConfig: ImagePipelineProps = {
     throughput: 1000,
   },
   enableCrossAccountDistribution: true,
-  distributionAccountIDs: ['111222333444', '222444666888'],
-  distributionRegions: ['us-east-1', 'us-west-2'],
+  distributionAccountIDs: ['111222333444'],
+  distributionRegions: ['us-east-1'],
 };
 
 beforeAll(() => {
@@ -163,6 +163,14 @@ test('Infrastructure Configuration is built with provided EBS volume properties'
   templateWithVolume.hasResourceProperties('AWS::ImageBuilder::DistributionConfiguration', {
     Name: 'TestImageRecipe-distribution-config',
     Description: 'Cross account distribution settings for TestImageRecipe',
+    Distributions: [{
+      Region: 'us-east-1',
+      AmiDistributionConfiguration: {
+        Name: 'TestImageRecipe-us-east-1-{{imagebuilder:buildDate}}',
+        Description: 'copy AMI TestImageRecipe to us-east-1',
+        TargetAccountIds: ['111222333444'],
+      },
+    }],
   });
 });