fix(cli): deployment continues if ECR asset fails to build or publish

Fixes #26048, fixes #25827.
aws · Jun 20, 2023 · 393f8f0 · 393f8f0
1 parent 37e0c43
commit 393f8f0
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/packages/aws-cdk/lib/api/deployments.ts b/packages/aws-cdk/lib/api/deployments.ts
@@ -589,6 +589,9 @@ export class Deployments {
 
     const publisher = this.cachedPublisher(assetManifest, stackEnv, options.stackName);
     await publisher.buildEntry(asset);
+    if (publisher.hasFailures) {
+      throw new Error(`Failed to build asset ${asset.id}`);
+    }
   }
 
   /**
@@ -601,6 +604,9 @@ export class Deployments {
     // No need to validate anymore, we already did that during build
     const publisher = this.cachedPublisher(assetManifest, stackEnv, options.stackName);
     await publisher.publishEntry(asset);
+    if (publisher.hasFailures) {
+      throw new Error(`Failed to publish asset ${asset.id}`);
+    }
   }
 
   /**