linux-system-roles / vpn

Role for managing VPN/IPSec
https://linux-system-roles.github.io/vpn/
MIT License
8 stars 14 forks source link

use wait_for_connection instead of wait_for with ssh #31

Closed richm closed 2 years ago

richm commented 2 years ago

The check using wait_for to check the ssh port does not work reliably with qemu based testing, because the IP address inside the VM is not always resolvable externally, and the port number is different from 22 as well. Instead, use the wait_for_connection module which explicitly tests that Ansible can connection to the host using its transport mechanism, which I think is what this check is trying to do.

richm commented 2 years ago

I'm using timeout: 300 since that's the default timeout for wait_for https://docs.ansible.com/ansible/2.9/modules/wait_for_module.html#wait-for-module and I didn't want to change the existing timeout in the role.

richm commented 2 years ago

[citest pending]