Skip to content

Commit

Permalink
Fix memoryReservationLimit not being set in LoadBalancedEcsService
Browse files Browse the repository at this point in the history
  • Loading branch information
piradeepk committed May 2, 2019
1 parent ed39a8c commit b3ca5be
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions packages/@aws-cdk/aws-ecs/lib/load-balanced-ecs-service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -42,17 +42,18 @@ export class LoadBalancedEc2Service extends LoadBalancedServiceBase {
const container = taskDefinition.addContainer('web', {
image: props.image,
memoryLimitMiB: props.memoryLimitMiB,
environment: props.environment,
memoryReservationMiB: props.memoryReservationMiB,
environment: props.environment
});

container.addPortMappings({
containerPort: props.containerPort || 80,
containerPort: props.containerPort || 80
});

const service = new Ec2Service(this, "Service", {
cluster: props.cluster,
desiredCount: props.desiredCount || 1,
taskDefinition,
taskDefinition
});

this.addServiceAsTarget(service);
Expand Down

0 comments on commit b3ca5be

Please sign in to comment.