Remove more usage of pause with junos tests (#58153)
This removes more potential race conditions with junos testing. Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This commit is contained in:
parent
1539e0e855
commit
d010510835
2 changed files with 26 additions and 9 deletions
|
@ -81,9 +81,15 @@
|
|||
that:
|
||||
- "result.changed == true"
|
||||
|
||||
- name: wait for netconf server to come up
|
||||
pause:
|
||||
seconds: 10
|
||||
- name: wait for netconf port tcp/12345 to be closed
|
||||
wait_for:
|
||||
host: "{{ hostvars[item].ansible_host }}"
|
||||
port: 12345
|
||||
state: stopped
|
||||
with_inventory_hostnames: junos
|
||||
|
||||
- name: Reset ansible connections
|
||||
meta: reset_connection
|
||||
|
||||
- name: Ensure we can NOT communicate over non-default port
|
||||
include: "{{ role_path }}/tests/utils/junos_command.yaml ansible_connection=netconf ansible_port=12345 is_ignore_errors=true"
|
||||
|
|
|
@ -38,9 +38,14 @@
|
|||
that:
|
||||
- "result.changed == true"
|
||||
|
||||
- name: wait for persistent socket to timeout
|
||||
pause:
|
||||
seconds: 120
|
||||
- name: wait for netconf port tcp/830 to be open
|
||||
wait_for:
|
||||
host: "{{ hostvars[item].ansible_host }}"
|
||||
port: 830
|
||||
with_inventory_hostnames: junos
|
||||
|
||||
- name: Reset ansible connections
|
||||
meta: reset_connection
|
||||
|
||||
- name: Ensure we can communicate over netconf
|
||||
include: "{{ role_path }}/tests/utils/junos_command.yaml ansible_connection=netconf ansible_port=830 is_ignore_errors=false"
|
||||
|
@ -91,9 +96,15 @@
|
|||
that:
|
||||
- "result.changed == true"
|
||||
|
||||
- name: wait for netconf server to come up
|
||||
pause:
|
||||
seconds: 10
|
||||
- name: wait for netconf port tcp/830 to be closed
|
||||
wait_for:
|
||||
host: "{{ hostvars[item].ansible_host }}"
|
||||
port: 830
|
||||
state: stopped
|
||||
with_inventory_hostnames: junos
|
||||
|
||||
- name: Reset ansible connections
|
||||
meta: reset_connection
|
||||
|
||||
- name: Ensure we can NOT talk via netconf
|
||||
include: "{{ role_path }}/tests/utils/junos_command.yaml ansible_connection=netconf ansible_port=830 is_ignore_errors=true"
|
||||
|
|
Loading…
Reference in a new issue