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 <amul...@redhat.com>
Reviewed-on: http://gerrit.ovirt.org/17961
Reviewed-by: Saggi Mizrahi <smizr...@redhat.com>
Reviewed-by: Antoni Segura Puimedon <asegu...@redhat.com>
Reviewed-by: Federico Simoncelli <fsimo...@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 <amul...@redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com>
Gerrit-Reviewer: Assaf Muller <amul...@redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to