diff options
author | Nathaniel Case <this.is@nathanielca.se> | 2018-02-15 14:54:30 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-02-15 14:54:30 -0500 |
commit | 3d81f94349000c7223cdb71ff010745da3d1a563 (patch) | |
tree | b8aed21ddf4c5b58e16c2354cc343de88275b0eb | |
parent | a7a03bbf4a7e4d76e2665eade8170fc70c927ab7 (diff) | |
download | ansible-3d81f94349000c7223cdb71ff010745da3d1a563.tar.gz |
Update ios_ping.py to allow for count > 70. (#36142) (#36250)
* Update ios_ping.py to allow for count > 70.
Find 'Success...' output rather than relying on list index. Index -1 won't pass unit test.
(cherry picked from commit fed20b825f2edd0f6c5c147a32a494194dfa8164)
-rw-r--r-- | lib/ansible/modules/network/ios/ios_ping.py | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/lib/ansible/modules/network/ios/ios_ping.py b/lib/ansible/modules/network/ios/ios_ping.py index 743045bf2d..65c80a5608 100644 --- a/lib/ansible/modules/network/ios/ios_ping.py +++ b/lib/ansible/modules/network/ios/ios_ping.py @@ -143,7 +143,12 @@ def main(): ping_results = run_commands(module, commands=results["commands"]) ping_results_list = ping_results[0].split("\n") - success, rx, tx, rtt = parse_ping(ping_results_list[3]) + stats = "" + for line in ping_results_list: + if line.startswith('Success'): + stats = line + + success, rx, tx, rtt = parse_ping(stats) loss = abs(100 - int(success)) results["packet_loss"] = str(loss) + "%" results["packets_rx"] = int(rx) |