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