[node-patches] Change in ovirt-node[master]: Fix using "has_link()" to check nic physical connected faile...

jboggs at redhat.com jboggs at redhat.com
Wed Jan 29 19:29:47 UTC 2014


Joey Boggs has submitted this change and it was merged.

Change subject: Fix using "has_link()" to check nic physical connected failed issue
......................................................................


Fix using "has_link()" to check nic physical connected failed issue

Previously using the "has_link" of class "NIC" to detect the physical
connected state of a network cable/connector will fail.This is due to
it miss to execute "ip link set dev {dev} up" before check "1" in
/sys/class/net/{dev}/carrier.

Change-Id: I608af3dd35d979b2213a762d84b831e792417efc
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1029033
Signed-off-by: hadong <hadong0720 at gmail.com>
---
M src/ovirt/node/utils/network.py
1 file changed, 19 insertions(+), 5 deletions(-)

Approvals:
  Joey Boggs: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I608af3dd35d979b2213a762d84b831e792417efc
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: hadong <hadong0720 at gmail.com>
Gerrit-Reviewer: Joey Boggs <jboggs at redhat.com>
Gerrit-Reviewer: hadong <hadong0720 at gmail.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list