Skip to content

Commit

Permalink
Be more relaxed with delete_on_termination return values, we're seein…
Browse files Browse the repository at this point in the history
…g eventual-consistency related issues
  • Loading branch information
tremble committed Oct 8, 2024
1 parent e8c2b87 commit d990585
Showing 1 changed file with 6 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,9 @@
- ansible.builtin.assert:
that:
- result.changed
- result.interface.attachment.delete_on_termination == true
# Although we wait for the changethe API sometimes returns the old state *after* having
# returned the new state.
# - result.interface.attachment.delete_on_termination == true
- _interface_0.attachment.delete_on_termination == True
vars:
_interface_0: "{{ eni_info.network_interfaces[0] }}"
Expand Down Expand Up @@ -129,7 +131,9 @@
- ansible.builtin.assert:
that:
- result.changed
- result.interface.attachment.delete_on_termination == false
# Although we wait for the changethe API sometimes returns the old state *after* having
# returned the new state.
# - result.interface.attachment.delete_on_termination == false
- _interface_0.attachment.delete_on_termination == False
vars:
_interface_0: "{{ eni_info.network_interfaces[0] }}"
Expand Down

0 comments on commit d990585

Please sign in to comment.