Yaniv Bronhaim has posted comments on this change.

Change subject: Unified network persistence [4.2/4.*] - Upgrade updates running 
conf
......................................................................


Patch Set 16:

(1 comment)

....................................................
File lib/vdsm/tool/unified_persistence.py
Line 129: def unified_persistence():
Line 130:     """
Line 131:     Upgrade host networking persistence from ifcfg to unified if the
Line 132:     persistence model is set as unified in 
/usr/lib64/python2.X/site-packages/
Line 133:     vdsm/config.py
after seeing 
http://gerrit.ovirt.org/#/c/20803/3/lib/vdsm/tool/upgrade_300_networks.py , I 
understand that you are not the only one that provides such explanation here. 
I'll try to think about better solution for detailed and short description
Line 134:     """
Line 135:     unifiedPersistenceUpgrade = UnifiedPersistence()
Line 136:     upgradeName = UnifiedPersistence.__name__
Line 137:     upgrade = Upgrade(upgradeName)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I051b0f168b6357e60184409845ea410c5891b083
Gerrit-PatchSet: 16
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Assaf Muller <amul...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Assaf Muller <amul...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Giuseppe Vallarelli <gvall...@redhat.com>
Gerrit-Reviewer: Mark Wu <wu...@linux.vnet.ibm.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to