new feature edit ip note and add ipAddress table in detail

Author: caberos

SoftLayer/CLI/routes.py

@@ -289,6 +289,7 @@
     ('subnet:detail', 'SoftLayer.CLI.subnet.detail:cli'),
     ('subnet:list', 'SoftLayer.CLI.subnet.list:cli'),
     ('subnet:lookup', 'SoftLayer.CLI.subnet.lookup:cli'),
+    ('subnet:edit-ip', 'SoftLayer.CLI.subnet.edit_ip:cli'),
 
     ('tags', 'SoftLayer.CLI.tags'),
     ('tags:cleanup', 'SoftLayer.CLI.tags.cleanup:cli'),

SoftLayer/CLI/subnet/detail.py

@@ -25,7 +25,10 @@ def cli(env, identifier, no_vs, no_hardware):
     mgr = SoftLayer.NetworkManager(env.client)
     subnet_id = helpers.resolve_id(mgr.resolve_subnet_ids, identifier,
                                    name='subnet')
-    subnet = mgr.get_subnet(subnet_id)
+
+    mask = 'mask[ipAddresses[id, ipAddress], datacenter, virtualGuests,hardware]'
+
+    subnet = mgr.get_subnet(subnet_id, mask=mask)
 
     table = formatting.KeyValueTable(['name', 'value'])
     table.align['name'] = 'r'
@@ -45,6 +48,14 @@ def cli(env, identifier, no_vs, no_hardware):
     table.add_row(['usable ips',
                    subnet.get('usableIpAddressCount', formatting.blank())])
 
+    ip_address = subnet.get('ipAddresses')
+
+    ip_table = formatting.KeyValueTable(['ipAddress','value'])
+    for address in ip_address:
+        ip_table.add_row([address.get('id'),address.get('ipAddress')])
+
+    table.add_row(['ipAddresses', ip_table])
+
     if not no_vs:
         if subnet['virtualGuests']:
             vs_table = formatting.Table(['hostname', 'domain', 'public_ip', 'private_ip'])

SoftLayer/CLI/subnet/edit_ip.py

@@ -0,0 +1,27 @@
+"""Edit ip note"""
+# :license: MIT, see LICENSE for more details.
+
+import click
+
+import SoftLayer
+from SoftLayer.CLI import environment
+
+
+@click.command()
+@click.argument('identifier')
+@click.option('--ip', required=True,
+              help='Assume the ipAddress to set the note.')
+@click.option('--note', help="set ip address note of subnet")
+@environment.pass_env
+def cli(env, identifier, ip, note):
+    """Set the note of the ipAddress subnet"""
+
+    data = {
+        'note': note
+    }
+    mgr = SoftLayer.NetworkManager(env.client)
+    ips = mgr.get_subnet(identifier, mask='id,ipAddresses[id,ipAddress]').get('ipAddresses')
+
+    for address in ips:
+        if ip == address.get('ipAddress'):
+            mgr.set_subnet_ipddress_note(address.get('id'), data)

SoftLayer/fixtures/SoftLayer_Network_Subnet.py

@@ -25,5 +25,10 @@
         }
     ],
     'hardware': [],
-    'usableIpAddressCount': 22
+    'usableIpAddressCount': 22,
+    'ipAddresses': [
+        {'id': 123456,
+         'ipAddress': '16.26.26.25'},
+        {'id': 123457,
+         'ipAddress': '16.26.26.26'}]
 }

SoftLayer/fixtures/SoftLayer_Network_Subnet_IpAddress.py

@@ -7,3 +7,5 @@
     'isReserved': False,
     'subnetId': 5678,
 }
+
+editObject= True

SoftLayer/managers/network.py

@@ -685,3 +685,8 @@ def get_nas_credentials(self, identifier, **kwargs):
         """
         result = self.network_storage.getObject(id=identifier, **kwargs)
         return result
+
+    def set_subnet_ipddress_note(self, identifier, note):
+        """Set the ip address note of the subnet"""
+        result = self.client.call('SoftLayer_Network_Subnet_IpAddress','editObject', note, id=identifier)
+        return result

tests/CLI/modules/subnet_tests.py

@@ -36,6 +36,9 @@ def test_detail(self):
                         'private_ip': '10.0.1.2'
                     }
                 ],
+                 'ipAddresses': {
+                     '123456': '16.26.26.25',
+                     '123457': '16.26.26.26'},
                 'hardware': 'none',
                 'usable ips': 22
             },
@@ -134,3 +137,8 @@ def test_create_subnet_static_ipv6(self, confirm_mock):
         ]
 
         self.assertEqual(output, json.loads(result.output))
+
+    def test_editrou_Ip(self):
+        result = self.run_command(['subnet', 'edit-ip', '123456', '--ip=16.26.26.26', '--note=test'])
+        self.assert_no_fail(result)
+        self.assertTrue(result)