From 3ee962f1ac38f31d346f6af237fe9084c14d7860 Mon Sep 17 00:00:00 2001
From: Eddie Arrage <eddie.arrage@huawei.com>
Date: Wed, 11 Apr 2018 02:44:51 +0000
Subject: Modified validation script for tracing to support CI

- Changed default Jaeger ports to 16686 for use with basic
kubernetes port-forward and CI scripts
- Added CLI to validate script to disable istio service check
by default. This requires at least a single http request
to istio-ingress after Jaeger deployment. It can be enabled
with 'python validate.py -s'. Port and IP address for Jaeger
can optionally be specified with '-ip' and '-port' options
- Modified tracing doc to add k8s port-forward example in addition
to k8s expose

Change-Id: I10fb4d3cccfa50370d44ed7446f67a49c538bba9
Signed-off-by: Eddie Arrage <eddie.arrage@huawei.com>
---
 clover/tracing/validate.py | 39 +++++++++++++++++++++++++++++----------
 1 file changed, 29 insertions(+), 10 deletions(-)

(limited to 'clover/tracing/validate.py')

diff --git a/clover/tracing/validate.py b/clover/tracing/validate.py
index 9cbfdd0..0f6f0a8 100644
--- a/clover/tracing/validate.py
+++ b/clover/tracing/validate.py
@@ -6,12 +6,11 @@
 # http://www.apache.org/licenses/LICENSE-2.0
 
 from kubernetes import client, config
+import argparse
 
 from clover.tracing.tracing import Tracing
+# from tracing import Tracing
 
-JAEGER_IP = "localhost"
-# JAEGER_IP = "1.1.1.1"
-JAEGER_PORT = "30888"
 JAEGER_DEPLOYMENT = "jaeger-deployment"
 ISTIO_NAMESPACE = "istio-system"
 ISTIO_SERVICES = ["istio-ingress", "istio-mixer"]
@@ -36,10 +35,12 @@ def validateDeploy():
             validate = True
     return validate
 
-# Services in Jaeger will only be present when traffic passes through Istio
-# Requires a deployment in Istio service mesh with some traffic targeting nodes
-def validateServices():
-    t = Tracing(JAEGER_IP, JAEGER_PORT)
+
+# Services in Jaeger will only be present when traffic targets istio-ingress
+# Even a failed HTTP GET request to istio-ingress will add istio-ingress and
+# istio-mixer services
+def validateServices(args):
+    t = Tracing(args['ip'], args['port'])
     services = t.getServices()
     validate = True
     if services:
@@ -47,14 +48,20 @@ def validateServices():
             if s in services:
                 print("Service in tracing: {} present".format(s))
             else:
+                print("Service in tracing: {} not present".format(s))
                 validate = False
     else:
         validate = False
     return validate
 
 
-def main():
-    if validateDeploy() and validateServices():
+def main(args):
+    vdeploy = validateDeploy()
+    if args['s']:
+        vservice = validateServices(args)
+    else:
+        vservice = True
+    if vdeploy and vservice:
         print"Jaeger tracing validation has passed"
         return True
     else:
@@ -63,4 +70,16 @@ def main():
 
 
 if __name__ == '__main__':
-    main()
+    parser = argparse.ArgumentParser()
+    parser.add_argument(
+            '-s', action='store_true',
+            help='Validate istio services, \
+            which requires at least one http request to istio-ingress')
+    parser.add_argument(
+            '-ip', default='localhost',
+            help='IP address to access Jaeger')
+    parser.add_argument(
+            '-port', default='16686',
+            help='Port to acccess Jaeger')
+    args = parser.parse_args()
+    main(vars(args))
-- 
cgit 1.2.3-korg