[node-patches] Change in ovirt-node[master]: network: Don't remove bonding module on cleanup

[asegurap at redhat.com]

Antoni Segura Puimedon has posted comments on this change.

Change subject: network: Don't remove bonding module on cleanup
......................................................................


Patch Set 2: Verified-1

(1 comment)

....................................................
File src/ovirt/node/utils/network.py
Line 834:     def delete(self, mifname):
Line 835:         """Delete a bond master
Line 836:         """
Line 837:         if not self.is_bond(mifname):
Line 838:             raise RuntimeError("Can no delete '%s', is no bond" % mifname)
This should simply be:
write "-%s" %ifname to  bonding_masters_filename

so that it works on el6 as well as F18+
Line 839:         process.call(["ip", "link", "set", "dev", mifname, "down"])


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I184d3ce6318a4086528ae33fe67dfb6ce4aeed43
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-HasComments: Yes