Change in vdsm[master]: ifcfg.removeNic: ignore missing host interface

danken at redhat.com danken at redhat.com
Wed Feb 26 16:35:25 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: ifcfg.removeNic: ignore missing host interface
......................................................................


ifcfg.removeNic: ignore missing host interface

There may be cases where the Ifcfg configurator is asked to remove a Nic
that is no longer exist on the host. This can happen if the device was
renamed by an administrator or if it was unplugged from the system.

In this patch we do not try to ifup the missing device.

Change-Id: Ia892202b837f4515cd44a3b485fbe1b2a447457c
Bug-Url: https://bugzilla.redhat.com/1059773
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/25009
Reviewed-by: Antoni Segura Puimedon <asegurap at redhat.com>
---
M vdsm/netconf/ifcfg.py
1 file changed, 5 insertions(+), 1 deletion(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia892202b837f4515cd44a3b485fbe1b2a447457c
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list