Reduce logging in collect ips 54/65454/16
authorSam Hague <shague@redhat.com>
Sun, 12 Nov 2017 01:02:51 +0000 (20:02 -0500)
committerJamo Luhrsen <jluhrsen@redhat.com>
Mon, 13 Nov 2017 19:01:38 +0000 (19:01 +0000)
commitd8c13d7866321b7b2080cb15d8c6cfb02590887e
treeb9715bf3a01eb8e5b17b2225106fd026f25e5f91
parent22b769490a9169af12ad22e1ed2f56602889e28f
Reduce logging in collect ips

Created a new VM Collect IP keyword that reduces the
amount of logging and works faster.

Problems in old keyword:
- logged the whole console multiple times, per vm check and per
operation
- ran Verify Active extra times
- different suites replicated using code

This patch:
- moves all the common wrapping code to a single keyword
- reduces the extra Active calls
- reduces the console to a single time per vm check
- removed console log on error since the log is always captured
during the attempt to get the addresses already

This looks to have reduced the time in half for retrieving the
addresses as well as reducing the multiple loggin of long
console logs.

Only converted the connectivity suite and left the old keyword
in place. The next patches will convert the other suites.

Change-Id: I042c137165eae95946fa5197fbb52b1b85312a00
Signed-off-by: Sam Hague <shague@redhat.com>
csit/libraries/OpenStackOperations.robot
csit/suites/openstack/connectivity/01_l2_tests.robot
csit/suites/openstack/connectivity/02_l3_tests.robot
csit/suites/openstack/connectivity/03_external_network_tests.robot
csit/suites/openstack/connectivity/04_security_group_tests.robot
csit/variables/Variables.robot