fix(ecs-patterns): minHealthyPercent and maxHealthyPercent props validation

packages/aws-cdk-lib/aws-ecs-patterns/lib/fargate/application-load-balanced-fargate-service.ts

@@ -96,6 +96,9 @@ export class ApplicationLoadBalancedFargateService extends ApplicationLoadBalanc
  throw new Error('You must specify one of: taskDefinition or image');
  }
 
+ this.validateHealthyPercentage('minHealthyPercent', props.minHealthyPercent);
+ this.validateHealthyPercentage('maxHealthyPercent', props.maxHealthyPercent);
+
  const desiredCount = FeatureFlags.of(this).isEnabled(cxapi.ECS_REMOVE_DEFAULT_DESIRED_COUNT) ? this.internalDesiredCount : this.desiredCount;
 
  this.service = new FargateService(this, 'Service', {
@@ -120,4 +123,14 @@ export class ApplicationLoadBalancedFargateService extends ApplicationLoadBalanc
  });
  this.addServiceAsTarget(this.service);
  }
+
+ /**
+ * Throws an error if the specified percent is not an integer or negative.
+ */
+ private validateHealthyPercentage(name: string, value?: number) {
+ if (value === undefined) { return; }
+ if (!Number.isInteger(value) || value < 0) {
+ throw new Error(`${name}: Must be a non-negative integer; received ${value}`);
+ }
+ }
 }

packages/aws-cdk-lib/aws-ecs-patterns/test/fargate/load-balanced-fargate-service.test.ts

@@ -1208,3 +1208,37 @@ test('NetworkLoadBalancedFargateService multiple capacity provider strategies ar
  ]),
  });
 });
+
+test('should validate minHealthyPercent', () => {
+ // GIVEN
+ const stack = new cdk.Stack();
+ const vpc = new ec2.Vpc(stack, 'VPC');
+ const cluster = new ecs.Cluster(stack, 'Cluster', { vpc });
+
+ // WHEN
+ expect(() => new ecsPatterns.ApplicationLoadBalancedFargateService(stack, 'Service', {
+ cluster,
+ taskImageOptions: {
+ image: ecs.ContainerImage.fromRegistry('/aws/aws-example-app'),
+ dockerLabels: { label1: 'labelValue1', label2: 'labelValue2' },
+ },
+ minHealthyPercent: 0.5,
+ })).toThrow(/Must be a non-negative integer/);
+});
+
+test('should validate maxHealthyPercent', () => {
+ // GIVEN
+ const stack = new cdk.Stack();
+ const vpc = new ec2.Vpc(stack, 'VPC');
+ const cluster = new ecs.Cluster(stack, 'Cluster', { vpc });
+
+ // WHEN
+ expect(() => new ecsPatterns.ApplicationLoadBalancedFargateService(stack, 'Service', {
+ cluster,
+ taskImageOptions: {
+ image: ecs.ContainerImage.fromRegistry('/aws/aws-example-app'),
+ dockerLabels: { label1: 'labelValue1', label2: 'labelValue2' },
+ },
+ maxHealthyPercent: 0.5,
+ })).toThrow(/Must be a non-negative integer/);
+});