Change in vdsm[master]: ethtool: remove vdsm dependency on ethtool

danken at redhat.com danken at redhat.com
Wed Sep 3 20:24:27 UTC 2014


Dan Kenigsberg has posted comments on this change.

Change subject: ethtool: remove vdsm dependency on ethtool
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.ovirt.org/#/c/32382/1/lib/vdsm/netlink/__init__.py
File lib/vdsm/netlink/__init__.py:

Line 123: 
Line 124:     _nl_socket_alloc = CFUNCTYPE(c_void_p)(('nl_socket_alloc', LIBNL))
Line 125:     _nl_socket_free = _none_proto(('nl_socket_free', LIBNL))
Line 126: 
Line 127: except OSError:  # CDLL failed to lead libnl3, assume libnl-1
Lead-> load
Line 128:     # Change from handle to socket as it is now more accurately called in
Line 129:     # libnl-3
Line 130:     LIBNL_ROUTE = LIBNL = CDLL('libnl.so.1', use_errno=True)
Line 131: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I71f97e182f3fc74cfa9e8288b1071aad1082fb6f
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list