make bwlimit None if it is not set to a value. Previously, an empty value
authorStephen Soltesz <soltesz@cs.princeton.edu>
Thu, 6 Aug 2009 15:44:59 +0000 (15:44 +0000)
committerStephen Soltesz <soltesz@cs.princeton.edu>
Thu, 6 Aug 2009 15:44:59 +0000 (15:44 +0000)
would translate into zero.  This caused NM to enforce a zero bwlimit on the
node, which caused node to act very very slowly.

planetlab/common/actions.php

index 5649a8c..6b78666 100644 (file)
@@ -418,7 +418,11 @@ switch ($action) {
    foreach ($interface_details as $field) {
      $interface[$field]= $_POST[$field];
      if( in_array( $field, array( 'bwlimit', 'node_id' ) ) ) {
-       $interface[$field]= intval( $interface[$field] );
+       if ( intval($interface[$field]) != 0 ) {
+           $interface[$field]= intval( $interface[$field]);
+       } elseif ($field=='bwlimit' ) {
+           $interface[$field] = NULL;
+       }
      }
    }
    $result=$api->UpdateInterface( intval( $interface_id ), $interface );