Add ping retries and break on success 94/78394/12
authorSam Hague <shague@redhat.com>
Mon, 3 Dec 2018 19:58:11 +0000 (14:58 -0500)
committerSam Hague <shague@redhat.com>
Thu, 6 Dec 2018 02:42:45 +0000 (21:42 -0500)
commit6c7206b957f6c565e35b1d1bf37d2bf00dea6653
tree6465c4c0a66b6f826cdcb0be7588723872e2f1f6
parentf6a3676fdcc8d86fecba3f684980a79098c7be2a
Add ping retries and break on success

Change the ping command to take a retries value. This
allows the ping to be more independent of timing issues
with flow additions.

The cirros images and the SSHLibary have some weird quirks
that the patch had to workaround:

- The cirros images use a dash like shell and not bash. This
required catering the shell command to be posix compliant.
- The typical prompt of $ is in the shell command so it can't
be used to read until. This was changed to read until regexp
to look for the final rc=n from the command.
- The SSHLibrary Write keyword actually consumes and returns
the command itself, which was causing an early match. The regex
helped here also since the regex would only match on the rc=n -
so a number had to be in the output for it to match.

JIRA: NETVIRT-1399
Change-Id: I06d43fbd2ce7a0f32ccc60cdfd2783abaadd260b
Signed-off-by: Sam Hague <shague@redhat.com>
csit/libraries/OpenStackOperations.robot
csit/libraries/Utils.robot
csit/suites/openstack/connectivity/external_network.robot