[node-patches] Change in ovirt-node[master]: ping: Fix ping page

mburns at redhat.com mburns at redhat.com
Tue Jul 9 13:48:02 UTC 2013


Michael Burns has submitted this change and it was merged.

Change subject: ping: Fix ping page
......................................................................


ping: Fix ping page

Previously pinging while the network wasn't available or pinging an
unreachable host lead to problems on the ping page.
The code has now been refactored and simplified to prevent those
problems.

Change-Id: I7f3fcbb7fca8b904e3531d7b6334f3e4aa5bc918
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=980420
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M src/ovirt/node/setup/core/ping.py
M src/ovirt/node/utils/process.py
2 files changed, 13 insertions(+), 24 deletions(-)

Approvals:
  Michael Burns: Verified; Looks good to me, approved


-- 
To view, visit http://gerrit.ovirt.org/16570
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7f3fcbb7fca8b904e3531d7b6334f3e4aa5bc918
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list