Skip to content
This repository has been archived by the owner on Jan 16, 2019. It is now read-only.

Fix #203: not correctly tracking probes_sent #209

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Fix #203: not correctly tracking probes_sent #209

wants to merge 1 commit into from

Conversation

targuan
Copy link
Contributor

@targuan targuan commented Oct 6, 2017

Fix #203 : not correctly tracking probes_sent when the ip is not reachable.

@coveralls
Copy link

coveralls commented Oct 6, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.4%) to 72.242% when pulling e4d6504 on targuan:bugfix/ping_failed into 5137b2b on napalm-automation:develop.

@mirceaulinic mirceaulinic added this to the 0.8.1 milestone Oct 6, 2017
@ktbyers
Copy link
Contributor

ktbyers commented Oct 7, 2017

Need clarity on ping implementation:

napalm-automation/napalm-base#315

@ktbyers
Copy link
Contributor

ktbyers commented Oct 24, 2017

Will reimplement post-reunification.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ktbyers ktbyers Awaiting requested review from ktbyers

Assignees
No one assigned
Labels
bug hacktoberfest post_reunification
Projects
None yet
Milestone
0.8.1
Development

Successfully merging this pull request may close these issues.
4 participants
@targuan @coveralls @ktbyers @mirceaulinic