Assaf Muller has uploaded a new change for review.

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.

Change-Id: Ie92a4b4a95e9f1a7855f3d276b4e4d5b8b4c6ad2
Signed-off-by: Assaf Muller <[email protected]>
---
M vdsm/sourceRoute.sh
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/61/17961/1

diff --git a/vdsm/sourceRoute.sh b/vdsm/sourceRoute.sh
index bb8384e..1d388c4 100755
--- a/vdsm/sourceRoute.sh
+++ b/vdsm/sourceRoute.sh
@@ -11,7 +11,7 @@
 }
 
 sourceRoute_restore() {
-    if [ -n "$interface" ]; then
+    if [ -n "$interface" ] && [ ${reason} == "STOP" ]; then
       echo "remove" "$interface" > \
           /var/run/vdsm/sourceRoutes/$timeStamp
     fi


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie92a4b4a95e9f1a7855f3d276b4e4d5b8b4c6ad2
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Assaf Muller <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to