Change in vdsm[ovirt-3.3]: Only remove dhcp source routing if the removal reason is "STOP"

fsimonce at redhat.com fsimonce at redhat.com
Mon Aug 12 12:08:17 UTC 2013


Federico Simoncelli has submitted this change and it was merged.

Change subject: Only remove dhcp source routing if the removal reason is "STOP"
......................................................................


Only remove dhcp source routing if the removal reason is "STOP"

If the reason is FAIL then the IP configuration is not lost
and source routing should not be removed. Only STOP actually
removes the ip configuration from the device.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=995142
Change-Id: Ie92a4b4a95e9f1a7855f3d276b4e4d5b8b4c6ad3
Signed-off-by: Assaf Muller <amuller at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/17961
Reviewed-by: Saggi Mizrahi <smizrahi at redhat.com>
Reviewed-by: Antoni Segura Puimedon <asegurap at redhat.com>
Reviewed-by: Federico Simoncelli <fsimonce at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/17963
---
M vdsm/sourceRoute.sh
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Federico Simoncelli: Looks good to me, approved
  Assaf Muller: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie92a4b4a95e9f1a7855f3d276b4e4d5b8b4c6ad3
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Assaf Muller <amuller at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Assaf Muller <amuller at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list