Yaniv Bronhaim has posted comments on this change.

Change subject: unified pers: fix restoration when moving from ifcfg pers
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.ovirt.org/#/c/25063/4/init/vdsmd_init_common.sh.in
File init/vdsmd_init_common.sh.in:

Line 233:             tune_system \
Line 234:             test_space \
Line 235:             test_lo \
Line 236:             unified_network_persistence_upgrade \
Line 237:             restore_nets \
if this change of order is mandatory i'd add a comment that it will stay this 
way
Line 238:             upgrade_300_nets \
Line 239:             "
Line 240:         ;;
Line 241:     --post-stop)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I584b3b2ee953b508da23874c0adc79fe59e06856
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsl...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshz...@linux.vnet.ibm.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to