Merge pull request #31 from rererecursive/master

Don't set DesiredCount when using the daemon scheduling strategy.

ecs-service.cfndsl.rb

@@ -364,16 +364,18 @@
     desired_count = desired
   end
 
+  strategy = defined?(scheduling_strategy) ? scheduling_strategy : nil
+
   ECS_Service('Service') do
     Cluster Ref("EcsCluster")
     Property("HealthCheckGracePeriodSeconds", health_check_grace_period) if defined? health_check_grace_period
-    DesiredCount Ref('DesiredCount')
+    DesiredCount Ref('DesiredCount') if strategy != 'DAEMON'
     DeploymentConfiguration ({
         MinimumHealthyPercent: Ref('MinimumHealthyPercent'),
         MaximumPercent: Ref('MaximumPercent')
     })
     TaskDefinition Ref('Task')
-    SchedulingStrategy scheduling_strategy if defined? scheduling_strategy
+    SchedulingStrategy scheduling_strategy if !strategy.nil?
 
     if service_loadbalancer.any?
       Role Ref('Role') unless awsvpc_enabled

tests/scheduling-strategy-daemon.test.yaml

@@ -0,0 +1,14 @@
+test_metadata:
+  type: config
+  name: ecs service with daemon scheduling
+  description: Creates an ECS service with the daemon scheduling strategy
+
+scheduling_strategy: DAEMON
+
+task_definition:
+  sftp:
+    image: nginx
+    tag: latest
+    privileged: true
+    ports:
+      - 80

tests/scheduling-strategy-default.test.yaml

@@ -0,0 +1,12 @@
+test_metadata:
+  type: config
+  name: ecs service with replica scheduling
+  description: Creates an ECS service with the default (replica) scheduling strategy
+
+task_definition:
+  sftp:
+    image: nginx
+    tag: latest
+    privileged: true
+    ports:
+      - 80