ovs-ctl: Clean up temporary file used for force-reload-kmod.
authorBen Pfaff <blp@nicira.com>
Tue, 21 Jun 2011 17:13:04 +0000 (10:13 -0700)
committerBen Pfaff <blp@nicira.com>
Tue, 21 Jun 2011 22:13:32 +0000 (15:13 -0700)
Suggested-by: Andrew Evans <aevans@nicira.com>
utilities/ovs-ctl.in

index a127149a0e668f930bab5278ec848d1ec9cf9514..44afbd2727177f7c72f3b2383db685105ed163f1 100755 (executable)
@@ -233,7 +233,8 @@ force_reload_kmod () {
     stop
 
     script=`mktemp`
-    if action "Save interface configuration to $script" save_interfaces; then
+    trap 'rm -f "$script"' 0 1 2 13 15
+    if action "Saving interface configuration" save_interfaces; then
         :
     else
         log_warning_msg "Failed to save configuration, not replacing kernel module"
@@ -252,7 +253,16 @@ force_reload_kmod () {
 
     start
 
-    action "Restore interface configuration from $script" "$script"
+    action "Restoring interface configuration" "$script"
+    rc=$?
+    if test $rc = 0; then
+        level=debug
+    else
+        level=err
+    fi
+    log="logger -p daemon.$level -t ovs-save"
+    $log "force-reload-kmod interface restore script exited with status $rc:"
+    $log -f "$script"
 }
 
 ## ---- ##