From: Vratko Polak <vrpolak@cisco.com>
Date: Wed, 15 Aug 2018 07:51:40 +0000 (+0200)
Subject: PAL: Log and continue when out of colors
X-Git-Url: https://gerrit.fd.io/r/gitweb?p=csit.git;a=commitdiff_plain;h=399b58fe776d7b97ddf098d28093ad49c089aaea

PAL: Log and continue when out of colors

Previously, IndexError happened in worker, getting the job stuck.

Graphs should not need that many colors, so there is a bug to fix still.

Change-Id: I1af415119cf06004be5febbd03636f847a7001be
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
---

diff --git a/resources/tools/presentation/generator_CPTA.py b/resources/tools/presentation/generator_CPTA.py
index 7279ea345a..a4459bc003 100644
--- a/resources/tools/presentation/generator_CPTA.py
+++ b/resources/tools/presentation/generator_CPTA.py
@@ -318,13 +318,21 @@ def _generate_all_charts(spec, input_data):
                 logs.append(("WARNING", "No data for the test '{0}'".
                              format(test_name)))
                 continue
+            message = "index: {index}, test: {test}".format(
+                index=index, test=test_name)
             test_name = test_name.split('.')[-1]
-            trace, rslt = _generate_trending_traces(
-                test_data,
-                job_name=job_name,
-                build_info=build_info,
-                name='-'.join(test_name.split('-')[2:-1]),
-                color=COLORS[index])
+            try:
+                trace, rslt = _generate_trending_traces(
+                    test_data,
+                    job_name=job_name,
+                    build_info=build_info,
+                    name='-'.join(test_name.split('-')[2:-1]),
+                    color=COLORS[index])
+            except IndexError:
+                message = "Out of colors: {}".format(message)
+                logs.append(("ERROR", message))
+                logging.error(message)
+                continue
             traces.extend(trace)
             res.append(rslt)
             index += 1