Mark Wu has posted comments on this change.
Change subject: Unified network persistence [1/3] - Save running config
......................................................................
Patch Set 13: Code-Review-1
(1 comment)
....................................................
File vdsm/configNetwork.py
Line 399: elif config.get('vars', 'persistence') ==
'unified':
Line 400: logging.info('Removing network %s from running configuration',
Line 401: network)
Line 402: _removeRunningConfig(netinfo.NET_CONF_RUN_DIR + network)
Line 403: if bonding:
we still need check if the bonding has other users by netinfo.ifaceUsers(bonding). if it
still has other users, that means it's not removed, therefore we should not remove it
from persistence.
Line 404: logging.info('Removing bond %s from running configuration',
Line 405: bonding)
Line 406: _removeRunningConfig(netinfo.BOND_CONF_RUN_DIR + bonding)
Line 407:
--
To view, visit
http://gerrit.ovirt.org/16699
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I7137a96f84abd2c5e532c6c37737e36ef17567a9
Gerrit-PatchSet: 13
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap(a)redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com>
Gerrit-Reviewer: Assaf Muller <amuller(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Giuseppe Vallarelli <gvallare(a)redhat.com>
Gerrit-Reviewer: Livnat Peer <lpeer(a)redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Petr Šebek <psebek(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes