Merge pull request softlayer#1810 from caberos/issue1795

allmightyspiff · web-flow · commit a236652fffb1 · 2023-01-11T09:45:04.000-06:00
subnet route and subnet clear-route
diff --git a/SoftLayer/CLI/subnet/route.py b/SoftLayer/CLI/subnet/route.py
@@ -5,6 +5,7 @@
 
 import SoftLayer
 from SoftLayer.CLI import environment
+from SoftLayer.CLI import exceptions
 
 target_types = {'vlan': 'SoftLayer_Network_Vlan',
                 'ip': 'SoftLayer_Network_Subnet_IpAddress',
@@ -17,10 +18,25 @@
 @click.argument('identifier')
 @click.option('--target', type=click.Choice(['vlan', 'ip', 'hardware', 'vsi']),
               help='choose the type. vlan, ip, hardware, vsi')
-@click.option('--target-id', help='The identifier for the destination resource to route this subnet to. ')
+@click.option('--target-resource', help='Allows you to change the route of your secondary subnets.'
+                                        'Subnets may be routed as either Static or Portable, and that designation is '
+                                        'dictated by the routing destination specified.'
+                                        'Static subnets have an ultimate routing destination of a single IP address '
+                                        'but may not be routed to an existing subnet’s IP address whose '
+                                        'subnet is routed as a Static.'
+                                        'Portable subnets have an ultimate routing destination of a VLAN.'
+                                        'A subnet can be routed to any resource within the same "routing region"'
+                                        ' as the subnet itself, usually limited to a single data center.'
+                                        'See Also: '
+                                        'https://sldn.softlayer.com/reference/services/SoftLayer_Network_Subnet/route/')
 @environment.pass_env
-def cli(env, identifier, target, target_id):
+def cli(env, identifier, target, target_resource):
     """Assigns the subnet to a target."""
 
     mgr = SoftLayer.NetworkManager(env.client)
-    mgr.route(identifier, target_types.get(target), target_id)
+    route = mgr.route(identifier, target_types.get(target), target_resource)
+
+    if route:
+        click.secho("Route updated successfully.")
+    else:
+        raise exceptions.CLIAbort('Failed to route the subnet.')
