add the rebundant/degraded option

Author: Daniel Cabero Barrios

SoftLayer/CLI/hardware/edit.py

@@ -22,8 +22,10 @@
               help="Public port speed. -1 is best speed available")
 @click.option('--private-speed', default=None, type=click.Choice(['0', '10', '100', '1000', '10000', '-1']),
               help="Private port speed. -1 is best speed available")
+@click.option('--rebundant', is_flag=True,default=False, help="The desired state of redundancy for the interface(s)")
+@click.option('--degraded', is_flag=True,default=False, help="The desired state of degraded for the interface(s)")
 @environment.pass_env
-def cli(env, identifier, domain, userfile, tag, hostname, userdata, public_speed, private_speed):
+def cli(env, identifier, domain, userfile, tag, hostname, userdata, public_speed, private_speed, rebundant, degraded):
     """Edit hardware details."""
 
     if userdata and userfile:
@@ -51,7 +53,17 @@ def cli(env, identifier, domain, userfile, tag, hostname, userdata, public_speed
         raise exceptions.CLIAbort("Failed to update hardware")
 
     if public_speed is not None:
-        mgr.change_port_speed(hw_id, True, int(public_speed))
+        if rebundant:
+            mgr.change_port_speed(hw_id, True, int(public_speed), 'rebundant')
+        if degraded:
+            mgr.change_port_speed(hw_id, True, int(public_speed), 'degraded')
+        if not rebundant and not degraded:
+            raise exceptions.CLIAbort("Failed to update hardwar")
 
     if private_speed is not None:
-        mgr.change_port_speed(hw_id, False, int(private_speed))
+        if rebundant:
+            mgr.change_port_speed(hw_id, False, int(private_speed), 'rebundant')
+        if degraded:
+            mgr.change_port_speed(hw_id, False, int(private_speed), 'degraded')
+        if not rebundant and not degraded:
+            raise exceptions.CLIAbort("Failed to update hardware")

SoftLayer/managers/hardware.py

@@ -297,7 +297,7 @@ def rescue(self, hardware_id):
         """
         return self.hardware.bootToRescueLayer(id=hardware_id)
 
-    def change_port_speed(self, hardware_id, public, speed):
+    def change_port_speed(self, hardware_id, public, speed, rebundant):
         """Allows you to change the port speed of a server's NICs.
 
         :param int hardware_id: The ID of the server
@@ -319,11 +319,11 @@ def change_port_speed(self, hardware_id, public, speed):
         if public:
             return self.client.call('Hardware_Server',
                                     'setPublicNetworkInterfaceSpeed',
-                                    speed, id=hardware_id)
+                                    [rebundant, speed], id=hardware_id)
         else:
             return self.client.call('Hardware_Server',
                                     'setPrivateNetworkInterfaceSpeed',
-                                    speed, id=hardware_id)
+                                    [rebundant, speed], id=hardware_id)
 
     def place_order(self, **kwargs):
         """Places an order for a piece of hardware.

tests/CLI/modules/server_tests.py

@@ -526,7 +526,9 @@ def test_edit(self):
                                    '--tag=dev',
                                    '--tag=green',
                                    '--public-speed=10',
+                                   '--rebundant',
                                    '--private-speed=100',
+                                   '--degraded',
                                    '100'])
 
         self.assert_no_fail(result)
@@ -544,12 +546,12 @@ def test_edit(self):
         )
         self.assert_called_with(
             'SoftLayer_Hardware_Server', 'setPublicNetworkInterfaceSpeed',
-            args=(10,),
+            args=(['rebundant', 10],),
             identifier=100,
         )
         self.assert_called_with(
             'SoftLayer_Hardware_Server', 'setPrivateNetworkInterfaceSpeed',
-            args=(100,),
+            args=(['degraded', 100],),
             identifier=100,
         )
 

tests/managers/hardware_tests.py

@@ -335,20 +335,20 @@ def test_cancel_running_transaction(self):
                           12345)
 
     def test_change_port_speed_public(self):
-        self.hardware.change_port_speed(2, True, 100)
+        self.hardware.change_port_speed(2, True, 100, 'degraded')
 
         self.assert_called_with('SoftLayer_Hardware_Server',
                                 'setPublicNetworkInterfaceSpeed',
                                 identifier=2,
-                                args=(100,))
+                                args=(['degraded', 100],))
 
     def test_change_port_speed_private(self):
-        self.hardware.change_port_speed(2, False, 10)
+        self.hardware.change_port_speed(2, False, 10, 'rebundant')
 
         self.assert_called_with('SoftLayer_Hardware_Server',
                                 'setPrivateNetworkInterfaceSpeed',
                                 identifier=2,
-                                args=(10,))
+                                args=(['rebundant', 10],))
 
     def test_edit_meta(self):
         # Test editing user data