Change in vdsm[master]: netconf: Skip source routing cfg files when restoring device...

danken at redhat.com danken at redhat.com
Thu Jul 25 11:41:28 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: netconf: Skip source routing cfg files when restoring devices
......................................................................


netconf: Skip source routing cfg files when restoring devices

_sortModifiedIfcfgs() uses the cfg files under '/etc/sysconfig/network-scripts'
to get the devices which should be stopped and started during rollback. But the
backed up files could also include source routing cfg files after the source routing
feature was added. So this patch skips those files when it gets the devices list.

Change-Id: I6f72bd29f5f9d4f5d02d8cd2e19675d00eac4600
Signed-off-by: Mark Wu <wudxw at linux.vnet.ibm.com>
Reviewed-on: http://gerrit.ovirt.org/17213
Reviewed-by: Antoni Segura Puimedon <asegurap at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/netconf/ifcfg.py
1 file changed, 5 insertions(+), 3 deletions(-)

Approvals:
  Antoni Segura Puimedon: Looks good to me, but someone else must approve
  Mark Wu: Verified
  Dan Kenigsberg: Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6f72bd29f5f9d4f5d02d8cd2e19675d00eac4600
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Assaf Muller <amuller at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list