Mark Wu has posted comments on this change.

Change subject: Unified network persistence [3/3] - Restore network 
configuration
......................................................................


Patch Set 2:

(1 comment)

....................................................
File vdsm/vdsm-restore-net-config
Line 44:         nets[os.path.basename(netFile)] = json.load(open(netFile))
Line 45: 
Line 46:     for bondFile in os.listdir(netinfo.BOND_CONF_PERS_DIR):
Line 47:         bonds[os.path.basename(bondFile)] = json.load(open(bondFile))
Line 48:     setupNetworks(nets, bonds)
we also need delete the existing network before restore networks with 
setupNetworks, otherwise the new added network will not be deleted.
Line 49: 
Line 50: 
Line 51: def main():
Line 52:     if config.get('vars', 'persistence') == 'unified':


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I73462b160ecfbaa7efe71eed905a3bbd69ee6c23
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Assaf Muller <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Assaf Muller <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Giuseppe Vallarelli <[email protected]>
Gerrit-Reviewer: Livnat Peer <[email protected]>
Gerrit-Reviewer: Mark Wu <[email protected]>
Gerrit-Reviewer: Petr Ĺ ebek <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to