Skip to content

Commit 1eef1c6

Browse files
authored
fix assserts of deploymentCircuitBreaker (#1217)
fix assert of deploymentCircuitBreaker SUMMARY fix broken ecs_cluster integration test of #1215 cc @tremble @karcadia ISSUE TYPE COMPONENT NAME ADDITIONAL INFORMATION Reviewed-by: Mark Chappell <None>
1 parent a510cbb commit 1eef1c6

File tree

1 file changed

+6
-8
lines changed
  • tests/integration/targets/ecs_cluster/tasks

1 file changed

+6
-8
lines changed

tests/integration/targets/ecs_cluster/tasks/main.yml

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -225,23 +225,21 @@
225225
role: "ecsServiceRole"
226226
register: ecs_service
227227

228-
- name: check that placement constraint has been applied
228+
- name: check that ECS service creation changed
229229
assert:
230230
that:
231231
- ecs_service.changed
232-
- "ecs_service.service.placementConstraints[0].type == 'distinctInstance'"
233232

234-
- name: check that ECS service creation changed
233+
- name: check that placement constraint has been applied
235234
assert:
236235
that:
237-
- ecs_service.changed
238-
236+
- "ecs_service.service.placementConstraints[0].type == 'distinctInstance'"
237+
239238
- name: check that ECS service was created with deployment_circuit_breaker
240239
assert:
241240
that:
242-
- ecs_service.service.deploymentCircuitBreaker
243-
- ecs_service.service.deploymentCircuitBreaker.enable
244-
- ecs_service.service.deploymentCircuitBreaker.rollback
241+
- ecs_service.service.deploymentConfiguration.deploymentCircuitBreaker.enable
242+
- ecs_service.service.deploymentConfiguration.deploymentCircuitBreaker.rollback
245243

246244
- name: create same ECS service definition (should not change)
247245
ecs_service:

0 commit comments

Comments
 (0)