feat(ecs): Add support for ECS deployment circuit breaker (#6132)

* feat(ecs): Add support for ECS deployment circuit breaker

* ffeat(ecs): Formatting of code

* feat(ecs): remove unnecessary lines of code, oriented at enableExecuteCommand

* fix(ecs): used wrong syntax to test for enabled deploymentCircuitBreaker

---------

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

clouddriver-ecs/src/main/java/com/netflix/spinnaker/clouddriver/ecs/deploy/description/CreateServerGroupDescription.java

@@ -99,6 +99,7 @@ public class CreateServerGroupDescription extends AbstractECSDescription {
   String taskDefinitionArtifactAccount;
   Map<String, String> containerToImageMap;
   boolean enableExecuteCommand;
+  boolean enableDeploymentCircuitBreaker;
 
   /**
    * @deprecated this field only allows for one container to be specified. ECS supports the ability

clouddriver-ecs/src/main/java/com/netflix/spinnaker/clouddriver/ecs/deploy/ops/CreateServerGroupAtomicOperation.java

@@ -536,6 +536,12 @@ protected CreateServiceRequest makeServiceRequest(
     DeploymentConfiguration deploymentConfiguration =
         new DeploymentConfiguration().withMinimumHealthyPercent(100).withMaximumPercent(200);
 
+    DeploymentCircuitBreaker deploymentCircuitBreaker =
+        new DeploymentCircuitBreaker()
+            .withEnable(description.isEnableDeploymentCircuitBreaker())
+            .withRollback(false);
+    deploymentConfiguration.setDeploymentCircuitBreaker(deploymentCircuitBreaker);
+
     CreateServiceRequest request =
         new CreateServiceRequest()
             .withServiceName(newServerGroupName.getServiceName())

clouddriver-ecs/src/test/groovy/com/netflix/spinnaker/clouddriver/ecs/deploy/ops/CreateServerGroupAtomicOperationSpec.groovy

@@ -180,6 +180,7 @@ class CreateServerGroupAtomicOperationSpec extends CommonAtomicOperation {
       request.launchType == null
       request.platformVersion == null
       request.enableExecuteCommand == false
+      request.deploymentConfiguration.deploymentCircuitBreaker.enable == false
     }) >> new CreateServiceResult().withService(service)
 
     result.getServerGroupNames().size() == 1
@@ -246,7 +247,8 @@ class CreateServerGroupAtomicOperationSpec extends CommonAtomicOperation {
       securityGroupNames: ['helloworld'],
       associatePublicIpAddress: true,
       serviceDiscoveryAssociations: [serviceRegistry],
-      enableExecuteCommand: true
+      enableExecuteCommand: true,
+      enableDeploymentCircuitBreaker: true
     )
 
     def operation = new CreateServerGroupAtomicOperation(description)
@@ -311,6 +313,7 @@ class CreateServerGroupAtomicOperationSpec extends CommonAtomicOperation {
       request.launchType == 'FARGATE'
       request.platformVersion == '1.0.0'
       request.enableExecuteCommand == true
+      request.deploymentConfiguration.deploymentCircuitBreaker.enable == true
     } as CreateServiceRequest) >> new CreateServiceResult().withService(service)
 
     result.getServerGroupNames().size() == 1