From 1e749bd792081e3f1cb629988d849f1c9ec57577 Mon Sep 17 00:00:00 2001 From: Neetu Jain Date: Thu, 14 May 2015 10:41:18 -0500 Subject: [PATCH 1/3] correct arguement name --- SoftLayer/CLI/loadbal/service_add.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/SoftLayer/CLI/loadbal/service_add.py b/SoftLayer/CLI/loadbal/service_add.py index 5a9fa982a..0b0d0da9f 100644 --- a/SoftLayer/CLI/loadbal/service_add.py +++ b/SoftLayer/CLI/loadbal/service_add.py @@ -24,7 +24,7 @@ @click.option('--healthcheck-type', required=True, help="The health check type") -@click.option('--ip-address', '--ip', +@click.option('--ip-address', required=True, help="The IP of the service") @environment.pass_env From 8427c4370ee9716983f441a34af9ae079335eb8b Mon Sep 17 00:00:00 2001 From: Neetu Jain Date: Thu, 14 May 2015 10:47:19 -0500 Subject: [PATCH 2/3] change argument name --- SoftLayer/CLI/loadbal/service_add.py | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/SoftLayer/CLI/loadbal/service_add.py b/SoftLayer/CLI/loadbal/service_add.py index 0b0d0da9f..9b84c8505 100644 --- a/SoftLayer/CLI/loadbal/service_add.py +++ b/SoftLayer/CLI/loadbal/service_add.py @@ -24,11 +24,11 @@ @click.option('--healthcheck-type', required=True, help="The health check type") -@click.option('--ip-address', +@click.option('--ip', required=True, help="The IP of the service") @environment.pass_env -def cli(env, identifier, enabled, port, weight, healthcheck_type, ip_address): +def cli(env, identifier, enabled, port, weight, healthcheck_type, ip): """Adds a new load balancer service.""" mgr = SoftLayer.LoadBalancerManager(env.client) @@ -37,10 +37,11 @@ def cli(env, identifier, enabled, port, weight, healthcheck_type, ip_address): # check if the IP is valid ip_address_id = None - if ip_address: + if ip: ip_service = env.client['Network_Subnet_IpAddress'] - ip_record = ip_service.getByIpAddress(ip_address) - ip_address_id = ip_record['id'] + ip_record = ip_service.getByIpAddress(ip) + if len(ip_record)>0: + ip_address_id = ip_record['id'] mgr.add_service(loadbal_id, group_id, From 9ce8c323d5818746e44716aa34d5b5b40f04f74d Mon Sep 17 00:00:00 2001 From: Neetu Jain Date: Tue, 26 May 2015 13:27:37 -0500 Subject: [PATCH 3/3] cleaning --- SoftLayer/CLI/loadbal/service_add.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/SoftLayer/CLI/loadbal/service_add.py b/SoftLayer/CLI/loadbal/service_add.py index 9b84c8505..735aee75a 100644 --- a/SoftLayer/CLI/loadbal/service_add.py +++ b/SoftLayer/CLI/loadbal/service_add.py @@ -40,7 +40,7 @@ def cli(env, identifier, enabled, port, weight, healthcheck_type, ip): if ip: ip_service = env.client['Network_Subnet_IpAddress'] ip_record = ip_service.getByIpAddress(ip) - if len(ip_record)>0: + if len(ip_record) > 0: ip_address_id = ip_record['id'] mgr.add_service(loadbal_id,