commit c8224905c1630a69c6caaf38e53be3abf97e740e
author alshabib <alshabibi.ali@gmail.com> Thu Jan 26 11:59:52 2017 -0800
committer Zsolt Haraszti <zharaszt@ciena.com> Thu Jan 26 22:27:09 2017 -0800
tree 416f7e558697ee4441a085944a43790a21e97f98
parent accad4ac8c0e4603ce8b38b7aaec42147c9687b3

Cleaning up TODO items

removing TODO on line 162 of chameleon/grpc_client
and entering it into backlog as CORD-815

removing TODO on line 135 of cli/utils.py
and entering it into backlog as CORD-816

removing TODO on line 155 of cli/utils.py
and entering it into backlog as CORD-817

removing TODO on line 94 of common/utils/consulhelpers.py
and entering it into backlog as CORD-818

Change-Id: I736ea678b16c9b0e85a3cf0aabe96da19caae64d

grpc_client/grpc_client.py

diff --git a/grpc_client/grpc_client.py b/grpc_client/grpc_client.py
index 9b68912..234f24a 100644
--- a/grpc_client/grpc_client.py
+++ b/grpc_client/grpc_client.py
@@ -159,8 +159,8 @@
                         service_name=service_name)
             yield asleep(1.0)
 
-        # pick a random entry
-        # TODO should we prefer local IP addresses? Probably.
+        # pick local addresses when resolving a service via consul
+        # see CORD-815 (https://jira.opencord.org/browse/CORD-815)
 
         service = services[randint(0, len(services) - 1)]
         endpoint = '{}:{}'.format(service['ServiceAddress'],