Fix deployment group tagging

Author: rogperez

packages/cdk-blue-green-container-deployment/src/ecs-deployment-group.ts

@@ -125,6 +125,7 @@ export class EcsDeploymentGroup extends Resource implements IEcsDeploymentGroup,
           'codeDeploy:UpdateDeploymentGroup',
           'codeDeploy:DeleteDeploymentGroup',
           'codeDeploy:TagResource',
+          'codeDeploy:UntagResource',
         ],
         resources: ['*'],
       }),

packages/cdk-blue-green-container-deployment/src/lambdas/ecs-deployment-group/index.ts

@@ -157,22 +157,26 @@ export const handleUpdate: OnUpdateHandler = async (event): Promise<ResourceHand
     })
     .promise();
 
-  const newTagKeys: string[] = newProps.tags.map((t: any) => t.Key);
-  const removableTagKeys: string[] = oldProps.tags.map((t: any) => t.Key).filter((t) => !newTagKeys.includes(t));
-
-  await codeDeploy
-    .untagResource({
-      ResourceArn: newProps.arnForDeploymentGroup,
-      TagKeys: removableTagKeys,
-    })
-    .promise();
-
-  await codeDeploy
-    .tagResource({
-      ResourceArn: newProps.arnForDeploymentGroup,
-      Tags: newProps.tags,
-    })
-    .promise();
+  const newTagKeys: string[] = (newProps.tags || []).map((t: any) => t.Key);
+  const removableTagKeys: string[] = (oldProps.tags || []).map((t: any) => t.Key).filter((t) => !newTagKeys.includes(t));
+
+  if (removableTagKeys.length > 0) {
+    await codeDeploy
+      .untagResource({
+        ResourceArn: newProps.arnForDeploymentGroup,
+        TagKeys: removableTagKeys,
+      })
+      .promise();
+  }
+
+  if (newProps.tags.length > 0) {
+    await codeDeploy
+      .tagResource({
+        ResourceArn: newProps.arnForDeploymentGroup,
+        Tags: newProps.tags,
+      })
+      .promise();
+  }
 
   return {
     physicalResourceId: newProps.deploymentGroupName,