napalm-automation / napalm-ios

Apache License 2.0
31 stars 40 forks source link

napalm-ios ping is not correctly tracking probes_sent #203

Closed ktbyers closed 6 years ago

ktbyers commented 7 years ago

This was on a failure pinging an IP address that didn't exist using napalm-ansible:

        "results": {
            "success": {
                "packet_loss": -5, 
                "probes_sent": 0, 
                "results": [], 
                "rtt_avg": 0.0, 
                "rtt_max": 0.0, 
                "rtt_min": 0.0, 
                "rtt_stddev": 0.0
            }
        }
    }
}
mirceaulinic commented 6 years ago

Closing this as it seems that https://github.com/targuan/napalm-ios/commit/e4d6504d39430e1c84f7c42b6b165dddc37de4ad solved it.