Skip to content

Commit

Permalink
fix assserts of deploymentCircuitBreaker (ansible-collections#1217)
Browse files Browse the repository at this point in the history
fix assert of deploymentCircuitBreaker

SUMMARY
fix broken ecs_cluster integration test of ansible-collections#1215
cc @tremble  @karcadia
ISSUE TYPE

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell <None>
  • Loading branch information
markuman authored Jun 5, 2022
1 parent a510cbb commit 1eef1c6
Showing 1 changed file with 6 additions and 8 deletions.
14 changes: 6 additions & 8 deletions tests/integration/targets/ecs_cluster/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -225,23 +225,21 @@
role: "ecsServiceRole"
register: ecs_service

- name: check that placement constraint has been applied
- name: check that ECS service creation changed
assert:
that:
- ecs_service.changed
- "ecs_service.service.placementConstraints[0].type == 'distinctInstance'"

- name: check that ECS service creation changed
- name: check that placement constraint has been applied
assert:
that:
- ecs_service.changed

- "ecs_service.service.placementConstraints[0].type == 'distinctInstance'"
- name: check that ECS service was created with deployment_circuit_breaker
assert:
that:
- ecs_service.service.deploymentCircuitBreaker
- ecs_service.service.deploymentCircuitBreaker.enable
- ecs_service.service.deploymentCircuitBreaker.rollback
- ecs_service.service.deploymentConfiguration.deploymentCircuitBreaker.enable
- ecs_service.service.deploymentConfiguration.deploymentCircuitBreaker.rollback

- name: create same ECS service definition (should not change)
ecs_service:
Expand Down

0 comments on commit 1eef1c6

Please sign in to comment.